]> git.puffer.fish Git - matthieu/frr.git/commitdiff
ospfd: fix show_ip_ospf_gr_helper
authorteletajp <teletajp@yandex.ru>
Wed, 1 Nov 2023 09:25:07 +0000 (12:25 +0300)
committerteletajp <teletajp@yandex.ru>
Wed, 15 Nov 2023 13:18:46 +0000 (16:18 +0300)
Fix for the command "show ip ospf vrf NAME graceful-restart helper".
FRR did not show information by vrf's name.
If i have router ospf vrf red, vtysh's command
'show ip ospf vrf red graceful-restart helper' will not show anything.
But command 'show ip ospf vrf all graceful-restart helper' will work
normally. This fix fixes the display of information by vrf's name.

Example:
frr1# show ip ospf vrf vrf-1 graceful-restart helper
VRF Name: vrf-1

       OSPF Router with ID (192.168.255.81)

 Graceful restart helper support enabled.
 Strict LSA check is enabled.
 Helper supported for Planned and Unplanned Restarts.
 Supported Graceful restart interval: 1800(in seconds).

Signed-off-by: teletajp <teletajp@yandex.ru>
ospfd/ospf_vty.c

index ab412e360619198788cad7962356000d013d09fc..6c2520e4258ba176ca1fef0958c4037f53d7586d 100644 (file)
@@ -10807,38 +10807,23 @@ DEFPY (show_ip_ospf_gr_helper,
                }
 
                ospf = ospf_lookup_by_inst_name(inst, vrf_name);
-
-               if (ospf == NULL || !ospf->oi_running) {
-
-                       if (uj)
-                               vty_json(vty, json);
-                       else
-                               vty_out(vty,
-                                       "%% OSPF is not enabled in vrf %s\n",
-                                       vrf_name);
-
-                       return CMD_SUCCESS;
-               }
-
        } else {
                /* Default Vrf */
                ospf = ospf_lookup_by_vrf_id(VRF_DEFAULT);
+       }
 
-               if (ospf == NULL || !ospf->oi_running) {
-
-                       if (uj)
-                               vty_json(vty, json);
-                       else
-                               vty_out(vty,
-                                       "%% OSPF is not enabled in vrf default\n");
+       if (ospf == NULL || !ospf->oi_running) {
 
-                       return CMD_SUCCESS;
-               }
+               if (uj)
+                       vty_json(vty, json);
+               else
+                       vty_out(vty,
+                               "%% OSPF is not enabled in vrf %s\n", vrf_name ? vrf_name : "default");
 
-               ospf_show_gr_helper_details(vty, ospf, use_vrf, json, uj,
-                                           detail);
+               return CMD_SUCCESS;
        }
 
+       ospf_show_gr_helper_details(vty, ospf, use_vrf, json, uj, detail);
        if (uj)
                vty_json(vty, json);