X-Git-Url: https://git.donarmstrong.com/?a=blobdiff_plain;ds=sidebyside;f=3rdparty%2Fmodules%2Fneutron%2Fspec%2Fclasses%2Fneutron_agents_ml2_ovs_spec.rb;fp=3rdparty%2Fmodules%2Fneutron%2Fspec%2Fclasses%2Fneutron_agents_ml2_ovs_spec.rb;h=a09963a28db80ce5a7ad42387708ed94c4fdfb6e;hb=4631045ebb77ee8622f6fa09277a50c372bcc02e;hp=0000000000000000000000000000000000000000;hpb=3d4dc4fd9e59bd0e07646c99f6b356c7d9d859aa;p=dsa-puppet.git diff --git a/3rdparty/modules/neutron/spec/classes/neutron_agents_ml2_ovs_spec.rb b/3rdparty/modules/neutron/spec/classes/neutron_agents_ml2_ovs_spec.rb new file mode 100644 index 00000000..a09963a2 --- /dev/null +++ b/3rdparty/modules/neutron/spec/classes/neutron_agents_ml2_ovs_spec.rb @@ -0,0 +1,220 @@ +require 'spec_helper' + +describe 'neutron::agents::ml2::ovs' do + + let :pre_condition do + "class { 'neutron': rabbit_password => 'passw0rd' }" + end + + let :default_params do + { :package_ensure => 'present', + :enabled => true, + :bridge_uplinks => [], + :bridge_mappings => [], + :integration_bridge => 'br-int', + :enable_tunneling => false, + :local_ip => false, + :tunnel_bridge => 'br-tun', + :polling_interval => 2, + :l2_population => false, + :arp_responder => false, + :enable_distributed_routing => false, + :firewall_driver => 'neutron.agent.linux.iptables_firewall.OVSHybridIptablesFirewallDriver' } + end + + let :params do + {} + end + + shared_examples_for 'neutron plugin ovs agent with ml2 plugin' do + let :p do + default_params.merge(params) + end + + it { should contain_class('neutron::params') } + + it 'configures ovs_neutron_plugin.ini' do + should contain_neutron_plugin_ml2('agent/polling_interval').with_value(p[:polling_interval]) + should contain_neutron_plugin_ml2('agent/l2_population').with_value(p[:l2_population]) + should contain_neutron_plugin_ml2('agent/arp_responder').with_value(p[:arp_responder]) + should contain_neutron_plugin_ml2('ovs/integration_bridge').with_value(p[:integration_bridge]) + should contain_neutron_plugin_ml2('securitygroup/firewall_driver').\ + with_value(p[:firewall_driver]) + should contain_neutron_plugin_ml2('ovs/enable_tunneling').with_value(false) + should contain_neutron_plugin_ml2('ovs/tunnel_bridge').with_ensure('absent') + should contain_neutron_plugin_ml2('ovs/local_ip').with_ensure('absent') + end + + it 'configures vs_bridge' do + should contain_vs_bridge(p[:integration_bridge]).with( + :ensure => 'present', + :before => 'Service[neutron-ovs-agent-service]' + ) + should_not contain_vs_brige(p[:integration_bridge]) + end + + it 'installs neutron ovs agent package' do + if platform_params.has_key?(:ovs_agent_package) + should contain_package('neutron-ovs-agent').with( + :name => platform_params[:ovs_agent_package], + :ensure => p[:package_ensure] + ) + should contain_package('neutron-ovs-agent').with_before(/Neutron_plugin_ml2\[.+\]/) + else + end + end + + it 'configures neutron ovs agent service' do + should contain_service('neutron-ovs-agent-service').with( + :name => platform_params[:ovs_agent_service], + :enable => true, + :ensure => 'running', + :require => 'Class[Neutron]' + ) + end + + context 'when supplying a firewall driver' do + before :each do + params.merge!(:firewall_driver => false) + end + it 'should configure firewall driver' do + should contain_neutron_plugin_ml2('securitygroup/firewall_driver').with_ensure('absent') + end + end + + context 'when enabling ARP responder' do + before :each do + params.merge!(:arp_responder => true) + end + it 'should enable ARP responder' do + should contain_neutron_plugin_ml2('agent/arp_responder').with_value(true) + end + end + + context 'when enabling DVR' do + before :each do + params.merge!(:enable_distributed_routing => true, + :l2_population => true ) + end + it 'should enable DVR' do + should contain_neutron_plugin_ml2('agent/enable_distributed_routing').with_value(true) + end + end + + context 'when supplying bridge mappings for provider networks' do + before :each do + params.merge!(:bridge_uplinks => ['br-ex:eth2'],:bridge_mappings => ['default:br-ex']) + end + + it 'configures bridge mappings' do + should contain_neutron_plugin_ml2('ovs/bridge_mappings') + end + + it 'should configure bridge mappings' do + should contain_neutron__plugins__ovs__bridge(params[:bridge_mappings].join(',')).with( + :before => 'Service[neutron-ovs-agent-service]' + ) + end + + it 'should configure bridge uplinks' do + should contain_neutron__plugins__ovs__port(params[:bridge_uplinks].join(',')).with( + :before => 'Service[neutron-ovs-agent-service]' + ) + end + end + + context 'when enabling tunneling' do + context 'without local ip address' do + before :each do + params.merge!(:enable_tunneling => true) + end + it 'should fail' do + expect do + subject + end.to raise_error(Puppet::Error, /Local ip for ovs agent must be set when tunneling is enabled/) + end + end + context 'with default params' do + before :each do + params.merge!(:enable_tunneling => true, :local_ip => '127.0.0.1' ) + end + it 'should configure ovs for tunneling' do + should contain_neutron_plugin_ml2('ovs/enable_tunneling').with_value(true) + should contain_neutron_plugin_ml2('ovs/tunnel_bridge').with_value(default_params[:tunnel_bridge]) + should contain_neutron_plugin_ml2('ovs/local_ip').with_value('127.0.0.1') + should contain_vs_bridge(default_params[:tunnel_bridge]).with( + :ensure => 'present', + :before => 'Service[neutron-ovs-agent-service]' + ) + end + end + + context 'with vxlan tunneling' do + before :each do + params.merge!(:enable_tunneling => true, + :local_ip => '127.0.0.1', + :tunnel_types => ['vxlan'], + :vxlan_udp_port => '4789') + end + + it 'should perform vxlan network configuration' do + should contain_neutron_plugin_ml2('agent/tunnel_types').with_value(params[:tunnel_types]) + should contain_neutron_plugin_ml2('agent/vxlan_udp_port').with_value(params[:vxlan_udp_port]) + end + end + + context 'when l2 population is disabled and DVR enabled' do + before :each do + params.merge!(:enable_distributed_routing => true, + :l2_population => false ) + end + it 'should fail' do + expect do + subject + end.to raise_error(Puppet::Error, /L2 population must be enabled when DVR is enabled/) + end + end + end + end + + context 'on Debian platforms' do + let :facts do + { :osfamily => 'Debian' } + end + + let :platform_params do + { :ovs_agent_package => 'neutron-plugin-openvswitch-agent', + :ovs_agent_service => 'neutron-plugin-openvswitch-agent' } + end + + it_configures 'neutron plugin ovs agent with ml2 plugin' + end + + context 'on RedHat platforms' do + let :facts do + { :osfamily => 'RedHat' } + end + + let :platform_params do + { :ovs_cleanup_service => 'neutron-ovs-cleanup', + :ovs_agent_service => 'neutron-openvswitch-agent' } + end + + it_configures 'neutron plugin ovs agent with ml2 plugin' + + it 'configures neutron ovs cleanup service' do + should contain_service('ovs-cleanup-service').with( + :name => platform_params[:ovs_cleanup_service], + :enable => true + ) + should contain_package('neutron-ovs-agent').with_before(/Service\[ovs-cleanup-service\]/) + end + + it 'links from ovs config to plugin config' do + should contain_file('/etc/neutron/plugins/openvswitch/ovs_neutron_plugin.ini').with( + :ensure => 'link', + :target => '/etc/neutron/plugins/ml2/ml2_conf.ini' + ) + end + end +end