Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

confusing values entry: logLevel #216

Open
ShamrockOo4tune opened this issue Nov 1, 2024 · 2 comments
Open

confusing values entry: logLevel #216

ShamrockOo4tune opened this issue Nov 1, 2024 · 2 comments

Comments

@ShamrockOo4tune
Copy link

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"
...
  1. Go execute any query
  2. 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
Copy link
Owner

jp-gouin commented Nov 4, 2024

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
Copy link
Author

Hi,

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants