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

workflows: increase multicluster timeout to 30 minutes #15811

Merged
merged 1 commit into from
Apr 21, 2021
Merged

workflows: increase multicluster timeout to 30 minutes #15811

merged 1 commit into from
Apr 21, 2021

Conversation

nbusseneau
Copy link
Member

We are hitting the 20 minutes limit quite a lot:

installation-and-connectivity
The job running on runner GitHub Actions 8 has exceeded the maximum execution time of 20 minutes.

Bumping to 30 minutes should be enough to get us through until we switch to a more intelligent step-based timeout rather than using global job timeouts.

We are hitting the 20 minutes limit quite a lot:

> installation-and-connectivity
> The job running on runner GitHub Actions 8 has exceeded the maximum
> execution time of 20 minutes.

Bumping to 30 minutes should be enough to get us through until we switch
to a more intelligent step-based timeout rather than using global job
timeouts.

Signed-off-by: Nicolas Busseneau <nicolas@isovalent.com>
@nbusseneau nbusseneau requested review from a team as code owners April 21, 2021 14:41
@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 21, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Apr 21, 2021
@nbusseneau nbusseneau added the release-note/ci This PR makes changes to the CI. label Apr 21, 2021
@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 Apr 21, 2021
@nbusseneau nbusseneau 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 Apr 21, 2021
@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 Apr 21, 2021
@aanm aanm merged commit be573aa into cilium:master Apr 21, 2021
1.10.0 automation moved this from In progress to Done Apr 21, 2021
@nbusseneau nbusseneau deleted the pr/workflow-multicluster-increase-timeout branch April 29, 2021 14:53
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 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

5 participants