1723906 - Add Julian Descottes to Treeherder sheriff group

Status: RESOLVED FIXED

(Tree Management :: Treeherder: Infrastructure, task)

Description

[Julian Descottes [:jdescottes]]

I will be sheriffing performance alerts for the devtools tests on Perfherder, so I would like to be added to the Treeherder sheriff group.

My LDAP address is [e-mail]

Comment 1

[Julian Descottes [:jdescottes]]

Hi Sarah,

Not sure if those requests are triaged, so just pinging you in cased this request was missed.

Comment 2

[Sarah Clements [:sclements]]

Hi, is this request for being added to the Treeherder repo or for taskcluster permissions (retriggers and such)?

Comment 3

[Julian Descottes [:jdescottes]]

(In reply to Sarah Clements [:sclements] from comment #2)

Hi, is this request for being added to the Treeherder repo or for taskcluster permissions (retriggers and such)?

Forwarding the question to :davehunt . Do you know what I would need for DevTools performance sheriffing?

Comment 4

[Dave Hunt [:davehunt] [he/him] ⌚BST]

(In reply to Sarah Clements [:sclements] from comment #2)

Hi, is this request for being added to the Treeherder repo or for taskcluster permissions (retriggers and such)?

I believe it's TaskCluster permissions, but also Treeherder permissions. For example, according to bug 1506882 comment 1 we need the is_staff flag set.

Comment 5

[Sarah Clements [:sclements]]

Ah, right, thanks for the reminder. cloudOps manages our databases now, so I don't have any way to update the is_staff flag myself. The first thing is that Julian needs to have logged into Treeherder at least once so that a user profile is created and stored in the database.

Once that's done, someone from cloudOps will need to update the is_staff flag for Julian in the User table to true. Chris, can you help with this? I'd suggest starting with prototype, stage and then making the change in production.

Ben, can you give Julian the same taskcluster permissions as the other performance sheriffs please?

Comment 6

[[e-mail] (:bhearsum)]

(In reply to Sarah Clements [:sclements] from comment #5)

Ah, right, thanks for the reminder. cloudOps manages our databases now, so I don't have any way to update the is_staff flag myself. The first thing is that Julian needs to have logged into Treeherder at least once so that a user profile is created and stored in the database.

Once that's done, someone from cloudOps will need to update the is_staff flag for Julian in the User table to true. Chris, can you help with this? I'd suggest starting with prototype, stage and then making the change in production.

Ben, can you give Julian the same taskcluster permissions as the other performance sheriffs please?

This is all controlled by LDAP. I believe he'll need be added to the sheriff group.

Comment 7

[chris valaas [:cvalaas]]

Once that's done, someone from cloudOps will need to update the is_staff flag for Julian in the User table to true. Chris, can you help with this? I'd suggest starting with prototype, stage and then making the change in production.

I updated is_staff to 1 in the auth_user table (treeherder database) in the prototype/stage databases.
Can you log into prototype and/or stage and see if things are different, Julian?

Comment 8

[Julian Descottes [:jdescottes]]

I logged into https://treeherder.allizom.org/perfherder/alerts?status=0&framework=12&hideDwnToInv=1&page=1 (I assume allizom is using prototype or stage dbs?), and I seem to have sheriffing rights. I can now update alerts, add notes, tags, etc.

So seems to work fine in that environment, thanks!

Comment 9

[chris valaas [:cvalaas]]

Added to prod.