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

ospf: "no router ospf" has no effect using both vtysh and frr-reload #17011

Open
2 tasks done
c-po opened this issue Oct 6, 2024 · 3 comments
Open
2 tasks done

ospf: "no router ospf" has no effect using both vtysh and frr-reload #17011

c-po opened this issue Oct 6, 2024 · 3 comments
Labels
ospf triage Needs further investigation

Comments

@c-po
Copy link
Contributor

c-po commented Oct 6, 2024

Description

When instantiating a OSPF configuration, it is not possible to remove the configuration using no router ospf, it keeps on beeing output by vtysh -c "show run"

Version

LR1.wue3# show version
FRRouting 10.1.1 (LR1.wue3) on Linux(6.6.52).

Commit cf5b42e

How to reproduce

LR1.wue3# conf t
LR1.wue3(config)# router ospf
LR1.wue3(config-router)#  auto-cost reference-bandwidth 100
LR1.wue3(config-router)#  timers throttle spf 200 1000 10000
LR1.wue3(config-router)#  max-metric router-lsa on-startup 30
LR1.wue3(config-router)#  max-metric router-lsa on-shutdown 60
LR1.wue3(config-router)#  refresh timer 50
LR1.wue3(config-router)#  aggregation timer 100
LR1.wue3(config-router)#  distance 128
LR1.wue3(config-router)#  distance ospf intra-area 100 external 120
LR1.wue3(config-router)# exit
LR1.wue3(config)# end
LR1.wue3# show run
Building configuration...

Current configuration:
!
frr version 10.1.1
frr defaults traditional
hostname LR1.wue3
log syslog
log facility local7
service integrated-vtysh-config
!
router ospf
 auto-cost reference-bandwidth 100
 timers throttle spf 200 1000 10000
 max-metric router-lsa on-startup 30
 max-metric router-lsa on-shutdown 60
 refresh timer 50
 aggregation timer 100
 distance 128
 distance ospf intra-area 100 external 120
exit
!
end

Now remove configuration using vtysh

LR1.wue3# conf t
LR1.wue3(config)# no router ospf
LR1.wue3(config)# end

Expected behavior

LR1.wue3# show run
Building configuration...

Current configuration:
!
frr version 10.1.1
frr defaults traditional
hostname LR1.wue3
log syslog
log facility local7
service integrated-vtysh-config
!
end

Actual behavior

LR1.wue3# show run
Building configuration...

Current configuration:
!
frr version 10.1.1
frr defaults traditional
hostname LR1.wue3
log syslog
log facility local7
service integrated-vtysh-config
!
router ospf
 auto-cost reference-bandwidth 100
 timers throttle spf 200 1000 10000
 max-metric router-lsa on-startup 30
 max-metric router-lsa on-shutdown 60
 refresh timer 50
 aggregation timer 100
 distance 128
 distance ospf intra-area 100 external 120
exit
!
end

Additional context

No response

Checklist

  • I have searched the open issues for this bug.
  • I have not included sensitive information in this report.
@c-po c-po added the triage Needs further investigation label Oct 6, 2024
@ton31337 ton31337 added ospf bug triage Needs further investigation and removed triage Needs further investigation bug labels Oct 6, 2024
@ton31337
Copy link
Member

ton31337 commented Oct 6, 2024

As I see in the code, this is intentional due to the max-metric router-lsa on-shutdown command. The timer is activated for X seconds and the config is flushed only when the timer (event) expires.

@c-po
Copy link
Contributor Author

c-po commented Oct 6, 2024

Interesting - behavior on 9.1 was different. Thanks!

Will re-evaluate this and close if appropriate

@ton31337
Copy link
Member

ton31337 commented Oct 7, 2024

Interesting - behavior on 9.1 was different.

Yes, this 3e2f053 is since 10.0 :)

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

No branches or pull requests

2 participants