openstack/puppet-neutron
Code Issues Proposed changes

Ensure purge_config takes a boolean value

Browse Source 
The purge_config parameters only accept boolean values. This enforces
that using the typed parameters.

Change-Id: I8fffedd98f7bef5a9bf065ee3a7850fb0222869d
This commit is contained in:
Takashi Kajinami 2023-07-22 23:50:59 +09:00
parent 13128ddf6c
commit dbad032e33
20 changed files with 20 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/agents/bagpipe.pp
View File

@ -91,7 +91,7 @@ class neutron::agents::bagpipe (
$package_ensure = 'present', $package_ensure = 'present',
$peers = $facts['os_service_default'], $peers = $facts['os_service_default'],
$proxy_arp = false, $proxy_arp = false,
$purge_config = false, Boolean $purge_config = false,
$local_address = $facts['networking']['ip'], $local_address = $facts['networking']['ip'],
) { ) {

2
manifests/agents/bgp_dragent.pp
View File

@ -51,7 +51,7 @@ class neutron::agents::bgp_dragent(
Boolean $manage_service = true, Boolean $manage_service = true,
$bgp_speaker_driver = 'neutron_dynamic_routing.services.bgp.agent.driver.os_ken.driver.OsKenBgpDriver', $bgp_speaker_driver = 'neutron_dynamic_routing.services.bgp.agent.driver.os_ken.driver.OsKenBgpDriver',
$bgp_router_id = $facts['networking']['ip'], $bgp_router_id = $facts['networking']['ip'],
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/agents/dhcp.pp
View File

@ -145,7 +145,7 @@ class neutron::agents::dhcp (
Boolean $enable_force_metadata = false, Boolean $enable_force_metadata = false,
Boolean $enable_metadata_network = false, Boolean $enable_metadata_network = false,
$dhcp_broadcast_reply = $facts['os_service_default'], $dhcp_broadcast_reply = $facts['os_service_default'],
$purge_config = false, Boolean $purge_config = false,
$availability_zone = $facts['os_service_default'], $availability_zone = $facts['os_service_default'],
$ovs_integration_bridge = $facts['os_service_default'], $ovs_integration_bridge = $facts['os_service_default'],
$ovsdb_connection = $facts['os_service_default'], $ovsdb_connection = $facts['os_service_default'],

2
manifests/agents/l2gw.pp
View File

@ -83,7 +83,7 @@ class neutron::agents::l2gw (
$periodic_interval = $facts['os_service_default'], $periodic_interval = $facts['os_service_default'],
$max_connection_retries = $facts['os_service_default'], $max_connection_retries = $facts['os_service_default'],
$socket_timeout = '30', $socket_timeout = '30',
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/agents/l3.pp
View File

@ -138,7 +138,7 @@ class neutron::agents::l3 (
$ha_vrrp_auth_password = $facts['os_service_default'], $ha_vrrp_auth_password = $facts['os_service_default'],
$ha_vrrp_advert_int = '3', $ha_vrrp_advert_int = '3',
$agent_mode = 'legacy', $agent_mode = 'legacy',
$purge_config = false, Boolean $purge_config = false,
$availability_zone = $facts['os_service_default'], $availability_zone = $facts['os_service_default'],
$extensions = $facts['os_service_default'], $extensions = $facts['os_service_default'],
$report_interval = $facts['os_service_default'], $report_interval = $facts['os_service_default'],

2
manifests/agents/metadata.pp
View File

@ -91,7 +91,7 @@ class neutron::agents::metadata (
$nova_client_priv_key = $facts['os_service_default'], $nova_client_priv_key = $facts['os_service_default'],
$report_interval = $facts['os_service_default'], $report_interval = $facts['os_service_default'],
$rpc_response_max_timeout = $facts['os_service_default'], $rpc_response_max_timeout = $facts['os_service_default'],
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/agents/metering.pp
View File

@ -76,7 +76,7 @@ class neutron::agents::metering (
$report_interval = $facts['os_service_default'], $report_interval = $facts['os_service_default'],
$rpc_response_max_timeout = $facts['os_service_default'], $rpc_response_max_timeout = $facts['os_service_default'],
$agent_report_interval = $facts['os_service_default'], $agent_report_interval = $facts['os_service_default'],
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/agents/ml2/linuxbridge.pp
View File

@ -87,7 +87,7 @@ class neutron::agents::ml2::linuxbridge (
Array $physical_interface_mappings = [], Array $physical_interface_mappings = [],
Array $bridge_mappings = [], Array $bridge_mappings = [],
$firewall_driver = 'iptables', $firewall_driver = 'iptables',
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/agents/ml2/macvtap.pp
View File

@ -37,7 +37,7 @@ class neutron::agents::ml2::macvtap (
Boolean $manage_service = true, Boolean $manage_service = true,
$polling_interval = $facts['os_service_default'], $polling_interval = $facts['os_service_default'],
Array $physical_interface_mappings = [], Array $physical_interface_mappings = [],
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/agents/ml2/networking_baremetal.pp
View File

@ -121,7 +121,7 @@ class neutron::agents::ml2::networking_baremetal (
$status_code_retry_delay = $facts['os_service_default'], $status_code_retry_delay = $facts['os_service_default'],
$status_code_retries = $facts['os_service_default'], $status_code_retries = $facts['os_service_default'],
$valid_interfaces = $facts['os_service_default'], $valid_interfaces = $facts['os_service_default'],
$purge_config = false, Boolean $purge_config = false,
$report_interval = $facts['os_service_default'], $report_interval = $facts['os_service_default'],
) { ) {

2
manifests/agents/ml2/ovn.pp
View File

@ -122,7 +122,7 @@ class neutron::agents::ml2::ovn (
$root_helper = 'sudo neutron-rootwrap /etc/neutron/rootwrap.conf', $root_helper = 'sudo neutron-rootwrap /etc/neutron/rootwrap.conf',
$root_helper_daemon = $facts['os_service_default'], $root_helper_daemon = $facts['os_service_default'],
$state_path = '/var/lib/neutron', $state_path = '/var/lib/neutron',
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/agents/ml2/ovs.pp
View File

@ -290,7 +290,7 @@ class neutron::agents::ml2::ovs (
$tun_peer_patch_port = $facts['os_service_default'], $tun_peer_patch_port = $facts['os_service_default'],
$datapath_type = $facts['os_service_default'], $datapath_type = $facts['os_service_default'],
$vhostuser_socket_dir = $facts['os_service_default'], $vhostuser_socket_dir = $facts['os_service_default'],
$purge_config = false, Boolean $purge_config = false,
Boolean $enable_dpdk = false, Boolean $enable_dpdk = false,
$enable_security_group = $facts['os_service_default'], $enable_security_group = $facts['os_service_default'],
$permitted_ethertypes = $facts['os_service_default'], $permitted_ethertypes = $facts['os_service_default'],

2
manifests/agents/ml2/sriov.pp
View File

@ -107,7 +107,7 @@ class neutron::agents::ml2::sriov (
$report_interval = $facts['os_service_default'], $report_interval = $facts['os_service_default'],
$exclude_devices = $facts['os_service_default'], $exclude_devices = $facts['os_service_default'],
$extensions = $facts['os_service_default'], $extensions = $facts['os_service_default'],
$purge_config = false, Boolean $purge_config = false,
$number_of_vfs = $facts['os_service_default'], $number_of_vfs = $facts['os_service_default'],
$resource_provider_bandwidths = [], $resource_provider_bandwidths = [],
$resource_provider_hypervisors = [], $resource_provider_hypervisors = [],

2
manifests/agents/ovn_metadata.pp
View File

@ -145,7 +145,7 @@ class neutron::agents::ovn_metadata (
$root_helper = 'sudo neutron-rootwrap /etc/neutron/rootwrap.conf', $root_helper = 'sudo neutron-rootwrap /etc/neutron/rootwrap.conf',
$root_helper_daemon = $facts['os_service_default'], $root_helper_daemon = $facts['os_service_default'],
$state_path = '/var/lib/neutron', $state_path = '/var/lib/neutron',
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/agents/vpnaas.pp
View File

@ -44,7 +44,7 @@ class neutron::agents::vpnaas (
$vpn_device_driver = 'neutron.services.vpn.device_drivers.ipsec.OpenSwanDriver', $vpn_device_driver = 'neutron.services.vpn.device_drivers.ipsec.OpenSwanDriver',
$interface_driver = 'neutron.agent.linux.interface.OVSInterfaceDriver', $interface_driver = 'neutron.agent.linux.interface.OVSInterfaceDriver',
$ipsec_status_check_interval = $facts['os_service_default'], $ipsec_status_check_interval = $facts['os_service_default'],
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/init.pp
View File

@ -369,7 +369,7 @@ class neutron (
$ca_file = $facts['os_service_default'], $ca_file = $facts['os_service_default'],
$state_path = $facts['os_service_default'], $state_path = $facts['os_service_default'],
$lock_path = '$state_path/lock', $lock_path = '$state_path/lock',
$purge_config = false, Boolean $purge_config = false,
$notification_driver = $facts['os_service_default'], $notification_driver = $facts['os_service_default'],
$notification_topics = $facts['os_service_default'], $notification_topics = $facts['os_service_default'],
$notification_transport_url = $facts['os_service_default'], $notification_transport_url = $facts['os_service_default'],

2
manifests/plugins/ml2.pp
View File

@ -137,7 +137,7 @@ class neutron::plugins::ml2 (
$package_ensure = 'present', $package_ensure = 'present',
$physical_network_mtus = $facts['os_service_default'], $physical_network_mtus = $facts['os_service_default'],
$path_mtu = 0, $path_mtu = 0,
$purge_config = false, Boolean $purge_config = false,
$max_header_size = $facts['os_service_default'], $max_header_size = $facts['os_service_default'],
$overlay_ip_version = $facts['os_service_default'], $overlay_ip_version = $facts['os_service_default'],
) { ) {

2
manifests/plugins/ml2/nuage.pp
View File

@ -53,7 +53,7 @@ class neutron::plugins::ml2::nuage (
$nuage_cms_id, $nuage_cms_id,
$nuage_auth_resource = '/me', $nuage_auth_resource = '/me',
$nuage_server_ssl = true, $nuage_server_ssl = true,
$purge_config = false, Boolean $purge_config = false,
$nuage_default_allow_non_ip = false, $nuage_default_allow_non_ip = false,
) { ) {

2
manifests/policy.pp
View File

@ -52,7 +52,7 @@ class neutron::policy (
$policy_path = '/etc/neutron/policy.yaml', $policy_path = '/etc/neutron/policy.yaml',
$policy_default_rule = $facts['os_service_default'], $policy_default_rule = $facts['os_service_default'],
$policy_dirs = $facts['os_service_default'], $policy_dirs = $facts['os_service_default'],
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps

2
manifests/services/l2gw.pp
View File

@ -50,7 +50,7 @@ class neutron::services::l2gw (
$service_providers = $facts['os_service_default'], $service_providers = $facts['os_service_default'],
Boolean $sync_db = false, Boolean $sync_db = false,
$package_ensure = 'present', $package_ensure = 'present',
$purge_config = false, Boolean $purge_config = false,
) { ) {
include neutron::deps include neutron::deps