| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
The error was:
'import install_venv_common as install_venv' is a relative import
To fix it __init__.py was created and import string was fixed.
Requirements for hacking changed to >=0.5.6
Change-Id: I71aa7c7cb268c26aa00828306138a84b30289cb9
|
|
|
|
|
|
|
| |
We found that if tests run with testr --subunit option it can cause
wrong implementation of tests results.
Change-Id: I98c3f3ed90e2291ef984f07a93c89d24da7855d9
|
|\ |
|
| |
| |
| |
| |
| |
| |
| | |
There is no point to ignore this check because everything works
correctly without it.
Change-Id: I38b6f2314208b3bfb5898b2a08d7fe7c0f640e57
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
Config tox.ini updated to run testr instead of nosetests.
File test-requirements.txt modified respectively.
Config .testr.conf added.
Testr allows for more flexible testing, more consistent use of the
unittest protocol and, most importantly, faster testing via parallelism.
Part of blueprint run-tests-script.
Change-Id: Iba847d430a18458a1a68e6ef07541d29c0c01c64
|
|
|
|
|
|
| |
H404 - multi line docstring should start with a summary.
Change-Id: I2099e1ee81ff9657f7a07401b8e8f3327d03bdbd
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Rename tools/pip-requires to requirements.txt and tools/test-requires
to test-requirements.txt. These are standard files, and tools in the
general world are growing intelligence about them.
Fixes: bug #1179008
Change-Id: I033876ec304fea97c66365c16bcf0cccaa1d810e
|
| |
| |
| |
| |
| |
| | |
H306 - imports not in alphabetical order.
Change-Id: Iaa93bcafed562833318ac86241ad36662c2d8ca6
|
|\ \ |
|
| |/
| |
| |
| |
| |
| |
| |
| | |
Introduce py33 to tox.ini to make testing with
python3 easier.
Change-Id: I9a5a7e8258afc31b9d29b2845d98e7e9c552e88d
Signed-off-by: Chuck Short <chuck.short@canonical.com>
|
|/
|
|
|
|
| |
H402 one line docstring needs punctuation
Change-Id: Ie848453cace318d8310cdf0234c512f4c1121119
|
|\ |
|
| |
| |
| |
| |
| |
| | |
H403 multi line docstring end on new line
Change-Id: I33249651026b54ef346214965e909835288bb14e
|
|\| |
|
| |
| |
| |
| |
| |
| | |
H401 docstring should not start with a space
Change-Id: I761665ea9aff1f55de9f629ff8d108ba141eaf79
|
|\| |
|
| |
| |
| |
| |
| |
| | |
H201 no 'except:' at least use 'except Exception:'
Change-Id: Ibfab55ace63636d3507fbdad46d1436b8ec83475
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| | |
We're not setting the VIRTUAL_ENV environment variable, so 'tox -ecover'
is failing because patch_tox_venv uses the variable.
Change-Id: I3898d0dd1f8a9bf4216253bb2c7d8491e1e8ce56
|
| |
| |
| |
| |
| |
| |
| | |
Fix and enable test for 'formatting operation should be outside of
localization method call'
Change-Id: I0e707f9d4aacf75f9a9239b323a57d7d39f30c84
|
|/
|
|
|
|
|
| |
Fix and enable test for multiple positional placeholders in
localizations
Change-Id: Ie3b5e3113cd0f3340b7b0f5774001893b1e6857c
|
|
|
|
|
|
|
| |
Include hacking in test-requires and disable any failing tests.
Once failing tests are fixed they can be enabled to gate on them.
Change-Id: Id6a228441ec94d5d0bee1dba05118bb35ecaa001
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The eventlet on PyPI basically doesn't work on RHEL because we (Red Hat)
screwed up by not taking monkey patching into account when backporting
an API from python 3. This is the series of events:
1) We backported a patch to RHEL6 from Python 3 which added an
optional 'timeout' argument to subprocess.Popen:
http://pkgs.devel.redhat.com/cgit/rpms/python/commit/?h=rhel-6.3&id=fc9a3f0e07
This might sound innocuous but it means the wait() function get
called with a timeout argument
2) eventlet overrides this function, but doesn't know about the
timeout argument, so it fails. To reproduce, try applying this to
your python 2.7:
https://gist.github.com/markmc/5500967
3) eventlet doesn't yet support Python 3, so the only time this issue
crops up is if you use eventlet on RHEL6
4) We've fixed eventlet in EPEL6, but the issue still remains if you
install eventlet from upstream
5) Dave Malcolm sent a pull request to eventlet, but it has stalled:
https://bitbucket.org/eventlet/eventlet/pull-request/30/add-dummy-timeout-parameter-to
Nova has long had a nasty workaround to patch eventlet in our virtualenv
and we now need this workaround in oslo-incubator to test processutils.
Let's add it here and make nova sync it from oslo-incubator.
I really hope we can get eventlet upstream "fixed" and the fix released
soon. When that happens, we can kill all this nastiness.
Change-Id: I62ce43a330d7ae94eda4c7498782a655e63747fa
|
|
|
|
|
|
|
| |
flake8 is pluggable and handles pep8 and pyflakes, as well as configuration
through tox.ini. It also removes the need for flakes.py.
Change-Id: If5f7d8ad348b4fb8119fa4ec7b5e9d17bdc72a39
|
|
|
|
|
|
| |
Fixes bug #1157597
Change-Id: I77e0ddb1babf1eec0bb953b8bfa52cd1d415f709
|
|
|
|
|
|
|
| |
pyflakes 0.6.1 changes a module path - but we want to use it, because it
finally groks attributes properly.
Change-Id: Id35d1551c78212adb7ab470cefd9a53827167a71
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The cfg API is now available via the oslo-config library, so switch to
it and remove the copied-and-pasted version.
Add the 2013.1b3 tarball to tools/pip-requires - this will be changed
to 'oslo-config>=2013.1' when oslo-config is published to pypi. This
will happen in time for grizzly final.
Remove the 'deps = pep8==1.3.3' and 'deps = pyflakes' from tox.ini as
it means all the other deps get installed with easy_install which can't
install oslo-config from the URL.
Change-Id: I4815aeb8a9341a31a250e920157f15ee15cfc5bc
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Use 'nosetests --with-doctests' to run any doctests found. We currently
only use doctests in a handful of places, but we may as well run them
to ensure they work.
Make the cfg doctests avoid using the global CONF since we would need
to reset its state between each doctest.
Fix the cliutils doctests to actually pass.
Use 'nosetests --exclude-dir=tests/testmods' to avoid loading the
modules from this dir while discovering doctests. The cfg unit tests
rely on these modules not having been previously loaded.
Change-Id: I19ad70767fa5c8352b994dc963b5d3a7c9d9eb95
|
|
|
|
|
|
| |
Caesar's wife must be above reproach.
Change-Id: Iac85a57e71d403360f1567c07c8699057f0772fb
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Added both a tox test-env for pyflakes and fixed the current pyflakes errors.
This did actually fix a couple of bugs.
The CI team has started using pyflakes on its projects and also has started
using oslo for things, so ignoring pyflakes warnings on the oslo code was
starting to get old. However, additionally, pyflakes is pretty solid, so we
should maybe consider gating on it across the board. (% locals() is the
biggest thing that we do that it doesn't like)
Change-Id: Iac1ca62db301892b7863711162fcbc74807eb24f
|
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes bug #1082002
With sitepackages=True, I'm seeing the tests fail on my machine because
the older system version of nose gets used.
Looking at other projects, this is only set in the tox:jenkins section
where one exists.
Change-Id: I1ab5f337adf8fc668608146ccb2ab3f2590204be
|
|
|
|
|
|
|
|
|
| |
Exclude the .update-env directory, including anything
installed there from third-party packages, when running
pep8.
Change-Id: I2c2ee9146cbf2f5449f7f8d042b578ec3d8d2b83
Signed-off-by: Doug Hellmann <doug.hellmann@dreamhost.com>
|
|
|
|
|
|
|
| |
Fixed indents which violate pep8 1.3.3
Also added .mailmap file to coalesce commits
Change-Id: Ia7a788a28e6c0fd9cb52fb7ea477cddd7d589e58
|
|\ |
|
| |
| |
| |
| | |
Change-Id: Ieebefee89483e5abbcefe1bb9059d89c91dc04df
|
|/
|
|
|
|
| |
also,this patch turns off pep8 E125 check.
Change-Id: I22d2a94d89530586d614af39af17ae542c5e0cbf
|
|
|
|
| |
Change-Id: I0f403cfa8edec3795f65cf270a9cb2ad0a05d4f5
|
|
|
|
|
|
| |
This enables support for testing qpid (not available in pip)
Change-Id: I36bc0398ed8e8377dfc8af10ae0981e117a3ec20
|
|
|
|
|
|
|
| |
The contents of tox.ini were out of date. Update them in preparation
for Jenkins check and gate tests.
Change-Id: I7b6e33b118e0bdab492560c8e36d50c5f7c2c10b
|
|
|
|
|
|
|
| |
I noticed that pep8 wasn't running on the tests. This patch fixes that,
as well as a couple of pep8 errors in test_cfg.
Change-Id: I4429bfe6813a2e9394efb1753cbebbadb9f23833
|
|
|
|
| |
Change-Id: Ib1f768c6783db0e63cb51dfc767ae5a31a295c76
|
|
|