fence-agents: master - fence_rhevm: Incorrect status detected when VM is not in up/down

Marek GrĂ¡c marx at fedoraproject.org
Wed Feb 8 16:38:17 UTC 2012


Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=155370d65d5c027b10ef878c120e26c106625f7c
Commit:        155370d65d5c027b10ef878c120e26c106625f7c
Parent:        57ae9aa78b1d6bed89551944a8e0b5fa6c1f0075
Author:        Marek 'marx' Grac <mgrac at redhat.com>
AuthorDate:    Mon Jan 30 16:50:30 2012 +0100
Committer:     Marek 'marx' Grac <mgrac at redhat.com>
CommitterDate: Wed Feb 8 17:37:46 2012 +0100

fence_rhevm: Incorrect status detected when VM is not in up/down

For fencing purposes we have to focus on getting OFF status
when machine is really 100% down, conditions for ON status are much simplier
(as fact that machine is has power does not mean that it works). This patch
specify condition for OFF status - in fact only 'Down' is accepted now

Resolves: rhbz#769681
---
 fence/agents/rhevm/fence_rhevm.py |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/fence/agents/rhevm/fence_rhevm.py b/fence/agents/rhevm/fence_rhevm.py
index 65a7395..3578b5f 100644
--- a/fence/agents/rhevm/fence_rhevm.py
+++ b/fence/agents/rhevm/fence_rhevm.py
@@ -41,10 +41,10 @@ def get_power_status(conn, options):
 	else:
 		status = result.group(1)
 
-	if (status == "up"):
-		return "on"
-	else:
+	if (status.lower() == "down"):
 		return "off"
+	else:
+		return "on"
 
 def set_power_status(conn, options):
 	action = {


More information about the cluster-commits mailing list