Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(44)

Unified Diff: dashboard/dashboard/pinpoint/models/quest/run_test.py

Issue 3001163002: Pinpoint - Surface info from executions for display in UI. (Closed)
Patch Set: Created 3 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: dashboard/dashboard/pinpoint/models/quest/run_test.py
diff --git a/dashboard/dashboard/pinpoint/models/quest/run_test.py b/dashboard/dashboard/pinpoint/models/quest/run_test.py
index 3a4b90c2a3f80f75123a1c76aeccba43ed31adb0..7576f73e4429df0505de2b8c107325971431a458 100644
--- a/dashboard/dashboard/pinpoint/models/quest/run_test.py
+++ b/dashboard/dashboard/pinpoint/models/quest/run_test.py
@@ -88,6 +88,7 @@ class _RunTestExecution(execution_module.Execution):
self._dimensions = dimensions
self._extra_args = extra_args
self._isolate_hash = isolate_hash
+ self._output_isolate_hash = ''
dtu 2017/08/21 23:52:53 Similar thought as above. We can expose this by ex
shatch 2017/08/22 01:23:55 Done.
self._first_execution = first_execution
self._task_id = None
@@ -97,6 +98,14 @@ class _RunTestExecution(execution_module.Execution):
def bot_id(self):
return self._bot_id
+ def _AsDict(self):
+ return {
+ 'bot_id': self._bot_id or '',
+ 'task_id': self._task_id,
+ 'input_isolate_hash': self._isolate_hash,
+ 'output_isolate_hash': self._output_isolate_hash,
+ }
+
def _Poll(self):
if not self._task_id:
self._StartTask()
@@ -117,8 +126,9 @@ class _RunTestExecution(execution_module.Execution):
if result['failure']:
raise SwarmingTestError(self._task_id, result['exit_code'])
- result_arguments = {'isolate_hash': result['outputs_ref']['isolated']}
- self._Complete(result_arguments=result_arguments)
+ self._output_isolate_hash = result['outputs_ref']['isolated']
+ self._Complete(
+ result_arguments={'isolate_hash': self._output_isolate_hash})
def _StartTask(self):

Powered by Google App Engine
This is Rietveld 408576698