Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add ngc signing job for auto signing #739

Merged
merged 1 commit into from
Jun 24, 2024

Conversation

shivakunv
Copy link
Contributor

Hello!

Thanks for making this contribution! When contributing to this repository please keep in mind the following:

@shivakunv
Copy link
Contributor Author

@cdesiniotis @elezar PTAL

@shivakunv shivakunv force-pushed the ngc_signing_job branch 6 times, most recently from d57687d to fa4871c Compare June 12, 2024 08:55
Copy link
Contributor

@cdesiniotis cdesiniotis left a comment

Choose a reason for hiding this comment

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

@shivakunv For both the gpu-operator and gpu-operator-validator images, we push a short tag and a tag with the ubi8 suffix. I left some suggestions so that we sign both tags now. These changes align with the changes you made in the other projects.

.nvidia-ci.yml Outdated Show resolved Hide resolved
.nvidia-ci.yml Show resolved Hide resolved
.nvidia-ci.yml Show resolved Hide resolved
.nvidia-ci.yml Outdated Show resolved Hide resolved
.nvidia-ci.yml Show resolved Hide resolved
@shivakunv shivakunv force-pushed the ngc_signing_job branch 2 times, most recently from c90b88c to 5b98069 Compare June 13, 2024 06:51
@shivakunv
Copy link
Contributor Author

@cdesiniotis resolved all comments PTAL

@cdesiniotis cdesiniotis merged commit de55fb7 into NVIDIA:main Jun 24, 2024
9 of 11 checks passed
@shivakunv shivakunv self-assigned this Jul 5, 2024
@shivakunv shivakunv deleted the ngc_signing_job branch July 27, 2024 06:13
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