sensu-plugins / sensu-plugins-disk-checks

This plugin provides native disk instrumentation for monitoring and metrics collection, including: health, usage, and various metrics.

http://sensu-plugins.io

MIT License

27 stars 63 forks source link

Change platform to platform_family #136

Closed TheMysteriousX closed 4 years ago

TheMysteriousX commented 5 years ago

In the entity info, redhat systems are listed as:

{
  "entity_class": "agent",
  "system": {
    "hostname": "",
    "os": "linux",
    "platform": "redhat",
    "platform_family": "rhel",
    "platform_version": "7.7",
...

In order to filter as expected, platform_family should be used.

Pull Request Checklist

123

General

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

New Plugins

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

Purpose

Bugfix on centos

Known Compatibility Issues

Same change may need to be made on other platforms, but I do not have access to them

cwjohnston commented 4 years ago

Thanks for opening this PR. We needed to make additional changes for older versions of Centos, so this is closed by #141.