Skip to content

Fix clusterAllReplicas queries with external role when it was dropped #89068

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 10 commits into from
Oct 30, 2025
Merged

Fix clusterAllReplicas queries with external role when it was dropped #89068

merged 10 commits into from
Oct 30, 2025

Conversation

@pufit
Copy link
Member

@pufit pufit commented Oct 27, 2025 •
edited by clickhouse-gh bot
Loading

Changelog category (leave one):

  • Bug Fix (user-visible misbehavior in an official stable release)

Changelog entry (a user-readable short description of the changes that goes into CHANGELOG.md):

Fix ACCESS_ENTITY_NOT_FOUND error when trying to execute clusterAllReplicas from a user with a non-existing role. Resolves #87670

@clickhouse-gh
Copy link

clickhouse-gh bot commented Oct 27, 2025 •
edited
Loading

Workflow [PR], commit [3944d47]

Summary: ✅

@clickhouse-gh clickhouse-gh bot added the pr-bugfix Pull request with bugfix, not backported by default label Oct 27, 2025
@serxa serxa self-assigned this Oct 27, 2025
serxa
serxa approved these changes Oct 27, 2025
View reviewed changes
@pufit pufit added this pull request to the merge queue Oct 30, 2025
Merged via the queue into master with commit 33516ce Oct 30, 2025
124 checks passed
@pufit pufit deleted the pufit/dropped-external-roles-on-cluster branch October 30, 2025 13:22
@robot-ch-test-poll robot-ch-test-poll added the pr-synced-to-cloud The PR is synced to the cloud repo label Oct 30, 2025
@pufit pufit added pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-must-backport-cloud labels Oct 31, 2025
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added pr-backports-created-cloud deprecated label, NOOP pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR labels Oct 31, 2025
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 mentioned this pull request Oct 31, 2025
Merged
robot-clickhouse-ci-2 added a commit that referenced this pull request Oct 31, 2025
@robot-clickhouse-ci-2
Merge pull request #89287 from ClickHouse/cherrypick/25.3/89068
ca92478 
Cherry pick #89068 to 25.3: Fix `clusterAllReplicas` queries with external role when it was dropped
robot-clickhouse added a commit that referenced this pull request Oct 31, 2025
@robot-clickhouse
Backport #89068 to 25.3: Fix clusterAllReplicas queries with extern…
bd8d561 
…al role when it was dropped
This was referenced Oct 31, 2025
Merged
Merged
robot-clickhouse-ci-2 added a commit that referenced this pull request Oct 31, 2025
@robot-clickhouse-ci-2
Merge pull request #89289 from ClickHouse/cherrypick/25.8/89068
b090970 
Cherry pick #89068 to 25.8: Fix `clusterAllReplicas` queries with external role when it was dropped
robot-clickhouse added a commit that referenced this pull request Oct 31, 2025
@robot-clickhouse
Backport #89068 to 25.8: Fix clusterAllReplicas queries with extern…
870fd2b 
…al role when it was dropped
This was referenced Oct 31, 2025
Merged
Merged
robot-clickhouse-ci-2 added a commit that referenced this pull request Oct 31, 2025
@robot-clickhouse-ci-2
Merge pull request #89292 from ClickHouse/cherrypick/25.9/89068
df7c667 
Cherry pick #89068 to 25.9: Fix `clusterAllReplicas` queries with external role when it was dropped
robot-clickhouse added a commit that referenced this pull request Oct 31, 2025
@robot-clickhouse
Backport #89068 to 25.9: Fix clusterAllReplicas queries with extern…
5641953 
…al role when it was dropped
This was referenced Oct 31, 2025
Merged
Merged
robot-clickhouse-ci-2 added a commit that referenced this pull request Oct 31, 2025
@robot-clickhouse-ci-2
Merge pull request #89294 from ClickHouse/cherrypick/25.10/89068
436ab49 
Cherry pick #89068 to 25.10: Fix `clusterAllReplicas` queries with external role when it was dropped
robot-clickhouse added a commit that referenced this pull request Oct 31, 2025
@robot-clickhouse
Backport #89068 to 25.10: Fix clusterAllReplicas queries with exter…
4e8f23e 
…nal role when it was dropped
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 mentioned this pull request Oct 31, 2025
Merged
clickhouse-gh bot added a commit that referenced this pull request Oct 31, 2025
@clickhouse-gh
Merge pull request #89295 from ClickHouse/backport/25.10/89068
1e0f4bc 
Backport #89068 to 25.10: Fix `clusterAllReplicas` queries with external role when it was dropped
clickhouse-gh bot added a commit that referenced this pull request Oct 31, 2025
@clickhouse-gh
Merge pull request #89293 from ClickHouse/backport/25.9/89068
31bf6ba 
Backport #89068 to 25.9: Fix `clusterAllReplicas` queries with external role when it was dropped
@robot-ch-test-poll3 robot-ch-test-poll3 added the pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore label Oct 31, 2025
pufit added a commit that referenced this pull request Nov 7, 2025
@pufit
Merge pull request #89291 from ClickHouse/backport/25.8/89068
cc0e812 
Backport #89068 to 25.8: Fix `clusterAllReplicas` queries with external role when it was dropped
pufit added a commit that referenced this pull request Nov 7, 2025
@pufit
Merge pull request #89288 from ClickHouse/backport/25.3/89068
faf58fd 
Backport #89068 to 25.3: Fix `clusterAllReplicas` queries with external role when it was dropped
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@serxa serxa serxa approved these changes

Assignees

@serxa serxa

Labels

pr-backports-created Backport PRs are successfully created, it won't be processed by CI script anymore pr-backports-created-cloud deprecated label, NOOP pr-bugfix Pull request with bugfix, not backported by default pr-must-backport Pull request should be backported intentionally. Use this label with great care! pr-must-backport-synced The `*-must-backport` labels are synced into the cloud Sync PR pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Code: 492 ACCESS_ENTITY_NOT_FOUND after DROP ROLE; error persists after reconnect and any clusterAllReplicas() query fails

6 participants

@pufit @serxa @robot-ch-test-poll @robot-ch-test-poll3 @robot-clickhouse-ci-1