On 06/18/2012 04:18 PM, Robert Kukura wrote:
On 06/18/2012 08:56 AM, Gary Kotton wrote:
Hi,
I have yet to understand what happened. This afternoon I pushed a fix
and Jenkins failed. I got the following error:
/home/jenkins/workspace/gate-quantum-pep8/.tox/pep8/log$
/home/jenkins/workspace/gate-quantum-pep8/.tox/pep8/bin/pip install
--download-cache=/home/jenkins/workspace/gate-quantum-pep8/.tox/_download
pep8>/home/jenkins/workspace/gate-quantum-pep8/.tox/pep8/log/pep8-1.log
pep8 sdist-inst:
/home/jenkins/workspace/gate-quantum-pep8/.tox/dist/quantum-2012.2.zip
/home/jenkins/workspace/gate-quantum-pep8/.tox/pep8/log$
/home/jenkins/workspace/gate-quantum-pep8/.tox/pep8/bin/pip install
--download-cache=/home/jenkins/workspace/gate-quantum-pep8/.tox/_download
/home/jenkins/workspace/gate-quantum-pep8/.tox/dist/quantum-2012.2.zip>/home/jenkins/workspace/gate-quantum-pep8/.tox/pep8/log/pep8-2.log
pep8 runtests: commands[0]
/home/jenkins/workspace/gate-quantum-pep8$
/home/jenkins/workspace/gate-quantum-pep8/.tox/pep8/bin/pep8 --repeat
--show-source --exclude=.venv,.tox,dist,doc .
./quantum/plugins/cisco/tests/unit/test_ucs_driver.py:163:55: E251 no
spaces around keyword / parameter equals
expected_output=
This file was not touched by me. Any ideas?
I'm seeing these locally now too, and noticed tox is now picking up pep8
1.3.1 instead of 1.3.
Thanks!
I have addressed this in - https://review.openstack.org/#/c/8656/
Once Jenkins gives thumbs up can we get this approved to move forwards?
-Bob
Thanks
Gary
--
Mailing list: https://launchpad.net/~netstack
Post to : netstack@lists.launchpad.net
Unsubscribe : https://launchpad.net/~netstack
More help : https://help.launchpad.net/ListHelp
