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
[Merged by Bors] - Add timeouts to GH actions workflow #2167
Conversation
Also, remove first slack notification, this is unnecessary since this test is so quick
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've noticed this, too. I'm wondering why they hang... Don't think we've seen this behaviour on Travis 🤔
I think that timing out is probably better than just hanging, but still pretty frustrating if we don't know why.
Agree. Took another quick look just now and I'm not sure. Here are some recent timeouts:
All of these failed right after Let's merge this and investigate further if the timeouts continue. |
bors merge |
## Motivation I've noticed that these tests hang sometimes, this should make our lives a little easier by timing out tests if they're obviously hanging. I reviewed the times of the last few successful runs of each job and added a pretty large buffer in these timeouts. (see, e.g., https://github.com/spacemeshos/go-spacemesh/runs/1294892834?check_suite_focus=true) ## Changes - Adds `timeout-minutes` to each workflow job (per https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-syntax-for-github-actions#jobsjob_idstepstimeout-minutes) - Removes initial slack notification, as it's redundant since this test is so quick anyway ## Test Plan N/A
Build failed:
|
bors merge |
## Motivation I've noticed that these tests hang sometimes, this should make our lives a little easier by timing out tests if they're obviously hanging. I reviewed the times of the last few successful runs of each job and added a pretty large buffer in these timeouts. (see, e.g., https://github.com/spacemeshos/go-spacemesh/runs/1294892834?check_suite_focus=true) ## Changes - Adds `timeout-minutes` to each workflow job (per https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-syntax-for-github-actions#jobsjob_idstepstimeout-minutes) - Removes initial slack notification, as it's redundant since this test is so quick anyway ## Test Plan N/A
Pull request successfully merged into develop. Build succeeded: |
Motivation
I've noticed that these tests hang sometimes, this should make our lives a little easier by timing out tests if they're obviously hanging. I reviewed the times of the last few successful runs of each job and added a pretty large buffer in these timeouts. (see, e.g., https://github.com/spacemeshos/go-spacemesh/runs/1294892834?check_suite_focus=true)
Changes
timeout-minutes
to each workflow job (per https://docs.github.com/en/free-pro-team@latest/actions/reference/workflow-syntax-for-github-actions#jobsjob_idstepstimeout-minutes)Test Plan
N/A