openstack/puppet-neutron
Code Issues Proposed changes

Merge "Ensure resource_provider_bandwitdhs is unset by default"

Browse Source
This commit is contained in:
Zuul 2020-11-23 17:30:44 +00:00 committed by Gerrit Code Review
commit ddccc02855
4 changed files with 17 additions and 10 deletions

11
manifests/agents/ml2/ovs.pp

@ -318,10 +318,13 @@ class neutron::agents::ml2::ovs (
}
if ($resource_provider_bandwidths != []) {
$res_prov_bw_map_str = join(any2array($resource_provider_bandwidths), ',')
neutron_agent_ovs {
'ovs/resource_provider_bandwidths': value => $res_prov_bw_map_str;
}
$resource_provider_bandwidths_real = join(any2array($resource_provider_bandwidths), ',')
} else {
$resource_provider_bandwidths_real = $::os_service_default
}
neutron_agent_ovs {
'ovs/resource_provider_bandwidths': value => $resource_provider_bandwidths_real;
}
# TODO(tobias.urdin): Remove in V release.

10
manifests/agents/ml2/sriov.pp

@ -129,10 +129,12 @@ class neutron::agents::ml2::sriov (
}
if ($resource_provider_bandwidths != []) {
$res_prov_bw_map_str = join(any2array($resource_provider_bandwidths), ',')
neutron_sriov_agent_config {
'sriov_nic/resource_provider_bandwidths': value => $res_prov_bw_map_str;
}
$resource_provider_bandwidths_real = join(any2array($resource_provider_bandwidths), ',')
} else {
$resource_provider_bandwidths_real = $::os_service_default
}
neutron_sriov_agent_config {
'sriov_nic/resource_provider_bandwidths': value => $resource_provider_bandwidths_real;
}
}

3
spec/classes/neutron_agents_ml2_ovs_spec.rb

@ -65,7 +65,8 @@ describe 'neutron::agents::ml2::ovs' do
should contain_neutron_agent_ovs('ovs/tun_peer_patch_port').with_ensure('absent')
should contain_neutron_agent_ovs('agent/tunnel_types').with_ensure('absent')
should contain_neutron_agent_ovs('ovs/igmp_snooping_enable').with_value('<SERVICE DEFAULT>')
should_not contain_neutron_agent_ovs('ovs/resource_provider_bandwidths')
should contain_neutron_agent_ovs('ovs/resource_provider_bandwidths').\
with_value('<SERVICE DEFAULT>')
should contain_neutron_agent_ovs('agent/explicitly_egress_direct').with_value(['<SERVICE DEFAULT>'])
end

3
spec/classes/neutron_agents_ml2_sriov_spec.rb

@ -67,7 +67,8 @@ describe 'neutron::agents::ml2::sriov' do
end
it 'does not configure resource_provider_bandwidths by default' do
should_not contain_neutron_sriov_agent_config('sriov_nic/resource_provider_bandwidths')
should contain_neutron_sriov_agent_config('sriov_nic/resource_provider_bandwidths').\
with_value('<SERVICE DEFAULT>')
end
context 'when number_of_vfs is empty' do