search

sensu / sensu-chef

Sensu Chef cookbook.
https://supermarket.chef.io/cookbooks/sensu
Apache License 2.0
222 stars 280 forks source link

default.rb has a bug in master branch #561

Closed yangyinying closed 7 years ago

yangyinying commented 7 years ago

in master branch, file /recipes/default.rb line 96, it is: if node["sensu"]["rabbitmq"].port == 5671

When running it, it raises an error complaining about no such function/method as "port".

While in developer branch, it is correct: if node["sensu"]["rabbitmq"]["port"] == 5671

Since the master branch is the one that will be included in the wrapper cookbook, it needs to be fixed.

Thanks.

majormoses commented 7 years ago

@cwjohnston where are you at with prepping for a release? If we are not close I am more than happy to submit a hotfix for it.

salimkapadia commented 7 years ago

@cwjohnston +1 bumping this. Do you have a release date ?

cwjohnston commented 7 years ago

4.0.2 was released to the supermarket today and includes this syntax fix for Chef 13 compatibility.