commit 76caf9ac8c8b4a416422f75c3317e0fbf6cf1a7f
author Guillermo Maturana <maturana@google.com> Thu Apr 08 14:04:15 2021 -0700
committer Srikrishna Iyer <46467186+sriyerg@users.noreply.github.com> Tue Apr 13 23:03:48 2021 -0700
tree 76f76d65f75422bde05fd030a0463da8a0f324d7
parent 9312b9e529636114dc33dbd2937b489fe338af98

[dv/dvsim] Adds failure bucketizer for triage.

Changes _post_finish to take an ErrorMessage as second argument.
Moves class SimCfg's Results to its own file and renames it to SimResults.
The bucketizer is called when initializing SimResults.
This will change when reruns are implemented, since we would want to
rerun one test per bucket as soon as the failing test completes.

Signed-off-by: Guillermo Maturana <maturana@google.com>

util/dvsim/LocalLauncher.py

diff --git a/util/dvsim/LocalLauncher.py b/util/dvsim/LocalLauncher.py
index 70d7839..0a09682 100644
--- a/util/dvsim/LocalLauncher.py
+++ b/util/dvsim/LocalLauncher.py
@@ -6,7 +6,7 @@
 import shlex
 import subprocess
 
-from Launcher import Launcher, LauncherError
+from Launcher import ErrorMessage, Launcher, LauncherError
 
 
 class LocalLauncher(Launcher):
@@ -99,7 +99,9 @@
         except subprocess.TimeoutExpired:
             self.process.kill()
 
-        self._post_finish('K', 'Job killed!')
+        self._post_finish('K', ErrorMessage(line_number=None,
+                                            message='Job killed!',
+                                            context=[]))
 
     def _post_finish(self, status, err_msg):
         super()._post_finish(status, err_msg)