6003 Commits

Author SHA1 Message Date
Zuul
e78f4330dd Merge "Use Docker healthchecks for keystone-fernet container" 2021-08-27 09:02:15 +00:00
Zuul
6362dfa942 Merge "Fix Masakari in multi-region deploys" 2021-08-27 08:55:06 +00:00
Zuul
ad093c01ee Merge "Use api-paste.ini from /etc/neutron" 2021-08-27 08:55:04 +00:00
Zuul
0f1c1981bb Merge "Remove an unused file" 2021-08-24 10:17:47 +00:00
Zuul
1a538cce0e Merge "Add ability to retry image pulling" 2021-08-23 13:22:31 +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
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
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
Zuul
ea008fd7fe Merge "Validate if running CentOS OS is CentOS Stream" 2021-08-16 10:25:16 +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
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
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
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
Zuul
bb1522e380 Merge "[manila] Drop tenant_id templating from v2 endpoint" 2021-08-06 14:12:15 +00:00
Zuul
295c69b5ee Merge "Remove tempest role" 2021-08-06 14:04:55 +00:00
Zuul
caf763f7c3 Merge "Use Docker healthchecks for kuryr services" 2021-08-06 14:04:53 +00:00
Zuul
762e840a50 Merge "Use Docker healthchecks for zun services" 2021-08-06 14:04:51 +00:00
Zuul
c6f7feec48 Merge "Use Docker healthchecks for ceilometer services" 2021-08-06 13:58:58 +00:00
Piotr Parczewski
30e0eae8b2 Remove deprecated Designate option
Change-Id: Ib9ea83dd0019a4c4703e673a783c45ab07afe4e7
2021-08-06 15:31:07 +02:00
Alexander Evseev
7f98238b64 Elevated privileges required to set owner/group/mode by ansible
Elevated (root) privileges are required to set owner/group/mode when
target owner does not math the user running Ansible. Without it the
playbook fails with 'Permission denied' error.

Change-Id: Ie7455a5f1ed709dfb9c9d7c653c6f808c00af4c2
2021-08-06 12:39:32 +00:00
Piotr Parczewski
0d79d25fe9 Remove support for Prometheus v1
Change-Id: I0d7c7f47e6653cf2903589a9c86798a8c6404af5
2021-08-05 21:07:22 +02:00
Zuul
01c62fa1b6 Merge "Fix nova deployment failure when rabbitmq is disabled" 2021-08-04 13:12:14 +00:00
Zuul
d5b7af30e8 Merge "Fix deployment failure when kolla_dev_mod is enabled" 2021-08-04 13:00:58 +00:00
Zuul
94d7311b57 Merge "Do not run timesync checks on deployment host" 2021-08-04 12:40:38 +00:00
Zuul
87777b3601 Merge "Fix freezed spice console in horizon" 2021-08-04 12:21:45 +00:00
Zuul
80758f44d4 Merge "Trivial fix horizon's healthcheck when SSL turned on" 2021-08-03 11:07:43 +00:00
Zuul
82441350f2 Merge "watcher: add missing become for copying configs" 2021-08-03 09:23:57 +00:00
Zuul
ef6698dc76 Merge "Fix incorrect config of linuxbridge multiple external networks" 2021-08-02 20:07:07 +00:00
Michal Arbet
6ac4638cfb Trivial fix horizon's healthcheck when SSL turned on
This patch is fixing docker healthcheck for horizon
by changing value of horizon_listen_port, so
both apache's virtualhost and healthcheck will have
same correct port always. Also removing useless
apache's redirect as all redirects are done on
haproxy side.

Closes-Bug: #1933846
Change-Id: Ibb5ad1a5d1bbc74bcb62610d77852d8124c4a323
2021-08-02 19:58:37 +02:00