Zhou Zheng Sheng has posted comments on this change. Change subject: vdsm-tool: speedup loading seboolsetup.py ......................................................................
Patch Set 1: RHEL Jenkins failure is not related to this patch. In http://jenkins.ovirt.org/job/vdsm_unit_tests_gerrit_el/2337/console , the log is /usr/bin/pep8 --exclude="config.py,constants.py" --filename '*.py,*.py.in' \ client lib/cpopen/*.py lib/vdsm/*.py lib/vdsm/*.py.in tests vds_bootstrap vdsm-tool vdsm/*.py vdsm/*.py.in vdsm/netconf vdsm/sos/vdsm.py.in vdsm/storage vdsm/vdsm vdsm_api vdsm_hooks vdsm_reg vdsm_hooks/openstacknet/after_device_create.py:56:8: E128 continuation line under-indented for visual indent This is related to a previous patch merged, and that patch is actually clean. The pep8 tool on RHEL slave is too old. -- To view, visit http://gerrit.ovirt.org/16560 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7f961e2661159f671b17f3ab4fb3d5a7cbc3d6b9 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Zhou Zheng Sheng <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Zhou Zheng Sheng <[email protected]> Gerrit-Reviewer: oVirt Jenkins CI Server _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
