openstack/kolla-ansible
Code Issues Proposed changes

Merge "Adds feature flag for ironic-inspector in bifrost"

Browse Source
This commit is contained in:
Zuul 2024-02-29 17:05:13 +00:00 committed by Gerrit Code Review
commit 06e39af796
3 changed files with 19 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
ansible/roles/bifrost/defaults/main.yml
View File

@ -11,3 +11,10 @@ bifrost_deploy_image_full: "{{ bifrost_deploy_image }}:{{ bifrost_deploy_tag }}"
bifrost_deploy_container_proxy: "{{ container_proxy }}" bifrost_deploy_container_proxy: "{{ container_proxy }}"
bifrost_deploy_verbosity: "-vvvv" bifrost_deploy_verbosity: "-vvvv"
# Whether to enable the legacy ironic-inspector service
# NOTE(wszumski): Bifrost plans to remove this option once the native in-band
# inspection reaches feature parity. Please see:
# https://bugs.launchpad.net/kolla/+bug/2054685 which contains links for
# tracking the progress.
bifrost_enable_ironic_inspector: true

3
ansible/roles/bifrost/templates/bifrost.yml.j2
View File

@ -33,3 +33,6 @@ generate_tls: true
# NOTE: Needs to be world-readable, writeable by root, and persistent, which # NOTE: Needs to be world-readable, writeable by root, and persistent, which
# the default /etc/bifrost is not. # the default /etc/bifrost is not.
tls_root: "/etc/bifrost-certs" tls_root: "/etc/bifrost-certs"
# Whether to enable the legacy ironic-inspector service.
enable_inspector: "{{ bifrost_enable_ironic_inspector }}"

9
releasenotes/notes/expose-flag-to-enable-ironic-inspector-in-bifrost.yml-12215fb9b1f09528.yaml Normal file
View File

@ -0,0 +1,9 @@
---
features:
- |
Exposes a flag, ``bifrost_enable_ironic_inspector``, to enable
ironic-inspector in Bifrost. This option defaults to ``True`` as it can be
useful for backwards compatability. It is still possible to use native
in-band inspection when Ironic Inspector is enabled by setting
``inspect_interface`` to ``agent``. Please see the Ironic documentation for
more details.