Page MenuHomePhabricator

Configure the CampaignEvents extension to use the event-organizer group by default
Closed, ResolvedPublic2 Estimated Story Points

Description

The extension's default configuration (in extension.json) is that every user gets all the organizer rights. However, in WMF wikis, the default is only used in test wikis (and beta). With the extension being enabled on more and more wikis, it might make sense to have the event-organizer group by default, either in the extension config, or WMF config. This way, we wouldn't have to add all the boilerplate that is currently required when enabling the extension on a new wiki (see e.g. T376786).

Acceptance criteria

  • The event-organizer group is defined by default in the CampaignEvents extension
  • All the individual wiki definitions of the event-organizer group are removed from WMF config
  • Overall, no visible changes to the permission setup in WMF wikis

Event Timeline

Change #1111224 had a related patch set uploaded (by Mhorsey; author: Mhorsey):

[mediawiki/extensions/CampaignEvents@master] set sensible defaults for permission in the extension config.

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

Change #1111227 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[operations/mediawiki-config@master] test(2)wiki: Explicitly assign event organizer rights to all users

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

Change #1111227 merged by jenkins-bot:

[operations/mediawiki-config@master] test(2)wiki: Explicitly assign event organizer rights to all users

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

Mentioned in SAL (#wikimedia-operations) [2025-01-14T14:31:21Z] <lucaswerkmeister-wmde@deploy2002> Started scap sync-world: Backport for [[gerrit:1111227|test(2)wiki: Explicitly assign event organizer rights to all users (T376822)]]

Mentioned in SAL (#wikimedia-operations) [2025-01-14T14:37:47Z] <lucaswerkmeister-wmde@deploy2002> daimona, lucaswerkmeister-wmde: Backport for [[gerrit:1111227|test(2)wiki: Explicitly assign event organizer rights to all users (T376822)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2025-01-14T14:53:47Z] <lucaswerkmeister-wmde@deploy2002> Finished scap sync-world: Backport for [[gerrit:1111227|test(2)wiki: Explicitly assign event organizer rights to all users (T376822)]] (duration: 22m 26s)

Change #1008074 had a related patch set uploaded (by Mhorsey; author: Vaughn Walters):

[mediawiki/core@master] selenium: Add `api.addUserToGroup()` to add user to a user group

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

Change #1113491 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[mediawiki/extensions/CampaignEvents@master] [WIP] build: Update wdio-mediawiki to 2.7.0

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

Change #1008074 merged by jenkins-bot:

[mediawiki/core@master] selenium: Add `api.addUserToGroup()` to add user to a user group

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

Change #1113491 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] build: Update wdio-mediawiki to 2.7.0

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

Change #1111224 merged by jenkins-bot:

[mediawiki/extensions/CampaignEvents@master] Create event organizer group by default

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

Change #1116834 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[operations/mediawiki-config@master] core-Permissions: drop redundant CampaignEvents right assignments

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

Only thing left to do is cleaning up the WMF config, which needs to wait for r1111224 to reach production.

Scheduled for tomorrow 2025-02-11 14:00 UTC.

Change #1116834 merged by jenkins-bot:

[operations/mediawiki-config@master] core-Permissions: drop redundant CampaignEvents right assignments

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

Mentioned in SAL (#wikimedia-operations) [2025-02-11T14:05:22Z] <urbanecm@deploy2002> Started scap sync-world: Backport for [[gerrit:1118604|Drop obsolete CampaignEvents config flags (T380076 T381423)]], [[gerrit:1116834|core-Permissions: drop redundant CampaignEvents right assignments (T376822)]]

Mentioned in SAL (#wikimedia-operations) [2025-02-11T14:09:55Z] <urbanecm@deploy2002> daimona, urbanecm: Backport for [[gerrit:1118604|Drop obsolete CampaignEvents config flags (T380076 T381423)]], [[gerrit:1116834|core-Permissions: drop redundant CampaignEvents right assignments (T376822)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2025-02-11T14:19:34Z] <urbanecm@deploy2002> Finished scap sync-world: Backport for [[gerrit:1118604|Drop obsolete CampaignEvents config flags (T380076 T381423)]], [[gerrit:1116834|core-Permissions: drop redundant CampaignEvents right assignments (T376822)]] (duration: 14m 11s)

Change #1118820 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[operations/mediawiki-config@master] test(2)wiki: Re-assign event organizer rights to all users

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

Change #1118820 merged by jenkins-bot:

[operations/mediawiki-config@master] test(2)wiki: Re-assign event organizer rights to all users

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

Mentioned in SAL (#wikimedia-operations) [2025-02-11T14:35:59Z] <urbanecm@deploy2002> Started scap sync-world: Backport for [[gerrit:1118820|test(2)wiki: Re-assign event organizer rights to all users (T376822)]]

Mentioned in SAL (#wikimedia-operations) [2025-02-11T14:40:39Z] <urbanecm@deploy2002> urbanecm, daimona: Backport for [[gerrit:1118820|test(2)wiki: Re-assign event organizer rights to all users (T376822)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2025-02-11T14:48:19Z] <urbanecm@deploy2002> Finished scap sync-world: Backport for [[gerrit:1118820|test(2)wiki: Re-assign event organizer rights to all users (T376822)]] (duration: 12m 20s)

Hi Devs, Administrators and Bureaucrats on Wikidata and Meta-Wiki are not able to assign or remove the event-organizer permission anymore. I assume this is a mistake? Can someone have a look into that? I think that change is related to this Ticket.

Hi Devs, Administrators and Bureaucrats on Wikidata and Meta-Wiki are not able to assign or remove the event-organizer permission anymore. I assume this is a mistake? Can someone have a look into that? I think that change is related to this Ticket.

Ugh, yeah that's unexpected. Sorry about that! I'll try to get this fixed in the upcoming deployment window (14:00 UTC, in 50 minutes).

I see what happened. With r1111224, the CampaignEvents extension by default allows admins to add/remove the event-organizer group. However, for reasons that I don't fully grasp, our production config sets a different default for wgAddGroups, effectively erasing any defaults set by extensions. The only reason why I can see this being the case is that $wgAddGroups is empty by default in MediaWiki core, so whoever added the default probably 20 years ago thought it wouldn't make any difference. After all, there are only a few extensions that change this.


However:

Administrators and Bureaucrats on Wikidata and Meta-Wiki are not able to assign or remove the event-organizer permission anymore.

While administrators are definitely affected (I can confirm this myself on itwiki), can you double-check that bureaucrats also can't add users to the group? This should not be the case, as we only removed overrides for sysops, not bureaucrats.

Change #1119136 had a related patch set uploaded (by Daimona Eaytoy; author: Daimona Eaytoy):

[operations/mediawiki-config@master] Let sysop add/remove the event-organizer group by default

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

Change #1119136 merged by jenkins-bot:

[operations/mediawiki-config@master] Let sysops add/remove the event-organizer group by default

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

Mentioned in SAL (#wikimedia-operations) [2025-02-12T15:04:04Z] <lucaswerkmeister-wmde@deploy2002> Started scap sync-world: Backport for [[gerrit:1119136|Let sysops add/remove the event-organizer group by default (T376822)]]

Mentioned in SAL (#wikimedia-operations) [2025-02-12T15:06:59Z] <lucaswerkmeister-wmde@deploy2002> daimona, lucaswerkmeister-wmde: Backport for [[gerrit:1119136|Let sysops add/remove the event-organizer group by default (T376822)]] synced to the testservers (https://wikitech.wikimedia.org/wiki/Mwdebug)

Mentioned in SAL (#wikimedia-operations) [2025-02-12T15:16:58Z] <lucaswerkmeister-wmde@deploy2002> Finished scap sync-world: Backport for [[gerrit:1119136|Let sysops add/remove the event-organizer group by default (T376822)]] (duration: 12m 53s)

The issue with group management should be fixed now. I also filed T386210 for a related bug I found while testing locally.

Thanks for fixing it, assignment-option is "back again" on both metawiki and wikidatawiki :)

While administrators are definitely affected (I can confirm this myself on itwiki), can you double-check that bureaucrats also can't add users to the group? This should not be the case, as we only removed overrides for sysops, not bureaucrats.

I didn't ask anyone and I am not a crat on meta/wikidata, I was just looking at Special:ListGroupRights where event-organizer-assignment was missing everywhere, also at the crat-permission-list.

vaughnwalters subscribed.

Event-organizer group displaying, as well as the list of users with event-organizer right. sending to product sign off

pl wikipl wiki list of users with event organizer rightar wikiar wiki list of users with event org rights
Screenshot 2025-02-14 at 5.09.09 PM.png (190×1 px, 75 KB)
Screenshot 2025-02-14 at 5.31.35 PM.png (1×2 px, 477 KB)
Screenshot 2025-02-17 at 3.34.31 PM.png (142×1 px, 52 KB)
Screenshot 2025-02-17 at 3.35.11 PM.png (1×1 px, 319 KB)