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

Issue 697909 link

Starred by 1 user
Status: Archived
Owner:
davidri...@chromium.org
Closed: Mar 2017
Cc:
█ dshi@chromium.org
akes...@chromium.org
davidri...@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Re-enable suite timeline reports

Reported by jrbarnette@chromium.org, Mar 2 2017 
In looking for a way to stop problems described in
bug 695287, we temporarily disabled suite timeline
reports with this CL:
    https://chromium-review.googlesource.com/#/c/446897/

I believe we have solid evidence that that change had
no impact, so we can re-enable the code.
Project Member

Comment 1 by bugdroid1@chromium.org, Mar 3 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/5d14a5df1851ed34b5ad6571212de7525cffac7e

commit 5d14a5df1851ed34b5ad6571212de7525cffac7e
Author: David Riley <davidriley@chromium.org>
Date: Fri Mar 03 04:17:44 2017

Revert "[autotest] temporarily disable suite timeline report"

This reverts commit 229a76d01e67124606cebabaf62fc56f256a691e.

Reason for revert: We have evidence that this was not responsible for
the additional load experienced so undo this change.

Original change's description:
> [autotest] temporarily disable suite timeline report
>
> Temporarily disable the suite timeline report to test whether it causes
> the load on master db.
>
> BUG=chromium:695287
> TEST=None
>
> Change-Id: I8b58b17b8f0923dd6d4c663e79fd163662009d5b
> Reviewed-on: https://chromium-review.googlesource.com/446897
> Tested-by: Shuqian Zhao <shuqianz@chromium.org>
> Trybot-Ready: Shuqian Zhao <shuqianz@chromium.org>
> Reviewed-by: Aviv Keshet <akeshet@chromium.org>
> Commit-Queue: Shuqian Zhao <shuqianz@chromium.org>
>

TBR=akeshet@chromium.org,shuqianz@chromium.org
# Not skipping CQ checks because original CL landed > 1 day ago.
BUG= chromium:697909 

Change-Id: If059b97b5098fdab21873fbdc65429d0b2ea4d18
Reviewed-on: https://chromium-review.googlesource.com/448667
Commit-Ready: David Riley <davidriley@chromium.org>
Tested-by: David Riley <davidriley@chromium.org>
Reviewed-by: Richard Barnette <jrbarnette@google.com>

[modify] https://crrev.com/5d14a5df1851ed34b5ad6571212de7525cffac7e/scheduler/postjob_task.py

Comment 2 by autumn@chromium.org, Mar 7 2017

Owner: davidri...@chromium.org
FIxed already?

Comment 3 by davidri...@chromium.org, Mar 7 2017

Cc: akes...@chromium.org
Status: Fixed (was: Available)
Might be waiting for a prod push though.

Comment 4 by dchan@google.com, May 30 2017

Labels: VerifyIn-60

Comment 5 by dchan@chromium.org, Aug 1 2017

Labels: VerifyIn-61

Comment 6 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)
►

Sign in to add a comment