Skip to content

Share TLS config with the Prometheus plugin #479

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
Closed
ferozjilla opened this issue Nov 19, 2020 · 0 comments · Fixed by #533
Closed

Share TLS config with the Prometheus plugin #479

ferozjilla opened this issue Nov 19, 2020 · 0 comments · Fixed by #533
Labels
enhancement New feature or request

Comments

@ferozjilla
Copy link
Contributor

Is your feature request related to a problem? Please describe.
In #267, we shared TLS config provided in the custom resource spec to enable TLS for plugins - management, mqtt, stomp, web_mqtt, and web_stomp. Here, we missed the Prometheus plugin.

Prometheus does support connecting over TLS: https://www.rabbitmq.com/prometheus.html#tls

Describe the solution you'd like
Similar implementation pattern as #267.
@ferozjilla ferozjilla added the enhancement New feature or request label Nov 19, 2020
@ferozjilla ferozjilla changed the title Share TLS config for the Prometheus plugin Share TLS config with the Prometheus plugin Nov 19, 2020
Gsantomaggio added a commit that referenced this issue Dec 23, 2020
@Gsantomaggio
Add support for prometheus TLS
7803fe7 
Fixes #479
@Gsantomaggio Gsantomaggio mentioned this issue Dec 23, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add support for prometheus TLS rabbitmq/cluster-operator
1 participant
@ferozjilla