Skip to content

Commit

Permalink
enable ironic inspector
Browse files Browse the repository at this point in the history
for ironic-inspector to work with neutron, the following need to be set:

enabled_inspect_interfaces must contain `inspector`
each node must have its inspect interface set to `inspector` by
running `openstack baremetal node set --reset-inspect-interface --inspect-interface inspector`

OR, if `default_inspect_interface` is set to `inspector`, new nodes will have their interface
set to this automatically, as well as when `--reset-inspect-interface` is called.

Finally, inspection_network MUST be set, or ironic will return a cryptic error message about how
"driver IPMI does not support managed inspection".
  • Loading branch information
msherman64 committed Aug 29, 2023
1 parent d14a87d commit 72aeeae
Show file tree
Hide file tree
Showing 4 changed files with 35 additions and 1 deletion.
1 change: 1 addition & 0 deletions kolla/defaults.yml
Original file line number Diff line number Diff line change
Expand Up @@ -164,6 +164,7 @@ ironic_provisioning_network: ironic-provisioning
ironic_provisioning_network_cidr: 10.51.0.0/24
ironic_provisioning_network_gateway: "{{ ironic_provisioning_network_cidr | ipaddr('next_usable') }}"
ironic_cleaning_network: "{{ ironic_provisioning_network }}"
ironic_inspection_network: "{{ ironic_provisioning_network }}"
ironic_image_cache_size: 20480 # MB
ironic_console_serial_speed: 115200n8
encoded_ironic_pxe_root_password: "{{ ironic_pxe_root_password | password_hash('md5') | regex_replace( '(\\$)', '$\\1') }}"
Expand Down
23 changes: 23 additions & 0 deletions kolla/node_custom_config/inspector.conf
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
[processing]

# Which MAC addresses to add as ports during introspection. Possible
# values: all (all MAC addresses), active (MAC addresses of NIC with
# IP addresses), pxe (only MAC address of NIC node PXE booted from,
# falls back to "active" if PXE MAC is not supplied by the ramdisk).
add_ports = all

# Which ports (already present on a node) to keep after introspection.
# Possible values: all (do not delete anything), present (keep ports
# which MACs were present in introspection data), added (keep only
# MACs that we added during introspection). (string value)
keep_ports = present

# Whether to overwrite existing values in node database. Disable this
# option to make introspection a non-destructive operation. (boolean
# value)
overwrite_existing = False

# Whether to update the pxe_enabled value according to the
# introspection data. This option has no effect if
# [processing]overwrite_existing is set to False (boolean value)
#update_pxe_enabled = true
10 changes: 10 additions & 0 deletions kolla/node_custom_config/ironic.conf
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,9 @@ default_console_interface = ipmitool-socat

enabled_management_interfaces = ipmitool,noop

enabled_inspect_interfaces = no-inspect,inspector
default_inspect_interface = inspector

[console]
port_range = 40000:50000

Expand All @@ -43,9 +46,16 @@ deploy_ramdisk = "{{ ironic_deploy_ramdisk }}"
[deploy]
default_boot_option = local

[inspector]
require_managed_boot = True
# passed to ironic-python-agent, see :
# https://docs.openstack.org/ironic-python-agent/latest/admin/how_it_works.html#inspection
extra_kernel_params = "ipa-inspection-collectors=default,pci-devices,dmi-decode,numa-topology,logs ipa-collect-lldp=1"

[neutron]
cleaning_network = "{{ ironic_cleaning_network }}"
provisioning_network = "{{ ironic_provisioning_network }}"
inspection_network = "{{ ironic_inspection_network }}"

{% if ironic_neutron_status_code_retries is defined %}
status_code_retries = "{{ ironic_neutron_status_code_retries }}"
Expand Down
2 changes: 1 addition & 1 deletion site-config.example/inventory/hosts
Original file line number Diff line number Diff line change
Expand Up @@ -503,7 +503,7 @@ ironic
ironic

[ironic-inspector:children]
# Disabled; we use a separate mechanism for node property discovery
ironic

[ironic-pxe:children]
ironic
Expand Down

0 comments on commit 72aeeae

Please sign in to comment.