You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
When using this with sensu => 0.24.0 (which introduced named aggregates) some options of check-aggregate.rb don't work any more. I think the following options are now broken:
--limit
--summarize
--output
--pattern
--honor-stash
I've tried to fix --output but after looking into the code I've found out that Sensu Aggregates (2.0) API is rewritten from scratch:
I think it makes sense to drop sensu < 0.24 compatibility and provide only options that can work with sensu => 0.24
The text was updated successfully, but these errors were encountered:
ttarczynski
changed the title
check-aggregate.rb options doesn't work with named aggregates
check-aggregate.rb options don't work with named aggregates
Nov 10, 2016
At this point we need to get this fixed even if it is a breaking change. As long as we properly call it out in the changelog and the maintainers version it as a major release I am 👍 on dropping the older support.
When using this with sensu
=> 0.24.0
(which introduced named aggregates) some options ofcheck-aggregate.rb
don't work any more. I think the following options are now broken:I've tried to fix
--output
but after looking into the code I've found out that Sensu Aggregates (2.0) API is rewritten from scratch:I think it makes sense to drop sensu < 0.24 compatibility and provide only options that can work with sensu
=> 0.24
The text was updated successfully, but these errors were encountered: