Added git-hash-metrics builder and recipe

Added git-hash-metrics builder and recipe

BUG=602803
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300094

[pgervais, 2016-04-13 00:19:33]

pgervais@chromium.org changed reviewers:
+ sergeyberezin@chromium.org

[pgervais, 2016-04-13 00:19:33]

Waiting for a better solution...
ptal.

[pgervais, 2016-04-18 17:06:37]

On 2016/04/13 00:19:33, pgervais wrote:
> Waiting for a better solution...
> ptal.

ping. Or have we made any progress instrumenting luci-config?

[Sergey Berezin, 2016-04-18 17:14:22]

On 2016/04/18 17:06:37, pgervais wrote:
> On 2016/04/13 00:19:33, pgervais wrote:
> > Waiting for a better solution...
> > ptal.
> 
> ping. Or have we made any progress instrumenting luci-config?

I checked with nodir@, and luci-config is not the right place for that, it
doesn't know the revision of the master ref (only infra/config ref). So we still
may need a separate service to push that.

[Sergey Berezin, 2016-04-18 17:22:45]

Let's proceed with a buildbot cron for now, I don't have any better idea for a
quick solution. If it proves useful, we'll generalize.

https://codereview.chromium.org/1880343003/diff/1/scripts/slave/recipes/infra...
File scripts/slave/recipes/infra/git_hash_metric.py (right):

https://codereview.chromium.org/1880343003/diff/1/scripts/slave/recipes/infra...
scripts/slave/recipes/infra/git_hash_metric.py:20:
('--gauge=\'{"name":"repository/hash", '
You want --string.

[pgervais, 2016-04-19 00:43:24]

Fix filed, ptal.

[Sergey Berezin (google), 2016-04-21 17:18:14]

sergeyberezin@google.com changed reviewers:
+ sergeyberezin@google.com

[Sergey Berezin (google), 2016-04-21 17:18:14]

LGTM. BTW, why didn't expectation change with --string?

[pgervais, 2016-04-21 17:27:07]

On 2016/04/21 17:18:14, Sergey Berezin (google) wrote:
> LGTM. BTW, why didn't expectation change with --string?

Dammit. Forgot about it again.

[pgervais, 2016-04-21 17:29:35]

The CQ bit was checked by pgervais@chromium.org

[pgervais, 2016-04-21 17:29:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
sergeyberezin@google.com
Link to the patchset: https://chromiumcodereview.appspot.com/1880343003/#ps40001
(title: "Ahem. Updated expectations")

[commit-bot: I haz the power, 2016-04-21 17:29:40]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1880343003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1880343003/40001

[commit-bot: I haz the power, 2016-04-21 17:37:24]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-21 17:37:25]

Try jobs failed on following builders:
  Build Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

[pgervais, 2016-04-21 17:42:34]

The CQ bit was checked by pgervais@chromium.org

[pgervais, 2016-04-21 17:42:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
sergeyberezin@google.com
Link to the patchset: https://codereview.chromium.org/1880343003/#ps60001
(title: "Added missing import and rebase")

[commit-bot: I haz the power, 2016-04-21 17:42:45]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1880343003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1880343003/60001

[commit-bot: I haz the power, 2016-04-21 17:50:34]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-04-21 17:50:35]

Try jobs failed on following builders:
  Build Presubmit on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Build%20Presubmit/build...)

[pgervais, 2016-04-21 21:50:22]

The CQ bit was checked by pgervais@chromium.org

[pgervais, 2016-04-21 21:50:23]

The patchset sent to the CQ was uploaded after l-g-t-m from
sergeyberezin@google.com
Link to the patchset: https://chromiumcodereview.appspot.com/1880343003/#ps80001
(title: "Fixed presubmit errors")

[commit-bot: I haz the power, 2016-04-21 21:50:31]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1880343003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1880343003/80001

[commit-bot: I haz the power, 2016-04-21 21:54:30]

Description was changed from

==========
Added git-hash-metrics builder and recipe

BUG=602803
==========

to

==========
Added git-hash-metrics builder and recipe

BUG=602803

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=300094
==========

[commit-bot: I haz the power, 2016-04-21 21:54:31]

Committed patchset #5 (id:80001) as
http://src.chromium.org/viewvc/chrome?view=rev&revision=300094

[Matt Giuca, 2016-04-26 00:31:28]

On 2016/04/21 21:54:31, commit-bot: I haz the power wrote:
> Committed patchset #5 (id:80001) as
> http://src.chromium.org/viewvc/chrome?view=rev&revision=300094

This doesn't seem to be hooked up correctly:
https://bugs.chromium.org/p/chromium/issues/detail?id=606599