Bug 693428

Summary:	fence_ipmilan returns incorrect status on monitor op if chassis is powered off
Product:	Red Hat Enterprise Linux 6	Reporter:	Marek Grac <mgrac>
Component:	fence-agents	Assignee:	Marek Grac <mgrac>
Status:	CLOSED ERRATA	QA Contact:	Cluster QE <mspqa-list>
Severity:	high	Docs Contact:
Priority:	medium
Version:	6.2	CC:	bubble, cluster-maint, djansa, fdinitto, rsteiger
Target Milestone:	rc
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	fence-agents-3.1.4-1.el6	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:	690735	Environment:
Last Closed:	2011-12-06 12:22:43 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Bug Depends On:	690735
Bug Blocks:	693426, 693427

Description Marek Grac 2011-04-04 15:58:43 UTC

+++ This bug was initially created as a clone of Bug #690735 +++

Description of problem:
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.

Version-Release number of selected component (if applicable):
fence-agents-3.1.2-1.fc13.x86_64

--- Additional comment from bubble on 2011-03-25 04:24:11 EDT ---

Created attachment 487504 [details]
Patch to fix issue

--- Additional comment from mgrac on 2011-03-25 06:32:16 EDT ---

Created attachment 487518 [details]
Proposed patch

@Vladislav:

Thanks for reporting this bug. You are right, we should return 0 if the fence device is working. 

I have created a patch that should fix this. Yours patch will work correctly for 'monitor' but it will break 'status' as fence agent won't be able to return that chassis is powered off.

--- Additional comment from bubble on 2011-03-25 08:11:02 EDT ---

Looking through the code, it was probably just forgotten (ret is set to 0 for both on and off states but not used then).

I agree, your patch is cleaner.

Comment 10 errata-xmlrpc 2011-12-06 12:22:43 UTC

Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2011-1599.html