| Age | Commit message (Collapse) | Author | Files | Lines |
|
Change-Id: I1c0312aa7bdfb78fedef6c98463f25e6460f4587
Co-Authored-By: Pierre Riteau <pierre@stackhpc.com>
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
This reverts commit 28531dbdbe2b1985929a9a32a88b33a06d80067b.
Reason for revert: not needed anymore
Change-Id: Ic5806b2759f8dde4b6947e837b034f438d4955eb
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Change-Id: If6cb4f5a2796cd1eb48eecbb00b7f75c70941a6d
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
wait_discover_computes.yml had to be updated to pass ansible-lint
checks for undefined variables (ansible_facts.hostname/nodename was
rendered as UndefinedMarker type)
Change-Id: Iaded12e9bcf7ed558b7e35ea526ef1355cbca61a
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
Bumped collections in requirements-core.yml to Ansible 12 major versions,
see [1].
[1]: https://github.com/ansible-community/ansible-build-data/blob/main/12/ansible-12.1.0.yaml
Depends-On: https://review.opendev.org/c/openstack/kolla/+/960307
Co-authored-by: Doug Szumski doug@stackhpc.com
Change-Id: If8a29052d8a43ffc0fef11514adabb5e4ab9d977
Signed-off-by: Michal Nasiadka <mnasiadka@gmail.com>
|
|
This patch adds ansible-core as a dependency to
requirements.txt since there is really no reason
not to have it in the requirements.txt, given that
kolla-ansible project even has ansible in its name
and ansible-core is a crucial dependency.
Change-Id: I4d01b7eb944c4edf6a440f2b775221986509f7eb
|
|
Update the version of ansible because ansible-core was updated to 2.13-2.14
[1] https://review.opendev.org/c/openstack/kolla-ansible/+/883525
Change-Id: If3223fe17f9e480a9782c2696ed945d24a8dae5e
|
|
These are not needed for unit tests and we want to avoid conflicts when
running with older python versions that we still test against.
We intentionally duplicate ansible and reno as requirements in order to
avoid having to install more items than necessary.
Signed-off-by: Dr. Jens Harbott <harbott@osism.tech>
Change-Id: I0ec83c3e4ae38f911b922a3bade65429d91169eb
|
|
ansible-lint 6.0.0 introduced requirement to use FQCNs
Change-Id: I3f27c6bcdd200252ebb089f6377294f7e3a911a0
Signed-off-by: Maksim Malchuk <maksim.malchuk@gmail.com>
|
|
Also bumps the minimum to 4.
Change-Id: Ia373f9cc3bb69eba0288bbb3e497e8cadb7cc4d3
|
|
This change bumps up max supported Ansible version
to 4.x (ansible-core 2.11.x) and minimum to 2.10.
Change-Id: I8b9212934dfab3831986e8db55671baee32f4bbd
|
|
Min version stays as 2.9
Change-Id: I7ec8c5eb36757248c9aa016dc7d4e495ec5bb635
|
|
so that it matches what is tested otherwise.
Change-Id: I4eef9f89f1cd262ca2a96c20079756f5f1499ad3
|
|
Change-Id: I5befc72a4894d625ca352b27df9d3aa84a2f5b2c
|
|
See [1].
[1] https://github.com/ansible/ansible-lint/issues/969
Change-Id: I5ac8f20a2157d880b63a4c70bf6c782d06014f41
|
|
* Reworked tox pep8 into linters job, that runs:
- pep8
- bandit
- bashate
- doc8
- yamllint
- ansible-lint (validate-all-files.py + ansible-lint)
* Skip E701 - missing galaxy_info in meta and E602 see [1].
* Skip E301 and E503 - followup later in a separate change
* Added ansible-role-jobs to zuul.d/project.yaml which will run
openstack-tox-linters job in check queue
* Fixed remaining style issue
* Made tox and docs reference the new env for linters
* Dropped pype environment (not supported)
[1]: https://github.com/ansible/ansible-lint/issues/457
Change-Id: I494b4b151804aac8173120e6c6e42bc2fdb00234
|
|
flake8 new release 3.8.0 added new checks and gate pep8
job start failing. hacking 3.0.1 fix the pinning of flake8 to
avoid bringing in a new version with new checks.
Though it is fixed in latest hacking but 2.0 and 3.0 has cap for
flake8 as <4.0.0 which mean flake8 new version 3.9.0 can also
break the pep8 job if new check are added.
To avoid similar gate break in future, we need to bump the hacking min
version.
- http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014828.html
Change-Id: I4b11eaad9eac9985342a00e583f16e379a2ad04a
|
|
This causes non-local side effects that are hard to track down. E.g:
--- import errors ---
Failed to import test module: tests.test_merge_yaml
Traceback (most recent call last):
File "/home/will/.pyenv/versions/3.7.7/lib/python3.7/unittest/loader.py", line 436, in _find_test_path
module = self._get_module_from_name(name)
File "/home/will/.pyenv/versions/3.7.7/lib/python3.7/unittest/loader.py", line 377, in _get_module_from_name
__import__(name)
File "/home/will/code/kolla-ansible/tests/test_merge_yaml.py", line 19, in <module>
from ansible.errors import AnsibleModuleError
ModuleNotFoundError: No module named 'ansible.errors'; 'ansible' is not a package
This `'ansible' is not a package` message occurs because ansible is a Mock.
Depends-On: https://review.opendev.org/#/c/726768/
Change-Id: Iddbdd3d855daadbf12536cc990559e6b8e123051
|
|
Removes and/or replaces all mentions of py27.
Cleans up obsolete requirements and their lower-constraints.
Separates test-requirements.
Makes lower-constraints pass outside of CI (MarkupSafe).
Adds FIXMEs about some hacky Mocks that may misbehave.
Change-Id: Ifc090bf3c1db17d8542ee591c91e8225a597bfe2
|
|
The repo is Python 3 now, so update hacking to version 3.0 which
supports Python 3.
Fix problems found by updated hacking version.
Remove hacking and friends from lower-constraints, they are not needed
during installation.
Change-Id: I7ef5ac8a89e94f5da97780198619b6facc86ecfe
|
|
These should not be necessary.
Change-Id: Ide02b7da2a2a2d528c171772f5552f4529ffe880
|
|
Use latest release 1.1.0 and compatible changes w.r.t pep8
Change-Id: Ie5e5417a1d8481bb59d516a320bd7cc1dd89591e
|
|
According to Openstack summit session [1],
stestr is maintained project to which all Openstack projects should migrate.
Let's switch to stestr as other projects have already moved to it.
[1] https://etherpad.openstack.org/p/YVR-python-pti
Change-Id: I63079f352ff36dda4f3f64a8681431a319791f0e
|
|
The openstackdocstheme is included by ``doc/requirements.txt``. We
move all doc requirements to ``doc/requirements.txt``.
Change-Id: I3e3acf07826a496b3d8514941afa908624fb74f4
|
|
For compliance with the Project Testing Interface [1]
as described in [2]
[1]
https://governance.openstack.org/tc/reference/project-testing-interface.html
[2]
http://lists.openstack.org/pipermail/openstack-dev/2017-December/125710.html
doc8 command is dropped from docs tox envs.
So this affect nothing and run in PEP8.
Related-Bug: #1765348
Depends-On: Icc7fe3a8f9716281de88825e9d5b2fd84de3d00a
Change-Id: Idf9a16111479ccc64004eac9508da575822a3df5
|
|
hacking is not capped in g-r and it is in
blacklist for requirement as hacking new version
can break the gate jobs.
Hacking can break gate jobs because of various
reasons:
- There might be new rule addition in hacking
- Some rules becomes default from non-default
- Updates in pycodestyle etc
That was the main reason it was not added in g-r
auto sync also. Most of the project maintained the
compatible and cap the hacking version in
test-requirements.txt and update to new version when
project is ready. Bumping new version might need code
fix also on project side depends on what new in that
version.
If project does not have cap the hacking version then,
there is possibility of gate failure whenever new hacking
version is released by QA team.
Example of such failure in recent release of hacking 1.1.0
- http://lists.openstack.org/pipermail/openstack-dev/2018-May/130282.html
Change-Id: I05790c5f946a4127fe6c8d9867cc618cb83d67fb
|
|
Change-Id: Ib440c05aa88fcacbe64aadc6e1f9fd38c47d2d57
|
|
Change-Id: I836b0d6e915c32074d8062b5914277497ec4f444
|
|
Change-Id: Ie6fd05b52592ad48fd7acbd6b7e9a6f48ffd1150
|
|
Change-Id: Ia9a7b76a9db4a149d9e0df16ac576ecd706df22b
|
|
Change-Id: If0f42f15d8a92ce77ec7988bc04f795d795e2d89
|
|
Change-Id: I6cf2fce6afbaea4bb8c63c1dd0acc61bf1cf477d
|
|
Change-Id: I68a1f61af30f5cc3b0604fb988cf773107891b11
|
|
Change-Id: Ica929fd8d3ef1972fec0daeb4ae2c00b9a9be3a0
|
|
Change-Id: I89264d20cf97bab69ac6fb970c0a770a3b53bfe0
|
|
Change-Id: Ia70a5296c557ec54cdb47a28b163c9365d03506e
|
|
Change-Id: I8d097af4a75bdc4225e0df04bfbdc555d1529aae
|
|
Change-Id: I87ab7673dd74d9275dc043eb556ce980d353b810
|
|
Change-Id: If20101daba52df4a41abe5e4606b317761bc936b
|
|
Change-Id: Iaa85ff659906b438ef80efc70b51ee8b82755ddd
|
|
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: Id63951348211bdcb2b189455968bdfed40857815
|
|
Change-Id: I6bb90e40a4661663b2386506463dce4fb59018a8
|
|
Change-Id: Ife8a3f6acee02c4af965a044cba2074f36223cc3
|
|
Change-Id: I7e4c7fea9976dc854c212b0a2b79234fa73c1d1b
|
|
Change-Id: I14134cb07f7ca24ebb7e1c9a38f2d4f77cab3d73
|
|
|
|
|
|
|
|
With the split of kolla and kolla-ansible, it seems that this
dependency was carried over as it is used when generating
graphs for build sequences.
This patch removes it as the build process is not part of this
project anymore.
Change-Id: I7fd59fe703c2f06c003c8acda15f2a06d3fd7f3f
|
|
It seems that it was added back in Liberty where an effort was
started to setup functional tests, however it is not used at the
moment.
This patch removes it from the list of dependencies. With time,
Tempest should handle all integration checks.
Change-Id: I9e1990e93abc6e3001f499ccc8d40b6104ed2d93
|