[Feature request] Define admin username and passowrd using kubernetes secret, not directly in values #169

Closed
opened 2021-06-08 12:00:58 +00:00 by lord-kyron · 5 comments

Hi all,
I think in cases (like mine) when using FluxCD with Helm Operator and my helm values files are sitting in git repo, having the admin username and pass configured there is really insecure. That is why I think that it will be best if you guys make support for admin username and password to come from existing secret in kubernetes.
That way we can use the sealed secrets extension to encode the username and password in the git repo and this chart to take it from the created kubernets secret directly instead of hardcoding them in plain text!

Thanks!

Hi all, I think in cases (like mine) when using FluxCD with Helm Operator and my helm values files are sitting in git repo, having the admin username and pass configured there is really insecure. That is why I think that it will be best if you guys make support for admin username and password to come from existing secret in kubernetes. That way we can use the sealed secrets extension to encode the username and password in the git repo and this chart to take it from the created kubernets secret directly instead of hardcoding them in plain text! Thanks!
Member

True, there was some similiar request to this one. Haven't had time but I might have a nice idea in how to include it. Will try and prepare a PR for this :)

True, there was some similiar request to this one. Haven't had time but I might have a nice idea in how to include it. Will try and prepare a PR for this :)
Member

Done, feel free to test this :) #170

Done, feel free to test this :) #170
Author

Seems working fine! Thank you for the quick reaction!

Seems working fine! Thank you for the quick reaction!
Author

When can we expect this to be merged to master branch?

When can we expect this to be merged to master branch?
lunny closed this issue 2021-06-10 11:13:34 +00:00
Member

@lord-kyron just released 3.1.4 containing the update

@lord-kyron just released 3.1.4 containing the update
Sign in to join this conversation.
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: gitea/helm-chart#169
No description provided.