fence-agents: master - fence_ipmilan: Correct return code for diag operation

Lon Hohberger lon at fedoraproject.org
Mon Apr 4 20:15:21 UTC 2011


Gitweb:        http://git.fedorahosted.org/git/fence-agents.git?p=fence-agents.git;a=commitdiff;h=550a6ca4caee40bdf669e563e66371b23e564d6e
Commit:        550a6ca4caee40bdf669e563e66371b23e564d6e
Parent:        ed60a64a1eba5c941edaf7802e2d6ab3c638aaa5
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: Mon Apr 4 16:12:28 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#655764

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 040babf..892fc4e 100644
--- a/fence/agents/ipmilan/ipmilan.c
+++ b/fence/agents/ipmilan/ipmilan.c
@@ -1195,6 +1195,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