Daniel Erez has uploaded a new change for review.

Change subject: core: BaseImagesCommand - update disk volume type
......................................................................

core: BaseImagesCommand - update disk volume type

BaseImagesCommand -> endSuccessfully: update disk's VolumeType/VolumeFormat.
Specifically in bug context, needed when moving a raw/thin provisioned disk
from file domain to block domain.

Change-Id: I34f32dfdc0dcf8bad14d114371d9bd0a7314913c
Bug-Url: https://bugzilla.redhat.com/1098258
Signed-off-by: Daniel Erez <de...@redhat.com>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/13/27813/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java
index 3c235f4..2dca01f 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/BaseImagesCommand.java
@@ -320,6 +320,10 @@
 
                 if (newImageIRS != null) {
                     completeImageData(newImageIRS);
+
+                    // Set volume type/format before updating DB in the 
'finally' branch
+                    
getDestinationDiskImage().getImage().setVolumeType(newImageIRS.getVolumeType());
+                    
getDestinationDiskImage().getImage().setVolumeFormat(newImageIRS.getVolumeFormat());
                 }
             } catch (VdcBLLException e) {
                 // Logging only


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I34f32dfdc0dcf8bad14d114371d9bd0a7314913c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.4
Gerrit-Owner: Daniel Erez <de...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to