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

ospfd: OSPF multi-instance default origination fixes #17436

Merged

Conversation

aceelindem
Copy link
Collaborator

When originating a default AS-External LSA in one OSPF instance, it wasn't working if the criteria route was installed by another OSPF instance. This required more flexible processing of the OSPF external route information.

Copy link
Member

@ton31337 ton31337 left a comment

Choose a reason for hiding this comment

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

Can we cover this stuff by a topotest?

@ton31337
Copy link
Member

@Mergifyio backport stable/10.2 stable/10.1 stable/10.0 stable/9.1 stable/9.0

Copy link

mergify bot commented Nov 16, 2024

backport stable/10.2 stable/10.1 stable/10.0 stable/9.1 stable/9.0

✅ Backports have been created

When originating a default AS-External LSA in one OSPF instance,
it wasn't working if the criteria route was installed by another OSPF
instance. This required more flexible processing of the OSPF external
route information.

Also fix problem multi-instance display for "show ip ospf
 <instance> database ...".

Signed-off-by: Acee Lindem <[email protected]>
@aceelindem aceelindem force-pushed the aceelindem/ospf-multi-instance-default branch from 382b7a6 to 6afd56d Compare November 17, 2024 13:36
@aceelindem
Copy link
Collaborator Author

backport stable/10.2 stable/10.1 stable/10.0 stable/9.1 stable/9.0

🟠 Waiting for conditions to match

Note I've got a topotest that I'm going to add to this PR under a separate "tests" commit.

This change adds a topotest to test various case of OSPF multi-instance
origination including cases where the criteria route is from another
instance of OSPF, as well as the same OSPF instance (where a default
should not be originated).

Signed-off-by: Acee Lindem <[email protected]>
@frrbot frrbot bot added the tests Topotests, make check, etc label Nov 18, 2024
@github-actions github-actions bot added size/XL and removed size/L labels Nov 18, 2024
@aceelindem
Copy link
Collaborator Author

Can we cover this stuff by a topotest?

Pushed separate commit to PR with a topotest.

@aceelindem
Copy link
Collaborator Author

CI:rerun

4 similar comments
@aceelindem
Copy link
Collaborator Author

CI:rerun

@aceelindem
Copy link
Collaborator Author

CI:rerun

@aceelindem
Copy link
Collaborator Author

CI:rerun

@aceelindem
Copy link
Collaborator Author

CI:rerun

Copy link
Member

@riw777 riw777 left a comment

Choose a reason for hiding this comment

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

looks good

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants