search

apache / pulsar-helm-chart

Official Apache Pulsar Helm Chart
https://pulsar.apache.org/
Apache License 2.0
209 stars 221 forks source link

Add The Ability To Set Log Level #55

Open toneill818 opened 4 years ago

toneill818 commented 4 years ago

Is your feature request related to a problem? Please describe. Setting PULSAR_LOG_ROOT_LEVEL and PULSAR_ROOT_LOGGER environment variables using configData does not change the log level since it uses the conf/log4j2.yaml .

Describe the solution you'd like Ideally we would be able to only use environment variables, but that will not be possible until pulsar does not depend on the log4j2.yaml file. To get it working in its current state, we could create a ConfigMap with the contents of the logfile, then mount it as a volume and set the PULSAR_LOG_CONF to point to our mounted file instead of conf/log4j2.yaml.

rvashishth commented 4 years ago

until pulsar does not depend on the log4j2.yaml file

would it be good to make a PR in pulsar for this?

pisceslj commented 1 year ago

maybe just set these environment variables in values.yaml broker configData:

configData:
    PULSAR_LOG_LEVEL: "debug"