Change 418935 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/ContentTranslation@master] Add cx_version to cx_translations table.

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

Change 418937 had a related patch set uploaded (by Nikerabbit; owner: Nikerabbit):
[mediawiki/extensions/ContentTranslation@master] Send cx version when saving a draft

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

TODO: First patch needs DBA to check, need to check for the proper process. Third patch needs fixing.

Note to reviewers and testers: do check that things degrade gracefully when schema change is not applied.

Hi DBAs, please check https://gerrit.wikimedia.org/r/c/418935/

Please don't just add individual people to reviews- workflow is well documented at https://www.mediawiki.org/wiki/Development_policy#Database_patches (mediawiki policy) and detailed at https://wikitech.wikimedia.org/wiki/Schema_changes#Workflow_of_a_schema_change this is for your convenience, not intended to create unneeded bureaucracy, but to avoid unnecessary delays (eg. my work mates may not see calls for help if you only add me and I could be on vacation :-P).

Please add the DBA tag first if you need help from DBAs- it will then be scheduled for the next available person to work on it (it can be me, but it can be someone else, depending on the availability).

Another tip to speed up reviews is to add Manuel and me to the reviews as "reviewers".

I see the values can only be 1 or 2, so that invalidates most of my comments. We will be waiting for merge and you can add Schema-change-in-production after that happens here or on a separate ticket, as you prefer.

Thanks for the tips. I was reading https://wikitech.wikimedia.org/wiki/Schema_changes#Workflow_of_a_schema_change but not the other page and did not see that DBA should be used.

As for https://www.mediawiki.org/wiki/Development_policy#Database_patches:

• Update installer
• Not applicable to CX, tables are created manually
• Make schema change optional
• The new field has a default value
• The follow-up code has a fallback for missing fields and doesn't write if the field doesn't exist
• Search for input
• Done based on your comments. Thanks.
• Test on beta
• Q: Should we deploy this schema change on beta ourselves or is that dependent on you? I did not see this mentioned in the docs.

Yes, beta is fully "yours", you just merge and it gets deployed automatically, I think- I am not much involved on it. You test it, then you come back to us for the production deployment (you can reuse the ticket if you edit the description or you can file a separate one). To mark this is ready, you add the Schema-change-in-production (which means: "hey, dbas, we can do no more without the change, please deploy ASAP").

Sorry to be so strict, but we get many request over each week, the easier you put it to us, the faster you will get things done. Thank you for the understanding.

In addition to what Jaime said, once you add the Schema-change-in-production if you want to reuse the ticket, that is totally fine but edit the original task description to add the fields commented described on: https://wikitech.wikimedia.org/wiki/Schema_changes#Workflow_of_a_schema_change

The ALTER TABLEs to run (usually, a link to a commit diff is the best way)
Where to run those changes (specific dblist file or the complete list of wikis, in text, to apply the change to). "All wikis" or "everywhere" are not specific enough and has been the cause of outages in the past.
When to run those changes (if it depends on a particular code deployment or can be done at any time)
If the schema change is backwards compatible (is compatible with the current code deployed and can be performed at any time now)
If the schema change has been tested already on some of the test/beta wikis. Usually, as a last test, change should be applied to testwiki first.
If it involves new columns or tables, if the data should be made available on the labs replicas and/or dumps or not because they contain private or sensitive data (consult legal if you are unsure). Similar question if it involves deletion of data previously available on labs.

As Jaime said we get many requests every week and in order to avoid mistakes we prefer to use the same "format" for a schema change, as it is clearer for us and hence, faster to get it deployed as we avoid the back and forth, so it is a win-win :-)

Thanks!

Change 418935 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Add cx_version to cx_translations table.

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

Change 418936 had a related patch set uploaded (by Santhosh; owner: Nikerabbit):
[mediawiki/extensions/ContentTranslation@master] Use the stored cx_version in the dashboard

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

In T187986#4052729, @Nikerabbit wrote:

• Test on beta
• Q: Should we deploy this schema change on beta ourselves or is that dependent on you? I did not see this mentioned in the docs.

I updated schema in Beta (See: T190009) and Labs instances (cx-testing and cx2-testing).

Change 418936 merged by jenkins-bot:
[mediawiki/extensions/ContentTranslation@master] Use the stored cx_version in the dashboard

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

Change 418937 merged by Petar.petkovic:
[mediawiki/extensions/ContentTranslation@master] Send cx version when saving a draft

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