Nir Soffer has posted comments on this change.
Change subject: [WIP] core: Expose API for qemuimg commit
......................................................................
Patch Set 4:
(1 comment)
https://gerrit.ovirt.org/#/c/64222/4/lib/vdsm/qemuimg.py
File lib/vdsm/qemuimg.py:
Line 197: # If base volume is not provided, qemuimg commit will empty the top
Line 198: # volume after the operation has succeeded. Providing '-d'
option
Line 199: # will cause qemuimg commit not to empty the top volume. Note that
Line 200: # if a backing chain is provided, i.e. a base volume is provided,
Line 201: # '-d' is always implied.
This comment is repeating what we can read in the manual twice. What is missing here and
is more important, is why we don't want to empty the image - we may need to wipe the
top volume before deleting it, and emptying it may leave the data on the the underlying
storage (qemu probably mark sectors as zero instead of wiping the data).
Line 202: cmd.append("-d")
Line 203:
Line 204: if topFormat:
Line 205: cmd.extend(("-f", str(topFormat)))
--
To view, visit
https://gerrit.ovirt.org/64222
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: If7a13be40541fb268541bd8614a642263b96b487
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Ala Hino <ahino(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>
Gerrit-HasComments: Yes