Remove E402 from flake8 list, issues are fixed or addressed with noqa.
Remove E305 as comment, the warning is enabled.
Enable E241 and fix all problems
Change-Id: Ifd2bbce98be4206908da769df686a370328c7ee1
With python 3 the input function is safe.
The six dropping will re-introduce the usage of the input function
and bandit checks will fails.
We drop python 2.7 support so we ignore the bandit check B322 [1] before
starting to drop six support.
[1] https://bandit.readthedocs.io/en/latest/blacklists/blacklist_calls.html#b322-input
Change-Id: Ie956a65ebc4c781c5e661e8021f674833be8b323
The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.
Update local hacking checks for new flake8.
Ignore new warnings, they will be fixed in followup.
Remove hacking and friends from lower-constraints, they are not needed
to be installed at run-time.
Add Pygments to lower-constraints to pass requirements-check.
Change-Id: I20da1309e4d65707130fe517f013d3ed625bf94c
we no longer keep configuration for in-repo integration tests in the
repo, relying on heat-tempest-plugin instead.
As a result, currently the `tox -egenconfig` is broken as it tries to
generate config sample calling non-existing config.py module.
Change-Id: I48685d7c791d98db83a7cbcf4ef926ee25ca62a4
1. Sphinx 1.8 introduced [1] the '--keep-going' argument which, as its name
suggests, keeps the build running when it encounters non-fatal errors.
This is exceptionally useful in avoiding a continuous edit-build loop
when undertaking large doc reworks where multiple errors may be
introduced.
2. Add whitelist_externals to fix the warning
[1] https://github.com/sphinx-doc/sphinx/commit/e3483e9b045
Change-Id: I47683ab1313981a8202afe6b3fe1779d40340452
This change provides the necessary modifications to enable
building of pdf documentation. The resources.py extension
required modification to:
- prevent attributes from being rendered as stray \items
outside of a list environment
- prevent update policies from being rendered as stray
\items outside of a list environment
- remove the 'Note:' admonition around the 'Not implemented'
message when it applies to a sub-property, such as a list.
The admonition was not rendered as a list item, causing the
build to fail due to an empty list.
Change-Id: Id4e533430600b5ff03fd63cdc09b9a00741b8e4e
The api documentation is now published on docs.openstack.org instead
of developer.openstack.org. Update all links that are changed to the
new location.
Note that redirects will be set up as well but let's point now to the
new location.
For details, see:
http://lists.openstack.org/pipermail/openstack-discuss/2019-July/007828.html
Change-Id: I69a41aa850c87e92bf5c3e7fed32e0e961a3c6d3
As it was announced [1] global bindep-fallback.txt was removed and now
projects need to have a local bindep.txt to be able to install binary
dependencies for testing.
In test jobs the script tools/test-setup.sh is called which requires
mysql and postgres servers and clients to be installed.
[1] http://lists.openstack.org/pipermail/openstack-discuss/2019-June/007272.html
Change-Id: If9befe4115c64c2fda52321002ba5fe1124eaf7c
This goal is to implement the process set out in the 2018-10-24 Python
Update Process TC resolution[1], for the Train cycle to ensure unit
testing is in place for all of the Tested Runtimes for Train[2].
In practice, this generally means adding unit tests for Python 3.7 and dropping
unit tests for Python 3.5. Using the Zuul template for Train will ensure that
all projects that support Python3 will be tested against the agreed runtime
versions, and make it easier to update them in future.
[1]https://governance.openstack.org/tc/resolutions/20181024-python-update-process.html
[2]https://governance.openstack.org/tc/reference/runtimes/train.html
Change-Id: I62abb218bb314345dd7da1cbf9133d10db9696ff
Depends-On: https://review.opendev.org/#/c/641878/
Our cgit instance will be going away and opendev.org is the new
preferred URL for browsing our git repos. Redirects will exist for the
foreseeable future, but it's more efficient to just go directly to the
new locations.
Change-Id: Ic5fa1a8436f57836ad37b752a0cca1cd4f3a21a7
- Bumps some of the lower constraints
- Fixes common/endpoint_utils.py to fallback to use auth_uri.
Change-Id: Ief0868d5feef3ee6b0689c6be27649ff009fbbcc
We are updating all Python projects to publish artifacts to PyPI. The
name "heat" is already taken there by another project, and they have
rejected our request to claim the name. We therefore need to change the
dist name used to package heat. We have some other projects publishing
using an "openstack-" prefix, so I propose using the name
"openstack-heat". This will not change the imports or anything else
about how the code works, just how it is packaged.
Add a grenade plugin for upgrading between the packages with different
names so that we can clean up the old version of the code and avoid
discovering the heat plugins multiple times in different locations.
Change-Id: I59b55cffd9e648f842eb286b936f09c5b55a76db
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
Quotes around {posargs} cause the entire string to be combined into one
arg that gets passed to stestr. This prevents passing multiple args
(e.g. '--concurrency=16 some-regex')
Change-Id: Iccbd680731592abab4980a51aa97c60d6ef95005
This patch adds doc8 check for .rst files to pep8.
Files that fail doc8 check are also fixed.
Ignore D001 of doc8 check. This requires lot of changes and
should be done in a separated patch.
Change-Id: I7732abc55ec27026efbf56663ba02ff27e8ec847
We want to default to running all tox environments under python 3, so
set the basepython value in each environment.
We do not want to specify a minor version number, because we do not
want to have to update the file every time we upgrade python.
We do not want to set the override once in testenv, because that
breaks the more specific versions used in default environments like
py35 and py36.
Change-Id: I6dde50de599eb3b6b001cbc044929c64df044575
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
Python3 is smarter than Python2 about when to make use of .pyc files, so
there's no need to remove them before running unit tests just in case the
developer has switched branches.[1] To save recompiling before every test
run, disable this behaviour for Python3.
Also, use a consistent find command in all of the environments where we do
this.
[1] https://www.python.org/dev/peps/pep-3147/#case-3-pycache-foo-magic-pyc-with-no-source
Change-Id: Iabaa4a697329e7538fab93541f41fc208e591382
Create a tox environment for running the unit tests against the lower
bounds of the dependencies.
Create a lower-constraints.txt to be used to enforce the lower bounds
in those tests.
Add openstack-tox-lower-constraints job to the zuul configuration.
See http://lists.openstack.org/pipermail/openstack-dev/2018-March/128352.html
for more details.
Change-Id: I72fd5f8c87133713cd272de210b7e957880f48f9
Depends-On: https://review.openstack.org/555034
Signed-off-by: Doug Hellmann <doug@doughellmann.com>
We removed .testr.conf in 93746a9b2895d44fd105ba26ae77ddffbe2db167
and tox -ecover fails with error.
Change-Id: I31b00afffe38ab5c7591ba54c959349716df7c48
This patch removes the API, the next set of patches in the
series would remove stack watch service and related
WatchRule implementation.
Change-Id: I8b0472be862907298c8da51f435b5d8b19610ec3
Partial-Bug: #1743707
Regression tests will be maintained in-tree. See categries list:
https://etherpad.openstack.org/p/heat-integration-test-categories
We will seperate tests to tempest plugin or in-tree base on conditions
of each tests. So we need to rework on framework for integration tests.
This patch propose following changes:
* This partially reverts commit fff6518e69d19c7946942d1bd7c68e2b76c15966
* Allow running heat tempest plugin and heat regression tests.
* iniset configs to both heat_integrationtests.conf and tempest.conf
Change-Id: Ief31dc961bc108e2863119598dfb16581a38e9cf
Depends-On: I5e9325766ce166e62c731330c462c030cb1e11fc
Co-Authored-By: Zane Bitter <zbitter@redhat.com>
This adds the basic framework for registering and using default policy
rules. Rules should be defined and returned from a module in
heat/policies/, and then added to the list in heat/policies/__init__.py.
new policy wrapers `registered_identified_stack` and
`registered_policy_enforce` has been added for policy enforcement of
registered rules with same parameter as `identified_stack` and
`policy_enforce` besides set `is_registered_policy` flag to true.
This flag will decide to use new policy framework or not.
Now we can use `tox -e genpolicy` to check and generate policy file.
Change-Id: I7a232b3ea7ce0f69a5b7ffa278ceace7a76b666f
Partially-Implements: bp policy-in-code
Many deprecations are triggered early (on imports, for example).
To make sure all DeprecationWarning messages are emitted we enable
them via the PYTHONWARNINGS environment variable.
If we didn't setup this variable, the default action will be not
showing any python deprecation warning.
Change-Id: I78d1c68e51ee0833fd1ff0e9548c41bb9d1239a8
This makes tox use the latest version of the tool from os-testr repo,
which contains some valuable changes, like the one making the gate fail
if for some reason no tests are executed because all of them are
skipped.
Change-Id: If88be4b310d746ccc885e3d0c1db8a4dfabb2a28
Latest g-r version of bandit does not require a bandit.yaml. This
removes the bandit.yaml and skips the same tests that were not
run as defined by the profile before. This patch also puts running
of bandit within the pep8 testenv, consistent with other projects.
Change-Id: I7db8bc460e2051c635dc7b3fd4b152ece80c7a0c
The gating on python 3.4 is restricted to <= Mitaka. This is due
to the change from Ubuntu Trusty to Xenial, where only python3.5
is available. There is no need to continue to keep these settings.
Change-Id: Ic7b3fb3ff7b68cf280e36335aa9cacd644cc877a
Sphinx 1.5 is more picky about some code, it complaints if code-blocks
are not valid json and also about too short underlines.
Fix one json file, fix underlines, change code-blocks to none so that
Sphinx 1.5 builds this again.
Enable treating warnings as errors to avoid regressions. Note that with
current tools there are no double stanza warnings anymore.
Change-Id: I58d1452ffddf091d56dcfad8140caba084800a0e
This reverts commit d7f2d4f23eba13eea6c314bf5fde34df7a4df912.
ostestr spews what is IMHO an unmanageable amount of mostly worthless data when running tests - at least one line per test, of which there are 6351 at last count. There doesn't seem to be an option to quiet it down, so just revert it.
Change-Id: I7b12dc249e3e7041ab6f0ef36b3e5f0b082aeb7d
As a wrapper of testr, os-testr uses subunit-trace
for output and builds some helpful extra
functionality around testr
Reference link:
http://docs.openstack.org/developer/os-testr/index.html
Change-Id: I5eb1ea062334c06f5e95ef54367e427b98a17bac