-
Notifications
You must be signed in to change notification settings - Fork 129
slurm_scheduler: autodetect nomem setting #461
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
Conversation
@d4l3k has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@d4l3k has updated the pull request. You must reimport the pull request before landing. |
@d4l3k has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Codecov Report
@@ Coverage Diff @@
## main #461 +/- ##
==========================================
- Coverage 94.59% 94.58% -0.02%
==========================================
Files 65 65
Lines 3833 3858 +25
==========================================
+ Hits 3626 3649 +23
- Misses 207 209 +2
Continue to review full report at Codecov.
|
@d4l3k has updated the pull request. You must reimport the pull request before landing. |
@d4l3k has updated the pull request. You must reimport the pull request before landing. |
@d4l3k has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
This automatically infers
nomem
by usingsinfo
to get theMemory
setting for the specified partition. If we can't determine the memory setting or its <= 1000 MB we disable memory allocation checks.#405
Test plan:
CI
Slurm integration tests w/
nomem
removed