Skip to content
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.

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

4.1, by @Ayanda-D: stop QQ replicas when a QQ is forced to shrink to a single replica (backport #12468) #12482

Closed
wants to merge 5 commits into from

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Oct 8, 2024

This is #12427 by @Ayanda-D.

Shrinking operations did not stop QQ replicas. This was easy to miss because QQs are usually shrunk before a node is removed from the cluster.

However, there is a scenario where this is not the case. If some nodes (replicas) need to be replaced, in particular when a majority of nodes cannot be recovered for any reasons, the recovery process will involve shrinking a QQ to just one member so that it has an online quorum (of 1 node out of 1) before new replicas can be added.

For this to succeed, the older replicas must be stopped and deleted from the (QQ) cluster.


This is an automatic backport of pull request #12468 done by Mergify.

…r cluster

wide consistency, ensuring only the leader is active/running

(cherry picked from commit b675ce2)
(cherry picked from commit d9de6d9)
(cherry picked from commit 60ee35e)
(cherry picked from commit 10dbde1)

# Conflicts:
#	deps/rabbit/test/quorum_queue_SUITE.erl
(cherry picked from commit c26aa3b)
(cherry picked from commit b03637f)
(cherry picked from commit de0c0db)
(cherry picked from commit c9d97e6)

# Conflicts:
#	deps/rabbit/test/quorum_queue_SUITE.erl
(cherry picked from commit dd5ec3c)
(cherry picked from commit 16170d0)
@mergify mergify bot added the conflicts label Oct 8, 2024
Copy link
Author

mergify bot commented Oct 8, 2024

Cherry-pick of 10dbde1 has failed:

On branch mergify/bp/v3.13.x/pr-12468
Your branch is ahead of 'origin/v3.13.x' by 1 commit.
  (use "git push" to publish your local commits)

You are currently cherry-picking commit 10dbde1f71.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   deps/rabbit/test/quorum_queue_SUITE.erl

no changes added to commit (use "git add" and/or "git commit -a")

Cherry-pick of c9d97e6 has failed:

On branch mergify/bp/v3.13.x/pr-12468
Your branch is ahead of 'origin/v3.13.x' by 3 commits.
  (use "git push" to publish your local commits)

You are currently cherry-picking commit c9d97e61de.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   deps/rabbit/test/quorum_queue_SUITE.erl

no changes added to commit (use "git add" and/or "git commit -a")

To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants