Skip to content

Feature/frr GRE tunnel support#3561

Open
jbemmel wants to merge 6 commits into
ipspace:devfrom
jbemmel:feature/frr-gre-support
Open

Feature/frr GRE tunnel support#3561
jbemmel wants to merge 6 commits into
ipspace:devfrom
jbemmel:feature/frr-gre-support

Conversation

@jbemmel

@jbemmel jbemmel commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Building out the GRE tunnel support #3554, couple of highlights:

  • First plugin that requires a kernel module for Linux devices (with .-key issues); code adds a check for config keys such that kernel modules are only loaded if the plugin is actually used by at least 1 node
  • ipv6 support anticipated, not tested yet

@ipspace

ipspace commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Building out the GRE tunnel support #3554, couple of highlights:

Please rebase, the PR is showing tons of modified files from #3554

  • First plugin that requires a kernel module for Linux devices (with .-key issues); code adds a check for config keys such that kernel modules are only loaded if the plugin is actually used by at least 1 node

Interesting ;) Will review after the rebase.

@jbemmel jbemmel marked this pull request as draft July 2, 2026 13:53
jbemmel and others added 5 commits July 2, 2026 08:54
Register FRR device features and clab kernel modules in the plugin defaults, add an FRR configuration template, and load kmods for config plugins using the @ key convention.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Collect kernel modules from topology plugins before the per-node module loop, using the @ key convention for dotted plugin names.

Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Only collect kernel modules for plugins actually configured on a node, using the @ key convention for dotted plugin names.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jbemmel jbemmel force-pushed the feature/frr-gre-support branch from f6c1834 to b142242 Compare July 2, 2026 13:54
Declare features.tunnel.gre for FRR in the device definition instead of
the tunnel.gre plugin defaults, keeping device feature support with the
device.

Co-authored-by: Cursor <cursoragent@cursor.com>
@jbemmel jbemmel marked this pull request as ready for review July 2, 2026 14:04
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