Page MenuHomePhabricator

Review database schema of the WikimediaCampaignEvents extension
Closed, ResolvedPublic

Description

Disclaimer: there doesn't seem to be a request template in https://www.mediawiki.org/wiki/Writing_an_extension_for_deployment#Preparing_for_deployment, so I'm not quite sure what information is needed here. We'd be happy to provide any additional needed information, ça va sans dire.

We would like to request a DBA review of the schema of the WikimediaCampaignEvents extension.

Schema to review:

Location
The new table will leave together with the tables of the CampaignEvents extension. Currently, this means:

  • Beta: the wikishared database in x1
  • Prod: the wikishared database in x1, as well as the testwiki, test2wiki, and officewiki databases also in x1.

Deployment date
Tentatively December 2023, but it could be later if the deployment process takes longer.

Further information
An overview of the tool, team contacts etc. are also available in the security review task: T350900

Event Timeline

Marostegui triaged this task as Medium priority.
Marostegui moved this task from Triage to Pending comment on the DBA board.

My only question is wceg_grant_id seems to blob but it looks like an id? Can I have some examples?

What is the rough number of rows? I don't think it'll be a lot. Generally speaking this looks good to me.

My only question is wceg_grant_id seems to blob but it looks like an id? Can I have some examples?

Sure: these IDs are retrieved from Fluxx, and as far as they know, they are in the form XXXX-XXXXX (where Xs are numbers). We chose to store them without any processing because we did not have guarantees on the exact format IIRC (CC @cmelo), and also because Fluxx does dubious things with its IDs (e.g., add letters to the prefix depending on the status of the grant).

What is the rough number of rows? I don't think it'll be a lot. Generally speaking this looks good to me.

Yup, it should be really small. According to the current estimates, it would be around ~300 rows per year.

Then go ahead.

Thanks! Can this task be resolved then? I will create new tasks for the actual creation of these tables when we get there.

Sure but we don't create the tables ourselves so it's on your hands totally

Sure but we don't create the tables ourselves so it's on your hands totally

Yeah, sorry, I just meant to say that I will create new tasks for us to create the table, and just reference this one for DBA approval. Closing as resolved then, thanks again!