Skip to content

Commit e7fcd19

Browse files
authored
Merge pull request #3094 from ekouts/bugfix/login_shell_in_autodetect
[bugfix] Do not ignore the `use_login_shell` configuration option when generation the auto-detection script
2 parents 62ee308 + 3a30e58 commit e7fcd19

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

reframe/frontend/autodetect.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -131,12 +131,14 @@ def _is_part_local(part):
131131

132132

133133
def _remote_detect(part):
134+
use_login_shell = runtime.runtime().get_option('general/0/use_login_shell')
135+
134136
def _emit_script_for_source(job, env):
135137
commands = [
136138
'./bootstrap.sh',
137139
'./bin/reframe --detect-host-topology=topo.json'
138140
]
139-
job.prepare(commands, env, trap_errors=True)
141+
job.prepare(commands, env, trap_errors=True, login=use_login_shell)
140142

141143
def _emit_script_for_pip(job, env):
142144
commands = [
@@ -147,7 +149,7 @@ def _emit_script_for_pip(job, env):
147149
'reframe --detect-host-topology=topo.json',
148150
'deactivate'
149151
]
150-
job.prepare(commands, env, trap_errors=True)
152+
job.prepare(commands, env, trap_errors=True, login=use_login_shell)
151153

152154
getlogger().info(
153155
f'Detecting topology of remote partition {part.fullname!r}: '

0 commit comments

Comments
 (0)