Skip to content

Align JSON logs better with ECS#67898

Merged
pugnascotia merged 2 commits intoelastic:7.xfrom
pugnascotia:deperecation-logging-ecs-fixes-7x
Jan 26, 2021
Merged

Align JSON logs better with ECS#67898
pugnascotia merged 2 commits intoelastic:7.xfrom
pugnascotia:deperecation-logging-ecs-fixes-7x

Conversation

@pugnascotia
Copy link
Copy Markdown
Contributor

Backport / reimplementation of #67266.

The JSON logs that Elasticsearch produces are roughly in an ECS shape. This PR improves
that alignment. Since 7.x has it's own version of EcsJsonLayout, most of the changes are there, though I also had to backport ClusterIdConverter and NodeIdConverter.

The JSON logs that Elasticsearch produces are roughly in an ECS shape. This PR improves
that alignment.
@pugnascotia pugnascotia requested a review from pgomulka January 25, 2021 12:26
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Jan 25, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

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

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 7e093bb into elastic:7.x Jan 26, 2021
@pugnascotia pugnascotia deleted the deperecation-logging-ecs-fixes-7x branch January 26, 2021 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport :Core/Infra/Logging Log management and logging utilities >enhancement Team:Core/Infra Meta label for core/infra team v7.12.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants