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

Issue 771422 link

Starred by 1 user
Status: Fixed
Owner:
chanli@chromium.org
Closed: Oct 2017
Cc:
chanli@chromium.org
st...@chromium.org
█ robert...@chromium.org
lijeffrey@chromium.org
liaoyuke@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 1
Type: Bug

Blocking:
issue 772692



Sign in to add a comment

AssertionError in _CanRevert() (services/revert.py)

Project Member Reported by robert...@chromium.org, Oct 3 2017 
https://pantheon.corp.google.com/errors/71058473761960726?time=P1D&filter&project=findit-for-me

According to the trace, the error occurred in the line following
`culprit = WfSuspectedCL.Get(repo_name, revision)`
where culprit is asserted.

I have inserted snapshot points in the production instance and will re-run the analysis that we believe caused it, and see if we can figure out why the .Get call is returning None.

Comment 1 by robert...@chromium.org, Oct 6 2017

Owner: chanli@chromium.org
Status: Assigned (was: Started)
This seems to have been fixed by the revert: https://chromium-review.googlesource.com/c/infra/infra/+/699754

The issue seems to be different params/ordering for __init__() and run() in the create revert cl pipeline.

Assigning to Chan for her to reland with a fix.

Comment 2 by chanli@chromium.org, Oct 10 2017

Status: Fixed (was: Assigned)
Fix is landed at https://chromium-review.googlesource.com/c/infra/infra/+/707815

Comment 3 by chanli@chromium.org, Oct 10 2017

Blocking: 772692
►

Sign in to add a comment