New issue
Advanced search Search tips

Issue 601098 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Sheriffbot-Auto-triage Rule: Needs-Feedback Bugs, already received feedback 7 days ago

Project Member Reported by rsesek@chromium.org, Apr 6 2016

Issue description

Sheriffbot-Auto-triage Rule: Needs-Feedback Bugs, already received feedback 7 days ago.


Add Bug number which got updated: https://bugs.chromium.org/p/chromium/issues/detail?id=246096#c8


Issue/Concern or feedback:
According to the name of this sheriffbot rule, Needs-Feedback bugs should only be pinged with Needs-Review if the requested feedback is more than 7 days old. However I added Needs-Feedback on Tues April 5 @ 14:41, which was responded to by dbeam@ at Tues April 5 @ 16:12. Then sherriffbot comes along and adds the Needs-Review label at Wed April 6 @ 13:01.

Some simple math tells me that (April 6) - (April 5) < 7 days, so sheriffbot should not have done anything to this bug.

Any other important details:

 
We check for bugs which are modified in the last 7 days not received before 7 days. so if the reporter replies, the bot looks at it and adds "Needs-Review".
Modified the rules page to reflect this change - https://sites.google.com/a/chromium.org/dev/issue-tracking/autotriage [Needs-Feedback bugs, already received feedback in last 7 days]
What's the goal of this rule then? Having an automation bot interject itself in a non-stale discussion is not useful, it's disruptive. It seems like the useful thing would be to surface bugs in which the reporter provided the feedback but the bug hasn't been acted on in 7 days. What sheriffbot@ did at comment#8 does not seem correct.

Comment 4 Deleted

Anthony, could you please provide your input on this [As per rsesek@ update on #3, It seems like the useful thing would be to surface bugs in which the reporter provided the feedback but the bug hasn't been acted on in 7 days].
Status: Fixed (was: Assigned)
I believe this was addressed in other issues.

Sign in to add a comment