Page MenuHomePhabricator

Give Lea Voget (WMDE) "grafana-admin" LDAP group access
Closed, ResolvedPublic

Description

Phabrictor user: https://phabricator.wikimedia.org/p/Lea_WMDE/
LDAP User: Lea Voget (WMDE)

Lea needs access in order to be able to edit WMDE Grafana dashboards.

See T199967 for adding her to the WMDE ldap group & thus probably signing an NDA

Event Timeline

herron triaged this task as Medium priority.Jul 19 2018, 3:20 PM
herron added a project: SRE.
herron moved this task from Backlog to NDA Pending on the LDAP-Access-Requests board.

Lea has been added to the "wmde" LDAP group in the linked subtask.

This should also resolve this. The same comments on T199965#4477710 also apply here.

Dzahn claimed this task.

It doesn't look like lea has been added to the grafana-admin ldap group or the nda ldap group, so I don't believe she will currently be able to log in to grafana?
Or are the wikitech docs outdated?
The wmde ldap group certainly doesn't provide this access.

For convenience: https://tools.wmflabs.org/ldap/user/lea-wmde

I wasn't aware of any group called "grafana-admin". Apparently there is one though (with just 4 users in it??)

Addshore renamed this task from Give Lea Voget (WMDE) grafana-admin access to Give Lea Voget (WMDE) "grafana-admin" LDAP group access.Aug 13 2018, 6:31 PM

Indeed, as there is overlap with the NDA group (which has many more users)

As far as I understand it grafana-admin should be used when only grafana admin panel access is needed.

Well, if access to grafana requires an NDA and signing an NDA means being added to the group called "nda" and that group gives access to grafana.. then i don't know what grafana-admin would be used for.

Adding her to the "nda" group.

Well, if access to grafana requires an NDA and signing an NDA means being added to the group called "nda" and that group gives access to grafana.. then i don't know what grafana-admin would be used for.

This echos some of my thoughts on a previous phab ticket on a similar discussion :)

Change 452450 had a related patch set uploaded (by Dzahn; owner: Dzahn):
[operations/puppet@production] admins: add Lea Voget to ldap admins (wmde,nde)

https://gerrit.wikimedia.org/r/452450

Change 452450 merged by Dzahn:
[operations/puppet@production] admins: add Lea Voget to ldap admins (wmde,nde)

https://gerrit.wikimedia.org/r/452450