mergify icon indicating copy to clipboard operation
mergify copied to clipboard

Published 20 hours ago • verified publisher icon Mergifyio

Can't embark on merge train: `Unable to update: user X is unknown.`

Open JohanLorenzo opened this issue 1 year ago • 8 comments

Expected Behavior

Github users can use a Github label to get their PR embarked on a merge train.

Actual Behavior

Here's the error message we currently get:

This pull request cannot be embarked for merge

The merge queue pull request can't be updated Details:

Unable to update: user MozillaNoah is unknown.

Please make sure MozillaNoah has logged in Mergify dashboard.

Source: https://github.com/mozilla-mobile/firefox-android/pull/1217/checks?check_run_id=11997595835

Steps to Reproduce the Problem

  1. Have a user part of our Github organization mark a PR ready to embark on a merge train
  2. Wait for CI to complete
  3. When CI is ready, PR gets the error message above ⬆️

I'm not 100% sure but this looks like a recent regression. No one on the team has ever got this problem.

cc @rvandermeulen

Specifications

  • Pull Request URLs:
    • https://github.com/mozilla-mobile/firefox-android/pull/1217
    • https://github.com/mozilla-mobile/firefox-android/pull/1234
    • we had to manually rebase the following:
      • https://github.com/mozilla-mobile/firefox-android/pull/1204
      • https://github.com/mozilla-mobile/firefox-android/pull/1232
    • this one did work though https://github.com/mozilla-mobile/firefox-android/pull/1228
  • Mergify Config URL: https://github.com/mozilla-mobile/firefox-android/blob/2d0ce92593ecb04e24397c3947d1c8862c76041f/.mergify.yml

JohanLorenzo avatar Mar 14 '23 18:03 JohanLorenzo

This error is related to this change:

https://changelog.mergify.com/changelog/rebasing-pull-requests-with-a-random-user-is-deprecated

sileht avatar Mar 15 '23 07:03 sileht

TIL https://changelog.mergify.com/ 👍 Speaking of which, is there an RSS/Atom feed I can follow?

That sounds good, we'll look into how we can solve this on our side. Thanks a bunch for the quick answer and the pointer 👌

JohanLorenzo avatar Mar 15 '23 09:03 JohanLorenzo

ℹ️ We'll get this resolved on our side in bug 1822491.

JohanLorenzo avatar Mar 15 '23 09:03 JohanLorenzo

@sileht Is there a workaround for this that doesn't require a premium plan?

jcristau avatar Mar 15 '23 09:03 jcristau

TIL https://changelog.mergify.com/ 👍 Speaking of which, is there an RSS/Atom feed I can follow?

@JohanLorenzo no RSS feed for now, sorry 😢

We're working on a fix to make work again the queue action when using rebase. It should land by end of the week.

jd avatar Mar 15 '23 10:03 jd

@jd any update on a fix? Thanks :)

jcristau avatar Mar 21 '23 15:03 jcristau

Is there any update on this, till when it can get fixed. Thanks!

riya-singhal31 avatar Apr 11 '23 07:04 riya-singhal31

@jd, any workaround or suggestions on this one? We are using mergify with cephcsi to auto-merge PR's, we are currently blocked for PR merging. One example instance is here https://github.com/ceph/ceph-csi/pull/3738/checks?check_run_id=12653144572

Madhu-1 avatar Apr 12 '23 14:04 Madhu-1