This is the first in a series of commits to add support for codespell. This is continuning the process completed in ironic-python-agent.
Future Commits will add a Tox Target, CI support and potentially a git-blame-ignore-revs file if their are lots of spelling mistakes that could clutter git blame.
Change-Id: I41b2db35cdf21917ac20ed0866af3b9c94789f0b
It looks a little jarring at first so see keys ordered this way,
but once you get used to it, it reads so much better!
Also make indentation consistent, clean up epel install
and remove traces of Suse.
Change-Id: Id6279d681faf0c9a1893c00953b0b59d9319e08b
The epel packages are installed but their content is purged in the
centos nodes. Removing and reinstalling them when we run tests in
CI to reenable their content.
Change-Id: Ibf1d810a37811151449f79a24bb837db6663cce5
The epel repository contains among other things 'debootstrap',
which is needed by dib_host_required_packages, whenever
dib_os_element contains the word 'debian' (or 'ubuntu'), which
is the default[1] and install_dib is true which it also is by
default[2]
This improves upon I57513883c0fa8d6ffb2c70debb81d5e7e6d8b221
where the documentation[3] and procedure has probably been updated
since it was last followed. We can also remove the conditional
installation of dib_host_required_packages with enablerepo: epel
for centos
[1] bifrost-ironic-install/defaults/main.yml:dib_os_element: "debian"
[2] bifrost-ironic-install/defaults/main.yml:install_dib: "{{ create_image_via_dib | bool or create_ipa_image | bool }}"
bifrost-ironic-install/defaults/main.yml:create_image_via_dib: "{{ not use_cirros | default(false) | bool }}"
bifrost-ironic-install/defaults/main.yml:use_cirros: false
[3] https://docs.fedoraproject.org/en-US/epel/#_el9
Change-Id: Ia16b544df261b999b1503ee110ec88f39becd7dd
At least on the CI nodes it causes issues with grub-pc on Debian since
the CI nodes don't have bootloader configured. Download and extract
packages instead.
To account for Kolla (which has a split between the install and the
bootstrap phases), the downloaded files are cached in /use/lib/ironic.
Change-Id: I9307366db9579b194dcb88818ed0ce2fedb4baaf
The epel repository is used only when we need to build a debian-based
IPA image, so we enable it only in that case for CentOS Stream 8/9.
The configuration procedure is explained in the official guide [1].
[1] https://docs.fedoraproject.org/en-US/epel/#_el9
Change-Id: I57513883c0fa8d6ffb2c70debb81d5e7e6d8b221
There is currently a lot of duplication between nginx setup for keystone
(identity API) and ironic (httpboot directory). This change moves the
common code to a new role and makes the httpboot configuration a
separate nginx configuration file.
Change-Id: Ic5239aba764f632c48a422812fde7010ffb1a339
This provides a cleaner separation between installation and image
creation.
This allows Kolla to install all package dependencies at container image
build time rather than when using the container image to build disk
images.
Story: 2008839
Task: 42346
Change-Id: I5be8b355b92f0d9d907c743b522982f6de0398c1
This commit adds support to install the Ironic Prometheus Exporter,
it's necessary to set `enable_prometheus_exporter=True` or pass
`--enable-prometheus-exporter` when using the `bifrost-cli`
Also the ipe is now enabled in the
bifrost-integration-tinyipa-keystone-ubuntu-bionic/centos8
Change-Id: I3972e88162fba31ad6e262f47b67a37cd37a2076
This broke Kolla already.
One cannot assume systemd is being used during installation.
Package installations normally reload systemd themselves
if they need to.
Backport to Ussuri.
Change-Id: Iafc3e14eaf3e8341eb7d2af30c460bbee1c9c104
Story: 2008112
Task: 40828
Modifying the public zone is questionable, let's use our own zone.
Also let's make sure network_interface actually belongs to it.
Change-Id: I63f5fa4845aa8f1c90a0c73dd78deb45aaaa4fd1
Switch ironic-staging-drivers to source installation since
it's released very infrequently.
Do not clone keystone if it's not enabled
Change-Id: If01ae7c54d5d421a5c6b97a4d8b29bdaa722ddd9
Allows de-duplicating pip_install.yml between ironic and keystone,
fixes numerous inconsistencies and centralizes venv and skip_install
handling.
Change-Id: Id6a1a4f3ebe2010ab14f43a306b084f4525d5862
Apparently, it was needed for yum->dnf transition, which has happened
for all supported distributions already.
Change-Id: I603ae0b274d8a24b08af634012104f00e59e75f9
Shade is being phased out in favour of openstacksdk, which also allows
to simplify the authentication code.
Change-Id: I14210a720792c3f7f0669330ccb6d2f67d624246
Bifrost clones the DIB repo and installs it. However, after running
install.yaml pip freeze shows diskimage-builder 2.33 is installed.
This is caused by the recent addition of IPA builder, which has its own
dependency on diskimage-builder.
This change fixes the issue by swapping the order of installation of IPA
builder and DIB, to ensure DIB is installed last.
Change-Id: Ifd67adaa98e920f42c80d239349ae61d7d610f66
Story: 2007401
Task: 38999
Currently a few packages are installed without it, pulling in undesired
updates (e.g. cryptography 2.8 on stable/steain). This change fixes it.
The --upgrade and --force-reinstall flags are removed from all pip calls
since they're not compatible with using upper-constraints.
The explicit installation of pyOpenSSL is removed, >= 18.0 is anyway
used by Rocky and newer. A few explicit usages of -c argument are
also removed in favour of the implicit handling in pip_install.yml.
Change-Id: Ic776cb2ec0779c9b324e94bf387ee4f3f857cd72
Also uncapped the proliantutils maximum as upper
constraints is in place.
Set the ansible python interpreter as well for the
main process launch, which magically resolved
venv dependency issues.
Change-Id: Iae06a87e7dbcf76e4be7453e57ec5fd75389caa7
Attempts to clean up conditionals such that they are properly interpreted
and executed as intended.
Change-Id: I213f0db2e8d56590ba4b58908e538156edee0b1f
The goal of upper-constraints is to avoid breakages with versions
known not to work. Lack of upper-constraints for ironicclient is
currently breaking stable branches by pulling packages that are
not Python 2 compatible.
If we need recent versions of ironicclient, we should use source
installation instead.
To make it work with packages that are constrained themselves,
this patch splits the source-based installation into two steps:
installing requirements (constrained) and installing the project
itself (unconstrained).
Change-Id: Ie5f62655248c40885fdcd6fa87180a6952f145fc
* Remove definitions for ubuntu 15.04 and 15.10
* Changed binary packages to the python3 package names.
* Removed python-configparser as it is no longer required
Change-Id: I8f102bb6ea6238a7f5a638b89f01dc16852fc741
Also update the documentation to purge any mentions of CoreOS images
and the deprecated ironic-agent element.
Change-Id: I09018e451e698a7c6b99106b64c87a526cc959d5
The iscsi deploy interface depends on sgdisk for partioning.
Additionally mkfs.fat is needed for uefi partitions.
This commit installs said dependencies if the iscsi interface
is enabled.
Change-Id: Id400e5c7364eb8825234c49034d084c835c29776
This avoids an issue where the version installed could be incompatible
with ironic, or other installed libraries.
In particular, this was seen with proliantutils 2.8.2, which is
incompatible with the version of sushy-lib on the stable/rocky branch.
Change-Id: I02f1e247ca2333b2e90586e7311395e7bd94938c
Story: 2005310
Task: 30214