Change in ovirt-engine[master]: restapi: unable to add/remove slaves from bond using rest api

[juan.hernandez at redhat.com]

Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: unable to add/remove slaves from bond using rest api
......................................................................


restapi: unable to add/remove slaves from bond using rest api

A bug in the rest api did not allow to:
-add new slaves to a bond using the slave id
-specify exising slaves which are to remains in a bond using
the slave name
-remove existing slaves from a bond by either name or id
Another problem was that specyfing an existing slave in a rest
request caused the engine to try and add the slave again.

Change-Id: I9c55817d94e85fc10a33971043487dcad2c2c5af
Bug-Url: https://bugzilla.redhat.com/1262275
Signed-off-by: Marcin Mirecki <mmirecki at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostResource.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostNicMapper.java
2 files changed, 22 insertions(+), 0 deletions(-)

Approvals:
  Juan Hernandez: Looks good to me, approved
  Marcin Mirecki: Verified
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9c55817d94e85fc10a33971043487dcad2c2c5af
Gerrit-PatchSet: 11
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Marcin Mirecki <mmirecki at redhat.com>
Gerrit-Reviewer: Martin Mucha <mmucha at redhat.com>
Gerrit-Reviewer: automation at ovirt.org