Page MenuHomePhabricator

Update/add/remove LDAP entries based on changes to data.yaml
Closed, DeclinedPublic

Description

Cluster access is primarily defined in data.yaml, but additional group membership changes need to be made in LDAP. Having two data sources to edit causes overhead and is error-prone wrt stale data.

One way to solve this is to make data.yaml the authoritative source and auto-generate LDIFs based on changes made to data.yaml. These could then be merged/applied during the puppet-merge step (as long as the change touches modules/admin/data/data.yaml)

Event Timeline

I thought all our LDAP groups and all our production server groups were entirely separate?

I thought all our LDAP groups and all our production server groups were entirely separate?

Mostly, yes. cn=ops can/needs to be kept in sync, though. I have opened T142821 for for the followup work for generic groups.

demon triaged this task as Medium priority.Aug 31 2016, 6:34 PM

I'll extend the data.yaml file to also track users with privileged LDAP access (ops or wmf group), but no production shell access. That way we have the same properties on record as for shell access (full name, email address, optional expiry address for non-staff) and the access is properly reviewable/tracked in git.

Closing, this very much overlaps with T142819, tracking that one instead.