Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

enable ironic inspector #263

Merged
merged 1 commit into from
Sep 15, 2023
Merged

enable ironic inspector #263

merged 1 commit into from
Sep 15, 2023

Conversation

msherman64
Copy link
Contributor

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".

@msherman64 msherman64 force-pushed the wip/ironic_inspection branch 2 times, most recently from c185454 to 72aeeae Compare August 29, 2023 21:22
@msherman64
Copy link
Contributor Author

Before this can be merged, we need to be a little careful about what goes in node_custom_config.

Ironic-inspector supports a "standalone mode" that runs without neutron or glance, so it complains if pxe images aren't present in its own directory. I'd rather not stick 500mb of binary into the site config if we can avoid it, so this is still a work in progress ;)

@codyhammock
Copy link
Member

Can you be more specific about what settings in node_custom_config might be problematic?

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".
@msherman64
Copy link
Contributor Author

Can you be more specific about what settings in node_custom_config might be problematic?

Fixed the issues in this commit to kolla-ansible ChameleonCloud/kolla-ansible@6917bd6

Basically in the "standalone mode", ironic-inspector wants the ironic agent kernel+ramdisk in node_custom_config, but I was able to add a flag to skip this

@msherman64 msherman64 merged commit c6a82e3 into stable/xena Sep 15, 2023
1 check passed
@msherman64 msherman64 deleted the wip/ironic_inspection branch September 15, 2023 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants