Skip to content

rabbitmq-upgrade await_quorum_plus_one improvements (backport #12113)#12116

Merged
michaelklishin merged 4 commits intov4.0.xfrom
mergify/bp/v4.0.x/pr-12113
Aug 25, 2024
Merged

rabbitmq-upgrade await_quorum_plus_one improvements (backport #12113)#12116
michaelklishin merged 4 commits intov4.0.xfrom
mergify/bp/v4.0.x/pr-12113

Conversation

@mergify
Copy link
Copy Markdown

@mergify mergify Bot commented Aug 25, 2024

  1. If khepri_db is enabled, rabbitmq_metadata is a critical component
  2. When waiting for quorum+1, periodically log what doesn't have the quorum+1
    • for components: just list them
    • for queues: list how many we are waiting for and how to display them (because there could be a large number, logging that could be impractical or even dangerous)
  3. make the tests signficantly faster by using a single group
    This is an automatic backport of pull request rabbitmq-upgrade await_quorum_plus_one improvements #12113 done by Mergify.

mkuratczyk and others added 4 commits August 25, 2024 03:03
1. If khepri_db is enabled, rabbitmq_metadata is a critical component
2. When waiting for quorum+1, periodically log what doesn't have the
   quorum+1
   - for components: just list them
   - for queues: list how many we are waiting for and how to display
     them (because there could be a large number, logging that
     could be impractical or even dangerous)
3. make the tests signficantly faster by using a single group

(cherry picked from commit 6ca2022)
(cherry picked from commit f47daee)
Now the API endpoint can return Khepri as
a "queue" (or "stream") without the necessary
number of replicas online.

So don't expect the list to only have one element.

(cherry picked from commit d7d1397)
@michaelklishin michaelklishin merged commit b5fa75b into v4.0.x Aug 25, 2024
@michaelklishin michaelklishin deleted the mergify/bp/v4.0.x/pr-12113 branch August 25, 2024 03:03
michaelklishin added a commit that referenced this pull request Aug 25, 2024
rabbitmq-upgrade await_quorum_plus_one improvements (backport #12113) (backport #12116)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants