Skip to content

Commit f14ea9e

Browse files
committed
slurm2sql: Small cleanup of columns (remove obselete ReqGRES, etc)
1 parent e8844dd commit f14ea9e

File tree

1 file changed

+3
-12
lines changed

1 file changed

+3
-12
lines changed

slurm2sql.py

Lines changed: 3 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -561,15 +561,9 @@ def calc(row):
561561

562562
# Miscelaneous requested resources
563563
'ReqTRES': nullstr,
564-
'ReqGRES': nullstr, # Raw GRES string
565564
'NTasks': nullint,
566565
#'AllocGRES'
567566
'AllocTRES': nullstr,
568-
'TRESUsageInAve': nullstr,
569-
'TRESUsageInMax': nullstr,
570-
'TRESUsageInMin': nullstr,
571-
'TRESUsageInTot': nullstr,
572-
'TRESUsageOutTot': nullstr,
573567

574568
# CPU related
575569
'NCPUS': nullint, # === AllocCPUS
@@ -623,6 +617,9 @@ def calc(row):
623617
COLUMNS_EXTRA = ['JobID',
624618
'JobIDRaw',
625619
'ConsumedEnergyRaw',
620+
'TRESUsageInAve',
621+
'TRESUsageInTot',
622+
'TRESUsageOutTot',
626623
]
627624

628625

@@ -848,12 +845,6 @@ def slurm2sql(db, sacct_filter=['-a'], update=False, jobs_only=False,
848845
"""
849846
columns = COLUMNS.copy()
850847

851-
# Slurm > 20.11 deprecates ReqGRES, everything is put only into
852-
# ReqTRES. So don't fitch ReqGRES to avoid a warning.
853-
if slurm_version() >= (20, 11):
854-
del columns['ReqGRES']
855-
856-
857848

858849
def infer_type(cd):
859850
if hasattr(cd, 'type'): return cd.type

0 commit comments

Comments
 (0)