Skip to content

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 <bingshui@smail.nju.edu.cn>
@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 <bingshui@smail.nju.edu.cn>Signed-off-by: Shbinging <bingshui@smail.nju.edu.cn>

@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