Get rid of type *property* defaults and use of Puppet::Property::Boolean about puppet-network HOT 6 OPEN

@alexjfisher looks like there is an overarching issue with boolean properties in resource types: https://tickets.puppetlabs.com/browse/PUP-2368

Based on @DavidS's comment at the end of the ticket, this issue affects even the new Resource API.

from puppet-network.

@dhollinger Correct. I've bumped into this in a few other modules.
eg. theforeman/puppet-pulp#284

Shame about the resource API. I think after his talk at FOSDEM, David told me booleans properties were fixed in the new API. I guess not though.

from puppet-network.

Writing boolean properties, yes.

Can't do much about core puppet being a jerk, though. That's @joshcooper's sisyphus.

from puppet-network.

Is it nil, false, or :false... all I got is tears.

from puppet-network.

Is this why I see with:

network_config{ 'ifcfg-enp5s0f0':
  ensure => "present",
  onboot => true,
  family => "inet",
  method => "static",
  ipaddress => "138.137.160.71",
  netmask => "255.255.255.224",
  options => {
              "GATEWAY" => "138.137.160.65",
              "USERCTL" => "false"
              },
}

I get the error:

Error: Value 'true':TrueClass cannot be determined as a boolean value
Info: Unknown failure using insync_values? on type: Network_config[enp5s0f0] / property: hotplug to compare values [false] and absent

when there is no HOTPLUG entry in ifcfg-enp5s0f0

Property is defined as :

newproperty(:hotplug, required_features: :hotpluggable, parent: Puppet::Property::Boolean) do
    desc 'Allow/disallow hotplug support for this interface'
    defaultto :true
  end

Not possible to produce the error with a trivial puppet manifest and puppet apply - only a full puppet run.

from puppet-network.

We get the same for onboot:

Error: Value 'true':TrueClass cannot be determined as a boolean value
Info: Unknown failure using insync_values? on type: Network_config[eth2] / property: onboot to compare values [:absent] and absent
Error: /Stage[main]/Hg_cloud_compute::Nova::Compute/Network_config[eth2]/onboot: change from 'absent' to true failed: Value 'true':TrueClass cannot be determined as a boolean value

from puppet-network.