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

Issue 848836 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2018
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug



Sign in to add a comment

Dashboard: implement quarantine for bad reports

Project Member Reported by mmoroz@chromium.org, Jun 1 2018

Issue description

That would allow cron to recover from most of the errors. E.g. if a report with low coverage was uploaded, right now we'll report an error and that will need some manual work to investigate + unblock further reports to be published. Manual work for investigation is not avoidable, which is fine, but we can move "bad" report to a separate quarantine directory in order to let further reports to be automatically processed and published if they are good.
 
Owner: mmoroz@chromium.org
Status: Started (was: Untriaged)
Have a draft CL ready, will test it on the most recent error.
Status: Fixed (was: Started)
Just tested https://chrome-internal-review.googlesource.com/c/chrome/tools/code-coverage/+/634743 on prod, works fine.

No need to do any manual clean up over the weekend, bad reports should be automatically moved to the quarantine with errors reported, valid reports will be published as they should be.
Project Member

Comment 3 by bugdroid1@chromium.org, Jun 2 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/code-coverage/+/8b94b07376706b03a4518689088ec74d9f287b5d

commit 8b94b07376706b03a4518689088ec74d9f287b5d
Author: Max Moroz <mmoroz@google.com>
Date: Sat Jun 02 04:34:03 2018

Project Member

Comment 4 by bugdroid1@chromium.org, Jun 4 2018

The following revision refers to this bug:
  https://chrome-internal.googlesource.com/chrome/tools/code-coverage/+/5d41915709c43dcce82a276f37dd16545a8bd746

commit 5d41915709c43dcce82a276f37dd16545a8bd746
Author: Max Moroz <mmoroz@google.com>
Date: Mon Jun 04 06:04:53 2018

Sign in to add a comment