Change in ovirt-engine[master]: restapi: Update Storage-Domain - handle type

oliel at redhat.com oliel at redhat.com
Wed Aug 1 07:07:08 UTC 2012


Ori Liel has submitted this change and it was merged.

Change subject: restapi: Update Storage-Domain - handle type
......................................................................


restapi: Update Storage-Domain - handle type

restapi does not have 'Master' data type. A storage-domain of type master is
simply displayed as data. When user updates a storage-domain, he might send
back <type>data</type>. Backend would fail this, because it looks like the
user is updating the type field, which is forbidden (because the old value
is 'master', and user gave 'data').

This patch intercepts the update and replaces 'master' back to 'data' if needed,
so that the backend won't fail on ilegal update

Change-Id: Iabaaa2d262a55e073d4aebe32ac76bbc9ab92d0d
Signed-off-by: Ori Liel <oliel at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainResource.java
1 file changed, 6 insertions(+), 0 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iabaaa2d262a55e073d4aebe32ac76bbc9ab92d0d
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>



More information about the Engine-commits mailing list