Removing full OSPF protocol is not possible
Closed, WontfixPublicBUG
Actions

Assigned To

Authored By

	c-po
	Jul 11 2019, 9:17 AM

Description

Reproduce

set

set policy prefix-list ROUTE-OSPF-STATIC rule 10 action 'permit'
set policy prefix-list ROUTE-OSPF-STATIC rule 10 prefix '172.16.222.0/28'
set policy route-map ROUTE-MAP-OSPF-STATIC rule 10 action 'permit'
set policy route-map ROUTE-MAP-OSPF-STATIC rule 10 match ip address prefix-list 'ROUTE-OSPF-STATIC'
set protocols ospf area 0 area-type normal
set protocols ospf area 0 network '172.16.253.128/29'
set protocols ospf area 0 network '172.16.254.100/32'
set protocols ospf area 0 network '172.16.252.68/30'
set protocols ospf area 0 network '172.16.252.64/30'
set protocols ospf log-adjacency-changes
set protocols ospf parameters abr-type 'cisco'
set protocols ospf parameters router-id '172.16.254.100'
set protocols ospf passive-interface 'default'
set protocols ospf passive-interface-exclude 'tun100'
set protocols ospf passive-interface-exclude 'wg02'
set protocols ospf passive-interface-exclude 'wg01'
set protocols ospf redistribute static metric-type '2'
set protocols ospf redistribute static route-map 'ROUTE-MAP-OSPF-STATIC'

show

vyos@vyos# show protocols ospf
 area 0 {
     area-type {
         normal
     }
     network 172.16.253.128/29
     network 172.16.254.100/32
     network 172.16.252.68/30
     network 172.16.252.64/30
 }
 log-adjacency-changes {
 }
 parameters {
     abr-type cisco
     router-id 172.16.254.100
 }
 passive-interface default
 passive-interface-exclude tun100
 passive-interface-exclude wg02
 passive-interface-exclude wg01
 redistribute {
     static {
         metric-type 2
         route-map ROUTE-MAP-OSPF-STATIC
     }
 }

error

vyos@vyos# delete protocols ospf
vyos@vyos# commit
[ protocols ospf area 0 network 172.16.253.128/29 ]
Can't find specified network area configuration.

delete [ protocols ospf ] failed
Commit failed

vyos@vyos# show protocols ospf
-area 0 {
-    area-type {
-        normal
-    }
-    network 172.16.253.128/29
-    network 172.16.254.100/32
-    network 172.16.252.68/30
-    network 172.16.252.64/30
-}
-log-adjacency-changes {
-}
-parameters {
-    abr-type cisco
-    router-id 172.16.254.100
-}
-passive-interface default
-passive-interface-exclude tun100
-passive-interface-exclude wg02
-passive-interface-exclude wg01
-redistribute {
-    static {
-        metric-type 2
-        route-map ROUTE-MAP-OSPF-STATIC
-    }
-}
[edit]

FRR

Expected

vyos@vyos# vtysh -c "show run"
router ospf
 ospf router-id 172.16.254.100
 log-adjacency-changes
 redistribute static route-map ROUTE-MAP-OSPF-STATIC
 passive-interface default
 no passive-interface tun100
 no passive-interface wg01
 no passive-interface wg02
 network 172.16.252.64/30 area 0
 network 172.16.252.68/30 area 0
 network 172.16.253.128/29 area 0
 network 172.16.254.100/32 area 0

Got

vyos@vyos# vtysh -c "show run"
router ospf
 ospf router-id 172.16.254.100
 redistribute static route-map ROUTE-MAP-OSPF-STATIC
 passive-interface default
 no passive-interface tun100
 no passive-interface wg01
 no passive-interface wg02

Details

Version: 1.2.2, 1.3
Is it a breaking change?: Unspecified (possibly destroys the router)

Related Objects

Mentioned In: T5309: Issues when trying to remove OSPF configuration

Event Timeline

c-po created this task.Jul 11 2019, 9:17 AM

c-po triaged this task as High priority.

c-po updated the task description. (Show Details)Jul 11 2019, 9:19 AM

c-po updated the task description. (Show Details)

hagbard claimed this task.Jul 12 2019, 8:02 PM

hagbard added a project: VyOS 1.3 Equuleus.

hagbard changed Version from 1.2.2 to 1.2.2, 1.3.

Same issue present in 1.3

hagbard removed hagbard as the assignee of this task.Jul 15 2019, 2:28 PM

hagbard subscribed.

pasik subscribed.Jul 16 2019, 9:49 AM

syncer assigned this task to zsdc.Aug 30 2019, 11:45 PM

syncer removed a project: VyOS 1.2 Crux.

@c-po
I get other result.
After set from your example I see correct area
FRR

router ospf
 ospf router-id 172.16.254.100
 log-adjacency-changes
 redistribute static route-map ROUTE-MAP-OSPF-STATIC
 passive-interface default
 no passive-interface eth1
 network 172.16.252.64/30 area 0
 network 172.16.252.68/30 area 0
 network 172.16.253.128/29 area 0
 network 172.16.254.100/32 area 0

Try delete OSPF (get error ref https://github.com/vyos/vyatta-cfg-quagga/blob/current/templates/protocols/ospf/passive-interface/node.def)

vyos@1.2-roll-ns# delete protocols ospf 
[edit]
vyos@1.2-roll-ns# commit
[ protocols ospf passive-interface default ]
Error: delete passive-interface-exclude before deleting passive-interface default

delete [ protocols ospf ] failed
Commit failed
[edit]

FRR shows that the network x.x.x.x area 0 is deleted.

router ospf
 ospf router-id 172.16.254.100
 redistribute static route-map ROUTE-MAP-OSPF-STATIC
 passive-interface default
 no passive-interface eth1

So, some configurations were deleted in FRR and if we try execute commit again, we get

vyos@1.2-roll-ns# commit
[ protocols ospf area 0 network 172.16.253.128/29 ]
Can't find specified network area configuration.

delete [ protocols ospf ] failed
Commit failed
[edit]
vyos@1.2-roll-ns#

Protocol rewrite is more then overdue now ... :)

mbailey subscribed.Jan 24 2020, 1:50 PM

zakwan subscribed.Jun 10 2020, 10:50 AM

Not quite sure if this is the right thing to do but, if a "delete protocols ospf" command is given the equivalent in FRR should be "no router ospf".

I am unsure if that's what the current command does. I take it it does not, as it seems to only remove the networks and not remove anything else.

Edit:

I re-edited my post. I was wrong initially :(

jack9603301 subscribed.Sep 13 2020, 5:58 AM

This logic does not allow for the complete removal of the protocol.
https://github.com/vyos/vyatta-cfg-quagga/blob/32cbb1e5059c6c27449b7013f790aff1c50a9831/templates/protocols/ospf/passive-interface/node.def#L29-L35

dmbaturin added a project: test.Jul 29 2021, 12:30 PM

dmbaturin set Is it a breaking change? to Unspecified (possibly destroys the router).

syncer edited projects, added VyOS 1.3 Equuleus (1.3.0-epa1); removed test, VyOS 1.3 Equuleus.Sep 5 2021, 7:06 AM