This redirect does not work or won't work in the future.
Note: This adds ansible-core to test-requirements.txt,
we may need to add requirements.txt and ansible collections
in ansible-collections-requirements.yml in the future.
Change-Id: Iffb41b68283f77050d8073364156cc0aca85559e
A few cleanups:
* remove docs environment, this repo has not docs
* Cleanup setup.cfg, remove obsolete sections
* Update docs requirements to newer versions working properly with
python3, releasenotes are build using python3
Change-Id: I721b81e422501b841107d4d2d96458759448a326
* Kept old specific linter tox targets for backwards compatibility,
so developer can easily run a single linter if desired.
* Reused tox linters virtualenvs to avoid wasting space
* Moved linter config to their official location which enables
developer to call them in any way he wants (like his IDE)
* Updates yamllint to its newer version which supports strict
* Fixed yamllint errors
* Used parsable output format on yamllint which is more consistent
with other tools, being also recognized by IDEs and terminals.
Change-Id: I2bd44feb0a221dcd6f5402e070cf38b31fd8ac97
Depends-On: https://review.openstack.org/#/c/621930/
Story: https://tree.taiga.io/project/tripleo-ci-board/task/381
* Start running flake8 and ansible-lint via pre-commit
* Bumped hacking version to last release and fixed new errors
Change-Id: Iefe8794abba70660559fcb8cba12dc0b41737882
Story: https://tree.taiga.io/project/tripleo-ci-board/task/381
Migrates bashate linting task to pre-commit in order to allow fast
and transparent execution before uploading changes to gerrit.
Other linters can be run as well, including flake8 and ansible-linter.
This also lowers the footprint on developer machines because each
linter/version is stored only once on disk, regardless how many
repositories are using it.
Only action needed for enabling pre-commit as a developer is to
run `pip install pre-commit`, everything else is done transparently
after.
Partial-Bug: #1786286
Change-Id: I6ad834bbfbaf7645c542e4f922e89bc4449247c3
- adopts official syntax for 'when:' commands
- assures we use current linter which spots when errors
- reverts temporary linter rule disablement
- use parsable output (pep8 like) for ansible-lint
Change-Id: I6eb5719841c2f264e1efed4bf38726f71c840e36
Closes-Bug: #1797838
We have a probelm with current yaml version and ansible-lint
TypeError: unbound method construct_mapping() must be called with Constructor instance as first argument (got Loader instance instead)
Related-Bug: 1778865
Change-Id: I6d63bfb7eb06ff79d5f23d1f1b2b05198ec51257
As part of the docs migration work[0] for Pike we need to switch to use the
openstackdocstheme.
[0] https://review.openstack.org/#/c/472275/
Change-Id: I94aadd9514aa95e885517d9a5aae8efcc379791f
This will now run bashate on all shell scripts. We could ignore the
following list of bashate errors:
- E006: Line longer than 79 columns (as many scripts use jinja
templating, this is very difficult)
- E040: Syntax error determined using `bash -n` (as many scripts
use jinja templating, this will often fail and the syntax
error will be discovered in execution anyway)
Change-Id: I6f6d454ac54a0ac98d8b8a092fa522cba092a592
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
Also fix ansible-lint errors and disable flake8 checks on the files
within the roles until we fix them.
Change-Id: Iab40c0fea517b2f74b91cab7b0f443de89e453b8
Ghanshyam Mann