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

Unified Diff: infra/tools/builder_alerts/__main__.py

Issue 475943002: Add time and state to builder revision data (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Fix a couple style issues and address comments Created 6 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
« no previous file with comments | « no previous file | infra/tools/builder_alerts/buildbot.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: infra/tools/builder_alerts/__main__.py
diff --git a/infra/tools/builder_alerts/__main__.py b/infra/tools/builder_alerts/__main__.py
index bd639cbf768fb50bf4b0db59a725fe98a9b51264..5e59a136e73acc691549b4ace16e5e3a672ebdb4 100755
--- a/infra/tools/builder_alerts/__main__.py
+++ b/infra/tools/builder_alerts/__main__.py
@@ -87,7 +87,7 @@ def main(args):
master_urls = fetch_master_urls(gatekeeper, args)
start_time = datetime.datetime.now()
- latest_revisions = {}
+ latest_builder_info = {}
cache = buildbot.BuildCache(CACHE_PATH)
@@ -101,9 +101,9 @@ def main(args):
# FIXME: This doesn't really belong here. garden-o-matic wants
# this data and we happen to have the builder json cached at
# this point so it's cheap to compute.
- revisions = buildbot.latest_revisions_for_master(cache,
+ builder_info = buildbot.latest_builder_info_for_master(cache,
master_url, master_json)
- latest_revisions.update(revisions)
+ latest_builder_info.update(builder_info)
print "Fetch took: %s" % (datetime.datetime.now() - start_time)
@@ -117,7 +117,7 @@ def main(args):
'alerts': alerts,
'reason_groups': reason_groups,
'range_groups': range_groups,
- 'latest_revisions': latest_revisions,
+ 'latest_builder_info': latest_builder_info,
})}
if not args.data_url:
« no previous file with comments | « no previous file | infra/tools/builder_alerts/buildbot.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698