buildbucket: instrument endpoints with gae_ts_mon

buildbucket: instrument endpoints with gae_ts_mon

Deployed to cr-buildbucket-dev and verified that it sends values.

R=sergeyberezin@chromium.org
BUG=559500
Committed: https://chromium.googlesource.com/infra/infra/+/a971e1fca1fe5d69bb9cdcddab936b63d72945c9

[nodir, 2016-04-18 20:02:38]

PTAL

[Sergey Berezin, 2016-04-18 20:24:19]

neat! LGTM.

[nodir, 2016-04-18 20:38:34]

The CQ bit was checked by nodir@chromium.org

[commit-bot: I haz the power, 2016-04-18 20:38:43]

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

[commit-bot: I haz the power, 2016-04-18 20:44:55]

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

[commit-bot: I haz the power, 2016-04-18 20:44:56]

Try jobs failed on following builders:
  Infra Linux Precise 32 Tester on tryserver.infra (JOB_FAILED,
https://build.chromium.org/p/tryserver.infra/builders/Infra%20Linux%20Precise...)

[nodir, 2016-04-18 21:52:55]

The CQ bit was checked by nodir@chromium.org

[nodir, 2016-04-18 21:52:55]

The patchset sent to the CQ was uploaded after l-g-t-m from
sergeyberezin@chromium.org
Link to the patchset: https://codereview.chromium.org/1893233003/#ps20001
(title: "specify time fn")

[commit-bot: I haz the power, 2016-04-18 21:52:58]

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

[commit-bot: I haz the power, 2016-04-18 21:57:17]

Description was changed from

==========
buildbucket: instrument endpoints with gae_ts_mon

Deployed to cr-buildbucket-dev and verified that it sends values.

R=sergeyberezin@chromium.org
BUG=559500
==========

to

==========
buildbucket: instrument endpoints with gae_ts_mon

Deployed to cr-buildbucket-dev and verified that it sends values.

R=sergeyberezin@chromium.org
BUG=559500

Committed:
https://chromium.googlesource.com/infra/infra/+/a971e1fca1fe5d69bb9cdcddab936...
==========

[commit-bot: I haz the power, 2016-04-18 21:57:18]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/infra/infra/+/a971e1fca1fe5d69bb9cdcddab936...

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

https://codereview.chromium.org/1893233003/diff/20001/appengine/cr-buildbucke...
File appengine/cr-buildbucket/api.py (right):

https://codereview.chromium.org/1893233003/diff/20001/appengine/cr-buildbucke...
appengine/cr-buildbucket/api.py:177: fn =
gae_ts_mon.instrument_endpoint(time_fn=ts_mon_time)(fn)
You shouldn't need to specify time here - it's only for testing.