Change in ovirt-engine[master]: restapi: Don't display PM type when PM disabled (#1191478)

oliel at redhat.com oliel at redhat.com
Wed Apr 29 08:29:09 UTC 2015


Ori Liel has submitted this change and it was merged.

Change subject: restapi: Don't display PM type when PM disabled (#1191478)
......................................................................


restapi: Don't display PM type when PM disabled (#1191478)

Currenly Power-Management type is disaplyed even when
Power-Managemnent is disabled for the host. This patch
fixed that.

This patch also declares this field as deprecated, since
speaking of the 'type' of Power-Management doesn't make
sense since intorduction of fence-agents. Type should be
associated with a specific agent, not the entire PM.
It was not deleted from PM to maintain backwards
compatibility.

Bug-Url: http://bugzilla.redhat.com/1191478
Change-Id: I0630ebde2b342964f44c3fad51de1ca5032b5c23
Signed-off-by: Ori Liel <oliel at redhat.com>
---
M backend/manager/modules/restapi/interface/definition/src/main/resources/api.xsd
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/DeprecatedPowerManagementMapper.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/DeprecatedPowerManagementMapperTest.java
4 files changed, 14 insertions(+), 4 deletions(-)

Approvals:
  Ori Liel: Verified; Looks good to me, approved
  Juan Hernandez: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/40382
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I0630ebde2b342964f44c3fad51de1ca5032b5c23
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list