New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 766023 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit > 30 days ago
Closed: Sep 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 766193



Sign in to add a comment

[cr-audit-commits] Set deadline for auditor.

Project Member Reported by robert...@chromium.org, Sep 18 2017

Issue description

The auditor handler should set a 5 minute deadline to make it reliably terminate before the hard cron job timeout of 10 minutes.
 
Blocking: 766193
Status: Started (was: Untriaged)
Project Member

Comment 3 by bugdroid1@chromium.org, Sep 21 2017

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

commit f7a79859831c2f6257fe8dcdfeb6017a9561448d
Author: Roberto Carrillo <robertocn@google.com>
Date: Thu Sep 21 22:25:40 2017

[cr-audit-commits] Set timeout for audits.

Cron jobs have a 10 minute hard limit. By setting a 5 minute timeout for
audits, we attempt having enough time to save partial results i.e. if
some of the commits were correctly audited within the allotted time.

R=iannucci,stgao
BUG= 766023 

Change-Id: I82ae06b7eccb87f1d601831b90a6f60e4b195eb0
Reviewed-on: https://chromium-review.googlesource.com/677568
Reviewed-by: Robbie Iannucci <iannucci@chromium.org>
Commit-Queue: Roberto Carrillo <robertocn@chromium.org>

[modify] https://crrev.com/f7a79859831c2f6257fe8dcdfeb6017a9561448d/go/src/infra/appengine/cr-audit-commits/app/commit_auditor.go

Status: Fixed (was: Started)

Sign in to add a comment