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

Issue 1868693002: ts_mon: catch more PubSub errors from socket. (Closed)

Created:
4 years, 8 months ago by Sergey Berezin
Modified:
4 years, 8 months ago
Reviewers:
M-A Ruel, pgervais
CC:
chromium-reviews, infra-reviews+infra_chromium.org
Base URL:
https://chromium.googlesource.com/infra/infra.git@master
Target Ref:
refs/heads/master
Project:
infra
Visibility:
Public.

Description

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -2 lines) Patch
M infra_libs/httplib2_utils.py View 1 chunk +1 line, -1 line 0 comments Download
M infra_libs/ts_mon/common/monitors.py View 2 chunks +4 lines, -1 line 0 comments Download

Messages

Total messages: 6 (2 generated)
Sergey Berezin
PTAL
4 years, 8 months ago (2016-04-06 19:36:11 UTC) #1
M-A Ruel
lgtm, thanks!
4 years, 8 months ago (2016-04-06 19:37:20 UTC) #2
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1868693002/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1868693002/1
4 years, 8 months ago (2016-04-06 19:38:14 UTC) #4
commit-bot: I haz the power
4 years, 8 months ago (2016-04-06 19:48:53 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://chromium.googlesource.com/infra/infra/+/5e49a19c485e78c3aaae9fd10b7eb...

Powered by Google App Engine
This is Rietveld 408576698