search

ipspace / netlab

Making virtual networking labs suck less
https://netlab.tools
Other
428 stars 63 forks source link

[BUG] Make sure nodes with 'vrf' module have 'vrfs' dictionary #1109

Closed ipspace closed 4 months ago

ipspace commented 4 months ago

This is a spinoff from #1105. Based on commit text, it looks like we could get a situation where a node has vrf module enabled but no vrfs dictionary. That makes no sense and just increases the complexity of the configuration templates.

@ssasso: Could you provide a sample topology in which you experienced this bug? Thank you!

ssasso commented 4 months ago

I add the default to "vrfs" because of the "netlab initial -o" issue, not because the node has vrf module. In fact, the node did not have vrf enabled.

(#1104)

ipspace commented 4 months ago

Thank you, closing this one.