Closed Bug 1032152 Opened 5 years ago Closed 5 years ago

Job classification needs to be permitted for any logged in user, not just the sheriffs

Categories

(Tree Management :: Treeherder, defect, P1, major)

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: emorley, Assigned: camd)

References

Details

Currently job classification has been limited to just those users with the sheriffing permissions set. However all developers need to be able to classify failures, since:
a) They may wish to do so on their own try pushes
b) Whilst most of the sheriffing is done by half a dozen people, their is a long tail of others that chip in occasionally or monitor their own project repos (and those people don't need to be able to perform the more admin-like tasks that I think the sheriffing permissions should be saved for). 
c) It doesn't seem very Mozilla to shut everyone out, including possible new-starting volunteer sheriffs.

As such, we should either:
1) Allow failure classification to any logged in user (with logging to be able to tell if someone is causing mischief), or
2) Open up failure classification to anyone who has a vouched mozillians.org account (again, with logging).
Blocks: 1032163
Severity: normal → major
(In reply to Ed Morley [:edmorley UTC+0] from comment #0)
> As such, we should either:
> 1) Allow failure classification to any logged in user (with logging to be
> able to tell if someone is causing mischief), or
> 2) Open up failure classification to anyone who has a vouched mozillians.org
> account (again, with logging).

I think we should go for #1 for now, to make this simpler to implement & so we don't prematurely optimise, given that we've had no issues with TBPL for several years, even though it doesn't even need a login at all.
Priority: -- → P1
Summary: Job classification needs to be permitted for most users, not just the sheriffs → Job classification needs to be permitted for any logged in user, not just the sheriffs
ok, will make this change.
Assignee: nobody → cdawson
Status: NEW → ASSIGNED
I didn't see Ed's request for fields with fixes before pushing the change.  I'll do those things in the future.  But here's the commit SHA

Github commit: fcbccc2f14bf9de2d1122942c3b92d0148676972
https://github.com/mozilla/treeherder-service/commit/fcbccc2f14bf9de2d1122942c3b92d0148676972
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Great - thank you :-)
No longer blocks: 1032163
You need to log in before you can comment on or make changes to this bug.