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

RGW SSL certificates are not reloaded after renewal/update #14069

Open
briend opened this issue Apr 12, 2024 · 1 comment
Open

RGW SSL certificates are not reloaded after renewal/update #14069

briend opened this issue Apr 12, 2024 · 1 comment
Labels

Comments

@briend
Copy link

briend commented Apr 12, 2024

Rook CephObjectStore CRD seems to have pretty good support for enabling SSL on the RGW pods by providing a sslCertificateRef to a tls Secret:

https://rook.io/docs/rook/v1.13/CRDs/Object-Storage/ceph-object-store-crd/#gateway-settings

We use Cert-Manager, which automatically renews these certs. In our testing it seems to work fine until the certs expire. Restarting an rgw pod fixes it (it begins using the updated cert). The newer Beast rgw does not appear to have any option to reload the certs without a restart (even though the cert is updated inside the pod via the Secret). Previously, civetweb rgw did have an option to handle short-lived ssl certs.

I've filed a ceph issue here: https://tracker.ceph.com/issues/65470

I'm wondering if Rook should be monitoring the certificate and automatically rolling the RGW deployment when changes occur on the Secret. Another idea might be a Liveness Probe that checks the cert validity and then restarts the rgw (with a long random delay?)

Perhaps I'm missing something else and there is another solution already?

@briend briend added the bug label Apr 12, 2024
@BlaineEXE
Copy link
Member

Let's see what the Ceph project plans to do with the issue there. It seems to me that the RGW itself should reload when the file changes without need to do anything special in Rook.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants