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

log4j configuration for slow log files(search and index) is removed in ES 8.0 ? #84405

Closed
amitmbm opened this issue Feb 28, 2022 · 4 comments
Closed
Assignees
Labels
>bug :Core/Infra/Logging Log management and logging utilities Team:Core/Infra Meta label for core/infra team

Comments

@amitmbm
Copy link
Contributor

amitmbm commented Feb 28, 2022

Elasticsearch Version

8.0

Installed Plugins

None

Java Version

bundled

OS Version

MacOS

Problem Description

We are using the text format not the JSON format to log our search and indexing slow logs, but I noticed that in ES 8.0 I am getting these logs only in JSON format, I researched more around the changes and found logging configuration for the text bases logging was removed as part of this PR but in its related issue it wasn't mentioned that these changes are meant to remove the support of text-based logging of slow search/index ?
I also tried to find out other official documentation like the release notes and all release notes of 8 but couldn't find the reasoning to remove them.

Steps to Reproduce

  1. Define the search slow log settings of any index created in ES 8.0
  2. Go to logs folder and you will notice only the JSON search slow log file is generated not the text one.

Logs (if relevant)

No response

@amitmbm amitmbm added >bug needs:triage Requires assignment of a team area label labels Feb 28, 2022
@amitmbm
Copy link
Contributor Author

amitmbm commented Feb 28, 2022

It was removed in 8.0 and added a note in the https://www.elastic.co/guide/en/elasticsearch/reference/8.0/migrating-8.0.html but missing in the release notes?
Shouldn't we add this in the release docs as well? Let me know if we should add that and I would be happy to open a PR for it.

@amitmbm
Copy link
Contributor Author

amitmbm commented Feb 28, 2022

@pgomulka could you please take a look at it?

@nik9000 nik9000 added :Core/Infra/Logging Log management and logging utilities team-discuss and removed needs:triage Requires assignment of a team area label labels Mar 1, 2022
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label Mar 1, 2022
@elasticmachine
Copy link
Collaborator

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

@pgomulka pgomulka self-assigned this Mar 21, 2022
@pgomulka
Copy link
Contributor

hi @amitmbm
you can find the release notes mentioning about the change of JSON format to comply with ECS here
https://www.elastic.co/guide/en/elasticsearch/reference/current/release-notes-8.0.0-alpha1.html

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
Projects
None yet
Development

No branches or pull requests

4 participants