Ironic pep8 tests in fact only run flake8. pep8 reports many issues with Ironic code.
Running the following on Ironic code base:
$ pep8 --version
1.5.7
$ pep8 | wc
1009 8071 101963
$ pep8 | head
./ironic/conductor/manager.py:86:9: E126 continuation line over-indented for hanging indent
./ironic/conductor/manager.py:110:20: E128 continuation line under-indented for visual indent
./ironic/conductor/manager.py:111:20: E128 continuation line under-indented for visual indent
./ironic/conductor/manager.py:136:20: E128 continuation line under-indented for visual indent
./ironic/conductor/manager.py:137:20: E128 continuation line under-indented for visual indent
./ironic/conductor/manager.py:144:20: E128 continuation line under-indented for visual indent
./ironic/conductor/manager.py:145:20: E128 continuation line under-indented for visual indent
./ironic/conductor/manager.py:148:25: E124 closing bracket does not match visual indentation
./ironic/conductor/manager.py:209:33: E126 continuation line over-indented for hanging indent
./ironic/conductor/manager.py:254:50: E127 continuation line over-indented for visual indent
$ pep8 | tail
./ironic/drivers/modules/ilo/management.py:35:32: E127 continuation line over-indented for visual indent
./ironic/drivers/modules/ilo/management.py:36:30: E124 closing bracket does not match visual indentation
./ironic/drivers/modules/ilo/management.py:148:18: E128 continuation line under-indented for visual indent
./ironic/drivers/modules/ilo/power.py:153:17: E128 continuation line under-indented for visual indent
./ironic/drivers/modules/ilo/power.py:159:20: E127 continuation line over-indented for visual indent
./ironic/drivers/modules/ilo/power.py:160:22: E127 continuation line over-indented for visual indent
./ironic/drivers/modules/ilo/power.py:172:21: E127 continuation line over-indented for visual indent
./doc/source/conf.py:63:1: E265 block comment should start with '# '
./doc/source/conf.py:64:1: E265 block comment should start with '# '
./doc/source/conf.py:65:1: E265 block comment should start with '# '
I will discuss this with Ironic team and see if they are amenable to a patch to fix these issues and then adding 'pep8' to the testing to prevent future issues.
Hi! I suspect that it's due to use hacking and some rules being disabled/modified. I think it should be fixed (if ever) on hacking level, not per-project. But please feel free to seek other's opinions.