search

sensu-plugins / sensu-plugins-opsgenie

Sensu plugins for OpsGenie
http://sensu-plugins.io
MIT License
4 stars 24 forks source link

allow check to override default handler #5

Closed duncanphillips closed 8 years ago

duncanphillips commented 8 years ago

the handler currently doesn't allow checks to override settings when creating an alert which is very limiting. This pr allows tags, recipients and teams to be configured on a per check basis.

tested in an isolated environment.

analytically commented 8 years ago

Thanks for your PR. Can you add an entry to the changelog?

eheydrick commented 8 years ago

In addition to a CHANGELOG entry, can you rebase this PR against master.

analytically commented 8 years ago

I've manually integrated this.

duncanphillips commented 8 years ago

sorry about the delay, thanks for merging.

On Thu, Nov 26, 2015 at 3:11 PM, Mathias Bogaert notifications@github.com wrote:

Closed #5 https://github.com/sensu-plugins/sensu-plugins-opsgenie/pull/5 .

— Reply to this email directly or view it on GitHub https://github.com/sensu-plugins/sensu-plugins-opsgenie/pull/5#event-475571472 .

JarenGlover commented 8 years ago

@analytically how should one invoke the handler to override the server config? I would like to only define one opsgenie handler and leverage it for a wide range of sensu checks?

seems like your code only overrides if I reference the check explicitly???