Bug 785796

Summary: [webadmin] Host's PM test - Wrong message when there is no fencing proxy [TEXT]
Product: [Retired] oVirt Reporter: zvi <zfridler>
Component: ovirt-engine-webadminAssignee: Alona Kaplan <alkaplan>
Status: CLOSED CURRENTRELEASE QA Contact:
Severity: low Docs Contact:
Priority: unspecified    
Version: unspecifiedCC: acathrow, ecohen, iheim, yeylon
Target Milestone: ---   
Target Release: 3.1   
Hardware: x86_64   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-09 08:02:14 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Attachments:
Description Flags
engine log none

Comment 1 zvi 2012-01-30 16:23:34 UTC
zvi 2012-01-30 11:18:44 EST

Created attachment 558399 [details]
engine log

Description of problem:
Wrong message when there is no fencing proxy .

Version-Release number of selected component (if applicable):


How reproducible:

Always
Steps to Reproduce:
1.Attempt to tests Host's PM while there are no other Hosts, or all the Hosts are non responsive
ones

2.
3.

Actual results:

Message: "Test Failed (unknown error)."

Expected results:

Message: "No running proxy Host was found."


Additional info:
22) DbUserCacheManager::refreshAllUserData() - entered
2012-01-30 17:57:23,401 ERROR [org.ovirt.engine.core.bll.FencingExecutor]
(http--0.0.0.0-8080-5) Failed to run Power Management command on Host , no
running proxy Host was found.
2012-01-30 17:57:52,245 ERROR [org.ovirt.engine.core.bll.FencingExecutor]
(http--0.0.0.0-8080-4) Failed to run Power Management command on Host , no
running proxy Host was found.

Comment 2 Einav Cohen 2012-02-05 10:47:56 UTC
In HostModel, Test() method: Currently, in case the GetNewVdsFenceStatus query has returned with failure, we are always showing the "Test Failed (unknown error)" message.

We need to check the FenceStatusReturnValue returned value also in case of failure (currently we do it only in case of query success). Only if, for some reason, there isn't an instance of FenceStatusReturnValue in the query returned value, then need to display the "Test Failed (unknown error)" message.

Comment 4 Alona Kaplan 2012-02-21 08:30:06 UTC
fixed in commit e61e9ae8880d95332c7abd59ecb6cc59a9152486

Comment 5 Itamar Heim 2012-08-09 08:02:14 UTC
closing ON_QA bugs as oVirt 3.1 was released:
http://www.ovirt.org/get-ovirt/

Comment 6 Itamar Heim 2012-08-09 08:03:32 UTC
closing ON_QA bugs as oVirt 3.1 was released:
http://www.ovirt.org/get-ovirt/