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

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


Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=9320b2797b90975db220e29493e7b7a1a8fa3b06
Commit:        9320b2797b90975db220e29493e7b7a1a8fa3b06
Parent:        1170dcaa2632b40df04e8f655ffa1b0eb22000ad
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: Mon Jan 30 16:50:30 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