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

Log total non-zero expvars on shutdown #2349

Merged
merged 1 commit into from
Aug 23, 2016
Merged

Conversation

tsg
Copy link
Contributor

@tsg tsg commented Aug 23, 2016

Replaces the similar functionality from Metricbeat and Winlogbeat,
generalizing it to all Beats. Part of #1931.

@tsg tsg added in progress Pull request is currently in progress. review labels Aug 23, 2016
@ruflin
Copy link
Contributor

ruflin commented Aug 23, 2016

LGTM. Worth a CHANGELOG entry

@tsg tsg force-pushed the log_expvars_at_end branch from bf26ac4 to 5d522df Compare August 23, 2016 11:39
@tsg tsg removed the in progress Pull request is currently in progress. label Aug 23, 2016
@tsg
Copy link
Contributor Author

tsg commented Aug 23, 2016

@ruflin Added now, along with docs and test updates.

@tsg tsg force-pushed the log_expvars_at_end branch from 5d522df to fa90047 Compare August 23, 2016 11:42
@@ -751,7 +751,8 @@ output.elasticsearch:

# If enabled, filebeat periodically logs its internal metrics that have changed
# in the last period. For each metric that changed, the delta from the value at
# the beginning of the period is logged. The default is true.
# the beginning of the period is logged. Also, the total values for
# all non-zero internal metrics are logged on shutdown.The default is true.
Copy link
Contributor

Choose a reason for hiding this comment

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

Space after . is missing

Replaces the similar functionality from Metricbeat and Winlogbeat,
generalizing it to all Beats. Part of elastic#1931.
@tsg tsg force-pushed the log_expvars_at_end branch from fa90047 to 808f09e Compare August 23, 2016 13:04
@ruflin ruflin merged commit fec9ed1 into elastic:master Aug 23, 2016
@tsg tsg deleted the log_expvars_at_end branch August 25, 2016 10:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants