Skip to content
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

Change deprecation.elasticsearch keyword to elasticsearch.deprecation #117933

Merged
merged 10 commits into from
Dec 4, 2024

Conversation

JVerwolf
Copy link
Contributor

@JVerwolf JVerwolf commented Dec 3, 2024

This PR changes the values of the "data_stream.dataset" and "event.dataset" fields to be "elasticsearch.deprecation" instead of "deprecation.elasticsearch".

Closes #83251

@JVerwolf JVerwolf added >bug :Core/Infra/Logging Log management and logging utilities labels Dec 3, 2024
@JVerwolf JVerwolf requested review from a team as code owners December 3, 2024 21:32
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team v9.0.0 labels Dec 3, 2024
@elasticsearchmachine
Copy link
Collaborator

Hi @JVerwolf, I've created a changelog YAML for you.

Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

Looks good. We just need to add some breaking change explanation.

distribution/src/config/log4j2.properties Show resolved Hide resolved
pr: 117933
summary: Change `deprecation.elasticsearch` keyword to `elasticsearch.deprecation`
area: Infra/Logging
type: bug
Copy link
Member

Choose a reason for hiding this comment

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

We should note this as a breaking change, since some users may be relying on the ECS dataset.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @rjernst - I've updated it now, WDYT?

@JVerwolf JVerwolf requested a review from rjernst December 3, 2024 22:49
Copy link
Member

@rjernst rjernst left a comment

Choose a reason for hiding this comment

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

LGTM

@elasticsearchmachine
Copy link
Collaborator

Hi @JVerwolf, I've updated the changelog YAML for you.

@JVerwolf JVerwolf merged commit b716a53 into elastic:main Dec 4, 2024
16 checks passed
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 Team:Core/Infra Meta label for core/infra team v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inconsistent value of event.dataset in ES deprecation logs
3 participants