Change in ovirt-engine[ovirt-engine-3.6]: core: Fix Storage allocation validation for live merge

tnisan at redhat.com tnisan at redhat.com
Thu May 5 08:50:04 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: core: Fix Storage allocation validation for live merge
......................................................................


core: Fix Storage allocation validation for live merge

Currently, same storage allocation validation is used both for cold
merge and live merge.  However, while for cold merge we check
allocation for disks and their child snapshots, for live merge we
only want to check allocation for disks. This patch fixes the
validation accordingly.

Change-Id: I7312f35a536ee8999a1ac73f84f7144e8a2fefa2
Bug-Url: https://bugzilla.redhat.com/1318050
Signed-off-by: Vered Volansky <vvolansk at redhat.com>
Signed-off-by: Ala Hino <ahino at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveSnapshotCommand.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RemoveSnapshotCommandTest.java
2 files changed, 6 insertions(+), 3 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved
  Ala Hino: Verified



-- 
To view, visit https://gerrit.ovirt.org/57001
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I7312f35a536ee8999a1ac73f84f7144e8a2fefa2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
Gerrit-Owner: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Ala Hino <ahino at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list