Skip to content

Commit 23f896d

Browse files
author
Vasileios Karakasis
authored
Merge pull request #1702 from vkarak/test/remove-jobopts-append
[test] Simply set job options in tests; do not append if not needed
2 parents 4c1749c + c7db92a commit 23f896d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

cscs-checks/system/slurm/slurm.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ def __init__(self):
112112

113113
@rfm.run_before('run')
114114
def set_memory_limit(self):
115-
self.job.options += ['--mem=120000']
115+
self.job.options = ['--mem=120000']
116116

117117

118118
@rfm.simple_test
@@ -171,7 +171,7 @@ def __init__(self):
171171
def set_slurm_constraint(self):
172172
cabinet = self.cabinets.get(self.current_partition.fullname)
173173
if cabinet:
174-
self.job.options += [f'--constraint={cabinet}']
174+
self.job.options = [f'--constraint={cabinet}']
175175

176176

177177
@rfm.simple_test
@@ -188,7 +188,7 @@ def __init__(self):
188188

189189
@rfm.run_before('run')
190190
def set_memory_limit(self):
191-
self.job.options += ['--mem=2000']
191+
self.job.options = ['--mem=2000']
192192

193193

194194
@rfm.simple_test

0 commit comments

Comments
 (0)