Disable l2vni trunk reconciliation

[TBExtent]

L2VNI trunk reconciliation was added to ironic neutron agent in gazpacho, and enabled by default. It assumes that neutron trunk and segment are enabled, which is not guaranteed, so it is safest if it is disabled by default instead.
See this patch: https://review.opendev.org/c/openstack/networking-baremetal/+/974619

[coderabbitai]

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 8a9cb699-4edb-4c6c-941e-5ede933fc2b7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Team

Run ID: 6424180c-8551-40c0-b686-9da38f4019bc

📥 Commits

Reviewing files that changed from the base of the PR and between a4e5b48 and 69e9e7b.

📒 Files selected for processing (1)

• etc/kayobe/environments/baremetal/kolla/config/neutron/ironic_neutron_agent.ini

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. (2)

• GitHub Check: Ansible 2.20 lint with Python 3.12
• GitHub Check: Ansible 2.19 lint with Python 3.12

🔇 Additional comments (1)

etc/kayobe/environments/baremetal/kolla/config/neutron/ironic_neutron_agent.ini (1)

1-5: LGTM!

---

📝 Summary

Add conditional Ironic Neutron agent configuration that disables L2VNI trunk reconciliation and its event handling when the agent is enabled.

Walkthrough

The change adds a conditional Neutron agent configuration. When the Ironic Neutron agent is enabled, it disables L2 VNI trunk reconciliation and its events.

Changes

Ironic Neutron agent configuration

Layer / File(s)	Summary
Disable L2 VNI reconciliation etc/kayobe/environments/baremetal/kolla/config/neutron/ironic_neutron_agent.ini	Adds an [l2vni] section. Both reconciliation settings are set to False when kolla_enable_ironic_neutron_agent is enabled.

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

Merge Risk: ⚪ Minimal · up to 69e9e

When enabled, the Ironic Neutron agent will no longer perform L2 VNI trunk reconciliation or emit its reconciliation events. The configuration is limited to the intended agent and is ready to merge.

🚥 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…
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.

Full details: Docstring Coverage

Explanation

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

---

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

[priteau]

Can you include more details on why this is necessary? Is there an upstream bug open (if this is a bug) or a configuration guide that we can link to?

[priteau]

For example, you can link to this change in the commit: https://review.opendev.org/c/openstack/networking-baremetal/+/974619

[TBExtent]

Done :)

[github-actions]

Happy Friday @TBExtent, this is a friendly reminder that this PR is waiting for your changes or response. Please take a look when you have a moment!

Note: Once your changes are ready, remove the waiting-author-response label and add the waiting-review label.