Skip to content

ci: remove bot account from default queue - #6591

Merged
nixpanic merged 1 commit into
ceph:develfrom
black-dragon74:feat/mergify-rm-bot-default
Sep 28, 2026
Merged

nixpanic merged 1 commit into
ceph:develfrom
black-dragon74:feat/mergify-rm-bot-default

Conversation

@black-dragon74

Copy link
Copy Markdown
Member

Describe what this PR does

This PR removes the bot account from default queue in mergify rules.

Signed-off-by: Niraj Yadav <niryadav@redhat.com>
@black-dragon74
black-dragon74 requested review from a team as code owners September 28, 2026 08:44
@mergify mergify Bot added the component/testing Additional test cases or CI work label Sep 28, 2026
@mergify

mergify Bot commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 All 2 merge protections satisfied — ready to merge.

Show 2 satisfied protections

🟢 📃 Configuration Change Requirements

Mergify configuration change

  • any of:
    • check-success = @mergify/Configuration changed
    • check-success = @mergify/Configuration has been deleted

🟢 🚦 Auto-queue

When all merge protections are satisfied, this pull request will be queued automatically.

@black-dragon74 black-dragon74 added the ci/skip/e2e skip running e2e CI jobs label Sep 28, 2026
@nixpanic

Copy link
Copy Markdown
Member

/queue

@mergify

mergify Bot commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again.

@ceph-csi-bot ceph-csi-bot added the ok-to-test Label to trigger E2E tests label Sep 28, 2026
@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/mini-e2e/k8s-1.37/nfs

@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.35

@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/mini-e2e/k8s-1.35

@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.34

@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.36

@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/mini-e2e/k8s-1.34

@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot

Copy link
Copy Markdown
Collaborator

/test ci/centos/mini-e2e/k8s-1.36

@ceph-csi-bot ceph-csi-bot added ci/in-progress/e2e This label acts like a guard and prevents Mergify from adding the `ok-to-test` label again. and removed ok-to-test Label to trigger E2E tests labels Sep 28, 2026
@mergify mergify Bot removed the ci/in-progress/e2e This label acts like a guard and prevents Mergify from adding the `ok-to-test` label again. label Sep 28, 2026
@mergify

mergify Bot commented Sep 28, 2026

Copy link
Copy Markdown
Contributor

queue

🛑 This pull request comes from a fork and cannot be rebased on behalf of a user

Details

GitHub refuses an OAuth token on its rebase API for a fork, so rebasing a fork's pull request on behalf of a GitHub user means impersonating that user to force-push the contributor's branch. Mergify does not do that.

Unset update_bot_account on the default queue to keep queueing pull requests from forks: with no account to impersonate, the update rebases through GitHub's API as Mergify, which the installation token is allowed to do. A fork pull request that changes GitHub Actions workflows, or that another GitHub App's bot opened, is the exception: GitHub lets Mergify queue one in place only with an update_bot_account, so a queue that receives those has to run its checks on a draft pull request instead. So is one that changes more than 3000 files: GitHub stops listing a pull request's files there, so Mergify cannot rule out a workflow change and treats it as one. update_method: merge works too, and merges the base branch into the pull request rather than rebasing it.

@nixpanic
nixpanic merged commit a3e42be into ceph:devel Sep 28, 2026
49 of 52 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/skip/e2e skip running e2e CI jobs component/testing Additional test cases or CI work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants