Changeset 65196 in vbox for trunk/src/VBox/ValidationKit/testboxscript

Timestamp:

Jan 9, 2017 11:07:28 AM (8 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

112700

Message:

common/utils.py,testboxscript,tdUnitTest1: Create separate windows (console) process groups for children so we can safely generate Ctrl-C signals without also committing suicide. (testdriver.base.spawn already did this)

Location:

trunk/src/VBox/ValidationKit/testboxscript

Files:

• testboxscript.py (modified) (5 diffs)
• testboxtasks.py (modified) (1 diff)

trunk/src/VBox/ValidationKit/testboxscript/testboxscript.py

@@ -33 +33 @@
 __version__ = "$Revision$"
 
-import subprocess
-import sys
-import os
-import time
+import platform;
+import subprocess;
+import sys;
+import os;
+import time;
 
 
@@ -42 +43 @@
 # @remarks These will _never_ change
 # @{
-TBS_EXITCODE_FAILURE        = 1         # RTEXITCODE_FAILURE
-TBS_EXITCODE_SYNTAX         = 2         # RTEXITCODE_SYNTAX
-TBS_EXITCODE_NEED_UPGRADE   = 9
+TBS_EXITCODE_FAILURE        = 1;        # RTEXITCODE_FAILURE
+TBS_EXITCODE_SYNTAX         = 2;        # RTEXITCODE_SYNTAX
+TBS_EXITCODE_NEED_UPGRADE   = 9;
 ## @}
 
@@ -59 +60 @@
         Init
         """
-        self.task = None
+        self.oTask = None
 
     def __del__(self):
@@ -65 +66 @@
         Cleanup
         """
-        if self.task is not None:
+        if self.oTask is not None:
             print 'Wait for child task...'
-            self.task.terminate()
-            self.task.wait()
+            self.oTask.terminate()
+            self.oTask.wait()
             print 'done. Exiting'
-            self.task = None;
+            self.oTask = None;
 
     def run(self):
@@ -107 +108 @@
         rcExit = TBS_EXITCODE_FAILURE;
         while True:
-            self.task = subprocess.Popen(asArgs, shell=False);
-            rcExit = self.task.wait();
-            self.task = None;
+            self.oTask = subprocess.Popen(asArgs,
+                                          shell = False,
+                                          creationflags = (0 if platform.system() != 'Windows'
+                                                           else subprocess.CREATE_NEW_PROCESS_GROUP)); # for Ctrl-C isolation
+            rcExit = self.oTask.wait();
+            self.oTask = None;
             if rcExit == TBS_EXITCODE_SYNTAX:
                 break;

trunk/src/VBox/ValidationKit/testboxscript/testboxtasks.py

@@ -423 +423 @@
         # Spawn child.
         try:
-            oChild = subprocess.Popen(asArgs,
-                                      shell      = False,
-                                      bufsize    = -1,
-                                      stdout     = subprocess.PIPE,
-                                      stderr     = subprocess.STDOUT,
-                                      cwd        = self._oTestBoxScript.getPathSpill(),
-                                      universal_newlines = True,
-                                      close_fds  = (False if utils.getHostOs() == 'win' else True),
-                                      preexec_fn = (None if utils.getHostOs() in ['win', 'os2']
-                                                    else os.setsid)); # pylint: disable=E1101
+            oChild = utils.processPopenSafe(asArgs,
+                                            shell      = False,
+                                            bufsize    = -1,
+                                            stdout     = subprocess.PIPE,
+                                            stderr     = subprocess.STDOUT,
+                                            cwd        = self._oTestBoxScript.getPathSpill(),
+                                            universal_newlines = True,
+                                            close_fds  = (False if utils.getHostOs() == 'win' else True),
+                                            preexec_fn = (None if utils.getHostOs() in ['win', 'os2']
+                                                          else os.setsid)); # pylint: disable=E1101
         except Exception, oXcpt:
             self._log('Error creating child process %s: %s' % (asArgs, oXcpt));