Remove inspector and add agent to enabled_inspect_interfaces for baremetal mixin

[claudia-lola]

Deploying the baremetal mixin, ironic_conductor container would show as unhealthy. docker logs show error:

  File "/var/lib/kolla/venv/lib64/python3.12/site-packages/ironic/common/driver_factory.py", line 381, in _missing_callback
    raise exception.DriverNotFoundInEntrypoint(
ironic.common.exception.DriverNotFoundInEntrypoint: Could not find the following items in the 'ironic.hardware.interfaces.inspect' entrypoint: inspector.

Which relates to the removal of standalone Ironic inspector. To fix, removed inspector and added agent to list for kolla_enabled_inspect_interfaces resolved the error. ironic_conductor container deploys healthy.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: d331af65-7efd-47ec-afa4-fd2d55dcd29a

📥 Commits

Reviewing files that changed from the base of the PR and between e6c050b and 06b319d.

📒 Files selected for processing (1)

• etc/kayobe/environments/baremetal/ironic.yml

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.

📜 Recent review details

⏰ Context from checks skipped due to timeout. (3)

• GitHub Check: stackhpc/check
• GitHub Check: Ansible 2.19 lint with Python 3.12
• GitHub Check: Ansible 2.20 lint with Python 3.12

🔇 Additional comments (1)

etc/kayobe/environments/baremetal/ironic.yml (1)

52-52: LGTM!

---

📝 Walkthrough

Updates the baremetal Ironic configuration to use the agent inspect interface instead of inspector. The no-inspect interface remains enabled.

Walkthrough

The bare-metal Ironic configuration now enables the agent inspect interface instead of inspector. The no-inspect interface remains enabled.

Changes

Ironic inspection configuration

Layer / File(s)	Summary
Update enabled inspect interface etc/kayobe/environments/baremetal/ironic.yml	The configuration replaces inspector with agent in the enabled inspect interfaces. It retains no-inspect.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to 06b31

This is a localized configuration change that removes the inspector and enables the agent for the baremetal mixin. No actionable merge-blocking risk remains beyond normal checks and review.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}