⚓ T226826 Drop old oathauth

Reedy created this task.Jun 28 2019, 11:45 AM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJun 28 2019, 11:45 AM

Reedy moved this task from Unsorted to Delete / Drop / Remove on the Schema-change board.Jun 28 2019, 11:45 AM

jcrespo moved this task from Triage to Pending comment on the DBA board.Jun 28 2019, 12:05 PM

I have altered this table on db2054 on centralauth and will leave the columns renamed for a few days to make sure nothing uses it.

root@db2054.codfw.wmnet[centralauth]> alter table oathauth_users change secret TO_DROP_secret varbinary(255) DEFAULT NULL, change scratch_tokens TO_DROP_scratch_tokens varbinary(511) DEFAULT NULL;
Query OK, 0 rows affected (0.09 sec)
Records: 0  Duplicates: 0  Warnings: 0

root@db2054.codfw.wmnet[centralauth]> show create table oathauth_users\G
*************************** 1. row ***************************
       Table: oathauth_users
Create Table: CREATE TABLE `oathauth_users` (
  `id` int(11) NOT NULL,
  `TO_DROP_secret` varbinary(255) DEFAULT NULL,
  `TO_DROP_scratch_tokens` varbinary(511) DEFAULT NULL,
  `module` varbinary(255) NOT NULL,
  `data` blob,
  PRIMARY KEY (`id`)
) ENGINE=InnoDB DEFAULT CHARSET=binary ROW_FORMAT=COMPRESSED KEY_BLOCK_SIZE=8
1 row in set (0.04 sec)

Marostegui claimed this task.Jul 1 2019, 5:18 AM

Marostegui triaged this task as Medium priority.

Marostegui moved this task from Pending comment to In progress on the DBA board.

Also changed on an eqiad host (so we can check if there is something reading from those):

root@db1094.eqiad.wmnet[centralauth]> set session sql_log_bin=0;
Query OK, 0 rows affected (0.00 sec)

root@db1094.eqiad.wmnet[centralauth]> alter table oathauth_users change secret TO_DROP_secret varbinary(255) DEFAULT NULL, change scratch_tokens TO_DROP_scratch_tokens varbinary(511) DEFAULT NULL;
Query OK, 0 rows affected (0.54 sec)
Records: 0  Duplicates: 0  Warnings: 0

In T226826#5295046, @Marostegui wrote:

Also changed on an eqiad host (so we can check if there is something reading from those):

root@db1094.eqiad.wmnet[centralauth]> set session sql_log_bin=0;
Query OK, 0 rows affected (0.00 sec)

root@db1094.eqiad.wmnet[centralauth]> alter table oathauth_users change secret TO_DROP_secret varbinary(255) DEFAULT NULL, change scratch_tokens TO_DROP_scratch_tokens varbinary(511) DEFAULT NULL;
Query OK, 0 rows affected (0.54 sec)
Records: 0  Duplicates: 0  Warnings: 0

Thanks!

There might be some stuff doing SELECT * (I'd have to double check the code, though that might be only the migration/back compat code I've seen), but there shouldn't be any explicit usages SELECT secret, scratch_tokens

Cool, I will give it some more 24h - so far nothing on logstash for db1094

Mentioned in SAL (#wikimedia-operations) [2019-07-03T07:06:10Z] <marostegui> Drop secret and scratch_tokens from fishbowl wiki list T226826

Marostegui updated the task description. (Show Details)Jul 3 2019, 7:07 AM

Mentioned in SAL (#wikimedia-operations) [2019-07-03T07:10:44Z] <marostegui> Drop secret and scratch_tokens from labswiki (wikitech) and labstestwiki - T226826

Marostegui updated the task description. (Show Details)Jul 3 2019, 7:11 AM

Mentioned in SAL (#wikimedia-operations) [2019-07-03T10:00:03Z] <marostegui> Drop secret and stratch_tokens columns from the private wiki list T226826

Marostegui updated the task description. (Show Details)Jul 3 2019, 10:07 AM

Mentioned in SAL (#wikimedia-operations) [2019-07-03T12:55:48Z] <marostegui> Drop secret and stratch_tokens columns from centralauth (s7) T226826