Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 1191478

Summary:	[backend] When Host PM is disabled it should not show 'type="apc"'
Product:	Red Hat Enterprise Virtualization Manager	Reporter:	Jiri Belka <jbelka>
Component:	ovirt-engine	Assignee:	Ori Liel <oliel>
Status:	CLOSED CURRENTRELEASE	QA Contact:	Jiri Belka <jbelka>
Severity:	low	Docs Contact:
Priority:	unspecified
Version:	3.5.0	CC:	gklein, lpeer, lsurette, oourfali, pstehlik, rbalakri, Rhev-m-bugs, srevivo, ykaul
Target Milestone:	ovirt-3.6.0-rc
Target Release:	3.6.0
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-04-20 01:29:15 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Infra	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Jiri Belka 2015-02-11 11:26:28 UTC

Description of problem:

When Host PM is disabled it should not show 'type="apc"', it does not make sense at all.

<power_management type="apc"><enabled>false</enabled><options><option name="secure" value="false"/></options><automatic_pm_enabled>true</automatic_pm_enabled><kdump_detection>true</kdump_detection></power_management>

Either it should not be shown at all (no <power_management>) or 'type="apc"' should not be visible.

Version-Release number of selected component (if applicable):
rhevm-backend-3.5.0-0.31.el6ev.noarch

How reproducible:
100%

Steps to Reproduce:
1. add host without configuring PM
2. check /api/hosts via restapi
3.

Actual results:
some default is present it the output which makes no sense

Expected results:

Either it should not be shown at all (no <power_management>) or 'type="apc"' should not be visible.

Additional info:

Comment 1 Oved Ourfali 2015-02-14 07:51:27 UTC

Martin, please make sure it is fixed in the UI rework for this feature.

Comment 2 Oved Ourfali 2015-04-12 11:44:56 UTC

My bad... as this is an issue with the API, not the UI.
Assigning to Ori to have a look.

Comment 3 Ori Liel 2015-04-29 08:15:35 UTC

This field should be deprecated altogether because 'type' is a property of the fence-agent, not of the entire Power-Management. I posted a patch which deprecates it and also fixes the problem described here:

  https://gerrit.ovirt.org/#/c/40382/

Comment 4 Ori Liel 2015-04-29 08:29:50 UTC

patch merged.

Comment 5 Jiri Belka 2015-06-18 09:31:16 UTC

ok, api is ok, admin portal has still the issue but that's different topic.

ovirt-engine-backend-3.6.0-0.0.master.20150519172219.git9a2e2b3.el6.noarch

<power_management><enabled>false</enabled><pm_proxies/><automatic_pm_enabled>true</automatic_pm_enabled><kdump_detection>true</kdump_detection></power_management>