From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 26977 invoked by alias); 27 Aug 2008 12:59:01 -0000 Received: (qmail 26965 invoked by alias); 27 Aug 2008 12:59:00 -0000 X-Spam-Status: No, hits=0.2 required=5.0 tests=AWL,BAYES_40,KAM_MX,SPF_HELO_PASS X-Spam-Check-By: sourceware.org X-Spam-Checker-Version: SpamAssassin 3.2.4 (2008-01-01) on bastion.fedora.phx.redhat.com X-Spam-Level: Subject: RHEL5 - [FENCE] Fix #448133 - LPAR/HMC fence agent To: cluster-cvs-relay@redhat.com X-Project: Cluster Project X-Git-Module: cluster.git X-Git-Refname: refs/heads/RHEL5 X-Git-Reftype: branch X-Git-Oldrev: 31331549456cc1418641a6783ad9f5ee7a188d8a X-Git-Newrev: 34db56d4f5ea7428c02b072a742c40ca9c574f1c From: =?utf-8?q?Marek_Gr=C3=A1c?= Message-Id: <20080827125755.6C3DB120369@lists.fedorahosted.org> Date: Wed, 27 Aug 2008 13:00:00 -0000 X-Scanned-By: MIMEDefang 2.58 on 172.16.52.254 Mailing-List: contact cluster-cvs-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Post: List-Help: , Sender: cluster-cvs-owner@sourceware.org X-SW-Source: 2008-q3/txt/msg00326.txt.bz2 Gitweb: http://git.fedorahosted.org/git/cluster.git?p=cluster.git;a=commitdiff;h=34db56d4f5ea7428c02b072a742c40ca9c574f1c Commit: 34db56d4f5ea7428c02b072a742c40ca9c574f1c Parent: 31331549456cc1418641a6783ad9f5ee7a188d8a Author: Marek 'marx' Grac AuthorDate: Wed Aug 27 14:37:31 2008 +0200 Committer: Marek 'marx' Grac CommitterDate: Wed Aug 27 14:55:10 2008 +0200 [FENCE] Fix #448133 - LPAR/HMC fence agent Minor fix (thanks to brking@us.ibm.com) in get_power_status(). If the state is not 'Running' then it is considered off, originally function returns undefined value in specific cases (like panic on machine - 'Error'). --- fence/agents/lpar/fence_lpar.py | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/fence/agents/lpar/fence_lpar.py b/fence/agents/lpar/fence_lpar.py index b2d722f..aeb6a9a 100755 --- a/fence/agents/lpar/fence_lpar.py +++ b/fence/agents/lpar/fence_lpar.py @@ -36,8 +36,7 @@ def get_power_status(conn, options): ## Transformation to standard ON/OFF status if possible if status == "Running": status = "on" - - if status == "Not Activated": + else: status = "off" return status