| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
As per this discussion on the mailing list:
http://lists.openstack.org/pipermail/openstack-dev/2014-September/047160.html
We are going to stop enforcing the correctness of the sample config
file, and move to a separate non-voting check test that notifies us when
it is no longer accurate, thus prompting a human to correct it.
This addresses the issue where config options in dependent libraries
(eg. keystone) repeatedly break our gate, without removing the sample
config file that deployers benefit from.
Change-Id: Ief88a693e4967b32f5f2f71b9a683599527a5129
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch was based on the Nova patch:
https://review.openstack.org/#/c/121044
This should reduce the number of times you need to completely delete
your tox dir because you have stale pyc files around.
In Ironic we have hit this problem a number of times, migrating to
oslo.message was a big one that caused a lot of confusion.
Closes-Bug: #1368661
Change-Id: I7dda3ad879beff1e494a36144c941a847177c2ef
|
|
|
|
|
|
|
|
|
|
|
|
| |
When we used the i18n markers (_, _LE, _LW, etc) from
ironic.openstack.common.gettextutils, we bypassed H302
hacking rule for them (via tox.ini file). Now that we've
converted to using the markers from oslo.i18n and are no
longer using the markers from gettextutils, we can remove
the bypass stuff.
Change-Id: I26147b69884cd9c075e1cfe38a92e38e861083e7
Closes-Bug: #1368425
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Updates the Ironic compute driver and scheduler manager so that they
subclass the now in-Nova-tree driver classes. This should de-duplicate
code maintenance across Nova and Ironic and also provide end users
an easier upgrade path (with Deprecation log messages).
This patch also removes the old Ironic compute driver tests and classes
since those now live in Nova.
This patch also removes a line from tox.ini which was copying the second
set of unit test outputs; those no longer exist, since we are no longer
running unit tests on nova code in this tree.
Co-Authored-By: Dan Prince <dprince@redhat.com>
Co-Authored-By: Devananda van der Veen <devananda.vdv@gmail.com>
Change-Id: Ic194f779eca470e4a4dd8a3fbd31ca5a6f6b4f7e
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This part fixes ironic/common folder and tox.ini file.
Some modules use _ function for log translation, even when the
translated string is passed to e.g. LOG.warn function. This patch
fixes it, using translator functions appropriate for corresponding
log levels. Translator functions, imported from
ironic.openstack.common.gettextutils are now imported from
ironic.common.i18n. Translator functions are imported the same way
across all modules. All places that are changed conform to
http://docs.openstack.org/developer/oslo.i18n/guidelines.html
Partial-bug: #1364813
Change-Id: I195d0e256c5412d6d75d2e6c35b19b44f2a0b09f
|
|
|
|
|
|
|
|
|
|
| |
Now that ironic supports oslo.i18n, using the install function to
inject _ into the builtin has been deprecated. New i18n common module
used instead. In one module _ was imported from gettextutils, now
also changed to i18n import.
Closes-bug: #1346406
Change-Id: I672a975d479991982c24031e834a26e2c2811890
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
We're seeing frequent failures of the docs Jenkins job that appear
to be caused by the tox 1.7.2 upgrade and the migration to using
PYTHONHASHSEED=random. This overrides the environment in the venv
testenv to set PYTHONHASHSEED=0 and allows docs to build
successfully. Note that we want to run our tests with a random seed
if possible, so this avoids disabling it for the base testenv and
only uses non-random for the testenv used by Jenkins for doc builds.
Change-Id: I665703c9a002500bc90b90cd268bd3ccff4e3534
Closes-bug: 1361295
|
|
|
|
|
|
|
|
|
|
| |
Switch ironic to use oslo.i18n. A new i18n module has been
added under ironic with translation globals and imports the
necessary functions from oslo.i18n.
gettextutils module is still needed in the nova ironic driver.
Change-Id: I682228e158132c628166e4d1e334c4003249d112
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Using the new logging translation markers (_LI(), _LW(), _LE(),
_LC())[1] is problematic because they are not modules. Since the
Openstack guidelines only allow modules to be imported[2], the entire
gettextutils module must imported in order to use the translation
markers.
This patch adds all four markers to import_exceptions in tox.ini, so
they can be imported individually without failing the pep8 tests. This
is already done with the _() marker. With this change, it will be
allowable to import marker functions like so:
from ironic.openstack.common.gettextutils import _LI
from ironic.openstack.common.gettextutils import _LW
from ironic.openstack.common.gettextutils import _LE
from ironic.openstack.common.gettextutils import _LC
[1]http://docs.openstack.org/developer/oslo.i18n/guidelines.html
[2]http://docs.openstack.org/developer/hacking/#imports
Change-Id: I78192dd7525624174deb8d377996ed3d61b36a8a
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
If someone tries to run generate_sample.sh outside a venv
or without installing project dependencies, you could encounter
error messages related to missing or outdated dependencies.
This change wraps generate_sample.sh execution in tox. Wrapping it
in a tox job makes sure all dependencies found in requirements.txt and
test-requirements.txt are properly installed prior to its execution.
This change does not propose the removal of the sample config file
already in place in the repository and it does not propose adding
it to the list of jobs automatically ran by tox.
This change is merely the introduction of a more convenient way
to execute generate_sample.sh in a proper venv.
You can now (re)generate the sample config file with this tox job:
tox -egenconfig
Change-Id: I1058a306a462ba8349a4433c78070fabd6d03e39
|
|
|
|
|
|
| |
Enabled check for PEP8 issue E713.
Change-Id: Id41f5f19796a7af8cf4cb52dd7afc2e6ec1ccbc4
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This updates testr.conf to allow passing of the tests directory as an
environment variable (defaulting to ironic/tests/). Also updates tox.ini
to run testr once for ironic/tests and once for ironic/nova/tests/, then
combine subunit output.
test-requirements.txt is updated to pull in Nova from a tarball, as well
as mox and python-ironicclient.
Updates the existing Nova driver unit tests to account for recent
refactoring and changes in the Nova virt API.
Change-Id: Ibefa2e0d3f97512f4da9ace780703fe47467232e
Closes-bug: #1299312
|
|/
|
|
|
|
| |
This gives compatibility with OS X in the test.
Change-Id: I483b4279d0d6a5a6fe4f81b9fb07a838e17e1b0f
|
|
|
|
|
|
|
|
|
| |
Perform minimal checks on po and pot files so that no broken files can
get imported.
Change-Id: I328fc11c688c16737ae88984e4778483ce5db5ab
Co-Authored-By: Clark Boylan <clark.boylan@gmail.com>
Related-Bug: #1298645
|
|
|
|
|
|
|
|
|
| |
Enable pep8 tests for the Nova Ironic Driver and fix the current pep8
errors. Being able to run pep8 tests against the driver helps the
development because it captures problems like syntax errors, unbound
variables, duplicated methods, etc...
Change-Id: If1da455bb8669b56267cb3ecf45369d5d83a86b7
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* tox.ini: The LANG, LANGUAGE and LC_ALL environment overrides were
introduced originally during the testr migration in an attempt to be
conservative about the possibility that locale settings in the
calling environment could cause consistency problems for test runs.
In actuality, this should be unnecessary and any place where it does
cause issues ought to be considered an actual bug. Also, having
these in the configuration actively causes older pip to have
problems with non-ASCII content in some package metadata files under
Python 3, so drop it now.
Change-Id: I732b6559574f98fceada099e5e77e5c91095d67c
Closes-Bug: #1277495
|
|
|
|
|
|
|
|
|
| |
Add 'config' to openstack-common so that it syncs from Oslo,
move tools/conf -> tools/config to be able to sync from Oslo,
and then sync the changes which are proposed in oslo-incubator
commit Iee6cc05cad70963f2a151f595ac376c9338f5774.
Change-Id: I32a80b0e2aa0e26b9fc78b7546c07e102639374e
|
|
|
|
|
|
|
| |
Ask pip to upgrade packages in the virtualenv so that a long-lived
virtualenv is still kept up to date as external dependencies change.
Change-Id: Ife76a83be95fed7cc4b99bb50a97277d0751e76c
|
|
|
|
|
|
|
|
|
|
|
|
| |
To ensure that our sample config file, etc/ironic/ironic.conf.sample,
does not get out of sync with the code, this patch introduces a check
to tox that will compare the proposed ironic.conf.sample to a
freshly generated one, and error if they are not identical.
Ported from Nova's tools/config/check_uptodate.sh script.
Change-Id: Ic2898142754547e32bc4c03d23eca5d6a33721fc
Closes-bug: 1255621
|
|
|
|
|
|
|
| |
The design is for 'assertThat(subject, verb(object))', putting module
names in there significantly impedes readability.
Change-Id: If508bb38dd9cbea3c1ed8f654531e013a2e24afb
|
|
|
|
|
|
|
|
|
|
|
|
| |
We run "python setup.py build_sphinx" to generate the API documents,
however encountered a lot of "NameError: name '_' is not defined",
which will cause our api.controllers.v1.* objects documents will not
be generated by Sphinx, so we are missing these in the document page
http://docs.openstack.org/developer/ironic/py-modindex.html. Import
missing gettext _ to fix such Sphinx error with this patch.
Change-Id: Ie53d96203109b8b95b9734b58d85da41a2d011cf
Closes-Bug: #1251011
|
|
|
|
|
|
|
|
| |
These method were just stubs.
This patch implements them in db/sqlalchemy/api.py, and a subsequent
patch will expose this functionality via the API.
Change-Id: Iae68cbe3bf4e84a0f47a0770695bed9dae78aa95
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
See I62ce43a330d7ae94eda4c7498782a655e63747fa for the gorey details on
why this exists.
As of this fix:
https://github.com/eventlet/eventlet/pull/34
which was released in eventlet 0.13, we no longer need the patch.
This has now been removed from oslo-incubator, so this is really just
syncing that removal.
Change-Id: I84267f3c6726cb2e750f615e107c48b12c6ed353
|
|
|
|
|
|
|
|
|
|
|
| |
tox 1.6 was released, which means that we can now take advantage of the
feature we added to it - which is using setup.py develop to install the
code into the virtualenv. The logic was taken from run_tests.sh - so the
performance issues around using tox vs. using install_venv should now be
gone.
Change-Id: Ib011a9add595696f5db5fe596db9d3d60f46d038
Fixes-Bug: #1233213
|
|
|
|
|
|
|
|
| |
This tests for nova components cannot be started
in Ironic. test_ironic_deploy_helper moved,
two incompatible tests disabled.
Change-Id: I456034bf7bbe3e86630c82ed0d1474c15a9aeee6
|
|
|
|
|
|
|
|
|
|
| |
pep8 flake8 and pyflakes are pinned as part of hacking. So they don't
need to be pinned here as well.
Also add posargs to flake8, so can do things like 'tox -epep8 --
--statistics'
Change-Id: Id37949caa5a487cd98b6b85c026689fbd63cb5d8
|
|
|
|
|
|
|
|
|
|
| |
Fix all existing pep8 errors.
Remove install_venv and friends, which were not needed.
Add a few ignores for nova code that is yet to be cleaned up.
Skip one failing test case, fixed by review 29394.
Import contrib/redhat-eventlet.patch from Nova.
Change-Id: I46b6ccaa272bd058757064672ce9221263ed7087
|
| |
|
|
|