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:fix the bug that the empty area was not free after no area range command was executed #17183

Merged
merged 1 commit into from
Oct 23, 2024

Conversation

Shbinging
Copy link
Contributor

When we use the no area X.X.X.X range A.B.C.D/M command, if the area no longer has an interface to which it belongs, then the area should be deleted from the LSDB. This processing logic is consistent with instructions such as no network area and no area authentication.

…e command was executed

    When we use the no area X.X.X.X range A.B.C.D/M command, if the area no longer has an interface to which it belongs, then the area should be deleted from the LSDB. This processing logic is consistent with instructions such as no network area and no area authentication.

Signed-off-by: Shbinging <[email protected]>
@Shbinging
Copy link
Contributor Author

This is consistent with the revision of #17101

@riw777 riw777 self-requested a review October 22, 2024 15:08
@riw777
Copy link
Member

riw777 commented Oct 22, 2024

code looks good ... commit needs to be formatted correctly ("signed off by")

Copy link
Collaborator

@aceelindem aceelindem 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.

@Shbinging
Copy link
Contributor Author

Shbinging commented Oct 23, 2024

code looks good ... commit needs to be formatted correctly ("signed off by")

I signed the commit
Signed-off-by: Shbinging <[email protected]>Signed-off-by: Shbinging <[email protected]>

@ton31337 ton31337 merged commit a6aa6b5 into FRRouting:master Oct 23, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants