Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

lag module should model peerlink as regular VLAN #1909

Open
jbemmel opened this issue Feb 8, 2025 · 2 comments
Open

lag module should model peerlink as regular VLAN #1909

jbemmel opened this issue Feb 8, 2025 · 2 comments
Assignees
Labels
enhancement New feature or request

Comments

@jbemmel
Copy link
Collaborator

jbemmel commented Feb 8, 2025

Most MLAG platforms support things like OSPF on the peerlink between the MLAG router pair; the lag module should create a subinterface as appropriate, and allow these modules to do their job

For example, this should work as expected:

module: [ ospf, lag, vlan ]

links:
- lag:
    members: [ s1, s2 ]
    mlag.peergroup: True

Likewise:

module: [ ospf, lag, vlan ]

links:
- lag:
    members: [ s1, s2 ]
    mlag.peergroup: True
  ospf: False
@jbemmel jbemmel added the enhancement New feature or request label Feb 8, 2025
@jbemmel jbemmel self-assigned this Feb 8, 2025
@ipspace
Copy link
Owner

ipspace commented Feb 9, 2025

As most platforms want to have some sort of VLANs on the peer link, what's stopping you from using a routed VLAN to run OSPF on it?

@jbemmel
Copy link
Collaborator Author

jbemmel commented Feb 9, 2025

With the benefit of hindsight, I'm now realizing that the way the lag templates are structured is sub-optimal. They replicate some of the logic for provisioning a custom VLAN and allocating IPs, instead of relying on the VLAN module to do that part. The latter has the benefit that other features - such as OSPF configs - can be applied too.

There is nothing "stopping" me, but I wanted to add OSPF support and I'm now thinking that's better done through refactoring (as opposed to adding OSPF config to the lag templates)

@jbemmel jbemmel changed the title lag module should model peerlink as L3 subinterface lag module should model peerlink as regular VLAN Feb 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants