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 GKE/EKS ginkgo timeout to 3 hours #10809

Merged
merged 1 commit into from Apr 2, 2020

Conversation

raybejjani
Copy link
Contributor

amusingly, as I make this, we are at the max 5 GKE jobs allowed. This doesn't fix that, and it may fail with "cannot select cluster" instead

We've been seeing more timeouts on GKE when our CI is under load and
jobs wait in the queue for a free executor. The ginkgo timeout is
intended to be shorter than the jenkins timeout so that cleanups can
happen and so ginkgo can correctly log the timeout (instead of just
being terminated).

We've been seeing more timeouts on GKE when our CI is under load and
jobs wait in the queue for a free executor. The ginkgo timeout is
intended to be shorter than the jenkins timeout so that cleanups can
happen and so ginkgo can correctly log the timeout (instead of just
being terminated).

Signed-off-by: Ray Bejjani <ray@isovalent.com>
@raybejjani raybejjani added pending-review area/CI Continuous Integration testing issue or flake labels Apr 1, 2020
@raybejjani raybejjani requested a review from a team as a code owner April 1, 2020 18:34
@maintainer-s-little-helper
Copy link

Please set the appropriate release note label.

2 similar comments
@maintainer-s-little-helper
Copy link

Please set the appropriate release note label.

@maintainer-s-little-helper
Copy link

Please set the appropriate release note label.

@raybejjani
Copy link
Contributor Author

raybejjani commented Apr 1, 2020

test-gke (istio flake :( https://jenkins.cilium.io/job/Cilium-PR-K8s-GKE/216/ )

@raybejjani raybejjani added the release-note/ci This PR makes changes to the CI. label Apr 1, 2020
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 45.53% when pulling e94786b on raybejjani:ci-longer-ginkgo-timeouts into 19b5b7c on cilium:master.

@raybejjani
Copy link
Contributor Author

test-gke

@nebril
Copy link
Member

nebril commented Apr 2, 2020

Let's merge it

@raybejjani raybejjani merged commit 3de0635 into cilium:master Apr 2, 2020
1.8.0 automation moved this from In progress to Merged Apr 2, 2020
@raybejjani raybejjani deleted the ci-longer-ginkgo-timeouts branch April 2, 2020 09:57
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
1.8.0
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

3 participants