Milo buildbot: Add in the rest of the blame/commits information.

milo/appengine/buildbucket/expectations/master.tryserver.infra/InfraPresubmit.json

Index: milo/appengine/buildbucket/expectations/master.tryserver.infra/InfraPresubmit.json
diff --git a/milo/appengine/buildbucket/expectations/master.tryserver.infra/InfraPresubmit.json b/milo/appengine/buildbucket/expectations/master.tryserver.infra/InfraPresubmit.json
index 0ebc5d8ce29189f93ac0ba619ffc2721562d9828..c8ee7811ab765ee94fef70839e9fd982044fe251 100755
--- a/milo/appengine/buildbucket/expectations/master.tryserver.infra/InfraPresubmit.json
+++ b/milo/appengine/buildbucket/expectations/master.tryserver.infra/InfraPresubmit.json
@@ -26,10 +26,17 @@
         {
           "AuthorName": "",
           "AuthorEmail": "recipe-roller@chromium.org",
+          "CommitTime": "0001-01-01T00:00:00Z",
           "Repo": "",
           "Branch": "",
-          "RequestRevision": "HEAD",
-          "Revision": "",
+          "RequestRevision": {
+            "Label": "HEAD",
+            "URL": ""
+          },
+          "Revision": {

[nodir, 2017/04/19 18:37:48]

this does not look right. Shouldn't this be nil?

also I think all these fields used in expectation files should be marked
omitempty

[hinoka, 2017/04/20 22:09:43]

Done.

+            "Label": "",
+            "URL": ""
+          },
           "Description": "",
           "Title": "",
           "Changelist": {
@@ -68,10 +75,17 @@
         {
           "AuthorName": "",
           "AuthorEmail": "recipe-roller@chromium.org",
+          "CommitTime": "0001-01-01T00:00:00Z",
           "Repo": "",
           "Branch": "",
-          "RequestRevision": "HEAD",
-          "Revision": "",
+          "RequestRevision": {
+            "Label": "HEAD",
+            "URL": ""
+          },
+          "Revision": {
+            "Label": "",
+            "URL": ""
+          },
           "Description": "",
           "Title": "",
           "Changelist": {