Change in ovirt-engine[master]: core: Enable to modify/remove PM agents via API.

emesika at redhat.com emesika at redhat.com
Mon Jun 2 14:58:25 UTC 2014


Eli Mesika has submitted this change and it was merged.

Change subject: core: Enable to modify/remove PM agents via API.
......................................................................


core: Enable to modify/remove PM agents via API.

After adding two power management agents to a host, impossible to
remove/override second agent

This patch enables to remove/modify PM agents via API.
The problem was that the Host entity given to the API mapper contained
the former settings and nothing cleans it in order to set the actual new
settings properly
This patch fixes the following flows

1) Remove PM agents
2) Add a Secondary PM agent when Primary PM agent is defined
3) Remove a Secondary PM agent
4) Remove Primary PM agent and turning the Secondary PM agent to be
primary by changing the order from 2 to 1
5) remove/modify with the backward-compatibility syntax directly from
   the power management node (a primary PM) without sending the agents
   node

Change-Id: I8a1025ccb4a50f5a5d45746b44b93c12fb14a268
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1099903
Signed-off-by: Eli Mesika <emesika at redhat.com>
---
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
1 file changed, 36 insertions(+), 5 deletions(-)

Approvals:
  Ori Liel: Looks good to me, approved
  Eli Mesika: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/28101
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8a1025ccb4a50f5a5d45746b44b93c12fb14a268
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika 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