Nir Soffer has posted comments on this change. Change subject: automation: Run make check over el7 in automation CI ......................................................................
Patch Set 2: pep8 still broken: 11:38:12 ./vdsm_hooks/checkips/after_get_stats.py:81:21: E126 continuation line over-indented for hanging indent 11:38:12 ./vdsm_hooks/checkips/after_get_stats.py:85:17: E121 continuation line under-indented for hanging indent 11:38:12 ./vdsm_hooks/checkips/after_get_stats.py:87:21: E126 continuation line over-indented for hanging indent 11:38:12 ./vdsm_hooks/checkips/after_get_stats.py:91:17: E121 continuation line under-indented for hanging indent 11:38:12 ./vdsm_hooks/checkips/after_get_stats.py:93:21: E126 continuation line over-indented for hanging indent 11:38:12 ./vdsm_hooks/checkips/after_get_stats.py:97:17: E121 continuation line under-indented for hanging indent Maybe there is an old version of pep8 there? We should first get rid of the platform pep8 and install pep8 using pip install pep8==x.y.z. -- To view, visit https://gerrit.ovirt.org/56122 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Iabbb09920de316a4e26447a0ca77ba9f30e517cd Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: Yaniv Bronhaim <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: No _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
