Change in ovirt-engine[master]: restapi: Null check on domain volume group when attempting t...

juan.hernandez at redhat.com juan.hernandez at redhat.com
Wed Nov 19 09:59:10 UTC 2014


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Null check on domain volume group when attempting to extend domain
......................................................................


restapi: Null check on domain volume group when attempting to extend domain

When editing a block domain via REST, the LUN passed in the new entity can
come either from the LUNs collection and in case it's empty from the volume
group property and it's LUNs, added a null check on the volume group since
when you do not provide LUNs it automatically falls back to volume group
which might be null

Change-Id: Ic762bdfb4c01d7b9344c94878d6662b9866a4090
Bug-Url: https://bugzilla.redhat.com/1115966
Signed-off-by: Tal Nisan <tnisan at redhat.com>restapi:
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainResource.java
1 file changed, 11 insertions(+), 2 deletions(-)

Approvals:
  Tal Nisan: Verified
  Juan Hernandez: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic762bdfb4c01d7b9344c94878d6662b9866a4090
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list