Skip to content

Conversation

jbemmel
Copy link
Collaborator

@jbemmel jbemmel commented Jun 10, 2025

For discussion - needs more documentation and validation tests, but I'd first like to agree on the general approach before putting more effort

@jbemmel jbemmel marked this pull request as draft June 10, 2025 22:23
1:
kind: stub
no_summarize: True
```
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggest to add also the param range as an array of subnets for area range config - to be applied only on ABR

@ipspace
Copy link
Owner

ipspace commented Jun 11, 2025

Yeah, I had that on the backburner for ages (https://github.com/ipspace/netlab/tree/ospf.areas), maybe it's time to finally get it done (after pushing out 25.06). However, it needs to address all types of OSPF areas, not just the stub areas. So we need:

  • kind - stub/nssa (or regular if kind is missing)
  • summary: False (default: true)
  • default: False (default: true) -- for stub/nssa areas
  • range: list of summarization ranges
  • filter: suppressed prefixes

I had the data model (almost) worked out in 711f49e, the real show-stopper (why this isn't already out) was the dictionary. I wanted to use both area numbers as well as area addresses ("0.0.0.0") as the dictionary entries, but of course "0.0.0.0" gets exploded by Box and needs to be recreated.

Looks like the only way to get that solved would be to have a list of areas, with every element having area key, and the plugin reverse-engineering the list from the dict if needed.

Give me a few days ;)

@jbemmel
Copy link
Collaborator Author

jbemmel commented Jun 11, 2025

Give me a few days ;)

I ran into the ipv4 thing as well, perhaps cdgriffith/Box#297 could help (if accepted)

@jbemmel
Copy link
Collaborator Author

jbemmel commented Jun 12, 2025

Replaced by #2363

@jbemmel jbemmel closed this Jun 12, 2025
@jbemmel jbemmel deleted the ospf_stub_plugin branch June 12, 2025 17:48
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.

3 participants