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
203 stars 121 forks source link

confusing values entry: logLevel #216

Open ShamrockOo4tune opened 6 days ago

ShamrockOo4tune commented 6 days ago

Describe the bug values entry: logLevel, specifically switching it to debug` does not have any effect and brings confusion to the chart users

To Reproduce Steps to reproduce the behavior:

  1. Use following values :

    
...
logLevel: debug
...
env:
# LDAP_LOGLEVEL: "256"
...
  2. Go execute any query
  3. See logs

Expected behavior Expected effect: same as when changing log level with LDAP_LOGLEVEL environment variable

Additional context looks like e648cf3 has removed this functionality

jp-gouin commented 3 days ago

Hi,

Yeah I agree it's confusing and it's from the old osixia image and is not available now with the Bitnami one.

Would you prefer to have the field remove or still have it instead of the env var ? (And how much configuration room ?)

ShamrockOo4tune commented 3 days ago

Hi,

Removing of logLevel would be sufficient to avoid confusion. Log verbosity still can be adjusted in env section