search

redhat-openstack / puppet-pacemaker

Puppet modules to manage pacemaker with corosync
9 stars 25 forks source link

Fix invalid boolean check for properties' values #73

Closed JAORMX closed 8 years ago

JAORMX commented 8 years ago

The property resource checks that if 'ensure' is set to present, then a value must be set; This check is wrong because false can actually be a valid value for the property. In reality, what we want to check for is if the value has been set at all. So this change introduces a check for undef.

mbaldessari commented 8 years ago

Ack. I had this right in my first version, but then I slipped this one in. I'll make sure to add this test on the unit tests I will be working on. Thanks