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

Use RabbitMQ management image variant #443

Merged
merged 2 commits into from
Nov 4, 2020
Merged

Use RabbitMQ management image variant #443

merged 2 commits into from
Nov 4, 2020

Conversation

Zerpet
Copy link
Collaborator

@Zerpet Zerpet commented Nov 3, 2020

This closes #441

Note to reviewers: remember to look at the commits in this PR and consider if they can be squashed

Summary Of Changes

  • Changed the default RabbitMQ image to use tag 3.8.9-management

Additional Context

The non-management image variant disables metrics collection by the management agent, resulting in very little information
shown in the management UI. We want to always enable the management UI as it is a common entry point to RabbitMQ
administration. It's also not straightforward to re-enable the metric collection in the non-management variant.

Local Testing

  • Made changes to unit tests. Those are passing.
  • No changes to integration or system tests. Those are also green.

The non-managemet variant disabled metric collection in RabbitMQ
configuration and it's not straightforward to re-enable them. The
managmenet image does not disable these metrics and we want to always
enable the management UI.
@Zerpet Zerpet added this to the GA milestone Nov 3, 2020
@Zerpet Zerpet added the enhancement New feature or request label Nov 3, 2020
@Zerpet Zerpet added this to Review in progress in RabbitMQ Cluster Kubernetes Operator via automation Nov 3, 2020
Copy link
Contributor

@ChunyiLyu ChunyiLyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. One small thing: this is something worth mentioning in README and docs? Your call.

Copy link
Member

@ansd ansd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could also modify the example in

@Zerpet
Copy link
Collaborator Author

Zerpet commented Nov 3, 2020

We don't mention the management UI in the Readme or in the docs. In the docs we actually make a vague reference to the "community RabbitMQ image on dockerhub". I don't think we need to be explicit because of this change.

@ansd I blame ag rabbitmq:3.8.9 for comming out empty 😎 Will do this change.

@ablease
Copy link
Contributor

ablease commented Nov 3, 2020

No need to squash IMO. One commit for default image change, another for helm chart.

@Zerpet
Copy link
Collaborator Author

Zerpet commented Nov 4, 2020

Two approvals, checks are green. Merging! 🚀

@Zerpet Zerpet merged commit 7620e52 into main Nov 4, 2020
RabbitMQ Cluster Kubernetes Operator automation moved this from Review in progress to Done Nov 4, 2020
@Zerpet Zerpet deleted the cluster-operator-441 branch November 4, 2020 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

Switch to RabbitMQ management docker image variant
4 participants