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
.github: Fix image digest job printing #16660
Conversation
Commit ba728f0 does not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
Commits ba728f0, 61e036a515b2c514676dc05908c31eba96584476 do not contain "Signed-off-by". Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin |
Great, looks like the fix works: https://github.com/cilium/cilium/pull/16660/checks?check_run_id=2934571036 |
Commit 044afab ("ci: Set up qemu in images workflow and build cilium-test") introduced a new dependency for the "Display image digests" job which provides a nice single page that developers can use to find all of the CI image artifacts generated from the PR. Unfortunately the new dependency is conditionally skipped, and when it is skipped, the display digests job is also skipped because the dependency is skipped. This is a known behaviour of GitHub actions/runner, issue 491. This commit works around the issue by explicitly running the job if the dependency was skipped, allowing the "display image digests" job to run even when the qemu job is skipped. Signed-off-by: Joe Stringer <joe@cilium.io>
61e036a
to
6dad17f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤦🏻
One more thing on the list of GH Actions quirks.
Commit 044afab ("ci: Set up qemu in images workflow and build
cilium-test") introduced a new dependency for the "Display image digests"
job which provides a nice single page that developers can use to find
all of the CI image artifacts generated from the PR. Unfortunately the
new dependency is conditionally skipped, and when it is skipped, the
display digests job is also skipped because the dependency is skipped.
This is a known behaviour of GitHub actions/runner, issue 491.
This commit works around the issue by explicitly running the job if the
dependency was skipped, allowing the "display image digests" job to run
even when the qemu job is skipped.
This PR proposes the workaround listed on actions/runner#491 .