Skip to content

Definition export: inject default queue type into virtual host metadata (backport #12821) #12839

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 11 commits into from
Nov 27, 2024
Merged

Definition export: inject default queue type into virtual host metadata (backport #12821) #12839

merged 11 commits into from
Nov 27, 2024

Conversation

@mergify
Copy link

@mergify mergify bot commented Nov 27, 2024

With this change, GET /api/definitions and GET /api/vhosts will inject a default queue type (DQT) into virtual host metadata when default_queue_type is set in rabbitmq.conf (v4.0.x-only) and the virtual host in question does not have DQT defined for it:

default_queue_type = quorum

Closes #12776.
Closes #12835.

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

michaelklishin and others added 8 commits November 27, 2024 19:55
@michaelklishin @mergify
Inject DQT into GET /api/definitions and /api/vhosts
6746fbf 
References #12776

(cherry picked from commit 51e6004)
@michaelklishin @mergify
Adapt a rabbit_mgmt_http_SUITE test to account for DQT
be2a712 
(cherry picked from commit 765f3d7)
@michaelklishin @mergify
HTTP API tests for injected default queue type
09ca903 
(cherry picked from commit 090d118)
@michaelklishin @mergify
Explain why list_vhosts does not list metadata such as DQT by default
8c0e23b 
(cherry picked from commit 3e9c6a9)
@michaelklishin @mergify
Rework virtual host-specific definition export
204ac01 
1. Add more metadata, see #12835
2. Include DQT

Closes #12835

(cherry picked from commit 48a1660)
@michaelklishin @mergify
Single virtual host definition export: export the entire metadata map
f5e7dd6 
(cherry picked from commit dd408a4)
@michaelklishin @mergify
Inject DQT into 'ctl export_definitions'
29fc39f 
(cherry picked from commit 8a5ea76)
@michaelklishin @mergify
Cosmetics
fb44100 
(cherry picked from commit f377e8f)
@michaelklishin michaelklishin added this to the 4.0.5 milestone Nov 27, 2024
@mergify mergify bot added the bazel label Nov 27, 2024
@michaelklishin michaelklishin merged commit 6cb9737 into v4.0.x Nov 27, 2024
193 of 194 checks passed
@michaelklishin michaelklishin deleted the mergify/bp/v4.0.x/pr-12821 branch November 27, 2024 22:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@michaelklishin michaelklishin

Labels

None yet

Projects

None yet

Milestone

4.0.5

Development

Successfully merging this pull request may close these issues.

2 participants

@michaelklishin