Change in ovirt-engine[master]: core: Introduce qcow compat to the engine.

Code Review gerrit at ovirt.org
Tue Nov 22 14:05:51 UTC 2016


>From Maor Lipchuk <mlipchuk at redhat.com>:

Maor Lipchuk has submitted this change and it was merged.

Change subject: core: Introduce qcow compat to the engine.
......................................................................


core: Introduce qcow compat to the engine.

1. Introduce QcowCompat enum
2. Add qcow compat field to indicate the compat level of all the
images with QCOW volume type.
3. Use an upgrade script to update all the RAW images undefined
compat level and all the QCOW volumes to the default compat 0.10.
4. Update dal layer with the new field and the dao tests.
5. Update unit tests with the new qcow compat field.

Change-Id: I5365e698b47e75c1a18ca44b58ff86b4ffedeb52
Signed-off-by: Maor Lipchuk <mlipchuk at redhat.com>
---
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/storage/disk/image/RemoveImageCommandTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/DiskImageBase.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/Image.java
A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/QcowCompat.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DiskImageDaoImpl.java
M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/ImageDaoImpl.java
M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/ImageDaoTest.java
M backend/manager/modules/dal/src/test/resources/fixtures.xml
M frontend/webadmin/modules/gwt-common/src/main/resources/org/ovirt/engine/core/Common.gwt.xml
M packaging/dbscripts/create_views.sql
M packaging/dbscripts/images_sp.sql
A packaging/dbscripts/upgrade/04_01_0510_qcow2v3_compat.sql
12 files changed, 110 insertions(+), 1 deletion(-)

Approvals:
  Maor Lipchuk: Verified; Looks good to me, approved; Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5365e698b47e75c1a18ca44b58ff86b4ffedeb52
Gerrit-PatchSet: 17
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list