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

ci: increase connectivity test timeout in GHA external workload #26975

Merged

Conversation

mhofstetter
Copy link
Member

This commit increases the timeout of the job Installation and Connectivity Test within the worklfow External Workload from 30m to 45m.

The current timeout is too low:

Installation and Connectivity Test (1.26, europe-north1-b, 5)
The job running on runner GitHub Actions 52 has exceeded the maximum execution time of 30 minutes.`

https://github.com/cilium/cilium/actions/runs/5604532568

This commit increases the timeout of the job
`Installation and Connectivity Test` within the worklfow `External
Workload` from 30m to 45m.

The current timeout is too low:

`Installation and Connectivity Test (1.26, europe-north1-b, 5)
The job running on runner GitHub Actions 52 has exceeded the maximum execution time of 30 minutes.`

https://github.com/cilium/cilium/actions/runs/5604532568

Signed-off-by: Marco Hofstetter <marco.hofstetter@isovalent.com>
@mhofstetter mhofstetter added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. labels Jul 21, 2023
@mhofstetter mhofstetter requested review from a team as code owners July 21, 2023 08:02
@mhofstetter
Copy link
Member Author

/test

@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 Jul 21, 2023
@nbusseneau
Copy link
Member

What's funny here is that it's always this specific combo that times out, are the instances in this region slower? :p

@aditighag aditighag merged commit b952c55 into main Jul 21, 2023
177 checks passed
@aditighag aditighag deleted the pr/mhofstetter/ci-ext-workload-increase-conntest-timeout branch July 21, 2023 22:17
@julianwiedmann julianwiedmann added needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch labels Aug 8, 2023
@joamaki joamaki mentioned this pull request Aug 8, 2023
9 tasks
@joamaki joamaki added backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. and removed needs-backport/1.14 This PR / issue needs backporting to the v1.14 branch backport-pending/1.14 The backport for Cilium 1.14.x for this PR is in progress. labels Aug 8, 2023
@joamaki joamaki mentioned this pull request Aug 9, 2023
9 tasks
@joamaki joamaki added backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. and removed needs-backport/1.13 This PR / issue needs backporting to the v1.13 branch labels Aug 9, 2023
@YutaroHayakawa YutaroHayakawa added backport-done/1.13 The backport for Cilium 1.13.x for this PR is done. and removed backport-pending/1.13 The backport for Cilium 1.13.x for this PR is in progress. labels Aug 17, 2023
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.13 The backport for Cilium 1.13.x for this PR is done. backport-done/1.14 The backport for Cilium 1.14.x for this PR is done. 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.

None yet

6 participants