Change in ovirt-engine[master]: restapi: Fix update cluster network (#822631)

mkolesni at redhat.com mkolesni at redhat.com
Mon Jul 9 07:18:11 UTC 2012


Mike Kolesnik has submitted this change and it was merged.

Change subject: restapi: Fix update cluster network (#822631)
......................................................................


restapi: Fix update cluster network (#822631)

https://bugzilla.redhat.com/822631

The REST should use UpdateNetworkOnCluster command when updating a
cluster's network assignment.

There is no need to filter solely based on display network, since the
assignment can have more characteristics to it, and it's the
responsibility of the engine to update & validate whatever is needed.

Change-Id: I3ba860433d0a77caad6405fb30b9b29b0af3a847
Signed-off-by: Mike Kolesnik <mkolesni at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendClusterNetworkResource.java
M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendClusterNetworkResourceTest.java
2 files changed, 10 insertions(+), 13 deletions(-)

Approvals:
  Mike Kolesnik: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I3ba860433d0a77caad6405fb30b9b29b0af3a847
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern at redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkolesni at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>



More information about the Engine-commits mailing list