Skip to content

Commit 8b230a9

Browse files
author
Bill Weide
authored
Merge pull request #884 from beardeddog/slurmjobids
fixed join of slurnjobids bug
2 parents 64708f0 + 98433dc commit 8b230a9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pylib/Utilities/Logger.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,7 @@ def log_to_elk(self, result, logtype):
141141
del result['stdout']
142142

143143
# convert ini_files, lists of lists into a comma separated string
144-
if 'ini_files' in result['options']:
144+
if 'options' in result and 'ini_files' in result['options']:
145145
result['options']['ini_files'] = ','.join(t[0] for t in result['options']['ini_files'])
146146

147147
if logtype == 'mtt-sec':
@@ -215,7 +215,7 @@ def log_execmd_elk(self, cmdargs, status, stdout, stderr, timedout, starttime, e
215215
'starttime': str(starttime),
216216
'endtime': str(endtime),
217217
'elapsed': elapsed_secs,
218-
'slurm_job_ids': ','.join(slurm_job_ids)
218+
'slurm_job_ids': ','.join([str(j) for j in slurm_job_ids])
219219
})
220220

221221
def stage_start_print(self, stagename):

0 commit comments

Comments
 (0)