Change the subscriber target to 'client:name' and add more info to log

betorvs commented 7 years ago

line 95: change to use client:name to subscribe the check remediation line 136: change the json to include creator info and reason and use subscriber with []

Pull Request Checklist

Is this in reference to an existing issue? No.

General

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

Purpose

Add more info into log, and change the target subscriber to client:name.

Known Compatibility Issues

Tested on sensu 0.29 or higher.

majormoses commented 7 years ago

Thanks for the contribution!

majormoses commented 7 years ago

Can you please fix the tests?

majormoses commented 7 years ago

released: https://rubygems.org/gems/sensu-plugins-sensu/versions/2.3.0

sensu-plugins / sensu-plugins-sensu