puppet-neutron/spec/classes/neutron_plugins_ml2_spec.rb
Takashi Kajinami c7d6291d57 Fix incomplete validation by validate_network_vlan_ranges
Fixes the validation to enforce the following condition.
 - Vlan id should not be 0 or a negative value
 - Single vlan id is never accepted. Both min and max should be set

Change-Id: I5335df96c3d1967841673c25fc427e9a0b9a9735
2024-05-04 14:12:36 +09:00

308 lines
10 KiB
Ruby

#
# Copyright (C) 2013 eNovance SAS <licensing@enovance.com>
#
# Author: Emilien Macchi <emilien.macchi@enovance.com>
#
# Licensed under the Apache License, Version 2.0 (the "License"); you may
# not use this file except in compliance with the License. You may obtain
# a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
# WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
# License for the specific language governing permissions and limitations
# under the License.
#
# Unit tests for neutron::plugins::ml2 class
#
require 'spec_helper'
describe 'neutron::plugins::ml2' do
let :pre_condition do
"class { 'neutron':
core_plugin => 'ml2'
}"
end
let :default_params do
{
:type_drivers => ['local', 'flat', 'vlan', 'gre', 'vxlan', 'geneve'],
:tenant_network_types => ['local', 'flat', 'vlan', 'gre', 'vxlan'],
:mechanism_drivers => ['openvswitch'],
:flat_networks => '*',
:network_vlan_ranges => 'datacentre:10:50',
:tunnel_id_ranges => '20:100',
:vxlan_group => '224.0.0.1',
:vni_ranges => '10:100',
:package_ensure => 'present',
:purge_config => false,
}
end
let :params do
{}
end
shared_examples 'neutron plugin ml2' do
let :p do
default_params.merge(params)
end
it { should contain_class('neutron::params') }
it 'configures neutron.conf' do
should contain_neutron_config('DEFAULT/core_plugin').with_value('ml2')
end
it 'passes purge to resource' do
should contain_resources('neutron_plugin_ml2').with({
:purge => false
})
end
it 'configures ml2_conf.ini' do
should contain_neutron_plugin_ml2('ml2/type_drivers').with_value(p[:type_drivers].join(','))
should contain_neutron_plugin_ml2('ml2/tenant_network_types').with_value(p[:tenant_network_types].join(','))
should contain_neutron_plugin_ml2('ml2/mechanism_drivers').with_value(p[:mechanism_drivers].join(','))
should contain_neutron_plugin_ml2('ml2/extension_drivers').with_value('<SERVICE DEFAULT>')
should contain_neutron_plugin_ml2('ml2/path_mtu').with_value('<SERVICE DEFAULT>')
should contain_neutron_plugin_ml2('ml2/physical_network_mtus').with_value('<SERVICE DEFAULT>')
should contain_neutron_plugin_ml2('ml2/overlay_ip_version').with_value('<SERVICE DEFAULT>')
should contain_neutron_plugin_ml2('securitygroup/enable_security_group').with_value('<SERVICE DEFAULT>')
end
it 'creates plugin symbolic link' do
should contain_file('/etc/neutron/plugin.ini').with(
:ensure => 'link',
:target => '/etc/neutron/plugins/ml2/ml2_conf.ini'
)
end
it 'installs ml2 package (if any)' do
if platform_params.has_key?(:ml2_server_package)
should contain_package('neutron-plugin-ml2').with(
:name => platform_params[:ml2_server_package],
:ensure => p[:package_ensure],
:tag => ['openstack', 'neutron-package'],
)
should contain_package('neutron-plugin-ml2').that_requires('Anchor[neutron::install::begin]')
should contain_package('neutron-plugin-ml2').that_notifies('Anchor[neutron::install::end]')
end
end
context 'when overriding security group options' do
before :each do
params.merge!(
:enable_security_group => true,
)
end
it 'configures enable_security_group' do
should contain_neutron_plugin_ml2('securitygroup/enable_security_group').with_value('true')
end
end
context 'when specifying IPv4 overlays' do
before :each do
params.merge!(:overlay_ip_version => 4)
end
it 'configures as IPv4' do
should contain_neutron_plugin_ml2('ml2/overlay_ip_version').with_value(4)
end
end
context 'when specifying IPv6 overlays' do
before :each do
params.merge!(:overlay_ip_version => 6)
end
it 'configures as IPv6' do
should contain_neutron_plugin_ml2('ml2/overlay_ip_version').with_value(6)
end
end
context 'when specifying an invalid overlay IP versions' do
before :each do
params.merge!(:overlay_ip_version => 10)
end
it 'fails to accept value' do
should raise_error(Puppet::Error)
end
end
context 'when using extension drivers for ML2 plugin' do
before :each do
params.merge!(:extension_drivers => ['port_security','qos'])
end
it 'configures extension drivers' do
should contain_neutron_plugin_ml2('ml2/extension_drivers').with_value(p[:extension_drivers].join(','))
end
end
context 'configure ml2 with bad driver value' do
before :each do
allow(scope).to receive(:warning).with('type_driver unknown.')
params.merge!(:type_drivers => ['foobar'])
end
end
context 'when using flat driver' do
before :each do
params.merge!(:flat_networks => ['eth1', 'eth2'])
end
it 'configures flat_networks' do
should contain_neutron_plugin_ml2('ml2_type_flat/flat_networks').with_value(p[:flat_networks].join(','))
end
end
context 'when using gre driver with valid values' do
before :each do
params.merge!(:tunnel_id_ranges => ['0:20', '40:60'])
end
it 'configures gre_networks with valid ranges' do
should contain_neutron_plugin_ml2('ml2_type_gre/tunnel_id_ranges').with_value(p[:tunnel_id_ranges].join(','))
end
end
context 'when using gre driver with invalid values' do
before :each do
params.merge!(:tunnel_id_ranges => ['0:20', '40:100000000'])
end
it { should raise_error(Puppet::Error, /tunnel id ranges are to large./) }
end
context 'when using vlan driver with valid values' do
before :each do
params.merge!(:network_vlan_ranges => ['net1:1:20', 'net2:400:4094'])
end
it 'configures vlan_networks with 1:20 and 400:4094 VLAN ranges' do
should contain_neutron_plugin_ml2('ml2_type_vlan/network_vlan_ranges').with_value(p[:network_vlan_ranges].join(','))
end
end
context 'when using vlan driver with invalid vlan id' do
before :each do
params.merge!(:network_vlan_ranges => ['net1:1:20', 'net2:400:4099'])
end
it { should raise_error(Puppet::Error, /invalid vlan ids are used in vlan ranges./) }
end
context 'when using vlan driver with invalid vlan range' do
before :each do
params.merge!(:network_vlan_ranges => ['net1:2938:1'])
end
it { should raise_error(Puppet::Error, /vlan ranges are invalid./) }
end
context 'when using vxlan driver with valid values' do
before :each do
params.merge!(:vni_ranges => ['40:300', '500:1000'], :vxlan_group => '224.1.1.1')
end
it 'configures vxlan_networks with 224.1.1.1 vxlan group' do
should contain_neutron_plugin_ml2('ml2_type_vxlan/vni_ranges').with_value(p[:vni_ranges].join(','))
should contain_neutron_plugin_ml2('ml2_type_vxlan/vxlan_group').with_value(p[:vxlan_group])
end
end
context 'when using vxlan driver with invalid vxlan group' do
before :each do
params.merge!(:vxlan_group => '192.1.1.1')
end
it { should raise_error(Puppet::Error, /is not valid for vxlan_group./) }
end
context 'when using vxlan driver with invalid vni_range' do
before :each do
params.merge!(:vni_ranges => ['2938:1'])
end
it { should raise_error(Puppet::Error, /vni ranges are invalid./) }
end
context 'when using geneve driver' do
before :each do
params.merge!(:type_drivers => ['local', 'flat', 'vlan', 'gre', 'vxlan', 'geneve'],
:vni_ranges => ['40:300','500:1000'],
:max_header_size => 50
)
end
it 'configures geneve with valid values' do
should contain_neutron_plugin_ml2('ml2/type_drivers').with_value(p[:type_drivers].join(','))
should contain_neutron_plugin_ml2('ml2_type_geneve/vni_ranges').with_value([p[:vni_ranges].join(',')])
should contain_neutron_plugin_ml2('ml2_type_geneve/max_header_size').with_value(p[:max_header_size])
end
end
context 'with path_mtu set' do
before :each do
params.merge!(:path_mtu => '9000')
end
it 'should set the path_mtu on the ml2 plugin' do
should contain_neutron_plugin_ml2('ml2/path_mtu').with_value(p[:path_mtu])
end
end
context 'with physical_network_mtus set' do
before :each do
params.merge!(:physical_network_mtus => ['physnet1:9000'])
end
it 'should set the physical_network_mtus on the ml2 plugin' do
should contain_neutron_plugin_ml2('ml2/physical_network_mtus').with_value(p[:physical_network_mtus].join(','))
end
end
context 'when overriding package ensure state' do
before :each do
params.merge!(:package_ensure => 'latest')
end
it 'overrides package ensure state (if possible)' do
if platform_params.has_key?(:ml2_server_package)
should contain_package('neutron-plugin-ml2').with(
:name => platform_params[:ml2_server_package],
:ensure => params[:package_ensure],
:tag => ['openstack', 'neutron-package'],
)
end
end
end
end
on_supported_os({
:supported_os => OSDefaults.get_supported_os
}).each do |os,facts|
context "on #{os}" do
let (:facts) do
facts.merge!(OSDefaults.get_facts())
end
let (:platform_params) do
case facts[:os]['family']
when 'Debian'
if facts[:os]['name'] == 'Ubuntu'
{
:ml2_server_package => 'neutron-plugin-ml2'
}
else
{}
end
when 'RedHat'
{
:ml2_server_package => 'openstack-neutron-ml2'
}
end
end
it_behaves_like 'neutron plugin ml2'
end
end
end