X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=blobdiff_plain;f=fuel-plugin-ovsnfv%2Fdeployment_scripts%2Fpuppet%2Fmodules%2Fovsdpdk%2Fmanifests%2Funinstall_ovs.pp;h=7c7a23d645bcbd092c107e3cea782b185d93b996;hb=0e14c4cb61521bdfc3cd597342f526e1c30ff405;hp=97afdd7d12ba338e4744fe0e8ffd7352bd75f72a;hpb=9c5d3a2ab72fa9553924ecaf70ae7ef0d1d562d0;p=ovsnfv.git

diff --git a/fuel-plugin-ovsnfv/deployment_scripts/puppet/modules/ovsdpdk/manifests/uninstall_ovs.pp b/fuel-plugin-ovsnfv/deployment_scripts/puppet/modules/ovsdpdk/manifests/uninstall_ovs.pp
index 97afdd7..7c7a23d 100755
--- a/fuel-plugin-ovsnfv/deployment_scripts/puppet/modules/ovsdpdk/manifests/uninstall_ovs.pp
+++ b/fuel-plugin-ovsnfv/deployment_scripts/puppet/modules/ovsdpdk/manifests/uninstall_ovs.pp
@@ -6,41 +6,38 @@ class ovsdpdk::uninstall_ovs (
   $openvswitch_service_name = $::ovsdpdk::params::openvswitch_service_name,
   $openvswitch_agent        = $::ovsdpdk::params::openvswitch_agent,
   $install_packages         = $::ovsdpdk::params::install_packages,
-  $openvswitch_agent        = $::ovsdpdk::params::openvswitch_agent,
 ) inherits ovsdpdk {
 
-  #Due to dependencies to other packages, we won't purge vanilla OVS  
+  #Due to dependencies to other packages, we won't purge vanilla OVS
   #package { $remove_packages: ensure => 'purged' }
 
   if $compute == 'True' {
-	  exec { "/usr/sbin/service ${openvswitch_service_name} stop":
-	    user => root,
-	  }
-
-# This is required for Liberty
-#	  exec { "/usr/sbin/service ${openvswitch_agent} stop":
-#	    user => root,
-#	    path => "/usr/bin:/bin",
-#	  }
+    exec { "/usr/sbin/service ${openvswitch_service_name} stop":
+      user   => root,
+      path   => ["/usr/bin", "/bin", "/sbin"],
+    }
+
+    exec { "/usr/sbin/service ${openvswitch_agent} stop":
+      user   => root,
+      path   => ["/usr/bin", "/bin", "/sbin"],
+    }
+
+    exec { '/sbin/modprobe -r openvswitch':
+      onlyif  => "/bin/grep -q '^openvswitch' '/proc/modules'",
+      user    => root,
+      require => Exec["/usr/sbin/service ${openvswitch_service_name} stop"],
+    }
   }
 
   if $controller == 'True' {
-	  exec { '/usr/sbin/service neutron-server stop':
-	    user => root,
-	    path => "/usr/bin:/bin",
-	    onlyif => "ps aux |  grep -vws grep | grep -ws neutron-server"
-	  }
+    exec { '/usr/sbin/service neutron-server stop':
+      user   => root,
+      path   => ["/usr/bin", "/bin", "/sbin"],
+      onlyif => "ps aux | grep -vws grep | grep -ws neutron-server",
+    }
   }
 
   package { $install_packages: ensure => 'installed' }
 
-  if $compute == 'True' {
-	  exec { '/sbin/modprobe -r openvswitch':
-	    onlyif  => "/bin/grep -q '^openvswitch' '/proc/modules'",
-	    user    => root,
-#	    require => Exec["/usr/sbin/service ${openvswitch_agent} stop"],
-	  }
-  }
-
 }