Skip to content

Commit 9955780

Browse files
committed
fix(slurmCI): clone args map to isolate parallel branch state
Parallel matrix branches share the same oneStep object from config.steps. The previous code held a direct reference to oneStep.args, so args.remove() and template resolution overwrote values visible to all sibling branches. Cloning the map gives each branch its own copy. Signed-off-by: Daniel Pressler <danielpr@nvidia.com>
1 parent e279ab9 commit 9955780

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

vars/slurmCI.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
#!/usr/bin/env groovy
22

33
int call(ctx, oneStep, config) {
4-
def args = oneStep.args ?: [:]
4+
def args = (oneStep.args ?: [:]).clone()
55

66
// Load the shared Slurm helper library from the pipeline shared library.
77
// refer to this doc for more details: https://www.jenkins.io/doc/book/pipeline/shared-libraries/#using-libraries

0 commit comments

Comments
 (0)