Change in vdsm[master]: Extend Image.move to move and copy subchains

ilvovsky at redhat.com ilvovsky at redhat.com
Mon Apr 2 08:19:16 UTC 2012


Igor Lvovsky has posted comments on this change.

Change subject: Extend Image.move to move and copy subchains
......................................................................


Patch Set 1: Looks good to me, but someone else must approve

(1 inline comment)

....................................................
File vdsm/storage/image.py
Line 634:                 #     dstVol.setInternal()
Maybe you are right. It looks like we already have proper type volume from _createTargetImage.
Just verify whether it's really works

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Icb11818f830c6f02bcb74d69f1c119efaf100cc2
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list