diff options
-rw-r--r-- | NEWS | 3 | ||||
-rw-r--r-- | ick2/actionenvs.py | 18 |
2 files changed, 13 insertions, 8 deletions
@@ -27,6 +27,9 @@ Version 0.46+git, not yet released you could leave it out and it would act as if `where: host` was specified. +* All commands run by worker manager now get their stdin redirected + from /dev/null. + Version 0.46, released 2018-04-22 ---------------------------------- diff --git a/ick2/actionenvs.py b/ick2/actionenvs.py index a21cef2..845d1d2 100644 --- a/ick2/actionenvs.py +++ b/ick2/actionenvs.py @@ -38,14 +38,16 @@ class Runner: for key in kwargs: logging.debug('Runner.runcmd: kwargs: %r=%r', key, kwargs[key]) assert all(argv is not None for argv in argvs) - exit_code, _, _ = cliapp.runcmd_unchecked( - *argvs, - stdout_callback=self.capture_stdout, - stderr=subprocess.STDOUT, - output_timeout=self._timeout, - timeout_callback=self.flush, - **kwargs - ) + with open('/dev/null', 'rb') as devnull: + exit_code, _, _ = cliapp.runcmd_unchecked( + *argvs, + stdin=devnull, + stdout_callback=self.capture_stdout, + stderr=subprocess.STDOUT, + output_timeout=self._timeout, + timeout_callback=self.flush, + **kwargs + ) self.flush() logging.debug('Runner.runcmd: finished, exit code: %d', exit_code) return exit_code |