Switches to the built-in inspection by default and deprecates support
for ironic-inspector. The actual removal should happen after we get
inspection rules in since it's the last feature that is relevant for
bifrost.
Depends-On: https://review.opendev.org/c/openstack/ironic/+/907398
Change-Id: Id157d5ada2ff3f87f3f4d161423b1f3989f73194
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
Fixes bifrost so it utilizes:
* The admin role, instead of the long deprecated baremetal_admin
role.
* Utilizes system scope and admin scoped access where needed
for during installation self-test commands to execute
as expected.
* The user utilized for inspector's keystone access
now utilizes the "admin" role instead of the
baremetal_admin role.
Closes-Bug: 2051168
Change-Id: I090520547846a7a8d85bd032a52d6da039761186
After the switch to newer ansible collection, the enrollment workflow
started running automated cleaning (before that, it would use the old
API version resulting in immediate "available"). However, the static
DHCP configuration only happened in the inspect and deploy workflows,
which are run after enrollment.
This change extracts a new small role for the DHCP config and includes
it in all 3 workflows that use DHCP.
While here, make sure that dnsmasq_dhcp_hostsdir is respected.
Change-Id: Idf6f24dde11d600698d45a218812cba8134fb73f
The enable_dhcp variable is still pointing to the removed variable
include_dhcp_server while it should use enable_dhcp instead.
Change-Id: I60992fa7286766f4fac935e53207f49a98cd77cd
Currently ironic deploys log to the usual system console, while
inspector discovery/enroll default to logging to serial console.
Change-Id: I6a97d2de5add07cb10853eefb70a88574df44242
Previously online data migrations were not performed in the default
configuration where the database address is 'localhost'. I'm not sure
why this condition was used, but it was present in the original
change I685efd14bf3567a126311b676a50b0abb0f043db submitted adding the
online data migrations back in the Stein release.
This change removes the condition to perform online data migrations
regardless of the database address.
Change-Id: I60d0f61a3c6fd3573b2031f7dba5d8a29e178394
Closes-Bug: #2036772
By default, QEMU uses "qemu64" model that does not support x86-64-v2
features required for CentOS/RHEL 9. Use the appropriate host model
option depending on whether KVM or emulation is used.
Change-Id: Idf191547044b12c7acdbd6d3962c541a1b8c6d03
Often it is beneficial to use the functionality provided by `use_cirros`
and `cirros_deploy_image_upstream_url` to download and test with images
of other OS distributions. This patch changes the variable names to
reflect that use case.
`use_cirros` -> `download_custom_deploy_image`
`cirros_deploy_image_upstream_url` -> `custom_deploy_image_upstream_url`
The functionality of the old variables is retained for backwards
compatability.
Co-Authored-By: Alex Welsh <alex@stackhpc.com>
Change-Id: I8b3a973bd5ef19333af563c170b2a28fbbfb5c76
Newest versions break libvirtd
For more info check https://bugs.launchpad.net/neutron/+bug/2029335
Disabling upgrade job until we fix CS9 in all stable branches
Change-Id: If5da0b1b81e17ad0dd6dc1ac62442a84510e35ce