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_vmstat input plugins #1165

Closed
wants to merge 4 commits into from

Conversation

robinpercy
Copy link

Input that gathers statistics from /proc/vmstat

@robinpercy-xm robinpercy-xm force-pushed the pr/input-kernel_vmstat branch 2 times, most recently from 1b14ab6 to 44b2297 Compare May 8, 2016 23:33
@sparrc
Copy link
Contributor

sparrc commented May 18, 2016

looks good but same revisions are needed as what I said in #1164

@robinpercy-xm
Copy link
Contributor

Revisions have been made. I didn't add a "notlinux" dummy file to this one, since it's part of the system package. Let me know if that's wrong.

}

func (k *KernelVmstat) SampleConfig() string {
return `[[inputs.kernel_vmstat]]`
Copy link
Contributor

Choose a reason for hiding this comment

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

this should be return ""

@sparrc
Copy link
Contributor

sparrc commented May 26, 2016

sorry for the hassle but please rebase this and #1164 and then I can merge

jshim-xm and others added 4 commits May 28, 2016 15:09
@robinpercy-xm
Copy link
Contributor

No worries. Glad to see so much activity here. Both have been rebased.

@sparrc sparrc closed this in a7dfbce May 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants