Skip to content

Replace VXLAN example with GENEVE - #70

Merged
priteau merged 1 commit into
mainfrom
vxlan-to-geneve
Sep 24, 2026
Merged

priteau merged 1 commit into
mainfrom
vxlan-to-geneve

Conversation

@priteau

@priteau priteau commented Sep 22, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • Configuration
    • Updated the example network configuration to use a demo GENEVE network instead of the demo VXLAN network.
    • Updated the associated network, subnet, name, CIDR, gateway, and pool values to match the GENEVE setup.
    • Updated the demo router configuration to connect to the GENEVE network.

@priteau priteau self-assigned this Sep 22, 2026
@priteau
priteau requested a review from a team as a code owner September 22, 2026 11:49
@coderabbitai

coderabbitai Bot commented Sep 22, 2026

Copy link
Copy Markdown

Review in Change Stack →

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 configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 6245d228-4294-48c4-9059-4ea5988bff16

📥 Commits

Reviewing files that changed from the base of the PR and between 2299581 and 2cec836.

📒 Files selected for processing (1)
  • examples/networks.yml

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


📝 Walkthrough

Walkthrough

The example network configuration replaces the demo VXLAN network with a demo GENEVE network. The network list, network and subnet variables, and demo router interface now use the GENEVE names and settings.

Changes

Demo GENEVE network configuration

Layer / File(s) Summary
Define and wire the demo GENEVE network
examples/networks.yml
The demo network and subnet variables now use provider_network_type: "geneve" and matching names. The network list and demo router interface reference the GENEVE network.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Feature

Suggested reviewers: alex-welsh

Merge Risk: ⚪ Minimal · up to 2cec8

The example now provisions a demo GENEVE network and connects the demo router to it; the supplied configuration is consistent and ready to merge.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: replacing the VXLAN example with a GENEVE example.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

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

@priteau
priteau merged commit 4210928 into main Sep 24, 2026
2 checks passed
@priteau
priteau deleted the vxlan-to-geneve branch September 24, 2026 15:37
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