Page MenuHomePhabricator

Write to agent_id columns in the CheckUserInsert service when migration configuration allows it
Closed, ResolvedPublic

Description

The CheckUserInsert service needs to support writing to the cuc_agent_id, cule_agent_id, and cupe_agent_id columns if the schema migration configuration value added in T361173: Add schema migration config for cu_useragent table indicates to write to these columns.

This will handle supporting not writing to the cuc_agent, cule_agent, and cupe_agent columns.

Acceptance criteria
  • The CheckUserInsert service should support writing to the agent_id columns if SCHEMA_COMPAT_WRITE_NEW is set
  • The CheckUserInsert service should only write to the agent columns if SCHEMA_COMPAT_WRITE_OLD is set

Related Objects

StatusSubtypeAssignedTask
Resolved TBolliger
OpenNone
OpenNone
OpenNone
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedDreamy_Jazz
ResolvedMarostegui
DeclinedNone
ResolvedPapaul
Resolvedtaavi
ResolvedMarostegui
ResolvedSnwachukwu

Event Timeline

Change #1219815 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] [WIP] Write to the new cu_useragent table depending on migration config

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

Change #1219815 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Write to cu_useragent table if configured to do so

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

Change #1223186 had a related patch set uploaded (by Dreamy Jazz; author: Dreamy Jazz):

[mediawiki/extensions/CheckUser@master] Support not writing old for user agent table migration

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

Change #1223186 merged by jenkins-bot:

[mediawiki/extensions/CheckUser@master] Support not writing old for user agent table migration

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