From 256b67bc8873f268d2cecd16183398352b9fcd16 Mon Sep 17 00:00:00 2001 From: Neale Ranns Date: Wed, 2 Sep 2020 14:46:53 +0000 Subject: teib: Add adj-fibs for peers/adjacencies on p2mp interface Type: fix Change-Id: I6fdc4e952097e92ac3aa53e0be3ef99e0d801b28 Signed-off-by: Neale Ranns --- src/vnet/gre/gre.c | 3 +- src/vnet/ipip/ipip.c | 3 +- src/vnet/ipsec/ipsec_tun.c | 4 +- src/vnet/teib/teib.c | 237 +++++++++++++++++++++++++++++++++++++++++---- src/vnet/teib/teib.h | 7 +- src/vnet/teib/teib_api.c | 17 +++- src/vnet/teib/teib_cli.c | 22 +++-- test/test_ipip.py | 49 +++++++++- test/vpp_ip_route.py | 7 +- 9 files changed, 310 insertions(+), 39 deletions(-) diff --git a/src/vnet/gre/gre.c b/src/vnet/gre/gre.c index 83f12833611..f4c41bd09b4 100644 --- a/src/vnet/gre/gre.c +++ b/src/vnet/gre/gre.c @@ -472,7 +472,8 @@ mgre_update_adj (vnet_main_t * vnm, u32 sw_if_index, adj_index_t ai) ti = gm->tunnel_index_by_sw_if_index[sw_if_index]; t = pool_elt_at_index (gm->tunnels, ti); - ne = teib_entry_find (sw_if_index, &adj->sub_type.nbr.next_hop); + ne = teib_entry_find (sw_if_index, + adj->ia_nh_proto, &adj->sub_type.nbr.next_hop); if (NULL == ne) // no NHRP entry to provide the next-hop diff --git a/src/vnet/ipip/ipip.c b/src/vnet/ipip/ipip.c index 089e2b659c7..138f4d22326 100644 --- a/src/vnet/ipip/ipip.c +++ b/src/vnet/ipip/ipip.c @@ -364,7 +364,8 @@ mipip_update_adj (vnet_main_t * vnm, u32 sw_if_index, adj_index_t ai) ti = gm->tunnel_index_by_sw_if_index[sw_if_index]; t = pool_elt_at_index (gm->tunnels, ti); - ne = teib_entry_find (sw_if_index, &adj->sub_type.nbr.next_hop); + ne = teib_entry_find (sw_if_index, + adj->ia_nh_proto, &adj->sub_type.nbr.next_hop); if (NULL == ne) { diff --git a/src/vnet/ipsec/ipsec_tun.c b/src/vnet/ipsec/ipsec_tun.c index fb530c8dc9f..ac8b24ec232 100644 --- a/src/vnet/ipsec/ipsec_tun.c +++ b/src/vnet/ipsec/ipsec_tun.c @@ -718,7 +718,9 @@ ipsec_tun_protect_update (u32 sw_if_index, ip_address_to_46 (nh, &peer); ipsec_tun_protect_update_from_teib - (itp, teib_entry_find (sw_if_index, &peer)); + (itp, teib_entry_find (sw_if_index, + ip_address_family_to_fib_proto + (ip_addr_version (nh)), &peer)); } if (is_l2) diff --git a/src/vnet/teib/teib.c b/src/vnet/teib/teib.c index e392b49e86b..0f33d087a50 100644 --- a/src/vnet/teib/teib.c +++ b/src/vnet/teib/teib.c @@ -19,12 +19,14 @@ #include #include #include +#include typedef struct teib_key_t_ { ip46_address_t tk_peer; u32 tk_sw_if_index; -} teib_key_t; + fib_protocol_t tk_proto; +} __clib_packed teib_key_t; struct teib_entry_t_ { @@ -33,9 +35,10 @@ struct teib_entry_t_ u32 te_fib_index; }; -static uword *teib_db; +static uword *teib_db[FIB_PROTOCOL_IP_MAX]; static teib_entry_t *teib_pool; static teib_vft_t *teib_vfts; +static vlib_log_class_t teib_logger; #define TEIB_NOTIFY(_te, _fn) { \ teib_vft_t *_vft; \ @@ -46,12 +49,29 @@ static teib_vft_t *teib_vfts; } \ } +#define TEIB_DBG(...) \ + vlib_log_debug (teib_logger, __VA_ARGS__); + +#define TEIB_INFO(...) \ + vlib_log_notice (teib_logger, __VA_ARGS__); + +#define TEIB_TE_DBG(_te, _fmt, _args...) \ + vlib_log_debug (teib_logger, "[%U]: " _fmt, format_teib_entry, _te - teib_pool, ##_args) +#define TEIB_TE_INFO(_te, _fmt, _args...) \ + vlib_log_notice (teib_logger, "[%U]: " _fmt, format_teib_entry, _te - teib_pool, ##_args) + u32 teib_entry_get_sw_if_index (const teib_entry_t * te) { return (te->te_key->tk_sw_if_index); } +fib_protocol_t +teib_entry_get_proto (const teib_entry_t * te) +{ + return (te->te_key->tk_proto); +} + u32 teib_entry_get_fib_index (const teib_entry_t * te) { @@ -83,15 +103,17 @@ teib_entry_get (index_t tei) } teib_entry_t * -teib_entry_find (u32 sw_if_index, const ip46_address_t * peer) +teib_entry_find (u32 sw_if_index, + fib_protocol_t fproto, const ip46_address_t * peer) { teib_key_t nk = { .tk_peer = *peer, + .tk_proto = fproto, .tk_sw_if_index = sw_if_index, }; uword *p; - p = hash_get_mem (teib_db, &nk); + p = hash_get_mem (teib_db[fproto], &nk); if (NULL != p) return teib_entry_get (p[0]); @@ -99,34 +121,104 @@ teib_entry_find (u32 sw_if_index, const ip46_address_t * peer) return (NULL); } +static void +teib_adj_fib_add (fib_protocol_t fproto, + const ip46_address_t * ip, u32 sw_if_index, u32 fib_index) +{ + if (FIB_PROTOCOL_IP6 == fproto && + ip6_address_is_link_local_unicast (&ip->ip6)) + { + ip6_ll_prefix_t pfx = { + .ilp_addr = ip->ip6, + .ilp_sw_if_index = sw_if_index, + }; + ip6_ll_table_entry_update (&pfx, FIB_ROUTE_PATH_FLAG_NONE); + } + else + { + fib_prefix_t pfx = { + .fp_len = (FIB_PROTOCOL_IP4 == fproto ? 32 : 128), + .fp_proto = fproto, + .fp_addr = *ip, + }; + fib_table_entry_path_add (fib_index, &pfx, FIB_SOURCE_ADJ, + FIB_ENTRY_FLAG_ATTACHED, + fib_proto_to_dpo (pfx.fp_proto), + &pfx.fp_addr, + sw_if_index, + ~0, 1, NULL, FIB_ROUTE_PATH_FLAG_NONE); + + + if (1 == hash_elts (teib_db[pfx.fp_proto])) + fib_table_lock (fib_index, pfx.fp_proto, FIB_SOURCE_ADJ); + } +} + +static void +teib_adj_fib_remove (fib_protocol_t fproto, + ip46_address_t * ip, u32 sw_if_index, u32 fib_index) +{ + if (FIB_PROTOCOL_IP6 == fproto && + ip6_address_is_link_local_unicast (&ip->ip6)) + { + ip6_ll_prefix_t pfx = { + .ilp_addr = ip->ip6, + .ilp_sw_if_index = sw_if_index, + }; + ip6_ll_table_entry_delete (&pfx); + } + else + { + fib_prefix_t pfx = { + .fp_len = (FIB_PROTOCOL_IP4 == fproto ? 32 : 128), + .fp_proto = fproto, + .fp_addr = *ip, + }; + + fib_table_entry_path_remove (fib_index, &pfx, FIB_SOURCE_ADJ, + fib_proto_to_dpo (pfx.fp_proto), + &pfx.fp_addr, + sw_if_index, + ~0, 1, FIB_ROUTE_PATH_FLAG_NONE); + + if (0 == hash_elts (teib_db[pfx.fp_proto])) + fib_table_unlock (fib_index, pfx.fp_proto, FIB_SOURCE_ADJ); + } +} + int teib_entry_add (u32 sw_if_index, + fib_protocol_t fproto, const ip46_address_t * peer, u32 nh_table_id, const ip46_address_t * nh) { - fib_protocol_t fproto; + fib_protocol_t nh_proto; teib_entry_t *te; u32 fib_index; index_t tei; - fproto = (ip46_address_is_ip4 (nh) ? FIB_PROTOCOL_IP4 : FIB_PROTOCOL_IP6); + nh_proto = (ip46_address_is_ip4 (nh) ? FIB_PROTOCOL_IP4 : FIB_PROTOCOL_IP6); - fib_index = fib_table_find (fproto, nh_table_id); + fib_index = fib_table_find (nh_proto, nh_table_id); if (~0 == fib_index) { return (VNET_API_ERROR_NO_SUCH_FIB); } - te = teib_entry_find (sw_if_index, peer); + te = teib_entry_find (sw_if_index, fproto, peer); if (NULL == te) { teib_key_t nk = { .tk_peer = *peer, + .tk_proto = fproto, .tk_sw_if_index = sw_if_index, }; teib_entry_t *te; + u32 fib_index; + + fib_index = fib_table_get_index_for_sw_if_index (fproto, sw_if_index); pool_get_zero (teib_pool, te); @@ -139,26 +231,43 @@ teib_entry_add (u32 sw_if_index, te->te_nh.fp_len = (te->te_nh.fp_proto == FIB_PROTOCOL_IP4 ? 32 : 128); te->te_fib_index = fib_index; - hash_set_mem (teib_db, te->te_key, tei); + hash_set_mem (teib_db[fproto], te->te_key, tei); + + /* we how have a /32 in the overlay, add an adj-fib */ + teib_adj_fib_add (te->te_key->tk_proto, + &te->te_key->tk_peer, sw_if_index, fib_index); TEIB_NOTIFY (te, nv_added); + TEIB_TE_INFO (te, "created"); } else - return (VNET_API_ERROR_ENTRY_ALREADY_EXISTS); - + { + TEIB_TE_INFO (te, "exists"); + return (VNET_API_ERROR_ENTRY_ALREADY_EXISTS); + } return 0; } int -teib_entry_del (u32 sw_if_index, const ip46_address_t * peer) +teib_entry_del (u32 sw_if_index, + fib_protocol_t fproto, const ip46_address_t * peer) { teib_entry_t *te; - te = teib_entry_find (sw_if_index, peer); + te = teib_entry_find (sw_if_index, fproto, peer); if (te != NULL) { - hash_unset_mem (teib_db, te->te_key); + TEIB_TE_INFO (te, "removed"); + + u32 fib_index; + + fib_index = fib_table_get_index_for_sw_if_index (fproto, sw_if_index); + + teib_adj_fib_remove (te->te_key->tk_proto, + &te->te_key->tk_peer, sw_if_index, fib_index); + + hash_unset_mem (teib_db[fproto], te->te_key); TEIB_NOTIFY (te, nv_deleted); @@ -166,8 +275,14 @@ teib_entry_del (u32 sw_if_index, const ip46_address_t * peer) pool_put (teib_pool, te); } else - return (VNET_API_ERROR_ENTRY_ALREADY_EXISTS); - + { + TEIB_INFO ("no such entry: %U, %U, %U", + format_vnet_sw_if_index_name, + vnet_get_main (), sw_if_index, + format_fib_protocol, fproto, + format_ip46_address, peer, IP46_TYPE_ANY); + return (VNET_API_ERROR_NO_SUCH_ENTRY); + } return 0; } @@ -183,7 +298,8 @@ format_teib_entry (u8 * s, va_list * args) s = format (s, "[%d] ", tei); s = format (s, "%U:", format_vnet_sw_if_index_name, vnm, te->te_key->tk_sw_if_index); - s = format (s, " %U", format_ip46_address, + s = format (s, " %U:", format_fib_protocol, te->te_key->tk_proto); + s = format (s, "%U", format_ip46_address, &te->te_key->tk_peer, IP46_TYPE_ANY); s = format (s, " via [%d]:%U", fib_table_get_table_id (te->te_fib_index, te->te_nh.fp_proto), @@ -219,6 +335,76 @@ teib_walk_itf (u32 sw_if_index, teib_walk_cb_t fn, void *ctx) /* *INDENT-ON* */ } +static void +teib_walk_itf_proto (u32 sw_if_index, + fib_protocol_t fproto, teib_walk_cb_t fn, void *ctx) +{ + index_t tei; + + /* *INDENT-OFF* */ + pool_foreach_index(tei, teib_pool, + ({ + if (sw_if_index == teib_entry_get_sw_if_index(teib_entry_get(tei)) && + fproto == teib_entry_get_proto(teib_entry_get(tei))) + fn(tei, ctx); + })); + /* *INDENT-ON* */ +} + +typedef struct teib_table_bind_ctx_t_ +{ + u32 new_fib_index; + u32 old_fib_index; +} teib_table_bind_ctx_t; + +static walk_rc_t +teib_walk_table_bind (index_t tei, void *arg) +{ + teib_table_bind_ctx_t *ctx = arg; + teib_entry_t *te; + + te = teib_entry_get (tei); + + TEIB_TE_INFO (te, "bind: %d -> %d", ctx->old_fib_index, ctx->new_fib_index); + + teib_adj_fib_remove (te->te_key->tk_proto, + &te->te_key->tk_peer, + te->te_key->tk_sw_if_index, ctx->old_fib_index); + teib_adj_fib_add (te->te_key->tk_proto, + &te->te_key->tk_peer, + te->te_key->tk_sw_if_index, ctx->new_fib_index); + + return (WALK_CONTINUE); +} + +static void +teib_table_bind_v4 (ip4_main_t * im, + uword opaque, + u32 sw_if_index, u32 new_fib_index, u32 old_fib_index) +{ + teib_table_bind_ctx_t ctx = { + .old_fib_index = old_fib_index, + .new_fib_index = new_fib_index, + }; + + teib_walk_itf_proto (sw_if_index, + FIB_PROTOCOL_IP4, teib_walk_table_bind, &ctx); +} + +static void +teib_table_bind_v6 (ip6_main_t * im, + uword opaque, + u32 sw_if_index, u32 new_fib_index, u32 old_fib_index) +{ + teib_table_bind_ctx_t ctx = { + .old_fib_index = old_fib_index, + .new_fib_index = new_fib_index, + }; + + teib_walk_itf_proto (sw_if_index, + FIB_PROTOCOL_IP6, teib_walk_table_bind, &ctx); +} + void teib_register (const teib_vft_t * vft) { @@ -228,7 +414,22 @@ teib_register (const teib_vft_t * vft) static clib_error_t * teib_init (vlib_main_t * vm) { - teib_db = hash_create_mem (0, sizeof (teib_key_t), sizeof (u32)); + fib_protocol_t fproto; + + FOR_EACH_FIB_IP_PROTOCOL (fproto) + teib_db[fproto] = hash_create_mem (0, sizeof (teib_key_t), sizeof (u32)); + + ip4_table_bind_callback_t cb4 = { + .function = teib_table_bind_v4, + }; + vec_add1 (ip4_main.table_bind_callbacks, cb4); + + ip6_table_bind_callback_t cb6 = { + .function = teib_table_bind_v6, + }; + vec_add1 (ip6_main.table_bind_callbacks, cb6); + + teib_logger = vlib_log_register_class ("teib", "teib"); return (NULL); } diff --git a/src/vnet/teib/teib.h b/src/vnet/teib/teib.h index 4a03eee844f..2a5da76ef93 100644 --- a/src/vnet/teib/teib.h +++ b/src/vnet/teib/teib.h @@ -37,6 +37,7 @@ typedef struct teib_entry_t_ teib_entry_t; extern u32 teib_entry_get_sw_if_index (const teib_entry_t * ne); extern u32 teib_entry_get_fib_index (const teib_entry_t * ne); extern const ip46_address_t *teib_entry_get_peer (const teib_entry_t * ne); +extern fib_protocol_t teib_entry_get_proto (const teib_entry_t * ne); extern const fib_prefix_t *teib_entry_get_nh (const teib_entry_t * ne); extern u8 *format_teib_entry (u8 * s, va_list * args); @@ -44,12 +45,16 @@ extern u8 *format_teib_entry (u8 * s, va_list * args); * Create a new TEIB entry */ extern int teib_entry_add (u32 sw_if_index, + fib_protocol_t fproto, const ip46_address_t * peer, u32 nh_table_id, const ip46_address_t * nh); -extern int teib_entry_del (u32 sw_if_index, const ip46_address_t * peer); +extern int teib_entry_del (u32 sw_if_index, + fib_protocol_t fproto, + const ip46_address_t * peer); extern teib_entry_t *teib_entry_find (u32 sw_if_index, + fib_protocol_t fproto, const ip46_address_t * peer); extern teib_entry_t *teib_entry_get (index_t nei); diff --git a/src/vnet/teib/teib_api.c b/src/vnet/teib/teib_api.c index 48bfee34fda..b885128baf4 100644 --- a/src/vnet/teib/teib_api.c +++ b/src/vnet/teib/teib_api.c @@ -39,19 +39,26 @@ static void vl_api_teib_entry_add_del_t_handler (vl_api_teib_entry_add_del_t * mp) { vl_api_teib_entry_add_del_reply_t *rmp; - ip46_address_t peer, nh; + ip46_address_t nh; + ip_address_t peer; int rv; VALIDATE_SW_IF_INDEX ((&mp->entry)); - ip_address_decode (&mp->entry.peer, &peer); + ip_address_decode2 (&mp->entry.peer, &peer); ip_address_decode (&mp->entry.nh, &nh); if (mp->is_add) - rv = teib_entry_add (ntohl (mp->entry.sw_if_index), &peer, - ntohl (mp->entry.nh_table_id), &nh); + rv = teib_entry_add (ntohl (mp->entry.sw_if_index), + ip_address_family_to_fib_proto (ip_addr_version + (&peer)), + &ip_addr_46 (&peer), ntohl (mp->entry.nh_table_id), + &nh); else - rv = teib_entry_del (ntohl (mp->entry.sw_if_index), &peer); + rv = teib_entry_del (ntohl (mp->entry.sw_if_index), + ip_address_family_to_fib_proto (ip_addr_version + (&peer)), + &ip_addr_46 (&peer)); BAD_SW_IF_INDEX_LABEL; diff --git a/src/vnet/teib/teib_cli.c b/src/vnet/teib/teib_cli.c index faf0d828963..63e3645f14a 100644 --- a/src/vnet/teib/teib_cli.c +++ b/src/vnet/teib/teib_cli.c @@ -20,8 +20,8 @@ teib_add (vlib_main_t * vm, unformat_input_t * input, vlib_cli_command_t * cmd) { unformat_input_t _line_input, *line_input = &_line_input; - ip46_address_t peer = ip46_address_initializer; ip46_address_t nh = ip46_address_initializer; + ip_address_t peer = IP_ADDRESS_V6_ALL_0S; u32 sw_if_index, nh_table_id; clib_error_t *error = NULL; int rv; @@ -38,7 +38,7 @@ teib_add (vlib_main_t * vm, if (unformat (line_input, "%U", unformat_vnet_sw_interface, vnet_get_main (), &sw_if_index)) ; - else if (unformat (line_input, "peer %U", unformat_ip46_address, &peer)) + else if (unformat (line_input, "peer %U", unformat_ip_address, &peer)) ; else if (unformat (line_input, "nh %U", unformat_ip46_address, &nh)) ; @@ -58,7 +58,7 @@ teib_add (vlib_main_t * vm, format_unformat_error, line_input); goto done; } - if (ip46_address_is_zero (&peer)) + if (ip_address_is_zero (&peer)) { error = clib_error_return (0, "peer required'", format_unformat_error, line_input); @@ -71,7 +71,10 @@ teib_add (vlib_main_t * vm, goto done; } - rv = teib_entry_add (sw_if_index, &peer, nh_table_id, &nh); + rv = teib_entry_add (sw_if_index, + ip_address_family_to_fib_proto (ip_addr_version + (&peer)), + &ip_addr_46 (&peer), nh_table_id, &nh); if (rv) { @@ -99,7 +102,7 @@ teib_del (vlib_main_t * vm, unformat_input_t * input, vlib_cli_command_t * cmd) { unformat_input_t _line_input, *line_input = &_line_input; - ip46_address_t peer = ip46_address_initializer; + ip_address_t peer = IP_ADDRESS_V6_ALL_0S; clib_error_t *error = NULL; u32 sw_if_index; int rv; @@ -115,7 +118,7 @@ teib_del (vlib_main_t * vm, if (unformat (line_input, "%U", unformat_vnet_sw_interface, vnet_get_main (), &sw_if_index)) ; - else if (unformat (line_input, "peer %U", unformat_ip46_address, &peer)) + else if (unformat (line_input, "peer %U", unformat_ip_address, &peer)) ; else { @@ -130,14 +133,17 @@ teib_del (vlib_main_t * vm, error = clib_error_return (0, "interface required'", format_unformat_error, line_input); } - if (ip46_address_is_zero (&peer)) + if (ip_address_is_zero (&peer)) { error = clib_error_return (0, "peer required'", format_unformat_error, line_input); goto done; } - rv = teib_entry_del (sw_if_index, &peer); + rv = teib_entry_del (sw_if_index, + ip_address_family_to_fib_proto (ip_addr_version + (&peer)), + &ip_addr_46 (&peer)); if (rv) { diff --git a/test/test_ipip.py b/test/test_ipip.py index 04034874ddd..8f18c07341d 100644 --- a/test/test_ipip.py +++ b/test/test_ipip.py @@ -45,7 +45,7 @@ class TestIPIP(VppTestCase): @classmethod def setUpClass(cls): super(TestIPIP, cls).setUpClass() - cls.create_pg_interfaces(range(2)) + cls.create_pg_interfaces(range(3)) cls.interfaces = list(cls.pg_interfaces) @classmethod @@ -54,8 +54,14 @@ class TestIPIP(VppTestCase): def setUp(self): super(TestIPIP, self).setUp() + self.table = VppIpTable(self, 1, register=False) + self.table.add_vpp_config() + for i in self.interfaces: i.admin_up() + + self.pg2.set_table_ip4(self.table.table_id) + for i in self.interfaces: i.config_ip4() i.config_ip6() i.disable_ipv6_ra() @@ -68,8 +74,11 @@ class TestIPIP(VppTestCase): for i in self.pg_interfaces: i.unconfig_ip4() i.unconfig_ip6() + i.set_table_ip4(0) i.admin_down() + self.table.remove_vpp_config() + def validate(self, rx, expected): self.assertEqual(rx, expected.__class__(expected)) @@ -475,7 +484,7 @@ class TestIPIP(VppTestCase): def test_mipip4(self): """ p2mp IPv4 tunnel Tests """ - for itf in self.pg_interfaces: + for itf in self.pg_interfaces[:2]: # # one underlay nh for each overlay/tunnel peer # @@ -579,8 +588,44 @@ class TestIPIP(VppTestCase): self.assertEqual(rx[IP].dst, itf._remote_hosts[ii].ip4) rx = self.send_and_expect(self.pg0, tx_i, self.pg0) + # + # we can also send to the peer's address + # + inner = (IP(dst=teib.peer, src="5.5.5.5") / + UDP(sport=1234, dport=1234) / + Raw(b'0x44' * 100)) + tx_e = [(Ether(dst=self.pg0.local_mac, + src=self.pg0.remote_mac) / + inner) for x in range(63)] + + rxs = self.send_and_expect(self.pg0, tx_e, itf) + + # + # with all of the peers in place, swap the ip-table of + # the ipip interface + # + table = VppIpTable(self, 2) + table.add_vpp_config() + + ipip_if.unconfig_ip4() + ipip_if.set_table_ip4(self.table.table_id) + ipip_if.config_ip4() + + # + # we should still be able to reach the peers from the new table + # + inner = (IP(dst=teib.peer, src="5.5.5.5") / + UDP(sport=1234, dport=1234) / + Raw(b'0x44' * 100)) + tx_e = [(Ether(dst=self.pg0.local_mac, + src=self.pg0.remote_mac) / + inner) for x in range(63)] + + rxs = self.send_and_expect(self.pg2, tx_e, itf) + ipip_if.admin_down() ipip_if.unconfig_ip4() + ipip_if.set_table_ip4(0) class TestIPIP6(VppTestCase): diff --git a/test/vpp_ip_route.py b/test/vpp_ip_route.py index cd70b38101a..4675db5c5a6 100644 --- a/test/vpp_ip_route.py +++ b/test/vpp_ip_route.py @@ -188,16 +188,19 @@ class VppIpTable(VppObject): def __init__(self, test, table_id, - is_ip6=0): + is_ip6=0, + register=True): self._test = test self.table_id = table_id self.is_ip6 = is_ip6 + self.register = register def add_vpp_config(self): self._test.vapi.ip_table_add_del(is_add=1, table={'is_ip6': self.is_ip6, 'table_id': self.table_id}) - self._test.registry.register(self, self._test.logger) + if self.register: + self._test.registry.register(self, self._test.logger) return self def remove_vpp_config(self): -- cgit 1.2.3-korg