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

Reduce Cygwin timeouts to 240m like the remaining workflows #449

Merged
merged 1 commit into from
Apr 7, 2024

Conversation

jmid
Copy link
Collaborator

@jmid jmid commented Apr 5, 2024

Earlier, we struggled with Cygwin workflows being slow.
This was a necessity in #305 and #313 as the Cygwin port initially was too slow to complete within the 6h timeout!
In #420 we then merged them into a single workflow, like the remaining targets.
This is running well, but still suffers from timeouts due to #307, adding 2 hours compared to the MinGW counterparts.
This PR therefore reduced the Cygwin workflow timeouts to 240m like the remaining workflows.

@jmid
Copy link
Collaborator Author

jmid commented Apr 7, 2024

CI summary

Out of 37 workflows 3 failed - all with genuine issues.

Note: the 5.2 and trunk Cygwin workflows completed in 107min and 105min, respectively. As such 240min should be safe.

@jmid jmid merged commit c65989d into main Apr 7, 2024
24 of 27 checks passed
@jmid jmid deleted the reduce-cygwin-timeout branch April 7, 2024 20:51
@jmid
Copy link
Collaborator Author

jmid commented Apr 8, 2024

CI summary for merge to main:

The latter is somewhat strange - it may be due to BSD CI machine overload. It made me look up the FreeBSD Lin Bigarray.Array1 (of ints) test with Domain tests for the initial 2 runs of this PR, where I found:

  • freebsd-amd64-5.0 spending 1182.2s Lin Bigarray.Array1 (of ints) test with Domain with roughly 600s in shrinking
  • freebsd-amd64-5.1 spending 2185.3s Lin Bigarray.Array1 (of ints) test with Domain with over 2000s in shrinking!

Looking at previous runs, this seems to be exceptional though.

Out of 38 workflows 7 failed with 4 genuine issues, 1 false alarm, and 2 borderline cases that may be due to CI infrastructure load

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant