openstack/openstack-ansible-os_trove - openstack-ansible-os_trove - OpenDev: Free Software Needs Free Tools

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: I7cd73a8e8eccf28e3991913a35aa3ac9e847774f

With commit [1] to collection output structure of networks_info module
has been changed. With that we adpot to the new format.
[1] 9272146cf7
Change-Id: Ia0d50b96a4af59352a89c566137d8b0208338b6b

Implement support for service_tokens. For that we convert
role_name to be a list along with renaming corresponding variable.
Additionally service_type is defined now for keystone_authtoken which
enables to validate tokens with restricted access rules
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible-plugins/+/845690
Change-Id: I7b01de89b459b4992ed9531e6c81259af21ba997

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: Iabdc99ad150ea318263b4c384b26160c4b6c816a

Change-Id: Id0a07d09b1029edc1b57f1dcab5a5be91e077107

Change-Id: I852dfddbf8aac10d31fbaff4561cb159bd6bdc06

Change-Id: I0078082f2701f4bec072c0c931996c2ee8c4af76

Change-Id: I02152618bd19800316b875f04da9e907f3446510

Use a first_found lookup instead of a with_first_found loop so that
the 'paths' parameter can be used.
This ensures that only vars from the role are included, and not vars
from a parent calling role. This can happen when a parent role has
a higher priority vars file available for inclusion than the role
it calls.
Change-Id: I41a360e6f99ecd0ca5e4c13888bff4f363d8bc33

Change-Id: I9a40cfff39ae1cd4c53e5f783b620298c1fa93b2

Change-Id: I0e76cf0c6060dbd14707fb8d4b8a4ff993ab20c1

- Implemented new variable ``connection_recycle_time`` responsible for SQLAlchemy's connection recycling
- Set new default values for db pooling variables which are inherited from the global ones.
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/819424
Change-Id: Ie9591c96d1e5c8f4dcf04524c04056934ceb00db

Since we still use ceph-ansible that has their own implementation of
config_template module it's worth to use mentioned module as a collection
explicitly.
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/819814
Change-Id: I377c7b182861a8c53c3ab43690c2bed0d648f3ae

Change-Id: Ie9778bbd241805b81699d447247a04a0a7323a6b

With PKI role in place in most cases you don't need to explicitly
provide path to the CA file because PKI role ensures that CA is trusted
by the system overall. In the meanwhile in PyMySQL [1] you must either
provide CA file or cert/key or enable verify.
Since current behaviour is to provide path to the custom CA we expect
certificate being trusted overall. Thus we enable cert verification when
galera_use_ssl is True.
[1] 78f0cf99e5/pymysql/connections.py (L267)
Change-Id: I27bbc7b97a2fec235d43b4ca502a3667450dc1fa

The patch bumps min version of tox to 3.18.0 in order to
replace tox's whitelist_externals by allowlist_externals option:
https://github.com/tox-dev/tox/blob/master/docs/changelog.rst#v3180-2020年07月23日
Change-Id: I84c80aa01542eef2904c8a620ed929b13963d7f2

We've created integrated linters check job a while back and it's successfully
working for several releases. At the moment we experience difficulties
with future maintenance of the linters check from the openstack-ansible-tests
repo. So instead of fixing current one, we replace it with modern version of
the test.
Change-Id: I2587d1c6ec05d5a31a69143eecc16f7b84dce669

We should make service restart handlers more universal, since
there might be cases where api and taskmanager are placed on different
hosts. So we should restart services only when they are present, and
filtered_trove_services helps us here as defines only services that
should exist on the host.
This also align handler with other services and fix taskmanager restart
as it is missing now but conductor is restarted twice instead.
Change-Id: Ia5c92f312a773c88563c61bba327f07ef369d62c

By default OSA shares `service` project across services. So we should
not override _project_description with services
Change-Id: I0e194c5bf0ae4de1c9dd906c36955b6217d3f601

Change-Id: Ib09b4341ebfe263490c9dde65ed7d0253e3ea170

Since Trove requires guest network to be passed to the rabbitmq it makes
sense to use standalone rabbitmq cluster and instruction about how to do
might be useful
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/784780
Change-Id: Ibbc6faa2a5d5b3af39db9ae214ba9087a777a0eb

We haven't payed attention to the trove for a while and we need to
align configuration with current state of the service.
trove-conductor and trove-taskmanager configs are not
loaded by trove services anymore (at least by default), so we merge all
options into trove.conf
Also defaults and deprecated options are dropped from the config.
We implement bunch of new variables to cover missed logic and
some usecases.
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/784565
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/784113
Change-Id: I2ba758de38ed3fee10fe1485aa25621eddbf9046

This migrate trove-api to usage of the uwsgi from native service to
align with other service way of deployment and since that's how service
is tested in devstack.
Change-Id: I83ad3af282942ff9714757e863f393894ac35f45

To match documentation and general pattern of service networks,
we change default subnets that will be used by trove instances.
Change-Id: I7a0f23b0cdd314206f299a860685e2f96ac3f625

Allow deployers to manage trove guestimages with trove role. This will
allow us to correctly test role with tempest, since we were not able to
upload images with tags otherwise
Change-Id: If2f550ef09cd01ec5eab485f0d81ecfaf32f924f

As per the community goal of migrating the policy file
the format from JSON to YAML[1], we need to replace policy.json to
policy.yaml and remove deprecated policy.json.
config_template has been choosen instead of the copy, since it can
properly handle content that has been lookuped.
We make a separate task not to restart service when it's not needed.
[1] https://governance.openstack.org/tc/goals/selected/wallaby/migrate-policy-format-from-json-to-yaml.html
Change-Id: I866fea573429ddcedf1547d9fa8a7caae448eca2

Change-Id: I5d12c7a1d5dbaa39175ffe318f998c43db9c8181

Since we copy all release notes to the integrated repo there is not need
in publishing release notes for each repository. We should only verify their
validity and linting.
Change-Id: I760b42b9f2c80932e3286d8c686d10b90f3f03b5

See https://github.com/ansible/ansible/issues/73654
Change-Id: I29de46e830e0a63c1da90043ec08c5dcc56b7e3e

For pretty weird reason, first request to create a network to the neutron
fails with 500 without reasonable stack trace. Re-running request succeeds.
So we adding retries to the task which resolves metnioned issue.
Change-Id: I5b09dc3bc52d98df756cdc60c5e94cd1e7344e9a

All references to Gentoo, SUSE, Debian stretch and Centos-7 are removed.
Conditional tasks, ternary operators and variables are simplified where possible
OS specific variables files are generalised where possible
Change-Id: Ibec329fa5dca39222519b99ad27d78b46c11633e

Change-Id: Iaced386a7123e2dce1e57b18ac39072bc7eed897

os_ prefixed module names are deprecated for a while and will be removed
Current usage generates a warning for users.
Change-Id: Ie473569d866d6d8a9995f73f7cf5472e60589523

This is necessary to use the new pip resolver
Change-Id: Icdc485613655ffd0c4a69839fdf555340435d2ae

Instead of overriding each service separatelly it might make
sense for deployers to define some higher level variable that
will be used first or fallback to default variable.
Change-Id: Ic2fa270a3582db3da01b3c449fbdd8c2d5045062

Even the most modest 4C/8T system would run with the maximum 16 processes
due to the calculation being VCPU*2.
We devide amount of CPUs to number of threads for hyperthreaded CPUs
Change-Id: I2ae9039871158f8a94e3f96694ddadea3005537b

UPPER_CONSTRAINTS_FILE is deprecated and TOX_CONSTRAINTS_FILE is
the new environment variable name that replaces it [1].
This allows to use upper-constraints file as more
readable way instead of UPPER_CONSTRAINTS_FILE=<lower-constraints file>.
[1] https://zuul-ci.org/docs/zuul-jobs/python-roles.html#rolevar-tox.tox_constraints_file
[2] https://review.opendev.org/#/c/722814/
Change-Id: Iad47cff6b33ccc82bf9c7f23c23c779568951724

Change-Id: If6ce227ac1e57905b1626a30d2227dffd86a5170