Page MenuHomePhabricator

Striker should use ID instead of username to identify SUL accounts
Open, MediumPublic

Description

Currently Striker stores the SUL user name only, it should also store the ID so it can be matched against the wikimediaGlobalAccountId attribute in the parent task.

Event Timeline

taavi triaged this task as Medium priority.
taavi moved this task from Backlog to Doing on the Striker board.

Change 1009310 had a related patch set uploaded (by Majavah; author: Majavah):

[labs/striker@master] labsauth: Add field for SUL account ID

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

Change 1009311 had a related patch set uploaded (by Majavah; author: Majavah):

[labs/striker@master] labsauth: Store SUL user ID like username

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

taavi removed taavi as the assignee of this task.Jun 25 2024, 3:35 PM

Change #1009310 merged by jenkins-bot:

[labs/striker@master] labsauth: Add field for SUL account ID

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

Change #1076798 had a related patch set uploaded (by Majavah; author: Majavah):

[operations/puppet@production] hieradata: Bump striker-toolsbeta to 2024-09-30-162529-production

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

Change #1076798 merged by Majavah:

[operations/puppet@production] hieradata: Bump striker-toolsbeta to 2024-09-30-162529-production

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

Change #1076799 had a related patch set uploaded (by Majavah; author: Majavah):

[operations/puppet@production] hieradata: Bump striker-tools to 2024-09-30-162529-production

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

Change #1076799 merged by Majavah:

[operations/puppet@production] hieradata: Bump striker-tools to 2024-09-30-162529-production

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

Change #1009311 merged by jenkins-bot:

[labs/striker@master] labsauth: Store SUL user ID like username

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

Change #1076800 had a related patch set uploaded (by Majavah; author: Majavah):

[operations/puppet@production] hieradata: Bump striker-toolsbeta to 2024-09-30-170045-production

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

Change #1076800 merged by Majavah:

[operations/puppet@production] hieradata: Bump striker-toolsbeta to 2024-09-30-170045-production

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

Change #1076809 had a related patch set uploaded (by Majavah; author: Majavah):

[labs/striker@master] register: Also filter on user SUL ID

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

Change #1076809 merged by jenkins-bot:

[labs/striker@master] register: Also filter on user SUL ID

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

Aklapper added subscribers: Arendpieter, Aklapper.

Arendpieter added a project: Infrastructure-Foundations.

@Arendpieter: Why?

@taavi What’s still left to do on this issue? All the pull requests are merged. I’m looking for an interesting Python issue to work on 😉

What's left is finding a way to set the new ID field for old accounts that only have an username set.

What's left is finding a way to set the new ID field for old accounts that only have an username set.

It is relatively easy to write a script to do this for SUL accounts that have not been renamed by looking up the SUL account's global id. For accounts that have been renamed things are a bit trickier in that you need to figure out what the new name is, possibly with multiple renames having happened in the intervening time. I did this for a number of accounts during the Wikitech SUL migration, but I used a very manual process of attempting to visit the User namespace page of the old name and hoping to be redirected.