13409 Commits

Author SHA1 Message Date
Zuul
be1ca8a910 Merge "Fix Bash variable expansion issues in openrc file" 2023-05-16 01:58:48 +00:00
Pierre Riteau
f1b4098261 Remove kolla-ansible-ubuntu-binary-upgrade job
This job is not required since Zed dropped binary images. The job
definition was already deleted.

Change-Id: Ie3b8653ec309146f8e354a7afe4ae66dfbe6f8cd
2023-05-15 21:59:30 +02:00
Zuul
bfd75b1f05 Merge "CI: retry smoke tests and instance creation" 2023-05-08 14:45:46 +00:00
Michal Nasiadka
2524ec422d CI: retry smoke tests and instance creation
Sometimes services have problems connecting to the database
or rmq on upgrade jobs, and we're not retrying the test.

Depends-On: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/882344

Change-Id: I9fca3d4336375b72d09633b51c8596e0c08abbf9
2023-05-08 10:05:49 +00:00
Matt Crees
46c2b60d9f Correct ovn-ctl --db-nb-pidfile usage in templates
The flags ``--db-nb-pid`` and ``--db-sb-pid`` are corected to be
``--db-nb-pidfile`` and ``--db-sb-pidfile`` respectively. See here for
reference:
6c6a7ad1c6/utilities/ovn-ctl (L1045)

Closes-Bug: #2018436
Change-Id: Ic1e8768374566eb2198302807ecc644a19cd3062
2023-05-04 10:33:51 +01:00
Zuul
92ddbdfbc1 Merge "octavia: Fix hm-interface after bumping openstack collection" 2023-04-27 09:28:36 +00:00
Sven Kieske
c899ff26f7
Deprecate Sahara and Vitrage
as agreed in the Kolla meeting:

https://meetings.opendev.org/meetings/kolla/2023/kolla.2023-04-19-13.00.html

Signed-off-by: Sven Kieske <kieske@osism.tech>

Change-Id: I099a5328e0837e1f5dcf7f21b7fd7bea1748456d
2023-04-26 15:35:01 +02:00
Michal Nasiadka
c0d0866785 octavia: Fix hm-interface after bumping openstack collection
New openstack collection modules have changed output
dicts/variable names - adapting to that.

Also changing octavia amphora image to focal, since bionic
hasn't been rebuilt since May 2021.

Closes-Bug: #2012255
Change-Id: Icf38a52472d02ef7d69bcd3716afb16e859d44a2
2023-04-26 13:13:31 +00:00
Maksim Malchuk
df6472e0af Fix Bash variable expansion issues in openrc file
Sometimes passwords, URLs and other values of the variables can
contain special symbols, for example the dollar sign, using these
values can lead to unpredictable attempts of the variable expansions
in the Bash scripts, such as openrc file, so we need to use single
quotes for all variables values.

Change-Id: Ib2aabadd0ffd6a8dc2591245f29b4478e03d92fc
Signed-off-by: Maksim Malchuk <maksim.malchuk@gmail.com>
2023-04-25 14:46:11 +03:00
Zuul
6c3738a84c Merge "Designate: provide certificates file to use for neutron client requests" 2023-04-24 14:40:38 +00:00
Zuul
3b24d77c48 Merge "Fix faulty precheck for RabbitMQ" 2023-04-24 09:55:10 +00:00
Maksim Malchuk
31e0d374bd Designate: provide certificates file to use for neutron client requests
Closes-Bug: #1919464
Change-Id: I64a5185627f98bcaf471ed8ccf0fd8f314769404
Signed-off-by: Maksim Malchuk <maksim.malchuk@gmail.com>
2023-04-23 01:13:32 +03:00
Zuul
e8091acefc Merge "Add check_mode: false to Nova upgrade checks" 2023-04-21 07:39:16 +00:00
Zuul
861e9467ad Merge "Revert "ansible: bump min to 2.13 and max to 2.14"" 2023-04-20 13:20:52 +00:00
Magnus Lööf
fdacf9d1d9 Fix faulty precheck for RabbitMQ
When using externally managed certificates, according to [1],
one should set `kolla_externally_managed_cert: yes` and ensure
that the certificates are in the correct place.

However, RabbitMQ precheck still expects the certificates to be
available on the controller node. This is incorrect.

Fix by not running the tasks in question when `kolla_externally_managed_cert: yes`

[1] https://docs.openstack.org/kolla-ansible/latest/admin/tls.html

Closes-Bug: 1999081
Related-Bug: 1940286
Signed-off-by: Magnus Lööf <magnus.loof@basalt.se>
Change-Id: I9f845a7bdf5055165e199ab1887ed3ccbfb9d808
2023-04-20 13:28:39 +02:00
Zuul
557b1272ff Merge "Trivial fix - add int filter for rabbitmq definitions" 2023-04-20 11:15:16 +00:00
Dr. Jens Harbott
b98a71e5d4 Revert "ansible: bump min to 2.13 and max to 2.14"
This reverts commit 9867060b6b3bd36aad121b53b9e5dddfca8a8e4c.

Reason for revert: seems this broke some jobs

Change-Id: I1ca81214ece403351c0a522ea05bf07802e4c4c0
2023-04-20 09:26:11 +00:00
Zuul
226eb2aa99 Merge "Configure coordination in default for masakari-api" 2023-04-20 09:11:13 +00:00
Michal Arbet
d1b24a4137 Trivial fix - add int filter for rabbitmq definitions
Change-Id: I1d8021a1bc780449e3ca96183c6f4abaed17b382
2023-04-19 21:27:20 +02:00
Zuul
6873931f1d Merge "cli: fix find globals.d" 2023-04-19 18:29:39 +00:00
Zuul
005c6fae38 Merge "Update notes about CentOS support" 2023-04-19 17:11:06 +00:00
Zuul
6421d8df07 Merge "Add precheck to fail if RabbitMQ HA needs configuring" 2023-04-19 17:11:04 +00:00
Matt Crees
e0191cb167 Add check_mode: false to Nova upgrade checks
The step "Run Nova upgrade checks" needs to be excluded from check mode.

Related-Bug: #2002657
Change-Id: Ida582606b03d1896a6c58d4b324d088dc3b71852
2023-04-19 16:19:59 +01:00
Zuul
9cfba97cf5 Merge "Use the upgraded image to run Nova upgrade checks" 2023-04-19 14:16:19 +00:00
Matt Crees
a5331d3208 Add precheck to fail if RabbitMQ HA needs configuring
Currently, the process of enabling RabbitMQ HA with the variable
``om_enable_rabbitmq_high_availbility`` requires some manual steps to
migrate from transient to mirrored queues. In preparation for setting
this variable to ``True`` by default, this adds a precheck that will
fail if a system is currently running non-mirrored queues and
``om_enable_rabbitmq_high_availbility`` is set to ``True``.

Includes a helpful message informing the operator of their choice.
Either follow the manual procedure to migrate the queues described in
the docs, or set ``om_enable_rabbitmq_high_availbility`` to ``False``.

The RabbitMQ HA section of the reference docs is updated to include
these instructions.

Change-Id: Ic5e64998bd01923162204f7bb289cc110187feec
2023-04-19 09:45:34 +01:00
Zuul
f5991df02b Merge "mariadb: add mariadb_datadir_volume parameter" 2023-04-19 08:27:42 +00:00
Zuul
17aa380bf8 Merge "ansible: bump min to 2.13 and max to 2.14" 2023-04-19 01:26:28 +00:00
Zuul
fab40de52f Merge "Remove RabbitMQ ha-all policy when not required" 2023-04-18 20:24:54 +00:00
Zuul
7a8f3d3212 Merge "Fix create sasl account before config file is ready" 2023-04-18 18:57:00 +00:00
Zuul
52611ecbab Merge "Fix maximum width of the DIB Multiline-YAML" 2023-04-18 14:51:39 +00:00
Michal Arbet
842adf6d2f Configure coordination in default for masakari-api
This patch introduces distributed lock for masakari-api
service when handle the concurrent notifications for the same
host failure from multiple masakari-hostmonitor services.

Change-Id: I46985202dc8da22601357eefe2727599e7a413e5
2023-04-17 20:02:42 +02:00
Michal Nasiadka
c0a45c7eb7 opensearch: default dashboards tag to opensearch_tag
Closes-Bug: #2016627

Change-Id: I5ae1d911c5df423e0b70dab306709320083b7b69
2023-04-17 12:37:33 +00:00
Maksim Malchuk
47862b56bd Fix maximum width of the DIB Multiline-YAML
The dib_env_vars variable in the Bifrost's dib.yml file can contain
the DIB_BLOCK_DEVICE_CONFIG environment variable which is always the
Multiline-YAML data. By default, the format of the data is not
preserved while the configuration is merged and saved for the
bifrost-deploy container.

This is because Ansible uses the PyYAML library which has a default
80 symbol string length limit. The official Ansible documentation [1]
recommends using to_yaml or to_nice_yaml filters with width parameter.
This change adds the same ability to the merge_yaml Ansible plugin.

1. https://docs.ansible.com/ansible/latest/user_guide/playbooks_filters.html#formatting-data-yaml-and-json

The related change for the diskimage-builder to solve the issue with
incorrect data provided by Kolla-Ansible is also provided:
I3b74ede69eb064ad813a9108ec68a228e549e8bb

Closes-Bug: #2014980
Related-Bug: #2014981
Change-Id: Id79445c0311916ac6c1beb3986e14f652ee5a63c
Signed-off-by: Maksim Malchuk <maksim.malchuk@gmail.com>
2023-04-14 16:36:22 +03:00
Zuul
b142b0ae7a Merge "Fix merge action plugins verbose output" 2023-04-14 10:36:45 +00:00
Zuul
be2e353da3 Merge "RabbitMQ use maintenance mode on container restart" 2023-04-13 21:47:41 +00:00
Zuul
4b27d19b63 Merge "Set RabbitMQ ha-promote-on-shutdown=always" 2023-04-13 15:05:48 +00:00
Michal Nasiadka
9867060b6b ansible: bump min to 2.13 and max to 2.14
Change-Id: Ibc9cc91f64b0450de3cae6e2830b4ff2c52c0395
2023-04-13 14:10:28 +00:00
Matt Crees
c85b64d158 Remove RabbitMQ ha-all policy when not required
With the addition of the variable
`om_enable_rabbitmq_high_availability`, this feature in the upgrade
task should be brought back. It is also now used in the deploy task. The
`ha-all` policy is cleared only when
`om_enable_rabbitmq_high_availability` is set to `false`.

Change-Id: Ia056aa40e996b1f0fed43c0f672466c7e4a2f547
2023-04-13 15:58:15 +02:00
Matt Crees
e709599ff0 RabbitMQ use maintenance mode on container restart
Puts the RabbitMQ node into maintenance mode before restarting the
container. This will make the node shutdown less disruptive. For details
on what maintenance mode does, see:
https://www.rabbitmq.com/upgrade.html#maintenance-mode

Change-Id: Ia61573f3fb95fe8fcde6b789ca77ef5b45fe0a65
2023-04-12 15:58:17 +01:00
Michal Nasiadka
b30c7bc87b rabbitmq: Do not stop containers on upgrade
Since RMQ 3.8 we can use rolling upgrade [1].

Depends-On: https://review.opendev.org/c/openstack/kolla/+/872393

[1]: https://www.rabbitmq.com/upgrade.html#rolling-upgrades

Change-Id: If6a7c6c12d9226a2406728108b3c87b3485ac55f
2023-04-12 11:12:30 +00:00
gamerslouis
46415123d5 Fix create sasl account before config file is ready
Add checking for container readiness before create sasl user

Closes-Bug: #2015589
Change-Id: Ic650ba6be1f192e3cbeaa94de3d00507636c1c92
2023-04-08 17:48:46 +00:00
Zuul
717eb8c14f Merge "nova: Fix live migration on RHEL9 derivatives" 2023-04-03 14:31:39 +00:00
Michal Nasiadka
7c32e6f3b3 nova: Fix live migration on RHEL9 derivatives
Closes-Bug: #2005119
Change-Id: I542f7ae19b4400355b04854f42a1d1802a6efeea
2023-04-03 10:28:48 +00:00
Zuul
3e556dad61 Merge "Add LimitRequestBody configuration for Horizon" 2023-04-03 10:25:55 +00:00
Maksim Malchuk
8972241dc9 Fix merge action plugins verbose output
This change fixes the output 'module_args' information of the plugins
'merge_configs' and 'merge_yaml' when Ansible is executed in maximum
verbose mode. Now all the plugin options are displayed instead of
standard 'copy' plugin options only.

Also, this change contains fixes already applied in the Kayobe
project to improve and synchronize the code of the plugins between
projects.

Change-Id: Ie2d9a0501fe29bfd854eb31258f282b197855948
Signed-off-by: Maksim Malchuk <maksim.malchuk@gmail.com>
2023-04-02 00:33:40 +03:00
Michal Nasiadka
35b92825b1 cli: fix find globals.d
Currently when the /etc/kolla/globals.d directory does not exist
it writes an error on stderr.

Change-Id: I2d4b3dd9dde3d383af213dc6fd376bc14c650a7d
2023-03-31 14:03:14 +02:00
Zuul
dbb4f3d0ae Merge "Add neutron_ovn_availability_zones parameter" 2023-03-29 16:48:23 +00:00
Zuul
0b7ed4dad9 Merge "[CI] Test Neutron with IPv6 networks" 2023-03-29 13:47:54 +00:00
Maksim Malchuk
d907790fff Add LimitRequestBody configuration for Horizon
Since CVE-2022-29404 is fixed [1,2] the default value for the
LimitRequestBody directive in the Apache HTTP Server has been changed
from 0 (unlimited) to 1 GiB. This limits the size of images (for
example) uploaded in Horizon. This change add the ability to
configure the limit.

1. https://access.redhat.com/articles/6975397
2. https://ubuntu.com/security/CVE-2022-29404

Closes-Bug: #2012588
Change-Id: I4cd9dd088cbcf38ff6f8d188ebcc56be7d9ea1c9
Signed-off-by: Maksim Malchuk <maksim.malchuk@gmail.com>
2023-03-29 14:51:28 +03:00
Zuul
406a090f3d Merge "Add note about removing leading tabs in ceph.conf files" 2023-03-29 11:37:14 +00:00