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

Unified Diff: scripts/slave/annotated_run.py

Issue 22449006: Second attempt at adding archive_layout_test_results to the blink trybots. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: Fix archive_layout_tests to take gs-bucket Created 7 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: scripts/slave/annotated_run.py
diff --git a/scripts/slave/annotated_run.py b/scripts/slave/annotated_run.py
index b9b47a0ed0a84132e0be5a566ece8c6c98c943ce..5800d4f9f718e1e39ad93a55eff1417e8c990687 100755
--- a/scripts/slave/annotated_run.py
+++ b/scripts/slave/annotated_run.py
@@ -94,6 +94,7 @@ class StepPresentation(object):
self._finalized = False
self._logs = collections.OrderedDict()
+ self._links = collections.OrderedDict()
self._perf_logs = collections.OrderedDict()
self._status = None
self._step_summary_text = ''
@@ -136,6 +137,13 @@ class StepPresentation(object):
return copy.deepcopy(self._logs)
@property
+ def links(self):
+ if not self._finalized:
+ return self._links
+ else:
+ return copy.deepcopy(self._links)
+
+ @property
def perf_logs(self):
if not self._finalized:
return self._perf_logs
@@ -152,6 +160,8 @@ class StepPresentation(object):
annotator_step.write_log_lines(name, lines)
for name, lines in self.perf_logs.iteritems():
annotator_step.write_log_lines(name, lines, perf=True)
+ for label, url in self.links.iteritems():
+ annotator_step.step_link(label, url)
status_mapping = {
'WARNING': annotator_step.step_warnings,
'FAILURE': annotator_step.step_failure,

Powered by Google App Engine
This is Rietveld 408576698