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

Issue 612733 link

Starred by 3 users
Status: Duplicate
Owner:
█ shey...@chromium.org
Email to this user bounced
Closed: Jun 2016
Cc:
█ shey...@chromium.org
tandrii@chromium.org
rmis...@google.com
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: ----
Type: ----



Sign in to add a comment

Ensure bugdroid ignores Gerrit ref patches

Project Member Reported by tandrii@chromium.org, May 18 2016 
Bugdroid posts messages on a bug even if it’s not committed to the master, e.g. when CQ fails: example  http://crbug.com/608665 .

Comment 1 by tandrii@chromium.org, May 18 2016 

the reason i think is htis:

https://git.repo/url.git/+refs?format=json (log in first!) now returns list of all the patches. We should filter them out in bugdroid.

Comment 2 by tandrii@chromium.org, May 23 2016 

Nope, that dosen't seem to be the reason, because infra repo config has a whitelist for refs: https://chrome-internal.googlesource.com/infra/infra_internal/+/eb5d05/py_issue_tracker/bugdroid_config.json#145

the last change to the file was ~7 weeks ago, so that shouldn't have affected the bugdroid, well unless it was NOT restarted for these 7 weeks

Comment 3 by tandrii@chromium.org, May 23 2016

Owner: shey...@chromium.org
Status: Assigned (was: Untriaged)
And even before that, there was the same whitelist. So, I think this is a bug in bugdroid. 

Sheng, can you please take a look at this?

Comment 4 by tandrii@chromium.org, Jun 7 2016 

 Issue 617726  has been merged into this issue.

Comment 5 by tandrii@chromium.org, Jun 17 2016

Mergedinto: 621066
Status: Duplicate (was: Assigned)
Found the reason thanks to Aaron: filed 621066
►

Sign in to add a comment