Added `-R` to check-process.rb for RSS checks - Githubissues

sensu-plugins / sensu-plugins-process-checks

This plugin provides native process instrumentation for monitoring and metrics collection, including: process status, uptime, thread count, and others.

http://sensu-plugins.io

MIT License

20 stars 55 forks source link

Added `-R` to check-process.rb for RSS checks #46

Closed jgnagy closed 7 years ago

jgnagy commented 7 years ago

Added -R to check-process.rb to address #45. Also includes some minor rubocop fixes

Pull Request Checklist

Is this in reference to an existing issue? Yes, #45.

General

[x] Update Changelog following the conventions laid out on Keep A Changelog
[ ] Update README with any necessary configuration snippets
[ ] Binstubs are created if needed
[x] RuboCop passes
[x] Existing tests pass

New Plugins

[ ] Tests
[ ] Add the plugin to the README
[ ] Does it have a complete header as outlined here

Purpose

Known Compatablity Issues

majormoses commented 7 years ago

This looks good now, I will take a second pass and hopefully do some testing and release tomorrow.

majormoses commented 7 years ago

closing per: https://github.com/sensu-plugins/sensu-plugins-process-checks/pull/46#discussion_r124585380