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

Provide more detail on log levels #4405

Merged
merged 1 commit into from May 30, 2017

Conversation

Projects
None yet
2 participants
@dedemorton
Copy link
Contributor

dedemorton commented May 26, 2017

This issue adds more detail about log levels as requested in #861.

I'm still being a bit vague in some of the descriptions (especially for debug) because I don't want to expose details about beat processing that might change over time.

Other changes:

  • Had to do a bit of refactoring of anchor IDs to add a link to the topic about command line options.
  • Removed defaults listed in to_file and to_syslog because they seemed to contradict what we say in the intro about the default logging output being platform-specific.
  • Didn't really understand what we meant by the following sentence, so I rewrote it to say what I think we mean: "In addition to the logging system, the logging output configuration can be
    modified from the command line."
@dedemorton

This comment has been minimized.

Copy link
Contributor Author

dedemorton commented May 26, 2017

This should be backported to 5.4.

@tsg

tsg approved these changes May 29, 2017

Copy link
Collaborator

tsg left a comment

LGTM.

@dedemorton dedemorton merged commit 0f2d9a6 into elastic:master May 30, 2017

5 of 6 checks passed

beats-ci Build finished.
Details
CLA Commit author is a member of Elasticsearch
Details
codecov/patch Coverage not affected when comparing 3a58887...5c0e38d
Details
codecov/project 63.14% (-0.03%) compared to 3a58887
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

dedemorton added a commit that referenced this pull request Jun 6, 2017

dedemorton added a commit that referenced this pull request Jun 21, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.