]> git.puffer.fish Git - mirror/frr.git/commitdiff
bgpd: Remove deprecated COMMUNITY_INTERNET community 14141/head
authorDonatas Abraitis <donatas@opensourcerouting.org>
Thu, 3 Aug 2023 14:12:19 +0000 (17:12 +0300)
committerDonatas Abraitis <donatas@opensourcerouting.org>
Thu, 3 Aug 2023 14:12:19 +0000 (17:12 +0300)
Signed-off-by: Donatas Abraitis <donatas@opensourcerouting.org>
bgpd/bgp_clist.c
bgpd/bgp_community.c
bgpd/bgp_community.h
bgpd/bgp_routemap.c

index 3b987c2f57e9e16d6020a713483dc2018bc2f9ef..3fd246397fd897a3da404f5ff9c9fd1cf923d69d 100644 (file)
@@ -449,13 +449,6 @@ static char *community_str_get(struct community *com, int i)
        comval = ntohl(comval);
 
        switch (comval) {
-#if CONFDATE > 20230801
-CPP_NOTICE("Deprecate COMMUNITY_INTERNET BGP community")
-#endif
-       case COMMUNITY_INTERNET:
-               str = XSTRDUP(MTYPE_COMMUNITY_STR, "internet");
-               zlog_warn("`internet` community is deprecated");
-               break;
        case COMMUNITY_GSHUT:
                str = XSTRDUP(MTYPE_COMMUNITY_STR, "graceful-shutdown");
                break;
@@ -660,9 +653,6 @@ bool community_list_match(struct community *com, struct community_list *list)
 
        for (entry = list->head; entry; entry = entry->next) {
                if (entry->style == COMMUNITY_LIST_STANDARD) {
-                       if (community_include(entry->u.com, COMMUNITY_INTERNET))
-                               return entry->direct == COMMUNITY_PERMIT;
-
                        if (community_match(com, entry->u.com))
                                return entry->direct == COMMUNITY_PERMIT;
                } else if (entry->style == COMMUNITY_LIST_EXPANDED) {
@@ -735,9 +725,6 @@ bool community_list_exact_match(struct community *com,
 
        for (entry = list->head; entry; entry = entry->next) {
                if (entry->style == COMMUNITY_LIST_STANDARD) {
-                       if (community_include(entry->u.com, COMMUNITY_INTERNET))
-                               return entry->direct == COMMUNITY_PERMIT;
-
                        if (community_cmp(com, entry->u.com))
                                return entry->direct == COMMUNITY_PERMIT;
                } else if (entry->style == COMMUNITY_LIST_EXPANDED) {
@@ -767,17 +754,14 @@ struct community *community_list_match_delete(struct community *com,
 
                for (entry = list->head; entry; entry = entry->next) {
                        if ((entry->style == COMMUNITY_LIST_STANDARD) &&
-                           (community_include(entry->u.com,
-                                              COMMUNITY_INTERNET) ||
-                            community_include(entry->u.com, val))) {
+                           community_include(entry->u.com, val)) {
                                if (entry->direct == COMMUNITY_PERMIT) {
                                        com_index_to_delete[delete_index] = i;
                                        delete_index++;
                                }
                                break;
                        } else if ((entry->style == COMMUNITY_LIST_EXPANDED) &&
-                                  community_regexp_include(entry->reg, com,
-                                                           i)) {
+                                  community_regexp_include(entry->reg, com, i)) {
                                if (entry->direct == COMMUNITY_PERMIT) {
                                        com_index_to_delete[delete_index] = i;
                                        delete_index++;
index f56bfc8bdc56b589bddb59d5b2366a9001924fed..8e4c4305552b9e941a9932279ecf9a77a99bab78 100644 (file)
@@ -228,13 +228,6 @@ static void set_community_string(struct community *com, bool make_json,
                comval = ntohl(comval);
 
                switch (comval) {
-#if CONFDATE > 20230801
-CPP_NOTICE("Deprecate COMMUNITY_INTERNET BGP community")
-#endif
-               case COMMUNITY_INTERNET:
-                       len += strlen(" internet");
-                       zlog_warn("`internet` community is deprecated");
-                       break;
                case COMMUNITY_GSHUT:
                        len += strlen(" graceful-shutdown");
                        break;
@@ -298,19 +291,6 @@ CPP_NOTICE("Deprecate COMMUNITY_INTERNET BGP community")
                        strlcat(str, " ", len);
 
                switch (comval) {
-#if CONFDATE > 20230801
-CPP_NOTICE("Deprecate COMMUNITY_INTERNET BGP community")
-#endif
-               case COMMUNITY_INTERNET:
-                       strlcat(str, "internet", len);
-                       if (make_json) {
-                               json_string =
-                                       json_object_new_string("internet");
-                               json_object_array_add(json_community_list,
-                                                     json_string);
-                       }
-                       zlog_warn("`internet` community is deprecated");
-                       break;
                case COMMUNITY_GSHUT:
                        strlcat(str, "graceful-shutdown", len);
                        if (make_json) {
@@ -680,16 +660,6 @@ community_gettoken(const char *buf, enum community_token *token, uint32_t *val)
 
        /* Well known community string check. */
        if (isalpha((unsigned char)*p)) {
-#if CONFDATE > 20230801
-CPP_NOTICE("Deprecate COMMUNITY_INTERNET BGP community")
-#endif
-               if (strncmp(p, "internet", strlen("internet")) == 0) {
-                       *val = COMMUNITY_INTERNET;
-                       *token = community_token_no_export;
-                       p += strlen("internet");
-                       zlog_warn("`internet` community is deprecated");
-                       return p;
-               }
                if (strncmp(p, "graceful-shutdown", strlen("graceful-shutdown"))
                    == 0) {
                        *val = COMMUNITY_GSHUT;
index e7af362ea845d3c0ff8b1443c8ba3ae0b4b3ad58..7c7e7af4d29dff311d0d09675898276a1360ef21 100644 (file)
@@ -30,10 +30,6 @@ struct community {
 };
 
 /* Well-known communities value.  */
-#if CONFDATE > 20230801
-CPP_NOTICE("Deprecate COMMUNITY_INTERNET BGP community")
-#endif
-#define COMMUNITY_INTERNET                      0x0
 #define COMMUNITY_GSHUT                         0xFFFF0000
 #define COMMUNITY_ACCEPT_OWN                    0xFFFF0001
 #define COMMUNITY_ROUTE_FILTER_TRANSLATED_v4    0xFFFF0002
index 0f86cfdd2d2c4e6a1abb588836abc7b7fac74e01..b7ac976e23bf156cc9a05eb96cf752bf8b75df11 100644 (file)
@@ -6282,16 +6282,6 @@ DEFUN_YANG (set_community,
                else
                        first = 1;
 
-#if CONFDATE > 20230801
-CPP_NOTICE("Deprecate COMMUNITY_INTERNET BGP community")
-#endif
-               if (strncmp(argv[i]->arg, "internet", strlen(argv[i]->arg))
-                   == 0) {
-                       buffer_putstr(b, "internet");
-                       vty_out(vty, "%% `internet` community is deprecated\n");
-                       zlog_warn("`internet` community is deprecated");
-                       continue;
-               }
                if (strncmp(argv[i]->arg, "local-AS", strlen(argv[i]->arg))
                    == 0) {
                        buffer_putstr(b, "local-AS");