-
Notifications
You must be signed in to change notification settings - Fork 25k
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
Remove DEBUG-level logging from actions in docker #57389
Conversation
In elastic#51459 DEBUG-level logging was removed from the default log4j configuration. However, our docker build has its own log4j configuration which was missed in that change. This commit removes the same from the docker log4j configuration. relates elastic#51459 relates elastic#51198
Pinging @elastic/es-core-infra (:Core/Infra/Packaging) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. I think it is also in the docker/config/oss version of the file too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM2, but echoing Nik's comment that distribution/docker/src/docker/config/oss/log4j2.properties
wants the same treatment.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🤘
@elasticmachine run elasticsearch-ci/packaging-matrix-windows |
In #51459 DEBUG-level logging was removed from the default log4j
configuration. However, our docker build has its own log4j configuration
which was missed in that change. This commit removes the same from the
docker log4j configuration.
relates #51459
relates #51198