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

[rabbitmq] Option to enable SSL for amqp server #7492

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fwiesel
Copy link
Member

@fwiesel fwiesel commented Nov 29, 2024

The option enableSsl configures a certificate with the same name as
the rabbitmq with the prefix tls-, which will in turn generate a
secret of the same name.

The only other missing value is the name of the cluster-issuer to
be specified with certificate.issuerRef.name

Erlang should automatically reload the certificate, when the file
changes. So, certificate rotation should also be handled.

@fwiesel fwiesel force-pushed the rabbitmq_certs branch 2 times, most recently from 859994a to 841c29f Compare November 29, 2024 11:55
@s10 s10 self-requested a review November 29, 2024 12:13
common/rabbitmq/templates/deployment.yaml Outdated Show resolved Hide resolved
common/rabbitmq/templates/statefulset.yaml Outdated Show resolved Hide resolved
common/rabbitmq/templates/certificate.yaml Show resolved Hide resolved
@fwiesel fwiesel force-pushed the rabbitmq_certs branch 3 times, most recently from f2486c8 to ca75b8a Compare December 9, 2024 09:12
The option `enableSsl` configures a certificate with the same name as
the rabbitmq with the prefix `tls-`, which will in turn generate a
secret of the same name.

The only other missing value is the name of the cluster-issuer to
be specified with `certificate.issuerRef.name`

Erlang should automatically reload the certificate, when the file
changes. So, certificate rotation should also be handled.
@sapcc-bot
Copy link
Contributor

Failed to validate the helm chart. Details. Readme.

@fwiesel fwiesel requested a review from joker-at-work December 9, 2024 09:12
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.

3 participants