From 3c11b43c5feed58d64dc2b178ced638ece2ee450 Mon Sep 17 00:00:00 2001
From: Robert <taeil.choi@kt.com>
Date: Fri, 19 Apr 2013 10:56:05 +0900
Subject: [PATCH] revert only_if commands and change to use short node.name

---
 attributes/default.rb |  4 ++--
 recipes/config.rb     | 38 +++++++++++++++++++-------------------
 2 files changed, 21 insertions(+), 21 deletions(-)

diff --git a/attributes/default.rb b/attributes/default.rb
index e672c56..65c71d1 100644
--- a/attributes/default.rb
+++ b/attributes/default.rb
@@ -1,3 +1,4 @@
+# Only use short hostname (not FQDN)
 default['pacemaker']['nodes'] = ['node1', 'node2']
 
 default['pacemaker']['primitive']['drbd']['agent'] = "ocf:linbit:drbd"
@@ -18,8 +19,7 @@ default['pacemaker']['primitive']['clvm']['op']['monitor']['interval'] = "5s"
 default['pacemaker']['primitive']['clvm']['op']['monitor']['on-fail'] = "restart"
 default['pacemaker']['primitive']['clvm']['active'] = "#{pacemaker['nodes']}"
 
-# Temporary attribute for vip resource.
-# Later, vip address will be set by 'ktc-cinder' cookcook and be fetched here.
+# Vip address might be replaced by cinder-volume's myip specified in the environment.
 default['pacemaker']['primitive']['vip']['agent'] = "ocf:heartbeat:IPaddr2"
 default['pacemaker']['primitive']['vip']['params']['ip'] = "10.5.2.200"
 default['pacemaker']['primitive']['vip']['params']['cidr_netmask'] = "24"
diff --git a/recipes/config.rb b/recipes/config.rb
index e6cebe9..1e98118 100644
--- a/recipes/config.rb
+++ b/recipes/config.rb
@@ -54,7 +54,7 @@ node['pacemaker']['primitive'].each do |name, attr|
     meta attr['meta']
     op attr['op']
     action :create
-    only_if attr['active'].include?(node.name)
+    only_if { attr['active'].include?(node.name.partition(".")[0]) }
   end
 end
 
@@ -64,7 +64,7 @@ node['pacemaker']['location'].each do |name, attr|
     priority attr['priority']
     loc attr['loc']
     action :create
-    only_if attr['active'].include?(node.name)
+    only_if { attr['active'].include?(node.name.partition(".")[0]) }
   end
 end
 
@@ -73,7 +73,7 @@ node['pacemaker']['ms'].each do |name, attr|
     rsc attr['rsc_name']
     meta attr['meta']
     action :create
-    only_if attr['active'].include?(node.name)
+    only_if { attr['active'].include?(node.name.partition(".")[0]) }
   end
 end
 
@@ -82,27 +82,27 @@ node['pacemaker']['clone'].each do |name, attr|
     rsc attr['rsc_name']
     meta attr['meta']
     action :create
-    only_if attr['active'].include?(node.name)
+    only_if { attr['active'].include?(node.name.partition(".")[0]) }
   end
 end
 
 node['pacemaker']['colocation'].each do |name, attr|
-    pacemaker_colocation name do
-      priority attr['priority']
-      multiple attr['is_multiple']
-      rsc attr['rsc']
-      with_rsc attr['with_rsc']
-      multiple_rscs attr['multiple_rscs']
-      action :create
-      only_if attr['active'].include?(node.name)
-    end
+  pacemaker_colocation name do
+    priority attr['priority']
+    multiple attr['is_multiple']
+    rsc attr['rsc']
+    with_rsc attr['with_rsc']
+    multiple_rscs attr['multiple_rscs']
+    action :create
+    only_if { attr['active'].include?(node.name.partition(".")[0]) }
+  end
 end
 
 node['pacemaker']['order'].each do |name, attr|
-    pacemaker_order name do
-      priority attr['priority']
-      resources attr['resources']
-      action :create
-      only_if attr['active'].include?(node.name)
-    end
+  pacemaker_order name do
+    priority attr['priority']
+    resources attr['resources']
+    action :create
+    only_if { attr['active'].include?(node.name.partition(".")[0]) }
+  end
 end