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

sisd: Redistributed routes supports 'route-map interface xxxx' filtering. #18370

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

huchaogithup
Copy link

sisd: Redistributed routes supports 'route-map interface xxxx' filtering.
old: ospf support match interface and ISIS does not support matching interfaces. There is a need to inject only routes related to VPN interfaces in VPN scenarios.
image
new:
image

@frrbot frrbot bot added the isis label Mar 12, 2025
@donaldsharp
Copy link
Member

Please actually include the analysis of the problem in the commit message. We want future developers to have this data. Why would you not include it?

Why did you open the same PR 2 times?

@huchaogithup huchaogithup changed the title isisd: Fix the issue where redistributed routes do not change when th… sisd: Redistributed routes supports 'route-map interface xxxx' filtering. Mar 12, 2025
@huchaogithup
Copy link
Author

Sorry for the wrong title. These two PRS are actually two functions

@huchaogithup
Copy link
Author

'Please actually include the analysis of the problem in the commit message. We want future developers to have this data. Why would you not include it?'
I’m not quite sure what additional analysis I need to include. Could it be that my previous title was incorrect, which led to some misunderstanding? I’d really appreciate your guidance on this.

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.

2 participants