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
Allow to give admin password via secret #1274
Conversation
@zyyw , @MinerYang can you take a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the review!
I solved readme issue and tried to answer on the secret part.
Instead of setting the admin password via a parameter in the helm chart, provide the ability to give it via an existing secret. Signed-off-by: Sylvain Desbureaux <sylvain.desbureaux@orange.com>
764e32f
to
18399c7
Compare
@pitwegner any movement on this? We would really like this feature as well. |
@rdelpret I'm not related to the goharbor project at all, I just added my 2 cents to the PR because I'm interested in the feature as well |
@MinerYang @zyyw Any movement on this? also looking for any help with maintaining this chart? |
closing it and then reopen it to run the ci |
closing and reopen this PR to re-run the ci. |
Instead of setting the admin password via a parameter in the helm chart,
provide the ability to give it via an existing secret.