From 54fddb2d9a834ab6655b4c0d3b8e93b23377fdb7 Mon Sep 17 00:00:00 2001
From: Dulanjalie Ganegedara <ddhanapa@cisco.com>
Date: Wed, 25 Feb 2015 08:43:17 -0800
Subject: [PATCH] Changing the default value of fastpath_flood to enable

Current default value of fastpath_flood in nexus1000v is set to disable.
Cisco1000v handles broadcast floods and unknown pkts in fastpath(KLM) by default.
Thus this variable need to be set to enale as the default.

Change-Id: If18ba553b70a896f0d9e38113831445bf3a08cbe
Closes-Bug: 1425589
(cherry picked from commit c90b465001b82005f656cc8af08e87111c884089)
---
 neutron/manifests/agents/n1kv_vem.pp                 |  4 ++--
 neutron/spec/classes/neutron_agents_n1kv_vem_spec.rb | 10 +++++-----
 2 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/neutron/manifests/agents/n1kv_vem.pp b/neutron/manifests/agents/n1kv_vem.pp
index dc740c0..6366b41 100644
--- a/neutron/manifests/agents/n1kv_vem.pp
+++ b/neutron/manifests/agents/n1kv_vem.pp
@@ -97,7 +97,7 @@
 #
 # [*fastpath_flood*]
 #   (optional) Handle broadcast floods and unknown pkts in fastpath(KLM)
-#   Defaults to disable
+#   Defaults to enable
 #
 class neutron::agents::n1kv_vem (
   $n1kv_vsm_ip          = '127.0.0.1',
@@ -113,7 +113,7 @@ class neutron::agents::n1kv_vem (
   $enable               = true,
   $manage_service       = true,
   $portdb               = 'ovs',
-  $fastpath_flood       = 'disable'
+  $fastpath_flood       = 'enable'
 ) {
 
   include neutron::params
diff --git a/neutron/spec/classes/neutron_agents_n1kv_vem_spec.rb b/neutron/spec/classes/neutron_agents_n1kv_vem_spec.rb
index 4eeee17..68863ef 100644
--- a/neutron/spec/classes/neutron_agents_n1kv_vem_spec.rb
+++ b/neutron/spec/classes/neutron_agents_n1kv_vem_spec.rb
@@ -68,7 +68,7 @@ describe 'neutron::agents::n1kv_vem' do
       :n1kv_vsm_domain_id => 900,
       :host_mgmt_intf     => 'eth9',
       :portdb             => 'ovs',
-      :fastpath_flood     => 'disable'
+      :fastpath_flood     => 'enable'
     }
     end
     it do
@@ -87,7 +87,7 @@ describe 'neutron::agents::n1kv_vem' do
       should contain_file('/etc/n1kv/n1kv.conf') \
         .with_content(/^node-type compute/)
       should contain_file('/etc/n1kv/n1kv.conf') \
-        .with_content(/^fastpath-flood disable/)
+        .with_content(/^fastpath-flood enable/)
     end
   end
 
@@ -122,14 +122,14 @@ describe 'neutron::agents::n1kv_vem' do
   context 'verify fastpath_flood' do
     let :params do
     {
-      :fastpath_flood     => 'enable',
+      :fastpath_flood     => 'disable',
     }
     end
     it do
       should contain_file('/etc/n1kv/n1kv.conf') \
-        .with_content(/^fastpath-flood enable/)
+        .with_content(/^fastpath-flood disable/)
       should contain_file('/etc/n1kv/n1kv.conf') \
-        .without_content(/^fastpath-flood disable/)
+        .without_content(/^fastpath-flood enable/)
     end
   end