EugenMayer / helm-charts

Helm charts for applications no official chart exists for
24 stars 19 forks source link

(Rundeck chart) Feature request: default admin secret #61

Closed callen-rti closed 1 month ago

callen-rti commented 2 months ago

I've been greatly enjoying the Rundeck chart, but the one thing that keeps it from being a simple install is the admin secret.

Currently this has to be created by the user, but I don't see why it couldn't be a "default", with the user/pass being "admin".

Something like this:

apiVersion: v1
kind: Secret
metadata:
  name: user-credentials-secret
type: Opaque
data:
  userCredentials: YWRtaW46YWRtaW4sdXNlcixhZG1pbixhcmNoaXRlY3QsZGVwbG95LGJ1aWxkCg==
EugenMayer commented 2 months ago

i understand your point, but this chart is big enough IMHO. The emphasis on securing your installation and not running it with admin/admin is one thing i love to do in general - or you end up with "i did not know" zombie instances.

If you would like to deploy those things fast, thus parameterizing your chart, use terraform with helm and create a module for this chart (this is how we do it with every sing chart). This way you wrap your common secrets/ingress/storage settings in the module and, if needed, just override some specifics like the domain during the deployment.

So IMHO, i would not like to add this one, sorry.

callen-rti commented 1 month ago

I had not thought of using terraform in that way, that sounds like a better way to do it. Thanks!