Add Milo URLs to list of flakes

appengine/chromium_try_flakes/model/build_run.py

Index: appengine/chromium_try_flakes/model/build_run.py
diff --git a/appengine/chromium_try_flakes/model/build_run.py b/appengine/chromium_try_flakes/model/build_run.py
index 3ba6f2ebadabc6a8556685b658d058c23bcb8d89..dcad8437c245529f83be591c004093eaf6bca3d6 100644
--- a/appengine/chromium_try_flakes/model/build_run.py
+++ b/appengine/chromium_try_flakes/model/build_run.py
@@ -43,6 +43,15 @@ class BuildRun(ndb.Model):  # pragma: no cover
             self.removeMasterPrefix(parent.master) + '/builders/' +
             parent.builder + '/builds/' + str(self.buildnumber))
 
+  def getMiloURL(self):
+    # In July 2016, protobuf changed and URLs for earlier builds do not open.
+    if self.time_finished < datetime.datetime(2016, 8, 1):
+      return
+    parent = self.key.parent().get()
+    return ('https://luci-milo.appspot.com/logdog/build/chromium/bb/' +

[hinoka, 2016/09/12 20:22:08]

I'd recommend the /buildbot/<master>/<builder>/<build #> URL instead.  The
logdog ones certainly work, but:
* Internal masters will need to be /chrome/ instead of /chromium/
* Lack of backlinks to builder view
* the /logdog/ urls wasn't originally intended for production use, just a way to
see annotation protos from logdog meanwhile we worked on integrating them into
their respective endpoints (/buildbot/ and /swarming/)
* The logdog view doesn't contain builds where it doesn't get as far as logdog
(or hung builds... apperently)
* the /buildbot/ url will be backed by logdog in a couple weeks anyways
(crbug.com/644363)

[Sergiy Byelozyorov, 2016/09/12 22:34:57]

Done.

+            self.removeMasterPrefix(parent.master) + '/' + parent.builder +
+            '/' + str(self.buildnumber) + '/+/annotations')
+
   buildnumber = ndb.IntegerProperty(required=True)
   result = ndb.IntegerProperty(required=True)
   time_finished = ndb.DateTimeProperty(required=True)