Page MenuHomePhabricator

Maintenance_bot not removing patch-for-review when all conditions are seemingly met
Closed, ResolvedPublicBUG REPORT

Description

Lately I'm noticing that @Maintenance_bot seems to have stopped removing the Patch-For-Review tag from lots of task in CampaignEvents. Some example tasks where this happened: T363187, T363135, T363200, T362400, T362895 (super simple example) and that's just a sample from this week.

I checked the original announcement in wikitech-l, which I remembered had a list of scenarios where the bot wouldn't remove the tag, but none of them are met in those tasks AFAICS:

  • The tag is always added by gerrit-bot
  • There are no links to github anywhere
  • All patches have been merged
  • Nothing weird from gerrit-bot

So I'm not sure what's going on. The bot is definitely removing patch-for-review from other tasks (T346190#9747416, T362913#9747076, T353444#9746493), so there seems to be something specific going on here. Note, I didn't check how many tasks are affected overall (i.e., across the whole phabricator), so it's possible that the successful examples linked above are just some rare outliers. I skimmed through the bot source code but nothing obvious stands out there either.

Details

Related Changes in GitLab:
TitleReferenceAuthorSource BranchDest Branch
patchforreview_remover: Add support for gitlab MRs in some reposladsgroup/Phabricator-maintenance-bot!6ladsgroupgitlab_mr_supportmaster
Update regex for gerrit change numbersladsgroup/Phabricator-maintenance-bot!5daimonawork/daimona/T363573master
Customize query in GitLab

Event Timeline

The regex to search for comment does not include # as added by T358940: GerritBot comments for 7-digit Gerrit changes conflict with Diffusion commit hashes for new comments to avoid autolinking of numbers.

Two of your examples are gitlab comments and the last one has a patch set with a comment before T358940 and that gets found by the regex.

Lol, my first conflict in gitlab.

Lol, my first conflict in gitlab.

I'm sorry 😅

We probably should actually fix the gitlab part before running it again

We probably should actually fix the gitlab part before running it again

Indeed, I was just now wondering why it removed patch-for-review from this task.

Ladsgroup claimed this task.
Nemoralis reopened this task as Open.EditedJul 9 2024, 3:25 PM
Nemoralis subscribed.

@Ladsgroup this is happening again (for different projects): T369342, T369485, T369319

Fixed, the docker container went on zombie mode. New changes should get it fixed. I'll run it to backfill old tickets too.