gortiz opened a new pull request, #9015:
URL: https://github.com/apache/pinot/pull/9015
Issue #8944 was divided in two different parts:
- Establishing k8s resources
- Be able to read logs with `kubectl logs`.
PR #9012 solves the former while this PR solves the later.
The change is quite simple:
- Each pod type (servers, controllers, etc) uses different `log4j2.xml`
files that are included in the docker image. These files have been changed to
read a new env variable called `LOG4J_CONSOLE_LEVEL`. That variable is used to
determine the minimum level that will be redirected to stdout.
- In case the variable is not found, `warn` will be used. That was the
previous value, so if the variable is not specified the behavior will not
change.
- The chart has been modified to set `LOG4J_CONSOLE_LEVEL` to `info` by
default. By doing that when the chart is installed without modifying that env
variable error, warn and info logs will be printed in the stdout, so they will
be shown by `kubectl logs`.
Rino Kadijk asked me to add the ability to log the info as json. To do so I
would need to add a new dependency, so I'm going to do it on another PR in case
we don't want to add dependencies.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
