Add time and state to builder revision data

infra/tools/builder_alerts/buildbot.py

 # Copyright 2014 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 import collections
 import datetime
 import json
 import logging
 import operator
 import os
@@ skipping 160 matching lines @@
     # parent_got_foo_revision instead, but non-tester builders
     # don't have the parent_ versions, so we have to fall back
     # to got_foo_revision in those cases!
     # Don't even think about using 'revision' that's wrong too.
     revision = _property_value(build_json, 'parent_' + buildbot_property)
     if not revision:
       revision = _property_value(build_json, buildbot_property)
     revisions[repo_name] = revision
   return revisions
 
+def latest_update_time_for_builder(last_build):
+  last_update = float(0)

[eseidel, 2014/08/18 16:35:43]

None?

[leviw_travelin_and_unemployed, 2014/08/18 18:39:02]

Sure!

+  if last_build['times'][1] != None:
+    last_update = float(last_build['times'][1])
+  else:
+    for step in last_build['steps']:
+      step_time = float(step['times'][0])
+      last_update = max(step_time, last_update)
+  return last_update
+
 
 # "line too long" pylint: disable=C0301
-def latest_revisions_for_master(cache, master_url, master_json):  # pragma: no cover
-  latest_revisions = collections.defaultdict(dict)
+def latest_builder_info_for_master(cache, master_url, master_json):  # pragma: no cover
+  latest_builder_info = collections.defaultdict(dict)
   master_name = master_name_from_url(master_url)
   for builder_name, builder_json in master_json['builders'].items():
     # recent_builds can include current builds
     recent_builds = set(builder_json['cachedBuilds'])
     active_builds = set(builder_json['currentBuilds'])
     last_finished_id = sorted(recent_builds - active_builds, reverse=True)[0]
     last_build = fetch_build_json(cache,
         master_url, builder_name, last_finished_id)
-    latest_revisions[master_name][builder_name] = \
-        revisions_from_build(last_build)
-  return latest_revisions
+    latest_builder_info[master_name][builder_name] = {
+      'revisions': revisions_from_build(last_build),
+      'state': builder_json['state'],
+      'lastUpdateTime': latest_update_time_for_builder(last_build),

[eseidel, 2014/08/18 16:35:43]

What does 0 mean here?

[leviw_travelin_and_unemployed, 2014/08/18 18:39:02]

It should never actually be zero. It should be None.

+    }
+  return latest_builder_info
 
 
 def warm_build_cache(cache, master_url, builder_name,
     recent_build_ids, active_builds):  # pragma: no cover
   # Cache active (in-progress) builds:
   match_builder_name = lambda build: build['builderName'] == builder_name
   actives = filter(match_builder_name, active_builds)
   for build in actives:
     key = cache_key_for_build(master_url, builder_name, build['number'])
     cache.set(key, build)
 
   active_build_ids = [b['number'] for b in active_builds]
   # recent_build_ids includes active ones.
   finished_build_ids = [b for b in recent_build_ids
       if b not in active_build_ids]
   last_build_id = max(finished_build_ids)
   cache_key = cache_key_for_build(master_url, builder_name, last_build_id)
 
   # We cache in-progress builds, so if the first finished build has a non-None
   # eta, then it's just the cached version from when it was in progress.
   cached_build = cache.get(cache_key)
   if not cached_build or cached_build.get('eta') is not None:
     # reason = 'in progress' if cached_build else 'missing'
     # logging.debug('prefill reason: %s %s' % (max(finished_build_ids), reason))
     prefill_builds_cache(cache, master_url, builder_name)