From 606d49a4caeb6cab6bb105c24e88118a4c93174c Mon Sep 17 00:00:00 2001 From: Donatas Abraitis Date: Tue, 16 Aug 2022 09:28:21 +0300 Subject: [PATCH] bgpd: Change warning message when BGP community-list is not found Before: ``` donatas-laptop# show bgp ipv4 unicast community-list testas % testas is not a valid community-list name donatas-laptop# con donatas-laptop(config)# bgp community-list standard testas permit internet donatas-laptop(config)# do show bgp ipv4 unicast community-list testas donatas-laptop(config)# ``` `is not a valid community-list name` is a misleading warning message. Doing the same for filter-list, access-list, prefix-list, route-map. Signed-off-by: Donatas Abraitis --- bgpd/bgp_route.c | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/bgpd/bgp_route.c b/bgpd/bgp_route.c index a8cf9ca3c0..04f955f97a 100644 --- a/bgpd/bgp_route.c +++ b/bgpd/bgp_route.c @@ -12464,8 +12464,7 @@ DEFPY(show_ip_bgp, show_ip_bgp_cmd, list = community_list_lookup(bgp_clist, clist_number_or_name, 0, COMMUNITY_LIST_MASTER); if (list == NULL) { - vty_out(vty, - "%% %s is not a valid community-list name\n", + vty_out(vty, "%% %s community-list not found\n", clist_number_or_name); return CMD_WARNING; } @@ -12483,8 +12482,7 @@ DEFPY(show_ip_bgp, show_ip_bgp_cmd, as_list = as_list_lookup(filter); if (as_list == NULL) { - vty_out(vty, - "%% %s is not a valid AS-path access-list name\n", + vty_out(vty, "%% %s AS-path access-list not found\n", filter); return CMD_WARNING; } @@ -12499,7 +12497,7 @@ DEFPY(show_ip_bgp, show_ip_bgp_cmd, plist = prefix_list_lookup(afi, prefix_list_str); if (plist == NULL) { - vty_out(vty, "%% %s is not a valid prefix-list name\n", + vty_out(vty, "%% %s prefix-list not found\n", prefix_list_str); return CMD_WARNING; } @@ -12514,7 +12512,7 @@ DEFPY(show_ip_bgp, show_ip_bgp_cmd, alist = access_list_lookup(afi, access_list_str); if (!alist) { - vty_out(vty, "%% %s is not a valid access-list name\n", + vty_out(vty, "%% %s access-list not found\n", access_list_str); return CMD_WARNING; } @@ -12529,8 +12527,7 @@ DEFPY(show_ip_bgp, show_ip_bgp_cmd, rmap = route_map_lookup_by_name(rmap_str); if (!rmap) { - vty_out(vty, "%% %s is not a valid route-map name\n", - rmap_str); + vty_out(vty, "%% %s route-map not found\n", rmap_str); return CMD_WARNING; } -- 2.39.5