Repository navigation
dataplane: default to neutron-ovn instead of neutron-metadata - #2129
Conversation
The standalone ovn-metadata-agent is deprecated in favor of the unified ovn-agent's metadata extension, which is now enabled by default (openstack-k8s-operators/edpm-ansible). Switch the default OpenStackDataPlaneNodeSet services list, samples, and docs from neutron-metadata to neutron-ovn accordingly. The nova-metadata-neutron-config secret carries nova_metadata_host and metadata_proxy_shared_secret, both needed by the metadata extension. Make it required for neutron-ovn, matching neutron-metadata, so reconciliation correctly waits on it instead of silently running without metadata config. Depends-On: openstack-k8s-operators/edpm-ansible#1289 Depends-On: openstack-k8s-operators/ci-framework#4218 Related-Issue: #OSPRH-37892 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
OpenStackControlPlane CRD Size Report
Threshold reference
|
|
Navigate logical layers of code changes, visualize relationships, and explore their blast radius. No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (15)
💤 Files with no reviewable changes (2)
Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 SummarySummary by CodeRabbit
WalkthroughThe default dataplane service list now uses ChangesDataplane neutron service selection
Priority: ⬇️ Low Estimated code review effort: 2 (Simple) | ~8 minutes Change: Feature Suggested reviewers: Merge Risk: ⚪ Minimal · up to The default dataplane service list now uses neutron-ovn instead of neutron-metadata, and docs and samples match. No concrete merge-blocking risk was found. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Build failed (check pipeline). Post ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 49m 52s |
|
recheck cs10 content provider unrelated |
@karelyatin you need to remove Depends-On: openstack-k8s-operators/ci-framework#4218 for the cs10 jobs to pass. |
|
recheck depends-on removed |
|
recheck dequeued |
|
Build succeeded (check pipeline). ✔️ openstack-k8s-operators-content-provider SUCCESS in 2h 54m 55s |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: karelyatin, stuggi The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
there are issues with tide right now, going to merge manually |
The standalone ovn-metadata-agent is deprecated in favor of the unified ovn-agent's metadata extension, which is now enabled by default (openstack-k8s-operators/edpm-ansible). Switch the default OpenStackDataPlaneNodeSet services list, samples, and docs from neutron-metadata to neutron-ovn accordingly.
The nova-metadata-neutron-config secret carries nova_metadata_host and metadata_proxy_shared_secret, both needed by the metadata extension. Make it required for neutron-ovn, matching neutron-metadata, so reconciliation correctly waits on it instead of silently running without metadata config.
Depends-On: openstack-k8s-operators/edpm-ansible#1289
Related-Issue: #OSPRH-37892