New issue
Advanced search Search tips

Issue 623696 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux
Pri: 2
Type: Bug



Sign in to add a comment

alerts dispatcher ignores chromium.infra.cron

Project Member Reported by martiniss@chromium.org, Jun 27 2016

Issue description

It currently ignores chromium.infra.cron, since it doesn't see it on CBE.

There's a TODO to make direct polling more configurable, which we might want to implement now. I could do it, but other help is welcome.

https://cs.chromium.org/chromium/infra/go/src/infra/monitoring/client/client.go?q=client.go&sq=package:chromium&l=219
 

Comment 1 by jpar...@google.com, Jun 28 2016

Owner: martiniss@chromium.org
Status: Assigned (was: Available)
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 28 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/infra/infra/+/8e2ded1ce266f2a242c8040fa0c2461b579b8d61

commit 8e2ded1ce266f2a242c8040fa0c2461b579b8d61
Author: Stephen Martinis <martiniss@chromium.org>
Date: Tue Jun 28 21:49:38 2016

alerts_dispatcher: Direct poll infra cron

This is a hack, and I'm sad :(

BUG= 623696 

Change-Id: Ibeabc09856db8bf68c73303294cca419efed5373
Reviewed-on: https://chromium-review.googlesource.com/356672
Reviewed-by: Stephen Martinis <martiniss@chromium.org>
Reviewed-by: Sean McCullough <seanmccullough@chromium.org>
Commit-Queue: Stephen Martinis <martiniss@chromium.org>

[modify] https://crrev.com/8e2ded1ce266f2a242c8040fa0c2461b579b8d61/go/src/infra/monitoring/client/client.go

Status: Fixed (was: Assigned)
Done and live.

Sign in to add a comment