8000 Improve OpenVZ.pm by ChrisLundquist · Pull Request #264 · collectd/collectd · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Improve OpenVZ.pm #264

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 9 commits into from
Jul 29, 2014
Merged

Improve OpenVZ.pm #264

merged 9 commits into from
Jul 29, 2014

Conversation

ChrisLundquist
Copy link
Contributor
  • Output now matches the regular interface plugin
    • before it was if_octets -> lo -> tx, now it is lo -> if_octets -> tx
  • Disables CPU plugin by default, as all guests would report the same info give or take a few milliseconds
    • with 32 core machines with 8 guests, that was a lot of duplicate data

Sample Graphite output can be seen here,
http://snag.gy/CXyUJ.jpg

Target was:
render?from=-1h&target=test-vps_chef_blueboxgrid_com.interface.venet0.*.*&width=1200&height=800

@pyr
Copy link
Member
pyr commented Jul 29, 2014

This is sensible, thanks @ChrisLundquist !

pyr added a commit that referenced this pull request Jul 29, 2014
@pyr
Merge pull request #264 from ChrisLundquist/cl/perl-openvz
84dd85d 
Improve OpenVZ.pm
@pyr pyr merged commit 84dd85d into collectd:master Jul 29, 2014
@ChrisLundquist ChrisLundquist deleted the cl/perl-openvz branch July 30, 2014 00:24
@ChrisLundquist
Copy link
Contributor Author

I forgot about this. cheers!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Patch
Projects
None yet
Milestone
Features
Development

Successfully merging this pull request may close these issues.
3 participants
@ChrisLundquist @pyr @mfournier
0