Dan Kenigsberg has submitted this change and it was merged. Change subject: remoteFileHandler: improve PYTHONPATH definition ......................................................................
remoteFileHandler: improve PYTHONPATH definition The recently added import for logUtils in misc.py broke the out of process (remoteFileHandler) subsystem (ImportError: No module named logUtils). Defining PYTHONPATH with relative paths is both unreliable and insecure. In this patch: * Remove an old sys.path modification in misc.py * Improve the PYTHONPATH definition in remoteFileHandler removing the relative path Change-Id: I29dd748bcc9658a98ad86313921bfd94c96b83a6 Signed-off-by: Federico Simoncelli <[email protected]> --- M vdsm/storage/misc.py M vdsm/storage/remoteFileHandler.py 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Dan Kenigsberg: Looks good to me, approved Zhou Zheng Sheng: Verified; Looks good to me, but someone else must approve Objections: Shu Ming: I would prefer that you didn't submit this -- To view, visit http://gerrit.ovirt.org/9193 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I29dd748bcc9658a98ad86313921bfd94c96b83a6 Gerrit-PatchSet: 3 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Federico Simoncelli <[email protected]> Gerrit-Reviewer: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Shu Ming <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
