Problem with empty string parameter not behaving as expected on puppet4.
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
puppet-vswitch |
Fix Released
|
Medium
|
Unassigned |
Bug Description
Hi,
puppet4 changed the semantic of the empty string.
With puppet3 empty sting meant false, with puppet4 it means true[1]
So now the semantic of this test[2] are false. If you look at [3] then puppet4 will happily creates the parameter:
"--socket-mem ''"
while puppet3 won't as empty string is false.
Same on [4].
There are some places where this has been taken care of [5] for instance.
The !empty should be added everywhere to make sure we have the same behavior for empty string in puppet3 and in puppet4.
As it happen only when user explicitly passes empty string as parameter, I'm setting this as medium priority, as this behavior is not the "normal" one.
[2] https:/
[3] https:/
[4] https:/
[5] https:/
I assume this was fixed by https:/ /review. opendev. org/c/openstack /puppet- vswitch/ +/432165 .
Please feel free to reopen this in case you still hit the same problem with a recent version.