summaryrefslogtreecommitdiffstats
path: root/ospfd (follow)
Commit message (Collapse)AuthorAgeFilesLines
* *: Properly use memset() when zeroingDonatas Abraitis2022-05-1112-36/+35
| | | | | | | Wrong: memset(&a, 0, sizeof(struct ...)); Good: memset(&a, 0, sizeof(a)); Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* *: Avoid casting to the same type as on the leftDonatas Abraitis2022-05-082-5/+5
| | | | | | Just not necessary. Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* *: remove the checking returned value for hash_get()anlan_cs2022-05-023-28/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Firstly, *keep no change* for `hash_get()` with NULL `alloc_func`. Only focus on cases with non-NULL `alloc_func` of `hash_get()`. Since `hash_get()` with non-NULL `alloc_func` parameter shall not fail, just ignore the returned value of it. The returned value must not be NULL. So in this case, remove the unnecessary checking NULL or not for the returned value and add `void` in front of it. Importantly, also *keep no change* for the two cases with non-NULL `alloc_func` - 1) Use `assert(<returned_data> == <searching_data>)` to ensure it is a created node, not a found node. Refer to `isis_vertex_queue_insert()` of isisd, there are many examples of this case in isid. 2) Use `<returned_data> != <searching_data>` to judge it is a found node, then free <searching_data>. Refer to `aspath_intern()` of bgpd, there are many examples of this case in bgpd. Here, <returned_data> is the returned value from `hash_get()`, and <searching_data> is the data, which is to be put into hash table. Signed-off-by: anlan_cs <vic.lan@pica8.com>
* *: Fix spelling of accomodateDonald Sharp2022-04-192-4/+4
| | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* *: Fix spelling of atleastDonald Sharp2022-04-191-1/+1
| | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* *: Fix spelling of comparisionDonald Sharp2022-04-191-1/+1
| | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* *: Fix spelling of specifedDonald Sharp2022-04-192-4/+4
| | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* *: Fix spelling of neccessaryDonald Sharp2022-04-191-1/+1
| | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* *: Fix spelling of GracefullDonald Sharp2022-04-021-1/+1
| | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* *: Fix spelling of FollowingDonald Sharp2022-04-018-41/+41
| | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* isisd, lib, ospfd, pathd: Null out free'd pointerDonald Sharp2022-03-311-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The commands: router isis 1 mpls-te on no mpls-te on mpls-te on no mpls-te on ! Will crash Valgrind gives us this: ==652336== Invalid read of size 8 ==652336== at 0x49AB25C: typed_rb_min (typerb.c:495) ==652336== by 0x4943B54: vertices_const_first (link_state.h:424) ==652336== by 0x493DCE4: vertices_first (link_state.h:424) ==652336== by 0x493DADC: ls_ted_del_all (link_state.c:1010) ==652336== by 0x47E77B: isis_instance_mpls_te_destroy (isis_nb_config.c:1871) ==652336== by 0x495BE20: nb_callback_destroy (northbound.c:1131) ==652336== by 0x495B5AC: nb_callback_configuration (northbound.c:1356) ==652336== by 0x4958127: nb_transaction_process (northbound.c:1473) ==652336== by 0x4958275: nb_candidate_commit_apply (northbound.c:906) ==652336== by 0x49585B8: nb_candidate_commit (northbound.c:938) ==652336== by 0x495CE4A: nb_cli_classic_commit (northbound_cli.c:64) ==652336== by 0x495D6C5: nb_cli_apply_changes_internal (northbound_cli.c:250) ==652336== Address 0x6f928e0 is 272 bytes inside a block of size 320 free'd ==652336== at 0x48399AB: free (vg_replace_malloc.c:538) ==652336== by 0x494BA30: qfree (memory.c:141) ==652336== by 0x493D99D: ls_ted_del (link_state.c:997) ==652336== by 0x493DC20: ls_ted_del_all (link_state.c:1018) ==652336== by 0x47E77B: isis_instance_mpls_te_destroy (isis_nb_config.c:1871) ==652336== by 0x495BE20: nb_callback_destroy (northbound.c:1131) ==652336== by 0x495B5AC: nb_callback_configuration (northbound.c:1356) ==652336== by 0x4958127: nb_transaction_process (northbound.c:1473) ==652336== by 0x4958275: nb_candidate_commit_apply (northbound.c:906) ==652336== by 0x49585B8: nb_candidate_commit (northbound.c:938) ==652336== by 0x495CE4A: nb_cli_classic_commit (northbound_cli.c:64) ==652336== by 0x495D6C5: nb_cli_apply_changes_internal (northbound_cli.c:250) ==652336== Block was alloc'd at ==652336== at 0x483AB65: calloc (vg_replace_malloc.c:760) ==652336== by 0x494B6F8: qcalloc (memory.c:116) ==652336== by 0x493D7D2: ls_ted_new (link_state.c:967) ==652336== by 0x47E4DD: isis_instance_mpls_te_create (isis_nb_config.c:1832) ==652336== by 0x495BB29: nb_callback_create (northbound.c:1034) ==652336== by 0x495B547: nb_callback_configuration (northbound.c:1348) ==652336== by 0x4958127: nb_transaction_process (northbound.c:1473) ==652336== by 0x4958275: nb_candidate_commit_apply (northbound.c:906) ==652336== by 0x49585B8: nb_candidate_commit (northbound.c:938) ==652336== by 0x495CE4A: nb_cli_classic_commit (northbound_cli.c:64) ==652336== by 0x495D6C5: nb_cli_apply_changes_internal (northbound_cli.c:250) ==652336== by 0x495D23E: nb_cli_apply_changes (northbound_cli.c:268) Let's null out the pointer. After this change. Valgrind no longer reports issues and isisd no longer crashes. Fixes: #10939 Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* ospfd: Show neighbor state in full format (e.g. Full/DR instead of Full)Donatas Abraitis2022-03-301-7/+8
| | | | | | | | | | | | | | | | | | | | Before: ``` $ vtysh -c 'show ip ospf neighbor json' | jq '.neighbors."100.100.200.100"[0].nbrState' "Full/DR" $ vtysh -c 'show ip ospf neighbor detail json' | jq '.neighbors."100.100.200.100"[0].nbrState' "Full" ``` After: ``` $ vtysh -c 'show ip ospf neighbor json' | jq '.neighbors."100.100.200.100"[0].nbrState' "Full/DR" $ vtysh -c 'show ip ospf neighbor detail json' | jq '.neighbors."100.100.200.100"[0].nbrState' "Full/DR" ``` Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* Merge pull request #10825 from fdumontet6WIND/ospfdefaultoriginatedissue2Russ White2022-03-291-1/+6
|\ | | | | ospfd: fix ospf default route wrongly sent back
| * ospfd: fix ospf default route wrongly sent backFrancois Dumontet2022-03-251-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | That commit aim is to fix an invalid behavior when default-information is activated on ospf router without always option. Consider an ASBR with: -one default route coming from ospf, -and another default route coming from another deaemon (such BGP or static). When the daemon bgp stops advertising its default route, -ospf continues to advertise its previous default route (with aging 0), -this may create default routing loops. Expected behavior: is to update the removed external default route with MAXAGING value. Updating with MAXAGING value will notify the fact the route is currently invalid. A later removal from ospf external LSA database will be made. Analysis: all default routes have their type overwritten by a DEFAULT_ROUTE type. Thus all default routes whatever its origin (ospf, bgp, static...) is treated in a same way. But this is not pertinent for ospf originated default routes. Fix: avoid overwiting of route type when default route is ospf type. Signed-off-by: Francois Dumontet <francois.dumontet@6wind.com>
* | Merge pull request #10848 from pguibert6WIND/ospf_srlb_configRuss White2022-03-291-7/+14
|\ \ | | | | | | Ospf srlb config
| * | ospfd: show running-config displays also srgb on some casesPhilippe Guibert2022-03-251-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following configuration: router ospf segment-routing global-block 16000 23999 local-block 1000 2000 ! appears in the 'show running-config' like below: router ospf local-block 1000 2000 ! When there are custom Adjacencies, display the SRGB even if SRGB has the default settings. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
| * | ospfd: silently remove prefix sid already stored in configPhilippe Guibert2022-03-251-4/+7
| | | | | | | | | | | | | | | | | | | | | | | | There is no need to have an interface available to configure SRGB. Conversely, it should be possible to remove the SRGB when no interfaces are available. Signed-off-by: Philippe Guibert <philippe.guibert@6wind.com>
* | | Merge pull request #10837 from ↵Russ White2022-03-291-1/+31
|\ \ \ | | | | | | | | | | | | | | | | opensourcerouting/fix/consistent_JSON_keys_for_ospf_neighbor ospfd: Use consistent JSON keys for `show ip ospf neighbor` and detai…
| * | | ospfd: Use consistent JSON keys for `show ip ospf neighbor` and detail versionDonatas Abraitis2022-03-211-1/+31
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At the moment it's inconsistent, and very annoying. Let's just fix this, and add a deprecation period to remove them after that. ``` vr_ib# show ip ospf neighbor json { "neighbors":{ "192.10.120.2":[ { "priority":1, "state":"Full\/DROther", "deadTimeMsecs":36543, "address":"192.10.120.2", "ifaceName":"VLINK0", "retransmitCounter":0, "requestCounter":0, "dbSummaryCounter":0 }, ``` ``` vr_ib# show ip ospf neighbor detail json { "neighbors":{ "192.10.120.2":[ { "ifaceAddress":"192.10.120.2", "areaId":"0.0.0.0", "ifaceName":"VLINK0", "nbrPriority":1, "nbrState":"Full", "stateChangeCounter":5, "lastPrgrsvChangeMsec":53367612, "routerDesignatedId":"0.0.0.0", "routerDesignatedBackupId":"0.0.0.0", "optionsCounter":66, "optionsList":"*|O|-|-|-|-|E|-", "routerDeadIntervalTimerDueMsec":33126, "databaseSummaryListCounter":0, "linkStateRequestListCounter":0, "linkStateRetransmissionListCounter":0, "threadInactivityTimer":"on", "threadLinkStateRequestRetransmission":"on", "threadLinkStateUpdateRetransmission":"on", "grHelperStatus":"None" }, ``` Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* / / ospfd: Default route becomes stale route in nbrs even after flush from ↵rgirada2022-03-211-3/+3
|/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | originator. Description: Default route is not getting flushed from neighbours though originator triggered flush and deleted LSA from its database. It become as stale LSA in neighbours databse forever. This could seen in the following sequence of configurations with less than a second interval b/w configs. And this could happen only when originator shouldnt have default route in its rib so it originates default route only when configure with 'always' option. step-1:default-information originate always step-2:no default-information originate always step-3:default-information originate In step-1, default route will be originated to AS. In step-2, default route will be flushed to AS, but neighbours will be discarding this update due to minlsainterval condition. And it is expected that DUT need to keep send this update until it receives the ack from neighbours by adding each neighbour's retransmission list. In Step-3: It is deleting the lsas from nbr's retransmission list by assuming it initiated the flush. This is cuasing to not send the lsa update anymore to neighbours which makes stale lsa in nbrs forever. Fix: Allowed to delete the lsa from retransmission list only when lsa is not in maxage during flushing procedure. Signed-off-by: Rajesh Girada <rgirada@vmware.com>
* | ospfd: better error messages when not enabled in a vrfJafar Al-Gharaibeh2022-03-161-25/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | before: ``` r1# show ip ospf vrf blue % OSPF instance not found r1# show ip ospf vrf default % OSPF instance not found r1# show ip ospf % OSPF instance not found ``` after: ``` r1# show ip ospf vrf blue % OSPF is not enabled in vrf blue r1# show ip ospf vrf default % OSPF is not enabled in vrf default r1# show ip ospf % OSPF is not enabled in vrf default ``` Signed-off-by: Jafar Al-Gharaibeh <jafar@atcorp.com>
* | Merge pull request #10447 from ton31337/fix/json_with_whitespacesDonald Sharp2022-03-132-4/+66
|\ \ | | | | | | *: Fix JSON keys with whitespaces and PascalCase
| * | *: Add camelCase JSON keys in addition to PascalCaseDonatas Abraitis2022-02-032-2/+30
| | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
| * | ospfd,ospf6d: Add JSON additional keys with no whitespaces insideDonatas Abraitis2022-02-031-2/+36
| | | | | | | | | | | | Signed-off-by: Donatas Abraitis <donatas.abraitis@gmail.com>
* | | ospfd,ospf6d: Add missing newline for `graceful-restart prepare` CLIDonatas Abraitis2022-03-091-1/+1
| |/ |/| | | | | Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
* | ospfd: Convert `enum event` to `enum ospf_apiserver_event`Donald Sharp2022-03-022-5/+5
| | | | | | | | | | | | | | Let's name this something more appropriate to what is being done. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | *: Add necessary new line for output of vty_out()anlan_cs2022-02-271-2/+2
| | | | | | | | Signed-off-by: anlan_cs <vic.lan@pica8.com>
* | ospfd: NULL passed instead of ei pointer in external lsa originationrgirada2022-02-241-1/+1
| | | | | | | | | | | | | | | | | | | | Description: NULL pointer wrongly passed instead of 'ei' pointer to ospf_external_lsa_originate() API in opaque capability enable/disable which always make it to fail in origination. Corrected it by passing actual ei pointer. Signed-off-by: Rajesh Girada <rgirada@vmware.com>
* | *: Change thread->func to return void instead of intDonald Sharp2022-02-2421-190/+98
| | | | | | | | | | | | | | The int return value is never used. Modify the code base to just return a void instead. Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | ospfd: Fix spelling mistakesDonald Sharp2022-02-145-10/+10
| | | | | | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | Merge pull request #10511 from anlancs/ospf-substituteIgor Ryzhov2022-02-081-37/+16
|\ \ | | | | | | ospfd: fix loss of mixed form in "range" command
| * | ospfd: fix loss of mixed form in "range" commandanlan_cs2022-01-261-37/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently "range" command can only accept `cost` or `substitute` individually, and `show running` will mix them. So need make it accept mixed form with both of them, otherwise configuration file in mixed form will fail to start. Signed-off-by: anlan_cs <vic.lan@pica8.com>
* | | Merge pull request #10293 from rgirada/ospf_lsidRuss White2022-02-083-101/+321
|\ \ \ | | | | | | | | ospfd: Modifying LSID generation algorithm
| * | | ospfd: Modifying LSID generation algorithmrgirada2022-01-223-101/+321
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Description: This LSID alogithm added as per rcf2328 Appendex-E recommendation. This applies only for AS-external lsas and summary lsas. As an example of the algorithm, consider its operation when the following sequence of events occurs in a single router (Router A). (1) Router A wants to originate an AS-external-LSA for [10.0.0.0,255.255.255.0]: (a) A Link State ID of 10.0.0.0 is used. (2) Router A then wants to originate an AS-external-LSA for [10.0.0.0,255.255.0.0]: (a) The LSA for [10.0.0,0,255.255.255.0] is reoriginated using a new Link State ID of 10.0.0.255. (b) A Link State ID of 10.0.0.0 is used for [10.0.0.0,255.255.0.0]. (3) Router A then wants to originate an AS-external-LSA for [10.0.0.0,255.0.0.0]: (a) The LSA for [10.0.0.0,255.255.0.0] is reoriginated using a new Link State ID of 10.0.255.255. (b) A Link State ID of 10.0.0.0 is used for [10.0.0.0,255.0.0.0]. (c) The network [10.0.0.0,255.255.255.0] keeps its Link State ID of 10.0.0.255. Signed-off-by: Rajesh Girada <rgirada@vmware.com>
* | | | ospfd: Use AF_UNSPEC instead of setting to 0Donald Sharp2022-02-074-10/+10
| |/ / |/| | | | | | | | Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | | ospfd: Core in ospf_if_down during shutdown.Tomi Salminen2022-02-041-0/+2
| |/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Skip marking routes as changed in ospf_if_down if there's now new_table present, which might be the case when the instance is being finished The backtrace for the core was: raise (sig=sig@entry=11) at ../sysdeps/unix/sysv/linux/raise.c:50 core_handler (signo=11, siginfo=0x7fffffffe170, context=<optimized out>) at lib/sigevent.c:262 <signal handler called> route_top (table=0x0) at lib/table.c:401 ospf_if_down (oi=oi@entry=0x555555999090) at ospfd/ospf_interface.c:849 ospf_if_free (oi=0x555555999090) at ospfd/ospf_interface.c:339 ospf_finish_final (ospf=0x55555599c830) at ospfd/ospfd.c:749 ospf_deferred_shutdown_finish (ospf=0x55555599c830) at ospfd/ospfd.c:578 ospf_deferred_shutdown_check (ospf=<optimized out>) at ospfd/ospfd.c:627 ospf_finish (ospf=<optimized out>) at ospfd/ospfd.c:683 ospf_terminate () at ospfd/ospfd.c:653 sigint () at ospfd/ospf_main.c:109 quagga_sigevent_process () at lib/sigevent.c:130 thread_fetch (m=m@entry=0x5555556e45e0, fetch=fetch@entry=0x7fffffffe9b0) at lib/thread.c:1709 frr_run (master=0x5555556e45e0) at lib/libfrr.c:1174 main (argc=9, argv=0x7fffffffecb8) at ospfd/ospf_main.c:254 Signed-off-by: Tomi Salminen <tsalminen@forcepoint.com>
* | Merge pull request #10373 from anlancs/ospf-add-asbrIgor Ryzhov2022-02-014-12/+16
|\ \ | | | | | | ospfd: fix missing "aggregation timer" in running configuration
| * | ospfd: adjust type of "aggr_delay_interval"anlan_cs2022-01-294-6/+6
| | | | | | | | | | | | | | | | | | | | | Adjust type of "aggr_delay_interval": Just replace `unsigned int` with `uint16_t` for range is (50..1800). Signed-off-by: anlan_cs <vic.lan@pica8.com>
| * | ospfd: fix missing "aggregation timer" in running configurationanlan_cs2022-01-291-0/+4
| | | | | | | | | | | | Signed-off-by: anlan_cs <vic.lan@pica8.com>
| * | ospfd: a few minor fixes on ospfd clianlan_cs2022-01-291-6/+6
| |/ | | | | | | | | | | | | | | Two minor fixes: - remove redundant "(a.b.c.d/m)" in "prefix" description - remove some annoying space in "summary-address" and "tag" Signed-off-by: anlan_cs <vic.lan@pica8.com>
* | ospfd: remove extra spaceckishimo2022-02-011-1/+1
| | | | | | | | Signed-off-by: ckishimo <carles.kishimoto@gmail.com>
* | ospfd: print maximum paths in show ip ospf jsonckishimo2022-02-011-0/+4
| | | | | | | | Signed-off-by: ckishimo <carles.kishimoto@gmail.com>
* | ospfd: print administrative distance in show ip ospfckishimo2022-02-011-0/+11
| | | | | | | | Signed-off-by: ckishimo <carles.kishimoto@gmail.com>
* | ospfd: restart spf when distance is updatedckishimo2022-02-011-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | if r1 has a route received from a neighbor with the default administrative distance configured r1# sh ip route Codes: K - kernel route, C - connected, S - static, R - RIP, O - OSPF, I - IS-IS, B - BGP, E - EIGRP, N - NHRP, T - Table, v - VNC, V - VNC-Direct, A - Babel, F - PBR, f - OpenFabric, > - selected route, * - FIB route, q - queued, r - rejected, b - backup t - trapped, o - offload failure O>* 1.1.1.1/32 [110/20] via 10.0.12.2, r1-r2-eth0, weight 1, 00:00:41 if we change the administrative distance r1(config)# router ospf r1(config-router)# distance 50 this is not applied as there are no changes in the routing table r1# sh ip route Codes: K - kernel route, C - connected, S - static, R - RIP, O - OSPF, I - IS-IS, B - BGP, E - EIGRP, N - NHRP, T - Table, v - VNC, V - VNC-Direct, A - Babel, F - PBR, f - OpenFabric, > - selected route, * - FIB route, q - queued, r - rejected, b - backup t - trapped, o - offload failure O>* 1.1.1.1/32 [110/20] via 10.0.12.2, r1-r2-eth0, weight 1, 00:00:13 This commit will force the update of the routing table with the new configured distance r1# sh ip route Codes: K - kernel route, C - connected, S - static, R - RIP, O - OSPF, I - IS-IS, B - BGP, E - EIGRP, N - NHRP, T - Table, v - VNC, V - VNC-Direct, A - Babel, F - PBR, f - OpenFabric, > - selected route, * - FIB route, q - queued, r - rejected, b - backup t - trapped, o - offload failure O>* 1.1.1.1/32 [50/20] via 10.0.12.2, r1-r2-eth0, weight 1, 00:00:14 Signed-off-by: ckishimo <carles.kishimoto@gmail.com>
* | ospfd: Convert output to host order from network order for route_tagDonald Sharp2022-01-311-1/+1
| | | | | | | | | | | | | | | | | | | | FRR stores the route_tag in network byte order. Bug filed indicates that the `show ip ospf route` command shows the correct value. Every place route_tag is dumped in ospf_vty.c the ntohl function is used first. Fixes: #10450 Signed-off-by: Donald Sharp <sharpd@nvidia.com>
* | Merge pull request #10408 from idryzhov/no-opaque-by-defaultDonatas Abraitis2022-01-282-4/+2
|\ \ | | | | | | *: do not send opaque data to zebra by default
| * | *: do not send opaque data to zebra by defaultIgor Ryzhov2022-01-242-4/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Opaque data takes up a lot of memory when there are a lot of routes on the box. Given that this is just a cosmetic info, I propose to disable it by default to not shock people who start using FRR for the first time or upgrades from an old version. Fixes #10101. Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
* | | Merge pull request #10411 from idryzhov/if-config-vrf-nameRuss White2022-01-251-7/+3
|\ \ \ | |/ / |/| | *: do not print vrf name for interface config when using vrf-lite
| * | *: do not print vrf name for interface config when using vrf-liteIgor Ryzhov2022-01-241-7/+3
| |/ | | | | | | | | | | | | VRF name should not be printed in the config since 574445ec. The update was done for NB config output but I missed it for regular vty output. Signed-off-by: Igor Ryzhov <iryzhov@nfware.com>
* | Merge pull request #10407 from idryzhov/ospf-opaque-crashRuss White2022-01-241-0/+3
|\ \ | | | | | | ospfd: fix crash on "ospf send-extra-data zebra"
generated by cgit v1.2.3 (git 2.39.1) at 2024-12-05 03:07:44 +0100