1283741 - Display the on duty sheriff in Treeherder's UI

Status: RESOLVED INCOMPLETE

(Tree Management :: Treeherder, defect, P3)

Description

[Carsten Book [:Tomcat]]

Its sometimes hard for developers to know what sheriff is on duty. Some sheriffs add |sheriffduty on their irc nick, some don't.

In order to make it more easier (and it raises also visibility), treeherder should have a field like 
"sheriff on duty is: [tomcat,kwierso,#developers]"

on the top pane (maybe next to the repos you have open). 

So when a sheriff is on duty she/he can add the name or change it after duty etc

Comment 1

[Carsten Book [:Tomcat]]

iris, wes, david what do you think, does this make sense to you ?

Comment 2

[Xidorn Quan [:xidorn] UTC+11]

I think editing some field may still be too much thing for all sheriffs to do for all the time. It should probably be as simple as a single button so that whenever you see it, you just push it, and everything is done in that one click.

I believe you all need to login to treeherder to get sheriff permission, right?

Comment 3

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

I think this would be best done in Treestatus. Treeherder could then poll this field along with the tree statuses. We could add a button to the sheriffing panel that submits something to Treestatus to automatically update this field.

In fact, we already have a bug for part of this.

Comment 4

[Iris Hsiao [:ihsiao]]

I'm ok for any ways to let developers know what sheriff is on duty, I'll follow your decision.
However, it's a great idea if there's a button to push, and update "sheriff on duty".

Comment 5

[Ed Morley [:emorley]]

As Wes said, this would be best stored/updated in TreeStatus IMO -- see the previous discussion in bug 1144589.

If someone adds that field to TreeStatus, Treeherder can then pull from that API and display in the UI.

Comment 6

[David Burns :automatedtester]

How would this be updated for someone coming on duty and going off duty? 
what happens if there are no paid sheriffs?

The idea sounds ok but the implementation needs a think about.

Comment 7

[Ed Morley [:emorley]]

Closing out until bug 1144589 implemented.