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

Rework docs on logging levels #111143

Conversation

DaveCTurner
Copy link
Contributor

Clarify that the default config is the recommended one, and that users
should not normally enable DEBUG or TRACE logging without looking at
the source code. Also reorders the information a bit for easier reading.

Clarify that the default config is the recommended one, and that users
should not normally enable `DEBUG` or `TRACE` logging without looking at
the source code. Also reorders the information a bit for easier reading.
@DaveCTurner DaveCTurner added >docs General docs changes :Core/Infra/Logging Log management and logging utilities labels Jul 22, 2024
Copy link
Contributor

Documentation preview:

@elasticsearchmachine elasticsearchmachine added Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team labels Jul 22, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-docs (Team:Docs)

@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@leemthompo leemthompo left a comment

Choose a reason for hiding this comment

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

Nice! I offered a few edits for readability :)

docs/reference/setup/logging-config.asciidoc Outdated Show resolved Hide resolved
docs/reference/setup/logging-config.asciidoc Show resolved Hide resolved
docs/reference/setup/logging-config.asciidoc Outdated Show resolved Hide resolved
@DaveCTurner DaveCTurner added auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport-and-merge labels Jul 22, 2024
@elasticsearchmachine elasticsearchmachine merged commit c8583cd into elastic:main Jul 22, 2024
5 checks passed
@DaveCTurner DaveCTurner deleted the 2024/07/22/configuring-logging-levels branch July 22, 2024 10:23
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Jul 22, 2024
Clarify that the default config is the recommended one, and that users
should not normally enable `DEBUG` or `TRACE` logging without looking at
the source code. Also reorders the information a bit for easier reading.
DaveCTurner added a commit to DaveCTurner/elasticsearch that referenced this pull request Jul 22, 2024
Clarify that the default config is the recommended one, and that users
should not normally enable `DEBUG` or `TRACE` logging without looking at
the source code. Also reorders the information a bit for easier reading.
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.15
8.14

elasticsearchmachine pushed a commit that referenced this pull request Jul 22, 2024
Clarify that the default config is the recommended one, and that users
should not normally enable `DEBUG` or `TRACE` logging without looking at
the source code. Also reorders the information a bit for easier reading.
elasticsearchmachine pushed a commit that referenced this pull request Jul 22, 2024
Clarify that the default config is the recommended one, and that users
should not normally enable `DEBUG` or `TRACE` logging without looking at
the source code. Also reorders the information a bit for easier reading.
ioanatia pushed a commit to ioanatia/elasticsearch that referenced this pull request Jul 22, 2024
Clarify that the default config is the recommended one, and that users
should not normally enable `DEBUG` or `TRACE` logging without looking at
the source code. Also reorders the information a bit for easier reading.
salvatore-campagna pushed a commit to salvatore-campagna/elasticsearch that referenced this pull request Jul 23, 2024
Clarify that the default config is the recommended one, and that users
should not normally enable `DEBUG` or `TRACE` logging without looking at
the source code. Also reorders the information a bit for easier reading.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Core/Infra/Logging Log management and logging utilities >docs General docs changes Team:Core/Infra Meta label for core/infra team Team:Docs Meta label for docs team v8.14.4 v8.15.1 v8.16.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants