12596 Commits

Author SHA1 Message Date
Zuul
1a538cce0e Merge "Add ability to retry image pulling" 2021-08-23 13:22:31 +00:00
Marc 'risson' Schmitt
ca6fc69ee8
docs: adding and removing host: add --all-projects option
When running without that option, the list of servers might be empty,
resulting in leftover instances on the host that is about to be
migrated.

Change-Id: I85915f0f86655f3a1eb5f54911d24e4e0e75f340
2021-08-22 11:34:40 +02:00
Zuul
8f6ab211af Merge "[CI] Fix Debian upgrade" 2021-08-21 11:26:46 +00:00
Radosław Piliszek
3c68e82585 Fix Masakari in multi-region deploys
to behave like it is most commonly expected - query Nova in the
same region.

Closes-Bug: #1939291
Change-Id: I584a83d352c747a799b5dab1d3b8159ba3805454
2021-08-20 18:53:46 +00:00
Radosław Piliszek
802f7c6218 Never make Docker registry insecure by default
To follow best security practices and help fellow operators.

More details inline and in the linked bug report.

Closes-Bug: #1940547
Change-Id: Ide9e9009a6e272f20a43319f27d257efdf315f68
2021-08-20 18:23:56 +00:00
Michal Arbet
4900bbf5a5 Use Docker healthchecks for neutron-bgp-dragent service
This change enables the use of Docker healthchecks for
neutron-bgp-dragent service.

Change-Id: I8c7e69b04d87acd1742c33269cde5b88ce75e619
2021-08-20 15:04:57 +02:00
Michal Arbet
ffd53512af Rename role haproxy to loadbalancer
For now role haproxy is maintaining haproxy
and keepalived. In follow-up changes there is also
proxysql added.

This patch is *only* renaming/moving stuff to more
prominent role loadbalancer, and moving also specific
templates to subdirectory.

This was done only to better diff in follow-up
changes.

Change-Id: I1d39d5bcaefc4016983bf267a2736b742cc3a555
2021-08-19 21:20:33 +02:00
Radosław Piliszek
cbb567cb86 Add ability to retry image pulling
Sometimes, the registries may intermittently fail to deliver the
images. This is often seen in the CI, though it also happens with
production deployments, even those with internal registries and/or
registry mirrors - due to sheer load when trying to pull the
images from many hosts.

This patchs adds two new vars to control retry behaviour.
The default has been set to make users happier by default. :-)

Change-Id: I81ad7d8642654f8474f11084c6934aab40243d35
2021-08-19 18:38:59 +00:00
Zuul
a98076f11c Merge "Use more RMQ flags for less busy wait" 2021-08-19 18:20:13 +00:00
Radosław Piliszek
16a4a9e5ac Remove an unused file
It seems to have been mistakenly introduced by
de00bf491dfbabc8e11009fce4410bce5c2110ed
"Simplify handler conditionals"

Change-Id: I65b6e322fa11a870f32099bbfd62150cbea4feb5
2021-08-19 13:23:44 +00:00
Zuul
98479f5e28 Merge "tools: use /usr/bin/env bash instead of /bin/bash" 2021-08-19 09:22:42 +00:00
Marc 'risson' Schmitt
0858d5487e
Fix haproxy precheck when kolla_externally_managed_cert is used
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
Closes-Bug: #1940286
Change-Id: I647f8615e9fd0cc0db7c737ce4afbd1bdd0d40d4
2021-08-18 16:07:09 +02:00
Michal Arbet
90fd9152a4 Use Docker healthchecks for keystone-fernet container
This change enables the use of Docker healthchecks for
keystone-fernet container. It checks if "key 0" has
right permissions, and if rsync is able to distribute
keys to other keystones.

Implements: blueprint container-health-check
Change-Id: I17bea723d4109e869cd05d211f6f8e4653f46e17
2021-08-17 15:07:01 +02:00
Marc 'risson' Schmitt
839ec629bf
tools: use /usr/bin/env bash instead of /bin/bash
Signed-off-by: Marc 'risson' Schmitt <marc.schmitt@risson.space>
Change-Id: I374f7427a4318d00ca474367818117e11789ec13
2021-08-17 14:29:33 +02:00
Skylar Kelty
8d5dde3723
Update Manila deploy steps for Wallaby
Manila has changed from using subfolders to subvolumes.
We need a bit of a tidy up to prevent deploy errors.
This change also adds the ability to specify the ceph FS
Manila uses instead of relying on the default "first found".

Closes-Bug: #1938285
Closes-Bug: #1935784
Change-Id: I1d0d34919fbbe74a4022cd496bf84b8b764b5e0f
2021-08-17 10:01:58 +01:00
Zuul
ea008fd7fe Merge "Validate if running CentOS OS is CentOS Stream" 2021-08-16 10:25:16 +00:00
Michal Arbet
2b599bdb83 Use Docker healthchecks for nova-spicehtml5proxy service
This change enables the use of Docker healthchecks for
nova-spicehtml5proxy service.

Implements: blueprint container-health-check
Change-Id: I584c588c20781e6c6567429811aecf97967baea3
2021-08-16 11:20:03 +02:00
Radosław Piliszek
b7e85d5282 [CI] Test Swift upgrades
Swift is a major OpenStack project. It could use testing upgrades.

New jobs are placed in the experimental pipeline to avoid
excessive CI load on general changes.

Change-Id: I8a089fdd1f21eb4c3e00c38ea9dfcecc77565bf5
Related-Bug: #1874691
2021-08-13 15:26:55 +00:00
Jan Horstmann
f8e3e169c3 Allow override of rabbitmq config in kolla toolbox
Certain overrides for rabbitmq may need to be set for `rabbitmqctl` in
kolla-toolbox aswell.
This commit allows to override `rabbitmq-env.conf` and `erl_inetrc` in
kolla-toolbox.

Change-Id: Idef6adcf9700f75a2db503444a8de093ee21a9c5
2021-08-13 12:57:16 +00:00
Radosław Piliszek
13200ace33 Fix kolla-toolbox with IPv6 and disabled RabbitMQ
Closes-Bug: #1939883
Change-Id: Ica311acba445cccac1d20757ced6f15a064ebcaa
2021-08-13 12:47:41 +00:00
Radosław Piliszek
b6d9cf7685 [CI] Fix Debian upgrade
Debian upgrades failed on using the ansible command to remove
chrony service because of broken python autodetection.
This patch uses the same workaround we have in globals-default.j2.
This is not nice long-term but there is no reason to keep the two
out of sync. We should remove this workaround from both places
when the situation fixes itself (possibly with newer Ansible).

Change-Id: I8b7f0c76d55cd31311285ce746acb6335e044470
2021-08-12 19:11:53 +00:00
Michał Nasiadka
d56dc34034 Validate if running CentOS OS is CentOS Stream
Change-Id: I2fe738249018e25e79fd726bc931be6b7bd91934
2021-08-12 12:45:45 +00:00
Michal Arbet
85879afc0b Trivial fix nova's healthchecks
Kolla-ansible upgrade task is calling different
handlers as deploy task and these handlers are
missing healthcheck key. This patch is fixing
this.

Closes-Bug: #1939679
Change-Id: Id83d20bfd89c27ccf70a3a79938f428cdb5d40fc
2021-08-12 13:39:50 +02:00
Pierre Riteau
bcfebaf87a Use api-paste.ini from /etc/neutron
CentOS binary packages for Xena have moved Neutron's api-paste.ini file
to /etc/neutron [1]. Ubuntu packages already provide this file at this
location. Source images should also have api-paste.ini installed there.

[1] https://review.rdoproject.org/r/c/openstack/neutron-distgit/+/34845

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

Change-Id: Ideb0247578afb8cc36bb6ac26d8ef7fda494bee3
2021-08-12 10:07:50 +02:00
Zuul
61f2bd86b0 Merge "fluentd: Fix check for external elasticsearch" 2021-08-11 14:32:59 +00:00
Zuul
e017e40968 Merge "Remove deprecated Designate option" 2021-08-11 13:09:06 +00:00
Zuul
6dff0305c0 Merge "Remove redundant Monasca Kafka client option" 2021-08-11 11:40:12 +00:00
Zuul
6d371d47ca Merge "Support multiple inventories" 2021-08-11 11:24:20 +00:00
Zuul
3be572f67f Merge "ironic: Follow up for ironic_enable_keystone_integration" 2021-08-11 09:50:58 +00:00
Zuul
578ca1d647 Merge "Add suppress_type_name to stop warnings with ES7" 2021-08-10 16:26:11 +00:00
Zuul
9cd7ed8074 Merge "Refactor and optimise image pulling" 2021-08-10 14:59:45 +00:00
Radosław Piliszek
9ff2ecb031 Refactor and optimise image pulling
We get a nice optimisation by using a filtered loop instead
of task skipping per service with 'when'.

Partially-Implements: blueprint performance-improvements
Change-Id: I8f68100870ab90cb2d6b68a66a4c97df9ea4ff52
2021-08-10 11:57:54 +00:00
Zuul
c0540760e0 Merge "monasca-thresh: Fix topology submission to storm" 2021-08-10 10:59:17 +00:00
Mark Goddard
46df30d871 ironic: Follow up for ironic_enable_keystone_integration
Follow up for I0c7e9a28876a1d4278fb2ed8555c2b08472864b9 which added a
ironic_enable_keystone_integration variable to support Ironic in
multi-region environments. This change skips Keystone service
registration based on ironic_enable_keystone_integration rather than
enable_keystone. It also updates the ironic-inspector.conf template to
use the new variable.

Change-Id: I2ecba4999e194766258ac5beed62877d43829313
2021-08-10 09:36:25 +01:00
Zuul
63be501318 Merge "baremetal: use docker_yum_gpgkey to fetch docker GPG key" 2021-08-09 15:36:49 +00:00
Zuul
a81c6dabdc Merge "neutron: fix neutron-server config.json with VMware" 2021-08-09 11:09:23 +00:00
Zuul
214d34ca7b Merge "Extra var ironic_enable_keystone_integration added." 2021-08-09 10:53:20 +00:00
Zuul
c1e14025ad Merge "Remove unused imports in merge_yaml" 2021-08-09 10:52:27 +00:00
Zuul
01a4868ba6 Merge "Fix release note for ansible_facts" 2021-08-09 10:26:01 +00:00
Doug Szumski
b692ce7af1 Support monitoring Fluentd with Prometheus
This patch adds support for integrating Prometheus with Fluentd.
This can be used to extract useful information about the status
of Fluentd, such as output buffer capacity and logging rate,
and also to extract metrics from logs via custom Fluentd
configuration. More information can be found here in [1].

[1] https://docs.fluentd.org/monitoring-fluentd/monitoring-prometheus

Change-Id: I233d6dd744848ef1f1589a462dbf272ed0f3aaae
2021-08-09 10:12:20 +01:00
Mark Goddard
bd3ad904d5 CI: stop setting ceph_nova_user
Once I6aa8db2214e07906f1f3e035411fc80ba911a274 has been merged and
backported, the default value of ceph_nova_user (cinder) should work.

Change-Id: I9769c4ee26393f8458537ad2ad543213cf8f57b0
2021-08-09 08:54:09 +00:00
Mark Goddard
a08aaf7d8e Fix release note for ansible_facts
Change-Id: I0103d7ef55e6aebe043a582d36f1a2efa137f447
2021-08-09 08:48:21 +00:00
Mark Goddard
2c786a0faf neutron: fix neutron-server config.json with VMware
Missing comma in config.json template rendered invalid JSON with VMware
plugin agent.

Closes-Bug: #1939080

Change-Id: I3d0c6421e3da7e1e753b99cf87c32fc238a21523
2021-08-09 09:29:51 +01:00
Victor Morales
d15d94300f Remove unused imports in merge_yaml
Dumper and Loader are classes seem to be loaded but not used in the merge_yaml
file. This change removes them for reducing the number of lines.

Change-Id: I87ef305903ab02226fcaa725ece622647d17811c
2021-08-06 14:45:45 -07:00
Zuul
a63b4ef85a Merge "Elevated privileges required to set owner/group/mode by ansible" 2021-08-06 15:12:52 +00:00
Ilya Popov
da4fd2d6a2 Extra var ironic_enable_keystone_integration added.
Basically, there are three main installation scenario:

Scenario 1:
Ironic installation together with other openstack services
including keystone. In this case variable enable_keystone
is set to true and keystone service will be installed
together with ironic installation. It is possible realise this
scenario, no fix needed

Scenario 2:
Ironic installation with connection to already installed
keystone. In this scenario we have to set enable_keystone
to “No” to prevent from new keystone service installation
during the ironic installation process. But in other hand,
we need to have correct sections in ironic.conf to provide
all information needed to connect to existing keystone.
But all sections for keystone are added to ironic.conf only
if enable_keystone var is set to “Yes”. It isn’t possible
to realise this scenario. Proposed fix provide support for
this scenario, where multiple regions share the same
keystone service.

Scenario 3:
No keystone integration. Ironic don't connect to Keystone.
It is possible realise this scenario, no fix needed

Proposed solution also keep the default behaviour: if no
enable_keystone_integration is manually defined by default
it takes value of enable_keystone variable and all behaviour
is the same. But if we don't want to install keystone and
want to connect to existing one at the same time, it will be
possible to set enable_keystone var to “No”
(preventing keystone from installation) and at the same
time set ironic_enable_keystone_integration to Yes to allow
needed section appear in ironic.conf through templating.

Change-Id: I0c7e9a28876a1d4278fb2ed8555c2b08472864b9
2021-08-06 17:58:48 +03:00
Zuul
1a4a8c1615 Merge "Reduce container metrics cardinality" 2021-08-06 14:47:38 +00:00
Zuul
be129d3100 Merge "Fix config action when OVN is enabled" 2021-08-06 14:25:55 +00:00
Zuul
3f67a96510 Merge "Blazar: Fix support for external keystone in multiregion deploy" 2021-08-06 14:12:21 +00:00
Zuul
bb05cf1150 Merge "Remove support for Prometheus v1" 2021-08-06 14:12:18 +00:00