provide explicit defaults for boolean values

When creating a neutron_network resource without explicitly setting the
router_external parameter, the network would be created with
router:external=True, which would result in a network to which it was
not possible to attach nova instances.

This change sets explicit defaults for all boolean options in the
neutron_network type.

Closes-bug: 1436133
Change-Id: Id0f950da791c65ba9034046177870f31f6f57c46
This commit is contained in:
Lars Kellogg-Stedman 2015-03-24 22:11:48 -04:00
parent 73d7599665
commit 5628b35fbd

View File

@ -17,6 +17,7 @@ Puppet::Type.newtype(:neutron_network) do
newproperty(:admin_state_up) do
desc 'The administrative status of the network'
newvalues(/(t|T)rue/, /(f|F)alse/)
defaultto 'True'
munge do |v|
v.to_s.capitalize
end
@ -25,6 +26,7 @@ Puppet::Type.newtype(:neutron_network) do
newproperty(:shared) do
desc 'Whether this network should be shared across all tenants or not'
newvalues(/(t|T)rue/, /(f|F)alse/)
defaultto 'False'
munge do |v|
v.to_s.capitalize
end
@ -61,6 +63,7 @@ Puppet::Type.newtype(:neutron_network) do
newproperty(:router_external) do
desc 'Whether this router will route traffic to an external network'
newvalues(/(t|T)rue/, /(f|F)alse/)
defaultto 'False'
munge do |v|
v.to_s.capitalize
end