Page MenuHomePhabricator

Drop `afl_ip` as the last step of the migration to `afl_ip_hex`
Closed, ResolvedPublic

Description

From T395612: AbuseFilter abuse_filter_log table: Store IP addresses as hex values.

Pre-reqs:

Acceptance Criteria:

  • The afl_ip column is removed once the pre-reqs are completed
  • Onwiki documentation for the table is updated to reflect the removal of the column
  • A Schema-change-in-production ticket is filed to apply the schema change to WMF production - T407997

Event Timeline

Dreamy_Jazz renamed this task from Deprecate `afl_ip` as the last step of the migration to `afl_ip_hex` to Drop `afl_ip` as the last step of the migration to `afl_ip_hex`.Jul 23 2025, 5:09 PM

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

[mediawiki/extensions/AbuseFilter@master] Drop afl_ip from the abuse_filter_log table

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

The two other acceptance criteria can only be done once the gerrit patch is merged

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

[mediawiki/extensions/AbuseFilter@master] Update PopulateAbuseFilterLogIPHex to support afl_ip being missing

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

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

[mediawiki/extensions/IPInfo@master] Fix abuse_filter_log index in TempUserIPLookup

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

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

[mediawiki/extensions/IPInfo@wmf/1.45.0-wmf.24] Fix abuse_filter_log index in TempUserIPLookup

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

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

[mediawiki/extensions/IPInfo@wmf/1.45.0-wmf.23] Fix abuse_filter_log index in TempUserIPLookup

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

Change #1198017 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@wmf/1.45.0-wmf.24] Fix abuse_filter_log index in TempUserIPLookup

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

Change #1198018 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@wmf/1.45.0-wmf.23] Fix abuse_filter_log index in TempUserIPLookup

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

Mentioned in SAL (#wikimedia-operations) [2025-10-22T10:58:32Z] <dreamyjazz@deploy2002> Started scap sync-world: Backport for [[gerrit:1198017|Fix abuse_filter_log index in TempUserIPLookup (T400280)]], [[gerrit:1198018|Fix abuse_filter_log index in TempUserIPLookup (T400280)]]

Change #1197998 merged by jenkins-bot:

[mediawiki/extensions/IPInfo@master] Fix abuse_filter_log index in TempUserIPLookup

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

Mentioned in SAL (#wikimedia-operations) [2025-10-22T11:03:01Z] <dreamyjazz@deploy2002> dreamyjazz: Backport for [[gerrit:1198017|Fix abuse_filter_log index in TempUserIPLookup (T400280)]], [[gerrit:1198018|Fix abuse_filter_log index in TempUserIPLookup (T400280)]] synced to the testservers (see https://wikitech.wikimedia.org/wiki/Mwdebug). Changes can now be verified there.

Mentioned in SAL (#wikimedia-operations) [2025-10-22T11:08:34Z] <dreamyjazz@deploy2002> Finished scap sync-world: Backport for [[gerrit:1198017|Fix abuse_filter_log index in TempUserIPLookup (T400280)]], [[gerrit:1198018|Fix abuse_filter_log index in TempUserIPLookup (T400280)]] (duration: 10m 01s)

Change #1197721 merged by jenkins-bot:

[mediawiki/extensions/AbuseFilter@master] Update PopulateAbuseFilterLogIPHex to support afl_ip being missing

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

Change #1197716 merged by jenkins-bot:

[mediawiki/extensions/AbuseFilter@master] Drop afl_ip from the abuse_filter_log table

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