search

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

I hope to customize the openldap administrator user name when deploying #122

Closed opencmit2 closed 9 months ago

opencmit2 commented 9 months ago

The default administrator user password of the openldap that is deployed through this chart package is admin. Our company requires that it be customized and we cannot use a fixed name.

jp-gouin commented 9 months ago

Hi @opencmit2 ,

Yes you should be able to set the admin name using env. LDAP_ADMIN_USERNAME variable Unfortunately by setting it to something else that admin replication will fail.

You can submit a PR to support the configuration of the admin username if you feeling it

Edit : just noticed you submitted a PR already ! Will take a look thanks

opencmit2 commented 9 months ago

Hi @jp-gouin

I have tested and confirmed that there is no problem. I hope you can merge it as soon as possible after testing.

jp-gouin commented 9 months ago

@opencmit2 thanks for the contribution, can you sync your PR with the main branch -> the pipeline has been fixed and merged

jp-gouin commented 9 months ago

Closed PR #123