Skip to content

Commit ccc1de5

Browse files
authored
Merge pull request #7 from NordicHPC/rkdarst/singularity-docker-image-order
Singularity/docker: change argument order in run stage
2 parents 7d9b7a7 + 16ef44b commit ccc1de5

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

envkernel.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -402,8 +402,8 @@ def setup(self):
402402
'docker',
403403
'run',
404404
'--connection-file', '{connection_file}',
405-
*unknown_args,
406405
args.image,
406+
*unknown_args,
407407
'--',
408408
*kernel['argv'],
409409
]
@@ -538,8 +538,8 @@ def setup(self):
538538
'singularity', 'run',
539539
'--connection-file', '{connection_file}',
540540
#*[ '--mount={}'.format(x) for x in args.mount],
541-
*unknown_args,
542541
args.image,
542+
*unknown_args,
543543
'--',
544544
*kernel['argv'],
545545
]

test_envkernel.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -292,14 +292,14 @@ def test_docker(d):
292292
kern = install(d, "docker --some-arg=AAA IMAGE1")
293293
#assert kern['argv'][0] == 'envkernel' # defined above
294294
assert kern['ek'][1:3] == ['docker', 'run']
295-
assert kern['ek'][-1] == 'IMAGE1'
295+
assert kern['ek'][-2] == 'IMAGE1'
296296
assert '--some-arg=AAA' in kern['ek']
297297

298298
def test_singularity(d):
299299
kern = install(d, "singularity --some-arg=AAA /PATH/TO/IMAGE2")
300300
#assert kern['argv'][0] == 'envkernel' # defined above
301301
assert kern['ek'][1:3] == ['singularity', 'run']
302-
assert kern['ek'][-1] == '/PATH/TO/IMAGE2'
302+
assert kern['ek'][-2] == '/PATH/TO/IMAGE2'
303303
assert '--some-arg=AAA' in kern['ek']
304304

305305

0 commit comments

Comments
 (0)