Skip to content

Conversation

@pugnascotia
Copy link
Contributor

PR #61474 reworked deprecation logging to rely more heavily on log4j. Unfortunately, the changes required to log4j's configuration were not applied to the version we ship with the Docker image.

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Logging)

@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Sep 2, 2020
Copy link
Contributor

@pgomulka pgomulka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pugnascotia pugnascotia merged commit 90d520c into elastic:master Sep 2, 2020
@pugnascotia pugnascotia deleted the deprecation-logging-fix-docker-properties branch September 2, 2020 11:02
pugnascotia added a commit that referenced this pull request Sep 2, 2020
PR #61474 reworked deprecation logging to rely more heavily on log4j. Unfortunately,
the changes required to log4j's configuration were not applied to the version we ship
with the Docker image.
@pugnascotia
Copy link
Contributor Author

Backported to 7.x in a8bbdd9

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Core/Infra/Logging Log management and logging utilities >non-issue Team:Core/Infra Meta label for core/infra team v7.10.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants