Skip to content

For 4.1.4: rabbitmqctl shovel_status: handle metrics (backport #14278) #14298

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 2 commits into from
Aug 5, 2025
Merged

For 4.1.4: rabbitmqctl shovel_status: handle metrics (backport #14278) #14298

merged 2 commits into from
Aug 5, 2025

Conversation

mergify[bot]
Copy link

@mergify mergify bot commented Jul 28, 2025

The rabbitmqctl command got broken a while back when metrics were added to the status e3430aa
The command stopped reporting any status.

This PR addresses that, and as we have the metrics available, we just include them in the status.

I don't think e3430aa made it into any release, otherwise this PR needs backporting.

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

dcorbacho added 2 commits July 28, 2025 12:44
@dcorbacho @mergify
Rabbitmqctl shovel_status: handle metrics
f92b6e5 
The ctl command got broken when metrics were added to the status.
As we have the metrics now, we just report them.

(cherry picked from commit 1a06bc0)
@dcorbacho @mergify
Shovel: status tests
052085d 
(cherry picked from commit 2a04016)
@michaelklishin michaelklishin added this to the 4.1.4 milestone Jul 28, 2025
@michaelklishin michaelklishin changed the title rabbitmqctl shovel_status: handle metrics (backport #14278) For 4.1.4: rabbitmqctl shovel_status: handle metrics (backport #14278) Jul 28, 2025
@michaelklishin
Copy link
Collaborator

We will merge this PR after 4.1.3 ships, so, it will ship in 4.1.4 some time in Sep 2025.

@michaelklishin michaelklishin merged commit 5de5a18 into v4.1.x Aug 5, 2025
274 checks passed
@michaelklishin michaelklishin deleted the mergify/bp/v4.1.x/pr-14278 branch August 5, 2025 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@dcorbacho dcorbacho

Labels
None yet
Projects
None yet
Milestone
4.1.4
Development

Successfully merging this pull request may close these issues.
2 participants
@michaelklishin @dcorbacho