Skip to content

Conversation

aanm
Copy link
Member

@aanm aanm commented Aug 14, 2025

This was also bothering me

Fixes #41138

@aanm aanm requested a review from joestringer August 14, 2025 14:11
@aanm aanm requested review from a team as code owners August 14, 2025 14:11
@aanm aanm added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Aug 14, 2025
@aanm aanm requested a review from christarazi August 14, 2025 14:11
@aanm aanm enabled auto-merge August 14, 2025 14:12
@aanm
Copy link
Member Author

aanm commented Aug 14, 2025

/test

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.

I think it's worth a try to see if this helps. If there's a temporary service outage <2.5m then this should improve reliability.

One discussion point below for CI builds.

@aanm
Copy link
Member Author

aanm commented Aug 19, 2025

/test

@aanm aanm added this pull request to the merge queue Aug 19, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 19, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 19, 2025
@aanm
Copy link
Member Author

aanm commented Aug 19, 2025

/test

@aanm aanm enabled auto-merge August 19, 2025 14:40
@joestringer joestringer added dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. and removed ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Aug 19, 2025
aanm added 3 commits August 20, 2025 11:10
Add composite action to decrease the number of duplicated code across
our build workflows.

Signed-off-by: André Martins <andre@cilium.io>
Make use of the new cosign composite image to decrease the amount of
duplicated code in our build image process.

Signed-off-by: André Martins <andre@cilium.io>
To prevent this action from failing, we should add a retry logic so that
steps that are prone to fail can be retried.

Signed-off-by: André Martins <andre@cilium.io>
@aanm
Copy link
Member Author

aanm commented Aug 20, 2025

/test

@aanm aanm added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed dont-merge/needs-rebase This PR needs to be rebased because it has merge conflicts. ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Aug 20, 2025
@aanm aanm added this pull request to the merge queue Aug 20, 2025
@maintainer-s-little-helper maintainer-s-little-helper bot added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Aug 20, 2025
Merged via the queue into main with commit abab821 Aug 20, 2025
360 of 361 checks passed
@aanm aanm deleted the pr/fix-cosign branch August 20, 2025 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make CI image signing reliable
3 participants