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

Add support for existing backup credentials #197

Closed
donch opened this issue Feb 28, 2024 · 4 comments · Fixed by #239
Closed

Add support for existing backup credentials #197

donch opened this issue Feb 28, 2024 · 4 comments · Fixed by #239
Labels
chart( cluster ) Related to the cluster chart

Comments

@donch
Copy link

donch commented Feb 28, 2024

Currently, the cluster chart create a secret containing backup credentials like S3.
There is no way to provide a existingSecret to avoid putting these credentials in the Helm Values.

@DASPRiD
Copy link
Contributor

DASPRiD commented Feb 29, 2024

I was just about to set up my first cluster and ran into that problem as well. Since we are using GitOps, it's anything but ideal to store the secret access key straight in Git.

@mrclrchtr
Copy link

Same here. I can not store the secret in the values file.

@SalvoRusso8
Copy link

Hi all, any update about this feature?
I think is very important to permit the use of an existing secret to allow the use of this chart with tools like fluxcd

@itay-grudev
Copy link
Collaborator

@SalvoRusso8 There is PR that implements this: #239. Would you be able to test that and confirm it works?

@itay-grudev itay-grudev added the chart( cluster ) Related to the cluster chart label May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment