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

Add kernel.failure_mode config option #4516

Merged
merged 1 commit into from
Jun 19, 2017

Conversation

andrewkroh
Copy link
Member

This make the kernel's failure mode configurable from Metricbeat (see the docs changes more details).

Another change was made to disable kernel auditing when Metricbeat stops since it will no longer be receiving messages.

This PR includes changes from elastic/go-libaudit#8.

This make the kernel's failure mode configurable from Metricbeat (see the docs changes more details).

Another change was made to disable kernel auditing when Metricbeat stops since it will no longer be receiving messages.
@andrewkroh andrewkroh force-pushed the feature/mb/kernel-failure-mode branch from 986c747 to 653cc0c Compare June 16, 2017 20:58
options are `silent`, `log`, or `panic`. `silent` basically makes the kernel
ignore the errors, `log` makes the kernel write the audit messages using
`printk` so they show up in system's syslog, and `panic` causes the kernel to
panic to prevent use of the machine. Metricbeat's default is `silent`.
Copy link
Contributor

Choose a reason for hiding this comment

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

By this description sounds like log would be a good choice for the default as well? Or do you think it might spam the kernel logs?

Copy link
Member Author

Choose a reason for hiding this comment

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

The spam to the kernel logs was my main motivation for choosing silent over log for the default value.

Copy link
Contributor

@tsg tsg left a comment

Choose a reason for hiding this comment

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

LGTM.

@tsg tsg merged commit d561c30 into elastic:master Jun 19, 2017
@andrewkroh andrewkroh deleted the feature/mb/kernel-failure-mode branch July 5, 2017 22:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants