search

georchestra / helm-georchestra

geOrchestra helm chart
3 stars 6 forks source link

openldap static password #40

Closed jeanmi151 closed 1 year ago

jeanmi151 commented 1 year ago

https://github.com/georchestra/helm-georchestra/blob/main/templates/ldap/openldap-deployment.yaml#L37 this do not take into account the env var

jeanpommier commented 1 year ago

Good catch @jeanmi151

Additionnally, it would be good to support using secrets

jeanpommier commented 1 year ago

this do not take into account the env var

Actually, rereading the chart, I'd say it should, if you define the env var in extra_environment: it should override the default env var (https://github.com/georchestra/helm-georchestra/blob/main/templates/ldap/openldap-deployment.yaml#L45-L47)

jeanpommier commented 1 year ago

But does not use the values defined in values.yaml indeed