Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add compatibility for Sensu 0.13 #209

Closed
superseb opened this issue Jul 31, 2014 · 4 comments
Closed

Add compatibility for Sensu 0.13 #209

superseb opened this issue Jul 31, 2014 · 4 comments

Comments

@superseb
Copy link
Contributor

Bumped into this, please add other findings for 0.13 compatibility in this issue:

@yeungda
Copy link
Contributor

yeungda commented Aug 26, 2014

I had trouble configuring my handler for sending metrics to graphite via ampq. It was a handler of type ampq, with an exchange property. In 0.13, I would expect this to become a handler of type transport with a pipe property. It seems that the transport type is now allowed by sensu::handler, but it doesn't seem possible to configure a pipe parameter for it.

As a temporary workaround, I'm manually adding the pipe part to the json file, after it's been created.

For an example of the desired state, see the Transport section in http://sensuapp.org/docs/0.13/handlers

Happy to make a patch for this if it would help.

@jamtur01
Copy link
Contributor

@yeungda Would welcome a patch.

@vorlock
Copy link

vorlock commented Sep 18, 2014

I just made some compatibility adjustments for 0.13.1 and created PR (#247), have a look at it pls.

@jamtur01
Copy link
Contributor

This is now fixed - I blipped on pipe being added.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants