cluster: RHEL57 - fence_ipmilan: Correct return code for diag operation

Lon Hohberger lon at fedoraproject.org
Fri Apr 8 17:10:12 UTC 2011


Gitweb:        http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=2ee5b3c42eb023692b3a6ca0d2b6241a2ef3b872
Commit:        2ee5b3c42eb023692b3a6ca0d2b6241a2ef3b872
Parent:        c303e88c2e81a10dd98583d8cbe5b6bbbf1bf0a8
Author:        Marek 'marx' Grac <mgrac at redhat.com>
AuthorDate:    Mon Apr 4 15:19:57 2011 +0200
Committer:     Lon Hohberger <lhh at redhat.com>
CommitterDate: Fri Apr 8 13:09:30 2011 -0400

fence_ipmilan: Correct return code for diag operation

Return code for diag operation should be 0.

Replaces commit 8a97abd04b8492310c1095413c5cf26bbf98e395

"diaf" != "diag"

Resolves: rhbz#678061

Signed-off-by: Lon Hohberger <lhh at redhat.com>
---
 fence/agents/ipmilan/ipmilan.c |    9 +++++++++
 1 files changed, 9 insertions(+), 0 deletions(-)

diff --git a/fence/agents/ipmilan/ipmilan.c b/fence/agents/ipmilan/ipmilan.c
index 7031982..b7b4c85 100644
--- a/fence/agents/ipmilan/ipmilan.c
+++ b/fence/agents/ipmilan/ipmilan.c
@@ -1448,6 +1448,15 @@ metaout:
 	
 	if (!strcasecmp(op, "monitor"))
           translated_ret = ret;
+        
+        if (!strcasecmp(op, "diag")) {
+          /** .. but when a system receive the DIAG signal , it 
+           ** switches to a kdump kernel, but the machine is always
+           ** "on" during the dump phase. It only become temporarily
+           ** "off" at the end of the dump just before rebooting.
+           **/
+          translated_ret = 0;
+        }
 
 	return translated_ret;
 }


More information about the cluster-commits mailing list