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

Follow up metrics renames in monitoring events && add debug messages #75

Merged
merged 1 commit into from
Mar 14, 2018

Conversation

kvch
Copy link
Contributor

@kvch kvch commented Mar 14, 2018

A few metrics were renamed in elastic/beats#6449
This PR follows up the changes in counter names.

I also added debug "messages", so it's visible if something is missing from the event.

@andrewkroh Is it possible to run this branch in Jenkins before merging it?

@kvch kvch added the review label Mar 14, 2018
@andrewkroh
Copy link
Member

No, we don't have a pull request builder for this project yet.

Copy link
Member

@andrewkroh andrewkroh left a comment

Choose a reason for hiding this comment

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

LGTM

@andrewkroh andrewkroh merged commit 1219085 into elastic:master Mar 14, 2018
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