Skip to content

Conversation

rastislavs
Copy link
Contributor

Extend timeout for the tests-e2e-upgrade workflow, as currently we sometimes hit this limit in some jobs.

Looks like it was lowered a bit too much in #34806

(e.g. https://github.com/cilium/cilium/actions/runs/11592950244/job/32283500641 - timeout while collecting sysdump)

Extend timeout for the tests-e2e-upgrade workflow
from 45 to 55 min, as currently we sometimes hit this
limit in some jobs.

Signed-off-by: Rastislav Szabo <rastislav.szabo@isovalent.com>
@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 Nov 1, 2024
@rastislavs rastislavs added the release-note/ci This PR makes changes to the CI. label Nov 1, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 1, 2024
@rastislavs rastislavs added area/CI Continuous Integration testing issue or flake dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Nov 1, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Nov 1, 2024
@rastislavs
Copy link
Contributor Author

/test

@rastislavs rastislavs marked this pull request as ready for review November 1, 2024 17:45
@rastislavs rastislavs requested review from a team as code owners November 1, 2024 17:45
@rastislavs rastislavs requested a review from brlbil November 1, 2024 17:45
@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 Nov 1, 2024
@julianwiedmann julianwiedmann added this pull request to the merge queue Nov 4, 2024
Merged via the queue into cilium:main with commit 0190c00 Nov 4, 2024
72 checks passed
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.

3 participants