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

Dell OS10: Fix test cases 33-vrf-common-ospf and 34- by redistributing imported OSPF routes too #1571

Open
wants to merge 3 commits into
base: dev
Choose a base branch
from

Conversation

jbemmel
Copy link
Collaborator

@jbemmel jbemmel commented Nov 26, 2024

Might be related to #1135

Fixes both 33-vrf-common-ospf and 34-vrf-common-mixed integration test cases

Technically this flag is only required if there are multiple VRFs running OSPF and they import from each other - but I think it's safe to enable whenever OSPF is used in VRFs

Note: Also fixes reference_bandwidth attribute

@jbemmel jbemmel changed the title Dell OS10: Fix test case 33-vrf-common-ospf by redistributing imported OSPF routes too Dell OS10: Fix test cases 33-vrf-common-ospf and 34- by redistributing imported OSPF routes too Nov 26, 2024
@jbemmel jbemmel marked this pull request as draft November 26, 2024 16:50
* Fix reference_bandwidth to be taken from the vdata, not global ospf
@jbemmel jbemmel marked this pull request as ready for review November 26, 2024 16:57
@ipspace ipspace requested a review from ssasso November 27, 2024 17:15
@ipspace
Copy link
Owner

ipspace commented Nov 27, 2024

@ssasso You know way more about OS10 than I do. Are these changes OK?

@jbemmel
Copy link
Collaborator Author

jbemmel commented Dec 2, 2024

@ssasso You know way more about OS10 than I do. Are these changes OK?

Not to put words in his mouth, but #1135 (comment) explicitly includes redistribute imported-ospf-routes

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