Page MenuHomePhabricator

Add retries, error handling and metrics to sync-gitlab-group-with-ldap
Open, MediumPublic

Description

With the recent events, we've seen that script failing when LDAP returns a 500 error. To improve things, it would be nice to:

  • add error handling to avoid crashing when LDAP returns a 500 error
  • add a retry policy (exponential backoff limited to 5 ?) to the script to avoid failing on the first error
  • add a metric set to track errors/success and the matching set of alerts

Details

Related Changes in GitLab:
TitleReferenceAuthorSource BranchDest Branch
group-management: add retry to gitlab helper _get methodrepos/releng/gitlab-settings!82lenapadd-error-handling-gitlab-helper-readonlymain
Customize query in GitLab

Event Timeline

ABran-WMF renamed this task from Improve sync-gitlab-group-with-ldap to Add retries, error handling and metrics to sync-gitlab-group-with-ldap.Apr 21 2026, 6:11 AM
LSobanski triaged this task as Medium priority.Apr 27 2026, 3:31 PM
LSobanski moved this task from Incoming to Backlog on the collaboration-services board.
Lenap subscribed.

I'm working on a patch in gitlab-settings, claiming the task.