13663 Commits

Author SHA1 Message Date
Juan Pablo Suazo Sarrocchi
95317e5df2 Fixes task name in notify module to the actual task name
An OVS-DPDK task had part of its name capitalized,
presumably due to the newly added ansible linter,
but this change was not reflected in the notify
module which calls the task. This results in an
inability to deploy openstack with OVS-DPDK.

This commit fixes the name of the task in the
notify module.

Closes-Bug: #2033413
Change-Id: Iae8a937bb2c35d2eb83e2c4522d829a6cfe25f0d
Signed-off-by: Juan Pablo Suazo <jsuazo@whitestack.com>
2023-08-29 12:51:44 -04:00
Pierre Riteau
f6e83f92c7 Remove duplicate whitespace
Change-Id: Ica155c5da29d36a3f944eb6a4a0ef5af88b01358
2023-08-29 14:43:16 +02:00
Pierre Riteau
e3e3046152 Fix Cyborg endpoints again
Cyborg endpoints were fixed with a /v2 suffix [1], but this was
accidentally reverted by the single external frontend patch [2].

[1] https://review.opendev.org/c/openstack/kolla-ansible/+/883495
[2] https://review.opendev.org/c/openstack/kolla-ansible/+/823395

Related-Bug: #2020080
Change-Id: Ic2cfaf739a65fbb02577be08c081a2a88360d5f5
2023-08-29 11:07:57 +00:00
Bartosz Bezak
cf8283928e Configure OVN to emit "need to frag" packets in case of MTU mismatch
It is useful when external network's MTU is lower then internal
geneve networks.
Host kernel needs to be in version >= 5.2 for this option to work.
All Kolla supported host operating systems have higher kernel version.

Change-Id: Id64e99b07e2bb5e6c97b784f4ffedafc7e7de188
2023-08-29 10:08:53 +00:00
Erik Olof Gunnar Andersson
8e3c168908 Fix designate-producers not running properly
Kolla is setting the producer tasks to None, and this
disables all producer tasks.

Starting with the next release of Designate a new task
was added to the producer named increment_serial that
is now critical path for incrementing the serial for each zone.
fb9c2da977

Closes-Bug: #1879557
Change-Id: Ief8dca4e27197c9576e215cbd960da75f6fdc20c
2023-08-29 10:07:32 +00:00
Bartosz Bezak
106a21fb6c Fix D001 Line too long in mariadb-guide
Change-Id: I0aff87f0392b43671c6e9fc3fae3cdb0f7ce118a
2023-08-29 11:27:10 +02:00
Michal Nasiadka
40cf91aab6 cleanup: fix cleanup containers
Currently the script only stops containers and removes services,
without disabling service or removing symlink - which renders the
service in a not-found state.
After we disable and stop - some of the services end up in failed
state (due to wrong exit codes for various reasons) - running
reset-failed fixes that.

Change-Id: I637783ce758dbf1c2a7b4b99aa6b61e2c5ca1460
2023-08-28 13:05:02 +00:00
Maksim Malchuk
b205dad892 Drop useless DUMMY_ENVIRONMENT environment for cron container
Since I033c51704bfb806f6515db26b8159fae12469ba5 the DUMMY_ENVIRONMENT
environment for cron container is useless, lets drop it.

Trivial-Fix

Change-Id: Id8bf642a57b54ecafe0ee6981e044b1aba872ef7
Signed-off-by: Maksim Malchuk <maksim.malchuk@gmail.com>
2023-08-27 02:20:46 +03:00
Matt Crees
b86c304a29 Enable RabbitMQ HA queues by default
Sets the variable ``om_enable_rabbitmq_high_availability`` to ``true``
by default. An upgrade will therefore require some manual steps to
migrate from transient to durable queues. Note that this will be
caught by this precheck:
https://review.opendev.org/c/openstack/kolla-ansible/+/880274

Also updates the CI upgrade jobs to perform this migration. This will
need to be removed in Caracal.

Related-Bug: #2031294

Change-Id: I26a70d4722aaa4663eced5f5337840474c7b961c
2023-08-25 10:10:04 +00:00
Matt Crees
60282285bd Add command to force reset the state of RabbitMQ
This command can be invoked with ``kolla-ansible rabbitmq-reset-state``.
This is primarily designed to be used when enabling HA queues[1].

As such, this also updates the RabbitMQ documentation to use this
command.

[1] https://docs.openstack.org/kolla-ansible/latest/reference/message-queues/rabbitmq.html#high-availability

Change-Id: I6ad95a3618fc1a34af56657ef99ef14dc979f17a
2023-08-25 10:09:58 +00:00
Michal Nasiadka
d1a52b5816 debian: Add Bookworm Host OS support
Depends-On: https://review.opendev.org/c/openstack/ansible-collection-kolla/+/892323

Change-Id: I11db700511233aa60229ee65d0cc96e46aafdf90
2023-08-24 19:08:36 +00:00
Adam Stackhouse
842e758221
Adding mariadb_port to wsrep sync status so alterative ports can be used
Closes-Bug: #2024554
Change-Id: Id6eae798784126d4dd53adef15bdce6b47b4601f
2023-08-22 12:00:49 +01:00
Zuul
0765e276fe Merge "glance/neutron: drop nbproc and add nbthread support" 2023-08-21 15:03:28 +00:00
Rodolfo Díaz
59204b2fee Fixes WEBSSO_KEYSTONE_URL Value
Change I60162b54bc06e158534d29311d4474b34750c64d
removed the `/v3` prefix from the WEBSSO_KEYSTONE_URL
variable. However, keystone endpoints do in fact
have the `/v3` prefix, and Horizon expects the
WEBSSO_KEYSTONE_URL variable to already contain
the prefix.

This patch adds the prefix again so that SSO
works with Horizon again.

Closes-Bug: #2028921
Change-Id: I5799f7a6fa4f52c2904b14fc02ed18443f4194bd
Signed-off-by: Juan Pablo Suazo <jsuazo@whitestack.com>
2023-08-21 14:19:01 +00:00
Michal Nasiadka
30fce65b2a glance/neutron: drop nbproc and add nbthread support
Change-Id: I42f9f182a2dab8563008e8b817ac58a69b72b062
2023-08-18 12:21:41 +00:00
Zuul
6af1754071 Merge "loadbalancer: remove support for haproxy_processes" 2023-08-18 10:28:23 +00:00
Zuul
0f7593a539 Merge "systemd: handle running container without systemd unit" 2023-08-18 07:39:25 +00:00
Léo Gillot-Lamure
b230858fb4 loadbalancer: remove support for haproxy_processes
Threads are the recommended way to scale CPU performance since HAProxy
1.8.

Official documentation says: « While "nbproc" historically used to be the only
way to use multiple processors, it also involved a number of shortcomings
related to the lack of synchronization between processes (health-checks, peers,
stick-tables, stats, ...) which do not affect threads. As such, any modern
configuration is strongly encouraged to migrate away from "nbproc" to
"nbthread". ».

Change-Id: I6f2e9d74e68703c8e0827e495945a75f020e1561
2023-08-18 06:39:58 +00:00
Michal Nasiadka
28e9f6e50c toolbox: use script handling signals
Since we use "sleep infinity" in kolla_toolbox container - stopping it
always ends up with exit code 143.

Let's use a wrapper script that handles signals correctly.

Closes-Bug: #2031692

Depends-On: https://review.opendev.org/c/openstack/kolla/+/891777
Change-Id: I84295b909d788ceb67bf09ef60fb0583108b111b
2023-08-17 19:41:21 +00:00
Michal Nasiadka
1497ab2ab3 systemd: handle running container without systemd unit
MariaDB bootstrap has a phase where the first MariaDB container
is running with Galera bootstrap - after a check that WSREP
is synced is successful - we restart the container.

The bootstrap container is named mariadb and running with
docker_restart_policy: "no" - the restarted container should be running
in systemd.

Before this patch the code created a systemd unit but it was initially
stopped - so stopping was always a success - and the container would be
killed with SIGKILL on removal (which obviously breaks MariaDB).

This patch also improves docker/systemd stops by waiting for real
unit/container stop and adds failing CI for containers that are
killed with signal 9.

Closes-Bug: #2029613

Change-Id: I0a03e509ce228a50e081fcab44d2b4831251190c
2023-08-17 14:57:39 +00:00
Zuul
3eec4b41aa Merge "loadbalancer: support cpu-map for threads" 2023-08-17 14:31:06 +00:00
Zuul
4cd8b77fd4 Merge "loadbalancer: support setting nbthread with variable haproxy_threads" 2023-08-17 14:11:28 +00:00
Zuul
dbad6c1f28 Merge "Fix improper designate-manage command usage." 2023-08-17 11:55:40 +00:00
Zuul
a34f1cd7e6 Merge "Add support for checking Octavia cert expiration" 2023-08-17 07:21:28 +00:00
Zuul
01a911b27d Merge "Fix an undefined variable bug for venus deployment" 2023-08-15 09:23:14 +00:00
Zuul
b64cd8bc3c Merge "CI: bump ansible version in linter" 2023-08-11 15:02:36 +00:00
Zuul
9b74433054 Merge "cephadm: log to file" 2023-08-11 10:59:00 +00:00
Michal Nasiadka
6e76ed2e40 CI: bump ansible version in linter
Change-Id: I2326225ddde2216cc94a0486366d5ef4782384d7
2023-08-11 09:36:57 +00:00
likui
8d98a89b64 update ansible version
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
2023-08-11 09:10:31 +08:00
Maksim Malchuk
e365f4b70d Deny access to public /server-status in http Openstack services
This change block access to the public /server-status url on all
http services exposed by HAProxy, also fixes an issue with Horizon
where 'Require all granted' open access to the /server-status in
the HAProxy-less configurations. Without this change the issue
affects only Ubuntu/Debian installations where mod_status in Apache2
enabled by default.

Closes-Bug: #1996913
Change-Id: I3ec1af6353c3ecc64589599abe375b0ae9b14d5c
Signed-off-by: Maksim Malchuk <maksim.malchuk@gmail.com>
2023-08-10 10:28:10 +00:00
Zuul
0a0eb30bc9 Merge "ironic: add ironic_agent_files_directory parameter" 2023-08-09 16:34:41 +00:00
Zuul
b49694ff99 Merge "rabbitmq: add rabbitmq_datadir_volume parameter" 2023-08-09 16:22:01 +00:00
Zuul
7575f4a485 Merge "Enable nova libvirt driver skip_cpu_compare_on_dest workaround" 2023-08-08 09:07:20 +00:00
Michal Nasiadka
b3ef7a9943 CI: venus - replace elastic with opensearch in gate deps
Change-Id: I4c5ade2d687684f37df83b44a852d88c77067680
2023-08-07 09:51:32 +02:00
Zuul
1938f555e3 Merge "Correct glance.conf example in external Ceph guide" 2023-08-04 11:49:34 +00:00
Matt Crees
08f408b526 Add genconfig to the Operating Kolla guide
While this section of the guide does not need to explain every command
available, ``kolla-ansible genconfig`` is a useful command for people
newer to using KA, so it seems worth mentioning in the guide.

Change-Id: I6f58bb395012b701ad349e967fb3d2f16e9373c7
2023-08-03 09:32:05 +01:00
Léo Gillot-Lamure
789ac7388c loadbalancer: support cpu-map for threads
The directive used has the same semantic as what is done above for nbproc > 1:
it binds each thread to a CPU. It is simpler and does not require a loop because
it uses the auto: syntax available in HAProxy 2.4.

Change-Id: I1ce124b678140f5f4737df557683bb67bc7cfc66
2023-08-02 13:25:55 +00:00
Léo Gillot-Lamure
e043828601 loadbalancer: support setting nbthread with variable haproxy_threads
Threads are the recommended way to scale CPU performance since HAProxy
1.8.

Official documentation says: « While "nbproc" historically used to be the only
way to use multiple processors, it also involved a number of shortcomings
related to the lack of synchronization between processes (health-checks, peers,
stick-tables, stats, ...) which do not affect threads. As such, any modern
configuration is strongly encouraged to migrate away from "nbproc" to
"nbthread". ».

While more recent versions of HAProxy automatically detect the number of
available CPU and enable threads for them, it can be useful to explicitely set
the value.

In this patch, setting cpu-map for threads is not supported.

Change-Id: Id917c70f3dbe52f24f25d9403ba8151729e8966b
2023-08-02 13:25:50 +00:00
Christian Berendt
58b5ca2783 ironic: add ironic_agent_files_directory parameter
With the parameter ironic_agent_files_directory it is possible to provide
the directory for the ironic-agent.kernel and ironic-agent.initramfs
files. By default the parameter is set to the value of node_custom_config.
This corresponds to the existing behaviour.

Change-Id: I53bb0eddc5380713a967356c85897d8df8ce505f
2023-08-02 14:14:47 +02:00
howardlee
7cb92baf05 Fix an undefined variable bug for venus deployment
The external_protocol variable does not exist, resulting in an error
during the venus deployment process. This commit will fix that.

Closes-Bug: #2029353

Change-Id: I2d983eecd8861689fdab7d60bdb9dd34ea0c159e
2023-08-02 18:15:01 +08:00
Zuul
e0b212d72d Merge "Fix L3 agent docs typo" 2023-08-02 09:48:56 +00:00
Zuul
4bfd9d85cd Merge "zun: Fix capsule functionality and tests" 2023-08-01 13:15:46 +00:00
Bartosz Bezak
bcf002509d Enable nova libvirt driver skip_cpu_compare_on_dest workaround
With the libvirt driver, during live migration,skip comparing guest CPU
with the destination host. When using QEMU >= 2.9 and libvirt >= 4.4.0,
libvirt will do the correct thing with respect to checking CPU
compatibility on the destination host during live migration.[1]

[1] 267a40663c

Change-Id: I947c94b59368c7a2740583bf57e407296473d75e
2023-08-01 10:44:24 +02:00
Michal Nasiadka
3431f124c8 cephadm: log to file
Currently we don't gather mon/mgr logs, which are essential for
debugging Ceph integration issues - this patch improves that.

Change-Id: I4ec34511b022707d0069d7a07f2f92b34acf0085
2023-07-28 14:58:48 +02:00
Jake Hutchinson
bb05f3b862 Fix L3 agent docs typo
Change-Id: I149bc646d2e5fb1e32be9c94c6a8d75fd92893e1
2023-07-28 10:46:59 +01:00
Paul Arthur
acf9375edc Fix loop label syntax error with ansible-core 2.15
Labels are supposed to be strings, but prior to
https://github.com/ansible/ansible/pull/80040 Ansible
did not enforce this.

Change-Id: Iefad160be12f1b5e689a74a82714857fa867d69a
Signed-off-by: Paul Arthur <paul.arthur@flowerysong.com>
2023-07-27 18:05:47 +00:00
Zuul
af2ccd1572 Merge "Trivial: Add deploy-containers for skyline" 2023-07-27 10:37:17 +00:00
Zuul
7dbe8184de Merge "haproxy: support single external frontend" 2023-07-25 10:11:17 +00:00
Doug Szumski
bacd6c7f4f Fix OpenSearch Dashboards health check
The OpenSearch Dashboards container does not have a health
check defined when created. This causes the container to always
restart when reconfigured, even if no change has been made.

Change-Id: I0b437a77aeb61bc5ae9238f900a1fa00cbc34e18
Partial-Bug: #2028362
2023-07-21 16:54:34 +01:00
Matt Crees
618d056369 Correct glance.conf example in external Ceph guide
Under the Glance section of the external Ceph guide, to enable
copy-on-write the config heading should be ``[DEFAULT]``, rather than
``[GLOBAL]``. For reference, see the Glance docs:
https://docs.openstack.org/glance/latest/configuration/glance_api.html#DEFAULT.show_image_direct_url

Closes-Bug: #2017643
Change-Id: I2d77e01637ecae0bd10319eac4bc75340dd6f61f
2023-07-17 22:14:36 +02:00