Francesco Romani has submitted this change and it was merged. Change subject: v2v: Running virt-v2v with some sane environment ......................................................................
v2v: Running virt-v2v with some sane environment Starting virt-v2v with (almost) empty environment no longer works and virt-v2v expects at least PATH to be set properly. Otherwise it will fail to find external tools. Still, even if virt-v2v was OK with it, having empty environment is not a good idea either. Since it depends on couple external tools they may also require some basic environment to work properly. Change-Id: Iae6455c3189b7fbbf0d9d3ad5369a06b3cd6017e Bug-Url: https://bugzilla.redhat.com/1368258 Backport-To: 4.0 Backport-To: 3.6 Signed-off-by: Tomáš Golembiovský <[email protected]> Reviewed-on: https://gerrit.ovirt.org/62765 Continuous-Integration: Jenkins CI Reviewed-by: Shahar Havivi <[email protected]> Reviewed-by: Francesco Romani <[email protected]> Reviewed-on: https://gerrit.ovirt.org/62920 Tested-by: Francesco Romani <[email protected]> --- M vdsm/v2v.py 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Shahar Havivi: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Francesco Romani: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/62920 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iae6455c3189b7fbbf0d9d3ad5369a06b3cd6017e Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Francesco Romani <[email protected]> Gerrit-Reviewer: Francesco Romani <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/admin/lists/[email protected]
