jp-gouin / helm-openldap

Helm chart of Openldap in High availability with multi-master replication and PhpLdapAdmin and Ltb-Passwd
Apache License 2.0
183 stars 115 forks source link

serviceAccount configuration not working. #81

Closed Jennas-Lee closed 1 year ago

Jennas-Lee commented 1 year ago

Describe the bug serviceAccount.name configuration is not working. It always set to default sa. This is my values.yaml.

serviceAccount:
  create: true
  name: "ldap-sa"

To Reproduce Steps to reproduce the behavior:

  1. Deploy using helm
  2. See StatefulSet information using kubectl describe statefulset openldap

Additional context ServiceAccount configuration feature has been added at https://github.com/jp-gouin/helm-openldap/commit/7b9c55fa9b313e2c3b699ce484164bd8472274d5. But why are these in the comment? Is there a special reason?

https://github.com/jp-gouin/helm-openldap/blob/43f5e39edb1069186197297af5aa92ddc0f4368d/templates/statefullset.yaml#L75-L77

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.