search

radical-cybertools / radical.pilot

RADICAL-Pilot
http://radical-cybertools.github.io/radical-pilot/index.html
Other
54 stars 23 forks source link

Review runs with `aprun` on Theta@ALCF #2821

Closed mtitov closed 1 year ago

mtitov commented 1 year ago

Check if the latest changes in subprocess.Popen call (within RP Executor Popen) affect launching process of aprun commands https://github.com/radical-cybertools/radical.pilot/blob/5889a458e82eb038636a22425e7acca1728e6934/src/radical/pilot/agent/executing/popen.py#L366-L377

Related to this PR #2430 (initial changes for subprocess.Popen)

mtitov commented 1 year ago

Confirmed by Tianle that it is required to have a configurable parameter start_new_session @andre-merzky should we put it into resource configuration as new_session_per_task = False for Theta? (start_new_session = self._cfg.get('new_session_per_task', True))