New issue
Advanced search Search tips

Issue 627121 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jul 2016
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Sheriffbot-Auto-triage Rule: Open bugs with CL landed which are not modified for 6 months - triggers too often on flaky test bugs

Project Member Reported by rdevlin....@chromium.org, Jul 11 2016

Issue description

Sheriffbot-Auto-triage Rule: Open bugs with CL landed which are not modified for 6 months.


Add Bug number which got updated: e.g. issue 403325, but the real problem is the number of these I see in a day (5? 10?)


Issue/Concern or feedback:
I like the rule for open bugs with cls landed, because I know that I've very often fixed something and forgotten to circle back and mark it as such.  However, I think that for flaky/broken tests, this triggers too much.  Almost always, a bug for a flaky/disabled test will have a CL landed, because it's the CL that disabled the test.  I'd prefer if we can exempt issues that look like test flakes from this rule.  Right now, it feels like make-work to open up every issue, remove the label so that the bug isn't auto closed, etc.  Naturally, we still want to look at these, but I think we can still catch them in the other rules, like not being updated in years, etc.
 
Cc: -cda...@chromium.org gov...@chromium.org
Owner: cda...@chromium.org
Agreed, I think that it would be reasonable to exclude flaky test. We can use component negation, in the query, to achieve that.

Comment 2 by cda...@chromium.org, Jul 12 2016

Status: Fixed (was: Assigned)
Fixed to ignore bugs from component:Tests>Flaky.
What about also from Tests>Fails, maybe Tests>Disabled, etc?
How about anything that is in the Tests parent component?

Comment 5 by cda...@chromium.org, Jul 13 2016

Updated query to ignore bugs from component:Tests
Components: Tests>Disabled
Labels: Test-Disabled

Sign in to add a comment