Change in ovirt-engine[ovirt-engine-3.3.1]: core: empty quota causes FK violation

[iheim at redhat.com]

Itamar Heim has submitted this change and it was merged.

Change subject: core: empty quota causes FK violation
......................................................................


core: empty quota causes FK violation

When creating a disk in Foreman, the disk object is sent via
REST with empty string in id attribute:
<quota id=""/>

REST creates an empty UUID (000...) out of empty string, and causes
FK violation, since there is no quota with that id.

This patch fixes it by ignoring empty quota for adding disk.

Change-Id: Ic488486edc7c5602d24e804c9a89aeccc98fc32b
Bug-Url: https://bugzilla.redhat.com/1022942
Signed-off-by: Gilad Chaplik <gchaplik at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddDiskCommand.java
1 file changed, 5 insertions(+), 2 deletions(-)

Approvals:
  Sandro Bonazzola: Looks good to me, but someone else must approve
  Itamar Heim: Verified; Looks good to me, approved
  Gilad Chaplik: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ic488486edc7c5602d24e804c9a89aeccc98fc32b
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3.1
Gerrit-Owner: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server