Change asset filter from platform to platform_family.

itachi17 commented 4 years ago

This filter will fail on centos & rhel, because:

entity.system.platform on centos does not returns rhel, but centos. entity.system.platform_family returns rhel. Both on rhel & centos, so it's more beneficial.

Pull Request Checklist

Is this in reference to an existing issue?

General

[x] Update Changelog following the conventions laid out here
[ ] Update README with any necessary configuration snippets
[ ] Binstubs are created if needed
[ ] 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 Compatibility Issues

bradpoulton commented 4 years ago

Bump, I ran into this too!

majormoses commented 4 years ago

Sorry for the long silence here, I have resolved the conflicts, let me take a look. Assuming CI passes :+1:

majormoses commented 4 years ago

Released in https://github.com/sensu-plugins/sensu-plugins-process-checks/releases/tag/4.2.1

sensu-plugins / sensu-plugins-process-checks