Change in ovirt-engine[master]: webadmin: Fix a bug in edit storage domain

tnisan at redhat.com tnisan at redhat.com
Wed Dec 31 14:07:13 UTC 2014


Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: Fix a bug in edit storage domain
......................................................................


webadmin: Fix a bug in edit storage domain

Patch I4e0d5a1c9158be1284ffe05d1bf1397c9113f6c0 introduced the
wipe_after_delete column in storage_domain_static table with a NOT NULL
constraint.
The edit storage domain operation in webadmin clones the editted object and
sends it to the engine for the entity updated, however, the clone storage
domain method did not clone the item correctly and omitted the wipe after
delete property from the clone thus sending null instead of the original
value, causing the update to fail with a NOT NULL constraint

Change-Id: Ia49b3460af6231356b7255d8b6322c98eef59406
Bug-url: https://bugzilla.redhat.com/1177931
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  Tal Nisan: Verified
  Allon Mureinik: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia49b3460af6231356b7255d8b6322c98eef59406
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vered Volansky <vvolansk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list