run_tests should run pep8 by default

[Ed Leafe]

Just got burned because my last trivial change introduced a pep8 violation. In nova, run_tests.sh runs the pep8 check if all tests pass; we should do this in keystone, too.

[Ed Leafe (ed-leafe)]

I will add a command-line option to skip the pep8 tests, as Ziad has informed me that gerrit runs pep8 separately.

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/2763

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/2763
Committed: http://github.com/openstack/keystone/commit/44e9bf7563170531760872983c17585da97d61ea
Submitter: Jenkins
Branch: master

commit 44e9bf7563170531760872983c17585da97d61ea
Author: Ed Leafe <email address hidden>
Date: Tue Jan 3 23:16:11 2012 +0000

    Changes run_tests.sh to also run pep8 by default.

    Can be turned off by passing -P on the command line.
    Fixes bug lp911438.

    Change-Id: Icf719702d496f08b14b7855a1d99ec5419ac63bf

[Thierry Carrez (ttx)]

Adding a CI task to track the potential need now to add "-P" to gerrit/jenkins calls to run_tests

[Monty Taylor (mordred)]

This is done now - keystone runs tox directly for unittests at this point.