Change Details

https://quarry.wmcloud.org/query/68437 Namespaces 0 (main), 2 (user), and 118 (draft) are expected to be in this table as these are namespaces PageTriage patrols. However there are a bunch of other namespaces in this table. Is this a bug? If so, let's track it down and fix it. {F35662310} * Fix the root cause of the bugs # [ ] Hooks.php -> onPageMoveComplete, the last conditional should check if page is being moved out of a namespace PageTriage patrols. If so, run `$pageTriage->deleteFromPageTriage()` # [ ] includes/Maintenance/RemoveOldRows.php should delete from all namespaces after 30 days but doesn't. The bug is on line 72, where the `$secondaryNamespaces` variable is declared. Despite a comment stating otherwise, it contains an incomplete list of namespaces. * Do cleanup # [ ] change maintenance/cleanupPageTriage.php to keep namespaces 0 (main) and 128 (draft), and delete the other namespaces # [ ] Ask DBAs to do a one time deletion of these entries, probably by running maintenance/cleanupPageTriage.php [ ] Keep an eye on this / query this after the DBA deletion. If it starts swelling again, track down the offending code and fix it. Rinse and repeat.

https://quarry.wmcloud.org/query/68437 Namespaces 0 (main), 2 (user), and 118 (draft) are expected to be in this table as these are namespaces PageTriage patrols. However there are a bunch of other namespaces in this table. Is this a bug? If so, let's track it down and fix it. {F35662310} * Fix the root cause of the bugs # [ ] Hooks.php -> onPageMoveComplete, the last conditional should check if page is being moved out of a namespace PageTriage patrols. If so, run `$pageTriage->deleteFromPageTriage()` # [ ] includes/Maintenance/RemoveOldRows.php should delete from all namespaces after 30 days but doesn't. The bug is on line 72, where the `$secondaryNamespaces` variable is declared. Despite a comment stating otherwise, it contains an incomplete list of namespaces. * [ ] Keep an eye on this / query this after the two patches are deployed and after 30 days elapse (giving the cron job time to delete the old entries). If it starts swelling again, track down the offending code and fix it. Rinse and repeat.

https://quarry.wmcloud.org/query/68437 Namespaces 0 (main), 2 (user), and 118 (draft) are expected to be in this table as these are namespaces PageTriage patrols. However there are a bunch of other namespaces in this table. Is this a bug? If so, let's track it down and fix it. {F35662310} * Fix the root cause of the bugs # [ ] Hooks.php -> onPageMoveComplete, the last conditional should check if page is being moved out of a namespace PageTriage patrols. If so, run `$pageTriage->deleteFromPageTriage()` # [ ] includes/Maintenance/RemoveOldRows.php should delete from all namespaces after 30 days but doesn't. The bug is on line 72, where the `$secondaryNamespaces` variable is declared. Despite a comment stating otherwise, it contains an incomplete list of namespaces. * Do cleanup # [ ] change maintenance/cleanupPageTriage.php to keep namespaces 0 (main) and 128 (draft), and delete the other namespaces # [ ] Ask DBAs to do a one time deletion of these entries, probably by running maintenance/cleanupPageTriage.php [ ] Keep an eye on this / query this after the DBA deletion[ ] Keep an eye on this / query this after the two patches are deployed and after 30 days elapse (giving the cron job time to delete the old entries). If it starts swelling again, track down the offending code and fix it. Rinse and repeat.