Skip to content

Fix deprecation logging config in Docker#61840

Merged
pugnascotia merged 1 commit intoelastic:masterfrom
pugnascotia:deprecation-logging-fix-docker-properties
Sep 2, 2020
Merged

Fix deprecation logging config in Docker#61840
pugnascotia merged 1 commit intoelastic:masterfrom
pugnascotia:deprecation-logging-fix-docker-properties

Conversation

@pugnascotia
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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