Shahar Havivi has posted comments on this change. Change subject: v2v: Log detailed output of virt-v2v ......................................................................
Patch Set 1: (1 comment) https://gerrit.ovirt.org/#/c/59834/1/lib/vdsm/v2v.py File lib/vdsm/v2v.py: Line 383: self._vmid = vmid Line 384: self._irs = irs Line 385: self._prepared_volumes = [] Line 386: self._passwd_file = os.path.join(_V2V_DIR, "%s.tmp" % vmid) Line 387: self._base_command = [_VIRT_V2V.cmd, '-v', '-x'] > Tomas, good point. The import process is sporadic enough that we can afford yes, I am convinced too, if its not cost too much it worth it!, I am seeing some mailing list with questions that with that log it will make easier investigation. Line 388: Line 389: def execute(self): Line 390: raise NotImplementedError("Subclass must implement this") Line 391: -- To view, visit https://gerrit.ovirt.org/59834 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I6a8d9284316a551edeaffdd66dfcd299fa02478e Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Tomas Golembiovsky <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Polednik <[email protected]> Gerrit-Reviewer: Milan Zamazal <[email protected]> Gerrit-Reviewer: Shahar Havivi <[email protected]> Gerrit-Reviewer: Tomas Golembiovsky <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
