Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Remove use of Slurm ReqGRES field #1479

Merged
merged 3 commits into from
Jan 7, 2021
Merged

Remove use of Slurm ReqGRES field #1479

merged 3 commits into from
Jan 7, 2021

Conversation

jtpalmer
Copy link
Contributor

@jtpalmer jtpalmer commented Jan 6, 2021 •
edited
Loading

Description

Removes shredding/ingestion of the Slurm ReqGRES field.

Motivation and Context

This field has been deprecated.

sacct: fatal: ReqGRES is deprecated, please use ReqTRES

Note that both ReqTRES and AllocTRES are already included in the list of Slurm fields ingested by xdmod-shredder.

https://app.asana.com/0/807770147051221/1198937557854870/f

Tests performed

Updated existing tests.

Checklist:

  • The pull request description is suitable for a Changelog entry
  • The milestone is set correctly on the pull request
  • The appropriate labels have been added to the pull request

@jtpalmer jtpalmer added enhancement Enhancement of the functionality of an existing feature Category:ETL Extract Transform Load labels Jan 6, 2021
@jtpalmer jtpalmer added this to the 9.5.0 milestone Jan 6, 2021
@jtpalmer jtpalmer merged commit eec93fc into ubccr:xdmod9.5 Jan 7, 2021
jtpalmer added a commit to jtpalmer/xdmod that referenced this pull request Jan 11, 2021
@jtpalmer
Remove use of Slurm ReqGRES field (ubccr#1479)
8b9d631
@treydock treydock mentioned this pull request Mar 10, 2021
Closed
@jtpalmer jtpalmer deleted the remove-slurm-gres branch June 15, 2021 11:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@jpwhite4 jpwhite4 jpwhite4 approved these changes

Assignees
No one assigned
Labels
Category:ETL Extract Transform Load enhancement Enhancement of the functionality of an existing feature
Projects
None yet
Milestone
9.5.0
Development

Successfully merging this pull request may close these issues.
2 participants
@jtpalmer @jpwhite4