cluster: RHEL57 - fence_ipmilan: returns incorrect status on monitor op if chassis is powered off

Marek Grác marx at fedoraproject.org
Mon Apr 4 16:27:33 UTC 2011


Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=48f5feec3bbd0375dff3024ce4deca7090608325
Commit:        48f5feec3bbd0375dff3024ce4deca7090608325
Parent:        3f19f28cc76c564f845fddd54a231f35410015fa
Author:        Marek 'marx' Grac <mgrac at redhat.com>
AuthorDate:    Mon Apr 4 15:11:59 2011 +0200
Committer:     Marek 'marx' Grac <mgrac at redhat.com>
CommitterDate: Mon Apr 4 18:15:32 2011 +0200

fence_ipmilan: returns incorrect status on monitor op if chassis is powered off

There is a bug in fence_ipmilan agent: it exits with result code 2 on
"monitor"/"status" command if chassis is powered off. Instead it should return
0 because "monitor" command should return fence device status, not chassis
status.

Reported and patch proposed by Vladislav Bogdanov

Resolves: rhbz#693427
---
 fence/agents/ipmilan/ipmilan.c |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/fence/agents/ipmilan/ipmilan.c b/fence/agents/ipmilan/ipmilan.c
index 4371143..7031982 100644
--- a/fence/agents/ipmilan/ipmilan.c
+++ b/fence/agents/ipmilan/ipmilan.c
@@ -1445,6 +1445,10 @@ metaout:
 	  else
 	    printf("Failed\n");
 	}
+	
+	if (!strcasecmp(op, "monitor"))
+          translated_ret = ret;
+
 	return translated_ret;
 }
 #endif /* fence */


More information about the cluster-commits mailing list