sensu / sensu-chef

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

properly lock to minimum required version of chef client. #611

Closed majormoses closed 6 years ago

majormoses commented 6 years ago

This fixes #605 which is an unfortunate scenerio

Signed-off-by: Ben Abrams me@benabrams.it

Description

Properly locks to 13.3 >= as thats what the code actually needs. See issue for more details.

Motivation and Context

Fixes #605

How Has This Been Tested?

Has not

Screenshots (if appropriate):

Types of changes

Checklist:

webframp commented 6 years ago

Change itself is fine, the CI failure appears to be due to this: https://github.com/sensu/sensu-chef/blob/develop/test/unit/default_spec.rb#L72

majormoses commented 6 years ago

wow I have not seen node.set in a long time.

majormoses commented 6 years ago

released: https://supermarket.chef.io/cookbooks/sensu/versions/5.1.2