diff --git a/manifests/network.pp b/manifests/network.pp index dc4d8974..a6a81cbd 100644 --- a/manifests/network.pp +++ b/manifests/network.pp @@ -55,6 +55,9 @@ # (optional) Syslog facility to receive log lines # Defaults value in params # +# [*dhcp_lease_duration*] +# (optionnal) DHCP Lease duration (in seconds) +# Defauls to '120' class cloud::network( $verbose = $os_params::verbose, @@ -66,7 +69,8 @@ class cloud::network( $provider_vlan_ranges = $os_params::provider_vlan_ranges, $provider_bridge_mappings = $os_params::provider_bridge_mappings, $use_syslog = $os_params::use_syslog, - $log_facility = $os_params::log_facility + $log_facility = $os_params::log_facility, + $dhcp_lease_duration = '120', ) { # Disable twice logging if syslog is enabled @@ -91,6 +95,7 @@ class cloud::network( core_plugin => 'neutron.plugins.ml2.plugin.Ml2Plugin', service_plugins => ['neutron.services.loadbalancer.plugin.LoadBalancerPlugin','neutron.services.metering.metering_plugin.MeteringPlugin','neutron.services.l3_router.l3_router_plugin.L3RouterPlugin'], log_dir => $log_dir, + dhcp_lease_duration => $dhcp_lease_duration, } class { 'neutron::agents::ovs': diff --git a/spec/classes/cloud_network_controller_spec.rb b/spec/classes/cloud_network_controller_spec.rb index e8a0ba9d..76bb4d61 100644 --- a/spec/classes/cloud_network_controller_spec.rb +++ b/spec/classes/cloud_network_controller_spec.rb @@ -32,6 +32,7 @@ describe 'cloud::network::controller' do verbose => true, debug => true, use_syslog => true, + dhcp_lease_duration => '10', log_facility => 'LOG_LOCAL0' }" end @@ -60,7 +61,8 @@ describe 'cloud::network::controller' do :bind_host => '10.0.0.1', :core_plugin => 'neutron.plugins.ml2.plugin.Ml2Plugin', :service_plugins => ['neutron.services.loadbalancer.plugin.LoadBalancerPlugin','neutron.services.metering.metering_plugin.MeteringPlugin','neutron.services.l3_router.l3_router_plugin.L3RouterPlugin'], - :log_dir => false + :log_dir => false, + :dhcp_lease_duration => '10' ) should contain_class('neutron::agents::ovs').with( :enable_tunneling => true, diff --git a/spec/classes/cloud_network_dhcp_spec.rb b/spec/classes/cloud_network_dhcp_spec.rb index 12d4c56a..fae78774 100644 --- a/spec/classes/cloud_network_dhcp_spec.rb +++ b/spec/classes/cloud_network_dhcp_spec.rb @@ -32,6 +32,7 @@ describe 'cloud::network::dhcp' do verbose => true, debug => true, use_syslog => true, + dhcp_lease_duration => '10', log_facility => 'LOG_LOCAL0' }" end @@ -55,7 +56,8 @@ describe 'cloud::network::dhcp' do :bind_host => '10.0.0.1', :core_plugin => 'neutron.plugins.ml2.plugin.Ml2Plugin', :service_plugins => ['neutron.services.loadbalancer.plugin.LoadBalancerPlugin','neutron.services.metering.metering_plugin.MeteringPlugin','neutron.services.l3_router.l3_router_plugin.L3RouterPlugin'], - :log_dir => false + :log_dir => false, + :dhcp_lease_duration => '10' ) should contain_class('neutron::agents::ovs').with( :enable_tunneling => true, diff --git a/spec/classes/cloud_network_l3_spec.rb b/spec/classes/cloud_network_l3_spec.rb index 854bc4b7..62539055 100644 --- a/spec/classes/cloud_network_l3_spec.rb +++ b/spec/classes/cloud_network_l3_spec.rb @@ -32,6 +32,7 @@ describe 'cloud::network::l3' do verbose => true, debug => true, use_syslog => true, + dhcp_lease_duration => '10', log_facility => 'LOG_LOCAL0' }" end @@ -55,7 +56,8 @@ describe 'cloud::network::l3' do :bind_host => '10.0.0.1', :core_plugin => 'neutron.plugins.ml2.plugin.Ml2Plugin', :service_plugins => ['neutron.services.loadbalancer.plugin.LoadBalancerPlugin','neutron.services.metering.metering_plugin.MeteringPlugin','neutron.services.l3_router.l3_router_plugin.L3RouterPlugin'], - :log_dir => false + :log_dir => false, + :dhcp_lease_duration => '10' ) should contain_class('neutron::agents::ovs').with( :enable_tunneling => true, diff --git a/spec/classes/cloud_network_lbaas_spec.rb b/spec/classes/cloud_network_lbaas_spec.rb index 527b7d40..3d8dcd24 100644 --- a/spec/classes/cloud_network_lbaas_spec.rb +++ b/spec/classes/cloud_network_lbaas_spec.rb @@ -32,6 +32,7 @@ describe 'cloud::network::lbaas' do verbose => true, debug => true, use_syslog => true, + dhcp_lease_duration => '10', log_facility => 'LOG_LOCAL0' }" end @@ -54,7 +55,8 @@ describe 'cloud::network::lbaas' do :bind_host => '10.0.0.1', :core_plugin => 'neutron.plugins.ml2.plugin.Ml2Plugin', :service_plugins => ['neutron.services.loadbalancer.plugin.LoadBalancerPlugin','neutron.services.metering.metering_plugin.MeteringPlugin','neutron.services.l3_router.l3_router_plugin.L3RouterPlugin'], - :log_dir => false + :log_dir => false, + :dhcp_lease_duration => '10' ) should contain_class('neutron::agents::ovs').with( :enable_tunneling => true, diff --git a/spec/classes/cloud_network_metadata_spec.rb b/spec/classes/cloud_network_metadata_spec.rb index ff95b600..4a0b68ed 100644 --- a/spec/classes/cloud_network_metadata_spec.rb +++ b/spec/classes/cloud_network_metadata_spec.rb @@ -32,6 +32,7 @@ describe 'cloud::network::metadata' do verbose => true, debug => true, use_syslog => true, + dhcp_lease_duration => '10', log_facility => 'LOG_LOCAL0' }" end @@ -62,7 +63,8 @@ describe 'cloud::network::metadata' do :bind_host => '10.0.0.1', :core_plugin => 'neutron.plugins.ml2.plugin.Ml2Plugin', :service_plugins => ['neutron.services.loadbalancer.plugin.LoadBalancerPlugin','neutron.services.metering.metering_plugin.MeteringPlugin','neutron.services.l3_router.l3_router_plugin.L3RouterPlugin'], - :log_dir => false + :log_dir => false, + :dhcp_lease_duration => '10' ) should contain_class('neutron::agents::ovs').with( :enable_tunneling => true, diff --git a/spec/classes/cloud_network_vpn_spec.rb b/spec/classes/cloud_network_vpn_spec.rb index bceee7cb..72c66186 100644 --- a/spec/classes/cloud_network_vpn_spec.rb +++ b/spec/classes/cloud_network_vpn_spec.rb @@ -32,6 +32,7 @@ describe 'cloud::network::vpn' do verbose => true, debug => true, use_syslog => true, + dhcp_lease_duration => '10', log_facility => 'LOG_LOCAL0' }" end @@ -50,7 +51,8 @@ describe 'cloud::network::vpn' do :bind_host => '10.0.0.1', :core_plugin => 'neutron.plugins.ml2.plugin.Ml2Plugin', :service_plugins => ['neutron.services.loadbalancer.plugin.LoadBalancerPlugin','neutron.services.metering.metering_plugin.MeteringPlugin','neutron.services.l3_router.l3_router_plugin.L3RouterPlugin'], - :log_dir => false + :log_dir => false, + :dhcp_lease_duration => '10' ) should contain_class('neutron::agents::ovs').with( :enable_tunneling => true,