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(a)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(a)redhat.com>
Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server