]> git.puffer.fish Git - matthieu/frr.git/commitdiff
pbrd: Remove table and rule range commands
authorDonald Sharp <sharpd@cumulusnetworks.com>
Mon, 26 Mar 2018 17:32:41 +0000 (13:32 -0400)
committerDonald Sharp <sharpd@cumulusnetworks.com>
Fri, 6 Apr 2018 17:22:43 +0000 (13:22 -0400)
Since PBR is meant to be for small deployments, allowing
end users to arbitrarily change rule and table ranges
without some more careful thought on what is going on
and how to do it, sets us up for issues.

At this time remove these knobs.

Signed-off-by: Donald Sharp <sharpd@cumulusnetworks.com>
pbrd/pbr_vty.c

index 9800baf6749ddab1ddaf033270a05be8c3c9e7a4..a4f3c54646412e62952c9d6d636b894a04700018 100644 (file)
@@ -295,44 +295,6 @@ DEFPY(pbr_map_nexthop, pbr_map_nexthop_cmd,
        return CMD_SUCCESS;
 }
 
-DEFPY (pbr_table_range,
-       pbr_table_range_cmd,
-       "[no]$no pbr table range (10000-65535)$start (11000-65535)$end",
-       NO_STR
-       "Policy based routing\n"
-       "Policy based routing table\n"
-       "Table range\n"
-       "Initial value of range\n"
-       "Final value of range\n")
-{
-       if (no)
-               pbr_nht_set_tableid_range(PBR_NHT_DEFAULT_LOW_TABLEID,
-                                         PBR_NHT_DEFAULT_HIGH_TABLEID);
-       else
-               pbr_nht_set_tableid_range(start, end);
-
-       return CMD_SUCCESS;
-}
-
-DEFPY (pbr_rule_range,
-       pbr_rule_range_cmd,
-       "[no] pbr rule range (300-900)$start (400-1000)$end",
-       NO_STR
-       "Policy based routing\n"
-       "Policy based routing rule\n"
-       "Rule range\n"
-       "Initial value of range\n"
-       "Final value of range\n")
-{
-       if (no)
-               pbr_nht_set_rule_range(PBR_NHT_DEFAULT_LOW_RULE,
-                                      PBR_NHT_DEFAULT_HIGH_RULE);
-       else
-               pbr_nht_set_rule_range(start, end);
-
-       return CMD_SUCCESS;
-}
-
 DEFPY (pbr_policy,
        pbr_policy_cmd,
        "[no] pbr-policy NAME$mapname",
@@ -606,8 +568,6 @@ void pbr_vty_init(void)
        install_element(CONFIG_NODE, &pbr_map_cmd);
        install_element(CONFIG_NODE, &no_pbr_map_cmd);
        install_element(INTERFACE_NODE, &pbr_policy_cmd);
-       install_element(CONFIG_NODE, &pbr_table_range_cmd);
-       install_element(CONFIG_NODE, &pbr_rule_range_cmd);
        install_element(PBRMAP_NODE, &pbr_map_match_src_cmd);
        install_element(PBRMAP_NODE, &pbr_map_match_dst_cmd);
        install_element(PBRMAP_NODE, &pbr_map_nexthop_group_cmd);