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 info Metricset to Docker module #2672

Merged
merged 1 commit into from
Oct 4, 2016

Conversation

ruflin
Copy link
Contributor

@ruflin ruflin commented Oct 4, 2016

@ruflin ruflin added review Metricbeat Metricbeat labels Oct 4, 2016
@ruflin
Copy link
Contributor Author

ruflin commented Oct 4, 2016

@douaejeouit FYI: I added this metricset as I think it was missing to get some overview information and it was good to get deeper into the docker module code. It still missed quite a few fields.

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.

@ruflin ruflin mentioned this pull request Oct 4, 2016
19 tasks
@tsg tsg merged commit de270e7 into elastic:master Oct 4, 2016
@douaejeouit
Copy link
Contributor

Ok @ruflin. It sounds good to me!

@monicasarbu monicasarbu deleted the mb-docker-info-metricset branch October 9, 2016 20:28
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.

3 participants