openstack/puppet-neutron
Code Issues Proposed changes

Merge "Unified agent section of ini files"

Browse Source
This commit is contained in:
Zuul 2019-05-10 10:53:16 +00:00 committed by Gerrit Code Review
commit dd3ba5ed0a
4 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/agents/dhcp.pp
View File

@ -157,7 +157,7 @@ class neutron::agents::dhcp (
'DEFAULT/dnsmasq_dns_servers': value => join(any2array($dnsmasq_dns_servers), ',');
'DEFAULT/dnsmasq_local_resolv': value => $dnsmasq_local_resolv;
'DEFAULT/ovs_integration_bridge': value => $ovs_integration_bridge;
'AGENT/availability_zone': value => $availability_zone;
'agent/availability_zone': value => $availability_zone;
}
if $ovsdb_connection =~ /^ssl:/ {

4
manifests/agents/l3.pp
View File

@ -134,8 +134,8 @@ class neutron::agents::l3 (
'DEFAULT/periodic_fuzzy_delay': value => $periodic_fuzzy_delay;
'DEFAULT/enable_metadata_proxy': value => $enable_metadata_proxy;
'DEFAULT/agent_mode': value => $agent_mode;
'AGENT/availability_zone': value => $availability_zone;
'AGENT/extensions': value => $extensions;
'agent/availability_zone': value => $availability_zone;
'agent/extensions': value => $extensions;
}
if $::neutron::params::l3_agent_package {

4
spec/classes/neutron_agents_dhcp_spec.rb
View File

@ -43,7 +43,7 @@ describe 'neutron::agents::dhcp' do
should contain_neutron_dhcp_agent_config('DEFAULT/dhcp_broadcast_reply').with_value('<SERVICE DEFAULT>');
should contain_neutron_dhcp_agent_config('DEFAULT/ovs_integration_bridge').with_value('<SERVICE DEFAULT>');
should contain_neutron_dhcp_agent_config('DEFAULT/dnsmasq_local_resolv').with_value('<SERVICE DEFAULT>');
should contain_neutron_dhcp_agent_config('AGENT/availability_zone').with_value('<SERVICE DEFAULT>');
should contain_neutron_dhcp_agent_config('agent/availability_zone').with_value('<SERVICE DEFAULT>');
should contain_neutron_dhcp_agent_config('OVS/ovsdb_connection').with_value('<SERVICE DEFAULT>');
should contain_neutron_dhcp_agent_config('OVS/ssl_key_file').with_value('<SERVICE DEFAULT>');
should contain_neutron_dhcp_agent_config('OVS/ssl_cert_file').with_value('<SERVICE DEFAULT>');
@ -146,7 +146,7 @@ describe 'neutron::agents::dhcp' do
params.merge!(:availability_zone => 'zone1')
end
it 'should configure availability zone' do
should contain_neutron_dhcp_agent_config('AGENT/availability_zone').with_value(p[:availability_zone]);
should contain_neutron_dhcp_agent_config('agent/availability_zone').with_value(p[:availability_zone]);
end
end

6
spec/classes/neutron_agents_l3_spec.rb
View File

@ -38,8 +38,8 @@ describe 'neutron::agents::l3' do
should contain_neutron_l3_agent_config('DEFAULT/periodic_interval').with_value('<SERVICE DEFAULT>')
should contain_neutron_l3_agent_config('DEFAULT/periodic_fuzzy_delay').with_value('<SERVICE DEFAULT>')
should contain_neutron_l3_agent_config('DEFAULT/enable_metadata_proxy').with_value('<SERVICE DEFAULT>')
should contain_neutron_l3_agent_config('AGENT/availability_zone').with_value('<SERVICE DEFAULT>')
should contain_neutron_l3_agent_config('AGENT/extensions').with_value('<SERVICE DEFAULT>')
should contain_neutron_l3_agent_config('agent/availability_zone').with_value('<SERVICE DEFAULT>')
should contain_neutron_l3_agent_config('agent/extensions').with_value('<SERVICE DEFAULT>')
end
it 'passes purge to resource' do
@ -106,7 +106,7 @@ describe 'neutron::agents::l3' do
end
it 'configures availability zone' do
should contain_neutron_l3_agent_config('AGENT/availability_zone').with_value(p[:availability_zone])
should contain_neutron_l3_agent_config('agent/availability_zone').with_value(p[:availability_zone])
end
end
end