1996 Commits

Author SHA1 Message Date
Zuul
e9a3733e24 Merge "Prevent matching unrelated images" 2021-06-24 11:23:01 +00:00
Pierre Riteau
f575f4b7fd Prevent matching unrelated images
Because regexes defined by Kayobe are not strict enough, Kolla can build
container images unrelated to the deployment. For example, the default
service configuration will build:

- centos-binary-barbican-keystone-listener (matches keystone)
- centos-binary-prometheus-memcached-exporter (matches memcached)

This commit makes regexes stricter to avoid unecessary builds. Note that
some images are still built unecessarily, such as nova-compute-ironic
when ironic is disabled or neutron-metadata-agent-ovn when ovn is
disabled.

Change-Id: I67f0e48ee03b150f6fe52b1a3e7e727b79207231
2021-06-17 16:24:54 +02:00
Pierre Riteau
bca2369894 Remove outdated upgrade step
Migrating to Ironic hardware types was required in Rocky. This can now
be removed from upgrading instructions.

Change-Id: I0510db9fd1b8dd72540c83173267491b32728330
2021-06-17 16:10:57 +02:00
Zuul
2445ec740e Merge "Fix url to ansible site in kolla-ansible.rst" 2021-06-12 02:07:20 +00:00
Zuul
018b60ab7a Merge "Revert "CI: Fix libvirt errors caused by latest edk2-ovmf"" 2021-06-11 20:24:35 +00:00
Ilya Popov
15d20154bc Fix url to ansible site in kolla-ansible.rst
Story: 2008961
Task: 42597
Change-Id: I2637085a1df22e39d64c33b823fc948063d3e06f
2021-06-10 18:17:52 +03:00
Zuul
79a3c47d0f Merge "docs: add variable for current branch" 2021-06-09 14:06:45 +00:00
Zuul
0adf7f996d Merge "docs: Stop setting os_distribution in development all-in-one" 2021-06-09 14:06:22 +00:00
Zuul
7ad589c84a Merge "CI: test base container image build in seed jobs" 2021-06-09 14:06:05 +00:00
Mark Goddard
034d436095 docs: Stop setting os_distribution in development all-in-one
kayobe-config-dev now sets os_distribution automatically.

Change-Id: Ic9edcf8168355eaf3ea9aff14947666d0ed46c97
2021-06-09 14:04:44 +02:00
Mark Goddard
c9cc6f4064 docs: add variable for current branch
This allows us to include the correct branch when cloning repositories
in documentation.

Method adapted from OpenStack Ansible documentation. A backslash is
added on purpose before repository URLs to bypass highlighting which
makes them less readable (dark text on dark background).

https://opendev.org/openstack/openstack-ansible/src/branch/master/doc/source/conf.py

Story: 2008851
Task: 42366
Change-Id: I4b4a8d1c848bb992f65860b058cb9fd9d77613d0
2021-06-09 14:01:13 +02:00
Zuul
e03c7d7c57 Merge "Ubuntu: CI: rely on os_distribution, remove interfaces pause" 2021-06-09 11:23:35 +00:00
Zuul
736c8c210b Merge "Fix overcloud post configure Python interpreter issues" 2021-06-09 11:23:27 +00:00
Zuul
814a377471 Merge "Add kayobe overcloud service prechecks command" 2021-06-09 11:23:19 +00:00
Zuul
95af2edc6b Merge "docs: improve automated seed development info" 2021-06-09 11:19:56 +00:00
Zuul
4faaebec0d Merge "[release] Set previous_release to wallaby" 2021-06-09 10:23:01 +00:00
Pierre Riteau
66e143ada2 Revert "CI: Fix libvirt errors caused by latest edk2-ovmf"
libvirt-6.0.0-36.el8 was built [0] and released for CentOS Stream 8. It
includes a backport of a commit [1] from libvirt 7.4.0 to avoid erroring
out on unknown firmware features, which fixes the issue seen with the
newer edk2-ovmf package.

This reverts commit 1e14fa3a24fa64b84cc993c985e696a9cba78dea.

[0] https://koji.mbox.centos.org/koji/buildinfo?buildID=17918
[1] 61d95a1073

Change-Id: Ibe177fe078769204c7e89c04e40870890fe501ba
2021-06-09 09:48:33 +02:00
Zuul
910dd452d9 Merge "Revert "Use OpenStack Wallaby release"" 2021-06-08 17:50:59 +00:00
Mark Goddard
c6c1cde3f9 CI: test base container image build in seed jobs
Currently we do not have any coverage of building container images. This
change adds a basic test of building the base image to the seed jobs.

A potential future enhancement could involve pushing to a local registry
running on the seed.

Change-Id: Id7378dd844aca7cb5634535308eba0f60342c81d
2021-06-08 13:47:04 +01:00
Zuul
05225342a8 Merge "Fix container image builds" 2021-06-07 23:59:03 +00:00
Zuul
71ad7bc11d Merge "Ubuntu: ensure /usr/bin/python is python3" 2021-06-07 18:05:24 +00:00
Zuul
da737b5e27 Merge "chrony: cleanup during overcloud host upgrade" 2021-06-07 18:05:20 +00:00
Mark Goddard
9c8f74993d Fix container image builds
Container image builds broken due to disabling docker iptables
management in
https://review.opendev.org/c/openstack/kolla-ansible/+/751795. This
affects Wallaby and later.

This patch configures kolla-build to use network_mode=host.

Change-Id: I608c0362e6eeea65ffafb32cda5c19ccb6ec8b5c
Story: 2008942
Task: 42560
2021-06-07 15:18:53 +00:00
Mark Goddard
1dc6d98be7 Revert "Use OpenStack Wallaby release"
This reverts commit 3f76d7d134d1f484d2ea21fddd0fbee00788e837.

The version of IPA images has been pinned to wallaby to ensure they are
compatible with the Wallaby Ironic in current Kolla master images.
Without this, bare metal tests fail with

    'Not found: Extension with id iscsi not found.'

Ironic removed the iscsi driver in Xena.

Change-Id: Ia4c70965d9f3f099046591bac7f702ee982fef71
2021-06-07 15:19:52 +01:00
Mark Goddard
82def9a516 Ubuntu: ensure /usr/bin/python is python3
/usr/bin/python may be python 2 on Focal, which causes problems with
Ansible on the control host. By installing the python-is-python3 package
we ensure that the correct interpreter is used.

This change updates the installation documentation and development
environment scripts.

Story: 2004960
Task: 42579

Change-Id: Ie94099075bae3c491f9cf830c38e6cfc8af605a6
2021-06-07 10:00:40 +01:00
Zuul
091a262908 Merge "Optimise order of inspector_processing_hooks" 2021-06-04 17:48:01 +00:00
Radosław Piliszek
a52b3b27be Optimise order of inspector_processing_hooks
Ironic Inspector docs [1] recommend to use lldp_basic before
local_link_connection.
It is also how TripleO applies them.

[1] https://docs.openstack.org/ironic-inspector/wallaby/user/usage.html#plugins

Change-Id: I4613e8973ccaef4534e44e3c79e910de5041add0
2021-06-04 10:13:57 +00:00
Zuul
8b1d5f059d Merge "ntp: check for ntp group in inventory" 2021-06-03 14:04:16 +00:00
Michał Nasiadka
9883726bf2 docs: Update Freenode to OFTC
Change-Id: I9a7f010760da1f6fee45c96705c1cf159c3a0ca3
2021-05-31 10:02:04 +02:00
Mark Goddard
9d3045dbf0 ntp: check for ntp group in inventory
The Wallaby release adds support for configuring a chrony daemon on
hosts in the ntp group. The kolla chrony container is disabled by
default, meaning that users could be left without an NTP client if they
forget to add hosts to the ntp group.

This change adds a check for the existence of an ntp group in the
inventory. The group may be empty.

Change-Id: Ic0ff71bb11692eb6c5699a1673df2d16b8f894ec
Story: 2007872
Task: 42533
2021-05-27 15:19:33 +01:00
Mark Goddard
d2e3f6f1ae docs: improve automated seed development info
* add 'bridge_type: linuxbridge' to tenks configuration to avoid
  dependency on OVS
* extend seed development environment testing to include overcloud
  provisioning and deployment
* remove seed hypervisor and seed VM environments. These are very
  stale, and largely replaced by a-universe-from-nothing. Add a link to
  that workshop on the same page

Change-Id: I9928e5912e6770bdcc1d5d0884d2f101c16ee6a9
2021-05-27 15:10:31 +01:00
Pierre Riteau
1e14fa3a24 CI: Fix libvirt errors caused by latest edk2-ovmf
Some CentOS Stream CI jobs started to fail when defining VMs with:

    libvirt.libvirtError: internal error: unknown feature amd-sev-es

This was introduced by edk2-ovmf-20200602gitca407c7246bf-5.el8.noarch.
This change excludes this package version in Zuul jobs, which fixes the
kayobe-seed-vm-centos8s job.

Overcloud centos8s jobs were also broken by the nova-libvirt Kolla image
containing this package, which is used by Tenks. Updated images using
the previous version of edk2-ovmf [1][2] are now available from quay.io.

This also needs to be fixed for Kayobe and Tenks in general, unless a
new package is released soon.

[1] https://review.opendev.org/c/openstack/kolla/+/792999
[2] https://review.opendev.org/c/openstack/kolla/+/793174

Change-Id: I5bc8197143ce763cdaf8a9ac0b782b524d5df258
2021-05-27 10:35:52 +02:00
Mark Goddard
dc53d29eaa chrony: cleanup during overcloud host upgrade
The Kolla Ansible chrony container is disabled by default in the Wallaby
release. A new kolla-ansible chrony-cleanup command can be used to clean
up the container.

This change extends the 'kayobe overcloud host upgrade' command to cover
cleaning up the chrony container (if disabled) and deploying a host
chrony daemon.

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

Change-Id: I275102ec6b5bab6982577b52fd29654c874446ce
2021-05-24 10:55:38 +01:00
Mark Goddard
b398e28dc5 Add kayobe overcloud service prechecks command
Runs 'kolla-ansible prechecks' without deploying services.

Change-Id: I3806b707593fc6ac9d7125e1b12d716ad40a2808
Story: 2008912
Task: 42503
2021-05-21 11:26:49 +01:00
Zuul
6b65b29395 Merge "Replace custom serial loop with throttle" 2021-05-20 10:28:21 +00:00
Zuul
211c68aa6c Merge "Support additional pass through host variables" 2021-05-18 13:24:03 +00:00
Zuul
7fc375e68c Merge "Make docker registry network mode configurable" 2021-05-18 11:28:08 +00:00
Zuul
7f41b14b7b Merge "Remove default time sources from customisable configuration" 2021-05-18 02:16:24 +00:00
Pierre Riteau
223e310bc2 Remove default time sources from customisable configuration
It seems unlikely that anyone would want to extend the time servers
variable, so let's keep it simple and not add an additional variable
where one isn't necessary. This is also consistent with how we configure
DNS.

These variables haven't made it into an official release yet, so now
is the time to clean up.

Change-Id: I3c15c6976296abf9c78f1480c1779b854145c7ca
2021-05-13 16:35:57 +01:00
Zuul
928d0a5cb3 Merge "Fixes for Ansible 2.10" 2021-05-13 11:20:52 +00:00
Zuul
bc4c16f004 Merge "CentOS Stream: update DNF repo file names" 2021-05-13 10:49:33 +00:00
Mark Goddard
0ce6ea2ee2 docs: add reno and upgrade info for CentOS Stream
Story: 2008795
Task: 42459

Change-Id: I1f55a2f47ac614fea983e19802e469efd810e849
2021-05-12 10:41:58 +01:00
Mark Goddard
4a59ea483b Fix overcloud post configure Python interpreter issues
Some time ago in Kayobe we switched from using set_fact to override the
ansible_python_interpreter variable to using a task variable. However,
not all of the galaxy roles were updated to match. This can lead to an
issue where a fact set by the os_networks role overrides the task
variable set in Kayobe.

Note that we haven't seen this in CI since the code path requires the
use of a VLAN provisioning network, and CI uses a flat network.

Change-Id: I14289df5e248b5b47fb8e60c6eea1ac845f50fd3
Story: 2008284
Task: 41158
2021-05-12 08:11:08 +00:00
Mark Goddard
ece41b7a67 CentOS Stream: update DNF repo file names
CentOS repositories were renamed following the separation of CentOS
Linux and Stream. This change updates their names when using a local
mirror. It also removes the old files.

Change-Id: Ifcc3781d771e150b5771f2bf20cd8bcbc5faa443
Story: 2008795
Task: 42435
2021-05-11 22:50:31 +02:00
Mark Goddard
e187ad7955 Make docker registry network mode configurable
Adds a new flag, 'docker_registry_network_mode', which defaults to
'host'. This may be used to set the network mode of the Docker registry
container.

This is a follow up to I404dd52701426a10c2e92727bd52b7fd7112abf6, which
changed the network mode from the default of bridge to host. It allows
that change to be backported to stable branches, without modifying the
default value.

Change-Id: Ic8ec3bb98f8f016e1d089bf10bd0538264394241
2021-05-11 22:48:21 +02:00
Mark Goddard
c44f4815cf Fixes for Ansible 2.10
os_image_facts renamed to os_image_info, and no longer sets a fact.
Instead we use the registered result. The same is true for
os_networks_facts and os_networks_info.

We did not catch the networks issue in CI because the code only runs
with a VLAN provisioning network, and we use a flat network.

Change-Id: I6a801e8bb5171bebd2b3f20d6140329d38ea36a5
2021-05-11 20:41:17 +00:00
Zuul
4daf293e78 Merge "CI: Remove unused Docker images after overcloud upgrade" 2021-05-11 17:05:14 +00:00
Zuul
0000a13118 Merge "Adds support for configuring chrony" 2021-05-11 16:51:35 +00:00
Pierre Riteau
93a8baca93 CI: Remove unused Docker images after overcloud upgrade
Overcloud upgrade jobs often fail when running on nodes with a 38G root
disk. The overcloud upgrade itself generally works, but post-upgrade
testing fails, for example when RabbitMQ gets in alarm state due to lack
of free disk space, which prevents Nova from functioning correctly and
makes it return "Unknown Error (HTTP 504)".

Change-Id: I536a526340f6d7dea7a69372904b777292774eba
2021-05-11 11:45:04 +02:00
Zuul
bd47fd025d Merge "docs: aio: add note with link to automated setup" 2021-05-10 11:21:58 +00:00