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: can not delete "segment-routing node-msd" when SR if off #15333

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

c-po
Copy link
Contributor

@c-po c-po commented Feb 8, 2024

This fixes the initial implementation of commit 7743f2f ("OSPFd: Update Segment Routing PR following review") where it wsa not possible to remove the "segment-routing node-msd" CLI nodes via vtysh once segment-routing got disabled.

Closes #14910

Should be backported to stable/9.0 and stable/9.1

This fixes the initial implementation of commit 7743f2f ("OSPFd: Update
Segment Routing PR following review") where it wsa not possible to remove
the "segment-routing node-msd" CLI nodes via vtysh once segment-routing got
disabled.

Closes FRRouting#14910

Signed-off-by: Christian Breunig <christian@breunig.cc>
@donaldsharp
Copy link
Member

LGTM.

@ton31337
Copy link
Member

ton31337 commented Feb 9, 2024

@Mergifyio backport stable/9.1 stable/9.0

Copy link

mergify bot commented Feb 9, 2024

backport stable/9.1 stable/9.0

✅ Backports have been created

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.

LGTM

@ton31337 ton31337 added this to the 10.0 milestone Feb 9, 2024
@ton31337 ton31337 merged commit b7987ec into FRRouting:master Feb 9, 2024
11 checks passed
donaldsharp added a commit that referenced this pull request Feb 9, 2024
ospfd: can not delete "segment-routing node-msd" when SR if off (backport #15333)
donaldsharp added a commit that referenced this pull request Feb 9, 2024
ospfd: can not delete "segment-routing node-msd" when SR if off (backport #15333)
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.

ospfd: OSPF SR is not turned on - "segment-routing node-msd" always enabled by default
3 participants