Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(402)

Issue 2705273003: Roll infra_libs and gae_ts_mon in luci-py, and add field_specs to all metrics (Closed)

Created:
3 years, 10 months ago by dsansome
Modified:
3 years, 9 months ago
Reviewers:
Vadim Sh., M-A Ruel
CC:
chromium-reviews, infra-reviews+luci-py_chromium.org, Sergey Berezin
Target Ref:
refs/heads/master
Project:
luci-py
Visibility:
Public.

Description

Roll infra_libs and gae_ts_mon in luci-py, and add field_specs to all metrics BUG=684974 TBR=maruel Review-Url: https://codereview.chromium.org/2705273003 Committed: https://github.com/luci/luci-py/commit/2dd95f76fcec9fc96c9412dbe396a7ad6de911d0

Patch Set 1 #

Patch Set 2 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+655 lines, -287 lines) Patch
M appengine/gce-backend/metrics.py View 1 1 chunk +8 lines, -4 lines 0 comments Download
M appengine/machine_provider/metrics.py View 1 chunk +10 lines, -10 lines 0 comments Download
M appengine/swarming/swarming_bot/bot_code/bot_main.py View 1 1 chunk +6 lines, -2 lines 0 comments Download
M appengine/swarming/ts_mon_metrics.py View 1 10 chunks +59 lines, -14 lines 0 comments Download
M appengine/third_party/gae_ts_mon/README.md View 2 chunks +6 lines, -3 lines 0 comments Download
M appengine/third_party/gae_ts_mon/__init__.py View 1 chunk +4 lines, -1 line 0 comments Download
M appengine/third_party/gae_ts_mon/config.py View 1 chunk +1 line, -1 line 0 comments Download
M appengine/third_party/gae_ts_mon/shared.py View 1 chunk +8 lines, -5 lines 0 comments Download
M client/third_party/infra_libs/event_mon/protos/goma_stats_pb2.py View 26 chunks +158 lines, -45 lines 0 comments Download
M client/third_party/infra_libs/httplib2_utils.py View 4 chunks +82 lines, -0 lines 0 comments Download
M client/third_party/infra_libs/logs/logs.py View 1 chunk +5 lines, -3 lines 0 comments Download
M client/third_party/infra_libs/ts_mon/__init__.py View 1 chunk +4 lines, -1 line 0 comments Download
M client/third_party/infra_libs/ts_mon/common/errors.py View 1 chunk +17 lines, -0 lines 0 comments Download
M client/third_party/infra_libs/ts_mon/common/http_metrics.py View 1 chunk +39 lines, -11 lines 0 comments Download
M client/third_party/infra_libs/ts_mon/common/interface.py View 3 chunks +8 lines, -26 lines 0 comments Download
M client/third_party/infra_libs/ts_mon/common/metrics.py View 17 chunks +138 lines, -115 lines 0 comments Download
M client/third_party/infra_libs/ts_mon/common/monitors.py View 6 chunks +78 lines, -37 lines 0 comments Download
M client/third_party/infra_libs/ts_mon/common/standard_metrics.py View 1 chunk +2 lines, -1 line 0 comments Download
M client/third_party/infra_libs/ts_mon/config.py View 1 chunk +4 lines, -3 lines 0 comments Download
M client/third_party/infra_libs/utils.py View 1 chunk +18 lines, -5 lines 0 comments Download

Messages

Total messages: 20 (11 generated)
dsansome
maruel: let me know when you're ready for me to commit this
3 years, 10 months ago (2017-02-22 03:41:55 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2705273003/1
3 years, 10 months ago (2017-02-24 02:39:40 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: Luci-py Presubmit on luci.infra.try (JOB_FAILED, https://luci-milo.appspot.com/swarming/task/3486eb378088d210)
3 years, 10 months ago (2017-02-24 03:41:40 UTC) #7
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2705273003/1
3 years, 10 months ago (2017-02-24 04:05:55 UTC) #9
commit-bot: I haz the power
Try jobs failed on following builders: Luci-py Presubmit on luci.infra.try (JOB_FAILED, https://luci-milo.appspot.com/swarming/task/34873982127b5710)
3 years, 10 months ago (2017-02-24 05:07:22 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2705273003/1
3 years, 9 months ago (2017-03-14 07:03:45 UTC) #13
commit-bot: I haz the power
Try jobs failed on following builders: Luci-py Presubmit on luci.infra.try (JOB_FAILED, https://luci-milo.appspot.com/swarming/task/34e48f20e291b110)
3 years, 9 months ago (2017-03-14 08:05:11 UTC) #15
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2705273003/20001
3 years, 9 months ago (2017-03-14 08:32:31 UTC) #17
commit-bot: I haz the power
3 years, 9 months ago (2017-03-14 08:37:03 UTC) #20
Message was sent while issue was closed.
Committed patchset #2 (id:20001) as
https://github.com/luci/luci-py/commit/2dd95f76fcec9fc96c9412dbe396a7ad6de911d0

Powered by Google App Engine
This is Rietveld 408576698