Adam Litke has posted comments on this change.

Change subject: storage: Use mismatched_vm_conf_disk workaround in copy_data
......................................................................


Patch Set 1:

(2 comments)

https://gerrit.ovirt.org/#/c/64231/1/vdsm/storage/sdm/api/copy_data.py
File vdsm/storage/sdm/api/copy_data.py:

Line 62
Line 63
Line 64
Line 65
Line 66
> If we add a is_bad_vm_conf() method to DivEndpoint:
I am not splitting the test from the fix in workaround.detect_format anymore.  
Therefore, I'll go back to putting this inside CopyDataDIVEndpoint.qemu_format.


Line 119:             if workarounds.mismatched_vm_conf_disk(self._vol):
Line 120:                 self._qemu_format = qemuimg.FORMAT.RAW
Line 121:             else:
Line 122:                 self._qemu_format = sc.fmt2str(self._vol.getFormat())
Line 123:         return self._qemu_format
> We need this check only for the source image, and then we want to apply the
I agreed with you earlier but in reality I think it needs to be here.  
workarounds.detect_format takes volume object but even if we changed it to 
accept the three scalar values needed to make the decision (vdsm md format, 
path, and size) this logic is not endpoint agnostic.  The only object which 
should supply a vdsm volume metadata value is a DIVEndpoint.  For this reason, 
I want to keep it here in the property since it is relevant only for vdsm 
volume source endpoints.  If we make the assumption that we are not changing 
the format on the fly when we copy then we will just use this value for the 
source and destination format value when calling qemuimg.convert.
Line 124: 
Line 125:     @property
Line 126:     def backing_path(self):
Line 127:         parent_vol = self._vol.getParentVolume()


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Iecfd3e0fd5923e9a333379ab21e01c9b12def78c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Adam Litke <ali...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org
To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org

Reply via email to