Closes Bug: #1635365

do not reboot compute on upgrade. Change-Id: I8250a2a25fbd0f3bde053c603047355e35c48b49
Juniper · Nov 2, 2016 · d3b9c94 · d3b9c94
1 parent 5ff2610
commit d3b9c94
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 5 deletions.
diff --git a/contrail/environment/modules/contrail/manifests/compute/config.pp b/contrail/environment/modules/contrail/manifests/compute/config.pp
@@ -52,6 +52,7 @@
   $kombu_ssl_ca_certs = $::contrail::params::kombu_ssl_ca_certs,
   $kombu_ssl_certfile = $::contrail::params::kombu_ssl_certfile,
   $kombu_ssl_keyfile  = $::contrail::params::kombu_ssl_keyfile,
+  $upgrade_needed = $::contrail::params::upgrade_needed,
 ){
   $config_ip_to_use = $::contrail::params::config_ip_to_use
   $keystone_ip_to_use = $::contrail::params::keystone_ip_to_use
@@ -348,11 +349,13 @@
   }
   ->
   class {'::contrail::compute::setup_compute_server_setup':}
-  ->
-  reboot { 'compute':
-    apply => "immediately",
-    subscribe       => Exec ["setup-compute-server-setup"],
-    timeout => 0,
+  if ($upgrade_needed != 1) {
+      Class ['::contrail::compute::setup_compute_server_setup'] ->
+      reboot { 'compute':
+        apply => "immediately",
+        subscribe       => Exec ["setup-compute-server-setup"],
+        timeout => 0,
+      }
   }
 
   Class['::contrail::compute::setup_compute_server_setup'] -> Nova_config <||>

diff --git a/contrail/environment/modules/contrail/manifests/compute/service.pp b/contrail/environment/modules/contrail/manifests/compute/service.pp
@@ -4,6 +4,7 @@
   $compute_ip_list = $::contrail::params::compute_ip_list,
   $nfs_server = $::contrail::params::nfs_server,
   $contrail_logoutput = $::contrail::params::contrail_logoutput,
+  $upgrade_needed = $::contrail::params::upgrade_needed,
 ) {
     if !('toragent' in $contrail::params::host_roles) {
         service { 'supervisor-vrouter':
@@ -35,4 +36,11 @@
          enable => true
        }
     }
+    if ($upgrade_needed == 1) {
+        exec { 'upgrade-vrouter-restart' :
+            command => "rmmod vrouter && service supervisor-vrouter restart",
+            provider => shell,
+            logoutput => $contrail_logoutput,
+        }
+    }
 }