search

Mergifyio / mergify

Mergify Community Issue Tracker
https://mergify.com
Apache License 2.0
318 stars 92 forks source link

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

Open JohanLorenzo opened 1 year ago

JohanLorenzo commented 1 year ago

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

sileht commented 1 year ago

This error is related to this change:

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

JohanLorenzo commented 1 year ago

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 commented 1 year ago

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

jcristau commented 1 year ago

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

jd commented 1 year ago

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.

jcristau commented 1 year ago

@jd any update on a fix? Thanks :)

riya-singhal31 commented 1 year ago

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

Madhu-1 commented 1 year ago

@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