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] Oversubscribe in OpenMPI jobs #790

Merged

Conversation

giordano
Copy link
Member

@giordano giordano commented Nov 3, 2023

This didn't use to be necessary, but probably recent changes in the GitHub Actions runners caused mpiexec -n 4 to complain about insufficient slots available.

This didn't use to be necessary, but probably recent changes in the GitHub
Actions runners caused `mpiexec -n 4` to complain about insufficient slots
available.
@giordano giordano added the CI label Nov 3, 2023
@giordano
Copy link
Member Author

giordano commented Nov 3, 2023

This is working: the test-system-apt (libopenmpi-dev, 1) job, which is currently failing on master before due to insufficient slots available, is now successful.

Edit: I'm now convinced the failures may have depended on the exact machine used during a CI run (with 2 vs 4 vCPUs), but this PR avoids hard failures in case we get a smaller machine anyway.

@giordano giordano merged commit 3f799ad into JuliaParallel:master Nov 3, 2023
39 of 44 checks passed
@giordano giordano deleted the mg/ci-openmpi-oversubscribe branch November 3, 2023 19:19
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.

1 participant