Changeset 65513 in vbox for trunk/src/VBox/ValidationKit

Timestamp:

Jan 29, 2017 8:22:01 PM (8 years ago)

Author:

vboxsync

Message:

ValidationKit: Collect information about the VM process state if pwoering off the VM failed

File:

• trunk/src/VBox/ValidationKit/testdriver/vbox.py (modified) (5 diffs)

trunk/src/VBox/ValidationKit/testdriver/vbox.py

@@ -2374 +2374 @@
         return fRc;
 
-    def annotateAndUploadProcessReport(self, sProcessReport):
+    def annotateAndUploadProcessReport(self, sProcessReport, sFilename, sKind, sDesc):
         """
         Annotates the given VM process report and uploads it if successfull.
@@ -2388 +2388 @@
                 sReportDbgSym = oResolver.annotateReport(sProcessReport);
                 if sReportDbgSym is not None:
-                    reporter.addLogString(sReportDbgSym, 'vmprocess.log', 'process/report/vm', 'Annotated VM process state');
+                    reporter.addLogString(sReportDbgSym, sFilename, sKind, sDesc);
                     fRc = True;
                 else:
@@ -2598 +2598 @@
         # requires memory too and seems to wedge.
         #
-        sHostProcessInfo    = None;
-        sLastScreenshotPath = None;
-        sOsKernelLog        = None;
-        sVgaText            = None;
-        asMiscInfos         = [];
+        sHostProcessInfo     = None;
+        sHostProcessInfoHung = None;
+        sLastScreenshotPath  = None;
+        sOsKernelLog         = None;
+        sVgaText             = None;
+        asMiscInfos          = [];
 
         if not oSession.fHostMemoryLow:
@@ -2706 +2707 @@
                 uPid = oSession.getPid();
                 if uPid is not None:
+                    #
+                    # Collect some information about the VM process first to have
+                    # some state information for further investigation why powering off failed.
+                    #
+                    sHostProcessInfoHung = utils.processGetInfo(uPid, fSudo = True);
+
+                    # Exterminate...
                     reporter.error('terminateVmBySession: Terminating PID %u (VM %s)' % (uPid, oSession.sName));
                     fClose = base.processTerminate(uPid);
@@ -2765 +2773 @@
         if sHostProcessInfo is not None:
             reporter.log('Trying to annotate the VM process report, please stand by...');
-            fRcTmp = self.annotateAndUploadProcessReport(sHostProcessInfo);
+            fRcTmp = self.annotateAndUploadProcessReport(sHostProcessInfo, 'vmprocess.log',
+                                                         'process/report/vm', 'Annotated VM process state');
             # Upload the raw log for manual annotation in case resolving failed.
             if not fRcTmp:
                 reporter.log('Failed to annotate VM process report, uploading raw report');
                 reporter.addLogString(sHostProcessInfo, 'vmprocess.log', 'process/report/vm', 'VM process state');
+
+        # Add the host process info for failed power off attempts if we were able to retrieve it.
+        if sHostProcessInfoHung is not None:
+            reporter.log('Trying to annotate the hung VM process report, please stand by...');
+            fRcTmp = self.annotateAndUploadProcessReport(sHostProcessInfoHung, 'vmprocess-hung.log',
+                                                         'process/report/vm', 'Annotated hung VM process state');
+            # Upload the raw log for manual annotation in case resolving failed.
+            if not fRcTmp:
+                reporter.log('Failed to annotate hung VM process report, uploading raw report');
+                reporter.addLogString(sHostProcessInfoHung, 'vmprocess-hung.log', 'process/report/vm',
+                                      'Hung VM process state');
 
         return fRc;