leezu commented on a change in pull request #18245:
URL: https://github.com/apache/incubator-mxnet/pull/18245#discussion_r420471171
##########
File path: ci/test_safe_docker_run.py
##########
@@ -408,7 +408,7 @@ def wait_for_container(name: str) -> bool:
# to the atexit
for sig in [None, signal.SIGTERM, signal.SIGINT]:
# Execute the safe docker run script in a different process
- proc = subprocess.Popen(['./safe_docker_run.py', "--name",
container_name, "ubuntu:18.04", "sleep 10"])
+ proc = subprocess.Popen(['python3 safe_docker_run.py', "--name",
container_name, "ubuntu:18.04", "sleep 10"])
Review comment:
May need to separate python3 and the script name into two elements of
the list?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
