Merge pull request #783 from UCL-INGI/run_student_wait_fix

[containers/run_student] Keep std file descriptors open in wait_until_finished
UCL-INGI · Feb 18, 2022 · d1b86b4 · d1b86b4
2 parents 7a8002a + 59289a0
commit d1b86b4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/base-containers/base/inginious_container_api/run_student.py b/base-containers/base/inginious_container_api/run_student.py
@@ -258,8 +258,8 @@ def wait_until_finished(both_dockers, zmq_socket, stdin, stdout, stderr, student
     # handle the student_container outputs and wait for final message
     message = None
     msg_type = None
-    stdout_file = os.fdopen(stdout, 'wb')
-    stderr_file = os.fdopen(stderr, 'wb')
+    stdout_file = os.fdopen(stdout, 'wb', closefd=False)
+    stderr_file = os.fdopen(stderr, 'wb', closefd=False)
 
     while msg_type != "run_student_retval":
         zmq_socket.send(msgpack.dumps({"type": "dummy_message"}, use_bin_type=True))  # ping pong socket