Skip to content

Conversation

aanm
Copy link
Member

@aanm aanm commented Aug 13, 2025

Without a timeout, this step could get stuck until the workflow timeout is reached. Thus, we add a sane timeout of 2 minutes for the step to be completed.

@aanm aanm added the kind/bug/CI This is a bug in the testing code. label Aug 13, 2025
@aanm aanm requested a review from a team as a code owner August 13, 2025 13:55
@aanm aanm added the area/CI Continuous Integration testing issue or flake label Aug 13, 2025
@aanm aanm requested a review from a team as a code owner August 13, 2025 13:55
@aanm aanm added the release-note/ci This PR makes changes to the CI. label Aug 13, 2025
@aanm aanm requested review from joestringer and brlbil August 13, 2025 13:55
@aanm aanm enabled auto-merge August 13, 2025 13:56
@aanm
Copy link
Member Author

aanm commented Aug 13, 2025

/test

Without a timeout, this step could get stuck until the workflow timeout
is reached. Thus, we add a sane timeout of 2 minutes for the step to be
completed.

Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm force-pushed the pr/fix-step-timeout branch from 6ff8708 to edd75d0 Compare August 14, 2025 14:59
@aanm aanm requested a review from joestringer August 14, 2025 14:59
@aanm
Copy link
Member Author

aanm commented Aug 14, 2025

/test

@aanm aanm added needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Aug 14, 2025
@aanm aanm added this pull request to the merge queue Aug 14, 2025
Merged via the queue into main with commit 4b8cec7 Aug 14, 2025
164 of 166 checks passed
@aanm aanm deleted the pr/fix-step-timeout branch August 14, 2025 17:57
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. labels Aug 14, 2025
@joamaki joamaki mentioned this pull request Aug 19, 2025
2 tasks
@joamaki joamaki added backport-pending/1.17 The backport for Cilium 1.17.x for this PR is in progress. and removed needs-backport/1.17 This PR / issue needs backporting to the v1.17 branch labels Aug 19, 2025
@joamaki joamaki mentioned this pull request Aug 19, 2025
19 tasks
@joamaki joamaki added backport-pending/1.18 The backport for Cilium 1.18.x for this PR is in progress. and removed needs-backport/1.18 This PR / issue needs backporting to the v1.18 branch labels Aug 19, 2025
@github-actions github-actions bot added backport-done/1.17 The backport for Cilium 1.17.x for this PR is done. backport-done/1.18 The backport for Cilium 1.18.x for this PR is done. and removed backport-pending/1.17 The backport for Cilium 1.17.x for this PR is in progress. backport-pending/1.18 The backport for Cilium 1.18.x for this PR is in progress. labels Aug 21, 2025
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 backport-done/1.17 The backport for Cilium 1.17.x for this PR is done. backport-done/1.18 The backport for Cilium 1.18.x for this PR is done. kind/bug/CI This is a bug in the testing code. 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.

4 participants