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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: Eli Mesika <[email protected]> Gerrit-Reviewer: Juan Hernandez <[email protected]> Gerrit-Reviewer: Ori Liel <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
