Change in vdsm[master]: v2v: add format for disk conversion

danken at redhat.com danken at redhat.com
Tue May 19 10:25:34 UTC 2015


Dan Kenigsberg has posted comments on this change.

Change subject: v2v: add format for disk conversion
......................................................................


Patch Set 5:

(1 comment)

https://gerrit.ovirt.org/#/c/40343/5/vdsm/v2v.py
File vdsm/v2v.py:

Line 418:     def _create_command(self):
Line 419:         cmd = [_VIRT_V2V.cmd,
Line 420:                '-ic', self._uri,
Line 421:                '-o', 'vdsm',
Line 422:                '-of', self._vminfo.get('format', 'raw')]
should be added as optional? on @ExternalVmInfo:
Line 423:         cmd.extend(self._generate_disk_parameters())
Line 424:         cmd.extend(['--password-file',
Line 425:                     self._passwd_file,
Line 426:                     '--vdsm-vm-uuid',


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I1da72771c9b02fa741cc84aa7159e862675fb577
Gerrit-PatchSet: 5
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Michal Skrivanek <mskrivan at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Shahar Havivi <shavivi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-HasComments: Yes


More information about the vdsm-patches mailing list