This pull request changes the Sensu API endpoint used by Sensu::Handler.event_exists? from the singular form event to the plural form events.
Motivation and Context
Sensu::Handler's event_exists? method uses the singular form of the events API. The plural form was added in Sensu 0.9, and the singular form was removed in Sensu 0.25.0.
Because the singular form was removed, event_exists? will always return false, effectively breaking the now-deprecated check dependency filtering in this library.
How Has This Been Tested?
It has not been tested.
Types of changes
[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
Checklist:
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have updated the Changelog following the conventions laid out on Keep A Changelog
Known Caveats
Breaks filter_dependencies for versions of Sensu prior to 0.9.
cwjohnston
commented
7 years ago
Description
This pull request changes the Sensu API endpoint used by
Sensu::Handler.event_exists?from the singular formeventto the plural formevents.Motivation and Context
Sensu::Handler's
event_exists?method uses the singular form of theeventsAPI. The plural form was added in Sensu 0.9, and the singular form was removed in Sensu 0.25.0.Because the singular form was removed,
event_exists?will always returnfalse, effectively breaking the now-deprecated check dependency filtering in this library.How Has This Been Tested?
It has not been tested.
Types of changes
Checklist:
Known Caveats
filter_dependenciesfor versions of Sensu prior to 0.9.