Commit Graph

1165 Commits

Author SHA1 Message Date
Dmitriy Rabotyagov d163850d7c Bump ansible-core to 2.12.8
On 30.09.2023, "old" ansible galaxy(galaxy.ansible.com) was replaced
with "galaxy ng"[1].
There was a bug [2][3][4] that was fixed only in a supported
ansible-core versions, so:
- v2.12.8
- v2.13.3
- v2.14.0 (and all newer versions)

All openstack-ansible versions up to Yoga are using older ansible-core
versions.
As a result, bootstrap-ansible.sh script fails with an error:

```
Skipping Galaxy server https://galaxy.ansible.com/api/. Got an
unexpected error when getting available versions of collection
ansible.netcommon:
'/api/v3/plugin/ansible/content/published/collections/index/ansible/netcommon/versions/'
```

This patch bumps ansible-core version to 2.12.8 that includes required
fix.

[1] https://www.ansible.com/blog/new-ansible-galaxy
[2] https://github.com/ansible/ansible/issues/81830
[3] https://github.com/ansible/ansible/issues/77911
[4] https://github.com/ansible/ansible/pull/78325

Change-Id: I2c15c220ca8c6b6f0cbedbf0da8e802b7c0f5e2d
2023-10-16 15:32:55 +00:00
Zuul ade6c0e8e0 Merge "Use curl to download u-c" 2022-12-30 18:56:53 +00:00
Dmitriy Rabotyagov 0c4f893cf7 Switch functional job to Jammy
As PTI for 2023.1 Antelope release Ubuntu Jammy was set as a target
platform. Thus, we should update our functional jobs to reflect that.

With that we also clean-out projects.yaml from jobs that are currently
undefined among zuul jobs to prevent any zuul config failures.

Change-Id: I72b4bb7eddb37579f9d33d365ded982e5db0a9ac
2022-12-28 23:15:17 +01:00
Dmitriy Rabotyagov c2c71cac9f Use curl to download u-c
In functional tests we don't ensure that wget exists. Also for Ubuntu
we ensure that curl is present only. So let's replace wget command with
curl and be consistent across repos.

Change-Id: If9106bd0a481079b9e676c5547e244a11a35fb73
2022-12-28 14:50:57 +01:00
Dmitriy Rabotyagov 747c5f6554 Update tox.ini to work with 4.0
With tox release of 4.0, some parameters were deprecated and are ignored now
which causes tox failures. One of the most spread issues we have is using
`whitelist_externals` isntead of `allowlist_externals`


Change-Id: Id2a4cb712b4099bff1c05c4d10502a7572517268
2022-12-28 09:14:53 +00:00
OpenStack Release Bot 7cf3536af1 Update master for stable/zed
Add file to the reno documentation build to show release notes for
stable/zed.

Use pbr instruction to increment the minor version number
automatically so that master versions are higher than the versions on
stable/zed.

Sem-Ver: feature
Change-Id: Iabfa5f742744a9fc69ba296d8953359891832779
2022-12-13 13:24:01 +00:00
Jonathan Rosser 5a80c5f145 Pin tox version to less than 4
Change-Id: Ie8d18b0d1a588e8a5619394be6c9c758e87b30cd
2022-12-12 08:34:44 +00:00
Dmitriy Rabotyagov 046ecd4abc Switch CentOS 8 Stream to 9 Stream
Change-Id: If030c646e9c79b55b206fdca51d0b5555286c31b
2022-12-01 09:20:54 +00:00
Dmitriy Rabotyagov 2ebc57082c Add SERVICES_BRANCH variable for better control
Currently TESTING_BRANCH serving 2 purposes - it controls over OSA
branch to use and services branch.
At the same time we need to have these separated from time to time, as
OSA is trailing.

Change-Id: Ib5ca8c7d3300f4e8efd81f87612c821594bebfac
2022-05-31 16:59:26 +01:00
Jonathan Rosser ecec4ecca7 Fix python installation for centos 8 stream
This must be python3.8 as we use this version in the integrated repo
and require it for ansible 2.12

Change-Id: Ia326d92c24b9cdfcf46011e0661a9b5081e11573
2022-05-31 16:58:53 +01:00
Jonathan Rosser 2c762b5daa Run main_pre.yml tasks as initial keystone installation step
Change-Id: I0b03f25a4682f1f084fb9405f3f09bc01fb09e10
2022-05-31 16:54:30 +01:00
Dmitriy Rabotyagov 87a65ffaf9 Switch sphinx language to en
With sphinx release of 5.0.0, they changed default for language variable
to 'en' from None. With that current None valuable is not valid and should
not be used.


Change-Id: I3fc1dd06c176c7004974b9cd989208f4ba7a37f2
2022-05-30 16:01:49 +02:00
Jonathan Rosser 20b54d669b Bump ansible version to 2.12.6
Change-Id: I55b12210a83b50deb2524d6b8face57800fbcdb4
2022-05-26 09:59:00 +01:00
Dmitriy Rabotyagov 5371795f68 Provide opportunity to override TESTING_BRANCH
Since OSA has trailing releases, we might want to override branch we're
testing. At the moment it's set based on the .gitreview file.

At same time we temporary override branch to yoga, as deploying from
master breaks installation.

Change-Id: Iacef9bec9ab8f2f15a56e3d95f71734277d7a8c7
2022-05-24 10:47:59 +02:00
Jonathan Rosser 2bd4e501a5 Do not filter setuptools from upper-constraints
This is now removed from openstack-ansible global-requirement-pins
[1] so there is no longer a conflict with the data in
upper-constraints. We use the version from u-c instead of the one
from global-requirement-pins.

Depends-On: https://review.opendev.org/c/openstack/openstack-ansible-os_nova/+/840412
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/840413
Depends-On: https://review.opendev.org/783720
Change-Id: I56305216a23b29a4a16fdd7b0658839a295d2f7c
2022-05-04 17:53:27 +00:00
Jonathan Rosser 3458ee0ee1 Initialise ssh and ssl certificates
This is needed later by the keystone, nova and repo_server roles

Depends-On: https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/840445
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible-os_keystone/+/840514
Change-Id: I3a3e9ee62fde37c65fa182bf59f12031c390d968
2022-05-04 16:42:51 +00:00
Marc Gariepy a504a35d29 Cleanup setup.py config
Change-Id: I8f09bb6e6752eced91a0276495c3b0cf5764e0c1
2022-03-31 10:41:50 -04:00
Neil Hanlon b4c12a25c7 Update tests for Rocky Linux
Change-Id: Ia2d5e704d3d4918f8d55f2102a3c4bef59dab303
Signed-off-by: Neil Hanlon <neil@shrug.pw>
2022-03-28 14:48:15 +00:00
Neil Hanlon 52acfb2c81
Disable setuptools auto discovery
Jobs are broken due to a release of setuptools 61.0 introducing
non-backwards compatible breaking changes [1]

Setting py_modules to an empty string here will disable autodiscovery
and should permit jobs to run as before.

[1] https://github.com/pypa/setuptools/issues/3197

Signed-off-by: Neil Hanlon <neil@shrug.pw>
Change-Id: I64c2850b6a414c68277270952676518aa7aaa85b
2022-03-28 10:45:05 -04:00
Jonathan Rosser 5da5356b43 Add ansible.utils collection requirement
ipaddr filter has been moved from netcommon to utils collection [1]

Based on that we must add that collection to requirements.

[1] db4920ebf6

Change-Id: Ifb7a043ce7c6a12c413641dde61c170ae9327dc4
2022-03-14 16:28:09 +00:00
Zuul 4d573f471d Merge "Gather /etc/pki directory" 2022-02-03 22:20:10 +00:00
Jonathan Rosser d3a67b0b68 Remove unused centos-8 functional job definitions
Depends-On: https://review.opendev.org/c/openstack/ansible-role-qdrouterd/+/820672
Change-Id: I7c4ef5da667cc157a9d263477afa50b8eb73149c
2022-01-21 20:58:31 +00:00
Zuul 32d88bd7f2 Merge "Do not syncronise DB/MQ/Service setup tasks to other roles" 2022-01-13 12:23:18 +00:00
Jonathan Rosser 439619bb28 Do not syncronise DB/MQ/Service setup tasks to other roles
These tasks are now available as roles in the openstack.osa collection.

Change-Id: I6096506967d288d67b47b442bbb13c31f6ccde63
2022-01-13 10:02:47 +00:00
Jonathan Rosser 9aa9119a6a Update ansible lint version
Change-Id: Ia01458f1f6183dd7d68e64645ad3829f2969e33b
2022-01-13 07:20:08 +00:00
Jonathan Rosser 57c07c0199 Fix rich version for ansible-lint
Ansible-lint prior to v5.3.1 has issue with reach version >=11.0.0.
In order to cherry-pick fix to stable branches we fix rich version.

We can't use requirements as rich is neither in blacklist nor in
global-requirements, so must be handled independently

This should be reverted with ansible-lint version bump.

Change-Id: I7ad62182178dd1eb285d6002b2852b457f678c7b
2022-01-12 16:12:48 +00:00
Jonathan Rosser 96b1a0e98c Gather /etc/pki directory
Change-Id: Ibc033f79a2ef434e1e57bd44de857c862edfd622
2022-01-04 13:06:11 +00:00
Jonathan Rosser 9044f21756 Clean out unused legacy tests
Remove job definitions for gentoo and bionic and any others which
are not used elsewhere in openstack-ansible.

Depends-On: Ic53886ee1833469daef0b31a46ea361e449d7697
Change-Id: I1fb55fd1f476139fcd4cc0b6c68b291454160132
2021-12-07 08:27:46 +00:00
Zuul 69b7fcc53c Merge "Allow to provide policy state" 2021-12-04 17:35:35 +00:00
Dmitriy Rabotyagov 2fe09cc698 Allow to provide policy state
It might be desired by deployer to remove already applied policy.
For that policy state should be explicitly passeda as absent
for the module.

Change-Id: I0db1e23ca0edf65aa1b4c8ca20153337044b6c4c
2021-11-30 21:07:47 +02:00
Dmitriy Rabotyagov f0f24a9ecc Add config_template to collections
Change-Id: I59e3e142363a9d254f70881d153bedc326fb8094
2021-11-30 17:54:16 +02:00
Dmitriy Rabotyagov 6345b19bca Bump ansible and collection versions
Change-Id: If67c448dcefb0fcd7a345fe48da5cebc372c2432
2021-11-10 09:54:12 +00:00
Zuul a238a9a5c6 Merge "setup.cfg: Replace dashes with underscores" 2021-08-03 09:28:05 +00:00
Zuul 9ef85442ba Merge "Remove linters from openstack-ansible-role-jobs" 2021-07-05 11:43:19 +00:00
Jonathan Rosser 3afe3b7448 Add ansible-role-pki to required-projects
Change-Id: Iba078102bdf5a7e7929a07ddfe9b73a63a70536f
2021-07-01 17:38:17 +01:00
Dmitriy Rabotyagov 0d95a0590e Remove linters from openstack-ansible-role-jobs
Once we've moved almost all repos to integrated linters jobs,
we shouldn't include linters inside functional tests template.

Change-Id: I890529cdd2fe3918e9d97e918da303c929dfe07d
2021-06-30 09:24:33 +00:00
Dmitriy Rabotyagov 76fe426827 Fix repo passing CI
Also remove linters from regular functional template

Depends-On: https://review.opendev.org/c/openstack/ansible-role-pki/+/798685
Depends-On: https://review.opendev.org/c/openstack/ansible-role-pki/+/798725
Depends-On: https://review.opendev.org/c/openstack/ansible-role-pki/+/798713
Change-Id: Ibfdb5185ee5da8f83858ef1f1db9ea3d332799ea
2021-06-30 08:51:06 +00:00
Dmitriy Rabotyagov 7da3518bf4 Bump ansible-lint
Since role-name (106) test has been added metadata tag in
ansible-lint 5.0.9, we shouldn't exclude it explicitly.

Change-Id: I84f8a97ecce0eebf1f0fbb5c8a4fc433eedc34fd
2021-05-24 12:40:45 +03:00
Dmitriy Rabotyagov 2030d6d1ac Add role-name lint check to warnings
role-name check is new one, which verifies role name regexp according to
the ansible guide [1]
based on the meta file, but does not have metadata tag on it.

[1] https://docs.ansible.com/ansible/devel/dev_guide/developing_collections.html#roles-directory

Change-Id: I2dc887a3932e40f634f3bcb0aec46faf314f1907
2021-05-20 18:34:02 +00:00
Dmitriy Rabotyagov d88998cd68 Bump ansible-lint and ansible version
ansible-lint 5.0.9 has merged our NoLogPasswordRule [1]
APTRepositoryCacheUpdateRule is not relevant since
ansible fix has been merged for several years now.
Also ansible-lint provides yamllint check.

This means, we shouldn't really carry custom lint
rules and can jsut use upstream ones.

We also update ansible and collections version to
match with the integrated repo.

[1] 8bef056d69

Change-Id: Ie85d6d0a17849bbc665a7f49644c38b5f6a96bb2
2021-05-19 18:25:57 +00:00
XinxinShen 86182ebe24 setup.cfg: Replace dashes with underscores
Setuptools v54.1.0 introduces a warning that the use of dash-separated
options in 'setup.cfg' will not be supported in a future version [1].
Get ahead of the issue by replacing the dashes with underscores. Without
this, we see 'UserWarning' messages like the following on new enough
versions of setuptools:

  UserWarning: Usage of dash-separated 'description-file' will not be
  supported in future versions. Please use the underscore name
  'description_file' instead

[1] https://github.com/pypa/setuptools/commit/a2e9ae4cb

Change-Id: I150f3362a3790ad6db4da443f582671e6d909f4b
2021-05-05 17:14:43 +08:00
Zuul efd8b4a1a4 Merge "Use ansible_facts[] instead of fact variables" 2021-05-05 09:07:50 +00:00
Dmitriy Rabotyagov 7013ee811e Set buster job to NV
Recent nova does not support libvirt version that is provided in
Debian Buster, so we setting job to not voting.

Change-Id: I2c0bcbcfe1f8b4bee60a6069e4b2417ad5f67cf6
2021-04-19 19:18:49 +03:00
Zuul be89e1210f Merge "Run notify setup when setup_host differs" 2021-04-19 09:47:28 +00:00
Dmitriy Rabotyagov 7466bc9309 Add is_metal variable
In functional tests, comparing to the integrtion ones, dynamic inventroy
is not used. Thus, ``is_metal`` variable is not defined, and we rely on
it in some roles.

To cover the gap, we're setting a variable based on the ansible_facts for
functional tests only.

Change-Id: Ifc9bb5bb65d599a39f562faaaa14a0756e493ba1
2021-04-15 15:21:58 +03:00
Dmitriy Rabotyagov d38c743552 Run notify setup when setup_host differs
There's a usecase, where all datasets might be the same, except
setup_hosts. For example, when we use isolated rabbitmq cluster for RPC
but shared one for notifications. And we have same set of users
and vhosts there. In current state setup won't be performed, since
vhosts and users are the same.

Change-Id: Ib5c44327d5f1edc25bace79fdd057adabb0b4c14
2021-04-07 18:32:33 +03:00
Marc Gariepy 567aa74319 Fix vagrantfile config
fix the config for focal

Change-Id: I9d73211137c1fc409db7f98e17416d90ec04fc96
2021-03-19 15:14:05 -04:00
Jonathan Rosser 84eea5b91d Use ansible_facts[] instead of fact variables
See https://github.com/ansible/ansible/issues/73654

Change-Id: I9841e5b7f32f452b7da0c643cb6c3e0e30d9e413
2021-03-17 12:20:20 +00:00
Zuul 5e46a70a74 Merge "Clean up code from unsupported operating systems" 2021-03-12 22:06:54 +00:00
Jonathan Rosser 4eac7fd2c1 Clean up code from unsupported operating systems
Remove SUSE, Gentoo, and references to yum package manager.

Change-Id: If591661b4145219f7837960f93bc9647b60d2a70
2021-03-11 23:08:21 +00:00