c7d6291d57
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
308 lines
10 KiB
Ruby
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
|