From: Giulio Fidente <gfidente@redhat.com>
Date: Mon, 18 May 2015 14:28:26 +0000 (-0400)
Subject: Enable VIPs via Pacemaker from step 2 instead of step 1
X-Git-Tag: opnfv-6.0.0~2548^2~5
X-Git-Url: https://gerrit.opnfv.org/gerrit/gitweb?a=commitdiff_plain;h=5f83a28bd43c85ff4355cbc33071c1eb97d3816e;p=apex-tripleo-heat-templates.git

Enable VIPs via Pacemaker from step 2 instead of step 1

Change-Id: I724c341f148fedf725f3b3da778e491741b754ae
---

diff --git a/puppet/manifests/overcloud_controller_pacemaker.pp b/puppet/manifests/overcloud_controller_pacemaker.pp
index 9f266b6a..4b956bfb 100644
--- a/puppet/manifests/overcloud_controller_pacemaker.pp
+++ b/puppet/manifests/overcloud_controller_pacemaker.pp
@@ -64,20 +64,6 @@ if hiera('step') >= 1 {
   class { '::pacemaker::stonith':
     disable => true,
   }
-  if $pacemaker_master {
-    $control_vip = hiera('tripleo::loadbalancer::controller_virtual_ip')
-    pacemaker::resource::ip { 'control_vip':
-      ip_address => $control_vip,
-    }
-    $public_vip = hiera('tripleo::loadbalancer::public_virtual_ip')
-    pacemaker::resource::ip { 'public_vip':
-      ip_address => $public_vip,
-    }
-  }
-
-  Class['::pacemaker::corosync'] -> Pacemaker::Resource::Ip <| |>
-  Class['::pacemaker::corosync'] -> Pacemaker::Resource::Ocf <| |>
-  Class['::pacemaker::corosync'] -> Pacemaker::Resource::Service <| |>
 
   # Only configure RabbitMQ in this step, don't start it yet to
   # avoid races where non-master nodes attempt to start without
@@ -162,6 +148,14 @@ if hiera('step') >= 1 {
 if hiera('step') >= 2 {
 
   if $pacemaker_master {
+    $control_vip = hiera('tripleo::loadbalancer::controller_virtual_ip')
+    pacemaker::resource::ip { 'control_vip':
+      ip_address => $control_vip,
+    }
+    $public_vip = hiera('tripleo::loadbalancer::public_virtual_ip')
+    pacemaker::resource::ip { 'public_vip':
+      ip_address => $public_vip,
+    }
     pacemaker::resource::service { 'haproxy':
       clone_params => true,
     }