Skip to content

Fix GPG KEY#1678

Merged
wangxingjun778 merged 13 commits into
modelscope:masterfrom
wangxingjun778:fix/preview
Apr 14, 2026
Merged

Fix GPG KEY#1678
wangxingjun778 merged 13 commits into
modelscope:masterfrom
wangxingjun778:fix/preview

Conversation

@wangxingjun778
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the GPG_KEY environment variable in the Dockerfiles to include an additional key and modifies the gpg --recv-keys command to support multiple keys. A redundant block of ARG and ENV definitions was identified in docker/Dockerfile.extra_install that should be removed to improve maintainability.

Comment on lines 38 to 42
#install and config python copy from https://github.com/docker-library/python/blob/1b7a1106674a21e699b155cbd53bf39387284cca/3.10/bookworm/Dockerfile
ARG PYTHON_VERSION={python_version}
ENV PATH /usr/local/bin:$PATH
ENV GPG_KEY A035C8C19219BA821ECEA86B64E628F8D684696D
ENV GPG_KEY="A035C8C19219BA821ECEA86B64E628F8D684696D 7169605F62C751356D054A26A821E680E5FA6305"
ENV PYTHON_VERSION {python_version}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

This block is a duplicate of lines 32-36. Redundant definitions of ARG and ENV variables should be removed to improve maintainability and reduce clutter in the Dockerfile.

@wangxingjun778 wangxingjun778 merged commit 9cc43a3 into modelscope:master Apr 14, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants