-
Notifications
You must be signed in to change notification settings - Fork 8
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
fix(agent): update the validate_job_state
validator to support data_parser 0.0.40
#739
fix(agent): update the validate_job_state
validator to support data_parser 0.0.40
#739
Conversation
…_parser 0.0.40 This commit modifies the validator function `validate_job_state` from the `SlurmJobData` model in order to support the data_parser 0.0.40. Essentialy, in Slurm 24.11, the `--json` flag from the `scontrol show job` command uses the data_parser 0.0.40. According to the official [documentation](https://slurm.schedmd.com/job_state_codes.html#overview), the job flags can be returned alongside the job state, which can make the `job_state` key to have multiple keys.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## release/5.5 #739 +/- ##
============================================
Coverage 93.00% 93.00%
============================================
Files 88 88
Lines 4859 4859
============================================
Hits 4519 4519
Misses 340 340
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
The code looks good, but please, add a news fragment so we can track it on new releases.
check 235c9ff please. |
…_parser 0.0.40 (#739) This commit modifies the validator function `validate_job_state` from the `SlurmJobData` model in order to support the data_parser 0.0.40. Essentialy, in Slurm 24.11, the `--json` flag from the `scontrol show job` command uses the data_parser 0.0.40. According to the official [documentation](https://slurm.schedmd.com/job_state_codes.html#overview), the job flags can be returned alongside the job state, which can make the `job_state` key to have multiple keys. (cherry picked from commit 1d97b8c)
…_parser 0.0.40 (#739) (#742) This commit modifies the validator function `validate_job_state` from the `SlurmJobData` model in order to support the data_parser 0.0.40. Essentialy, in Slurm 24.11, the `--json` flag from the `scontrol show job` command uses the data_parser 0.0.40. According to the official [documentation](https://slurm.schedmd.com/job_state_codes.html#overview), the job flags can be returned alongside the job state, which can make the `job_state` key to have multiple keys. (cherry picked from commit 1d97b8c) Co-authored-by: Matheus Tosta <[email protected]>
This PR modifies the validator function
validate_job_state
from theSlurmJobData
model in order to support the data_parser 0.0.40.Essentialy, in Slurm 24.11, the
--json
flag from thescontrol show job
command uses the data_parser 0.0.40. According to the official documentation, the job flags can be returned alongside the job state, which can make thejob_state
key to have multiple keys.