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

Tighten CI job timeout down to 15min #937

Merged
merged 1 commit into from
Aug 27, 2021
Merged

Conversation

stefanpenner
Copy link
Collaborator

@stefanpenner stefanpenner commented Aug 25, 2021

Speaking with the team, we would love to reach a 10 minute Commit -> CI
feedback loop goal. (Assuming the GH Actions queue depth isn't too deep)

We have spoken with GH to increase our concurrency, but in-order to
reach our goal each of our jobs cannot exceed 5->8minutes.

This is a first step in locking that in.


Note: I will wait until everyone is around to chat about this more.

@stefanpenner stefanpenner marked this pull request as draft August 25, 2021 19:18
@stefanpenner
Copy link
Collaborator Author

The first test failure appears completely unrelated, rather just a yarn -> registry connection failure

##[debug]/usr/bin/bash -e /home/runner/work/_temp/29af068a-7dc7-405e-bd92-171e9757ad02.sh
Volta error: Could not download [email protected]
from https://registry.npmjs.org/yarn/-/yarn-1.22.5.tgz

Please verify your internet connection and ensure the correct version is specified.
Volta error: Error cause: Io Error: Resource temporarily unavailable (os error 11)

Error cause: Io Error: Resource temporarily unavailable (os error 11)

@stefanpenner
Copy link
Collaborator Author

stefanpenner commented Aug 26, 2021

It looks like some of the windows tests exceed the 15 minute window. So we will likely need to split them, or reduce their times.

cc @thoov

@stefanpenner stefanpenner marked this pull request as ready for review August 26, 2021 00:29
Speaking with the team, we would love to reach a 10 minute Commit -> CI
feedback loop goal. (Assuming the GH Actions queue depth isn't too deep)

We have spoken with  GH to increase our concurrency, but in-order to
reach our goal each of our jobs cannot exceed 5->8minutes.

This is a first step in locking that in.
@stefanpenner stefanpenner merged commit a99af05 into master Aug 27, 2021
@stefanpenner stefanpenner deleted the tighten-timeouts branch August 27, 2021 19:48
@rwjblue rwjblue changed the title Tighten job timeouts Tighten CI job timeout down to 15min Aug 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants