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

cilium: Use build-and-push-with-qemu for builder #15679

Merged
merged 1 commit into from
Apr 13, 2021

Conversation

jrfastab
Copy link
Contributor

Fix images-legacy.yaml to use build-and-push-with-qemu.

Fixes: 044afab ("ci: Set up qemu in images workflow and build cilium-test")
Signed-off-by: John Fastabend john.fastabend@gmail.com

Fix images-legacy.yaml to use build-and-push-with-qemu.

Fixes: 044afab ("ci: Set up qemu in images workflow and build cilium-test")
Signed-off-by: John Fastabend <john.fastabend@gmail.com>
@jrfastab jrfastab requested review from a team as code owners April 13, 2021 20:07
@jrfastab jrfastab requested review from nebril and aanm April 13, 2021 20:07
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Apr 13, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Apr 13, 2021
Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Given that master is currently broken with the following message and this aligns the job requirement with the job introduced in the commit being fixed here, this LGTM and I will merge without waiting for CI to pass.

The workflow is not valid. Job image-digests depends on unknown job build-and-push-cilium-test.
https://github.com/cilium/cilium/actions/runs/745914737

@joestringer joestringer merged commit 36e45bf into master Apr 13, 2021
1.10.0 automation moved this from In progress to Done Apr 13, 2021
@joestringer joestringer deleted the pr/jrfastab/fix-images-legacy branch April 13, 2021 20:10
@joestringer
Copy link
Member

CC @nebril to be aware for future CI workflow testing (ref. #15605)

@joestringer joestringer added release-note/misc This PR makes changes that have no direct user impact. release-note/ci This PR makes changes to the CI. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. release-note/misc This PR makes changes that have no direct user impact. labels Apr 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants