On 10/27/20 6:18 PM, num...@ovn.org wrote: > From: Numan Siddique <num...@ovn.org> >
Hi Numan, > Presently to handle the load balancer hairpin traffic (the traffic destined > to the > load balancer VIP is dnatted to the backend which originated the traffic), > ovn-northd > adds a lot of logical flows to check this scenario. This patch attempts to > reduce the > these logical flows. Each ovn-controller will read the load balancers from > the newly added southbound Load_Balancer table and adds the load balancer > hairpin OF > flows in the table 68, 69 and 70. If suppose a below load balancer is > configured > > 10.0.0.10:80 = 10.0.0.4:8080, 10.0.0.5:8090, then the below flows are added > > table=68, ip.src = 10.0.0.4,ip.dst=10.0.0.4,tcp.dst=8080 > actions=load:1->NXM_NX_REG9[7] > table=68, ip.src = 10.0.0.5,ip.dst=10.0.0.5,tcp.dst=8090 > actions=load:1->NXM_NX_REG9[7] > table=69, ip.src = 10.0.0.4,ip.dst=10.0.0.10,tcp.src=8080 > actions=load:1->NXM_NX_REG9[7] > table=69, ip.src = 10.0.0.5,ip.dst=10.0.0.10,tcp.src=8090 > actions=load:1->NXM_NX_REG9[7] Shouldn't this be REG10[7]? > table=70, ct.trk && ct.dnat && ct.nw_dst == 10.0.0.10. actions=ct(commit, > zone=reg12, nat(src=10.0.0.5)) > > Upcoming patch will add OVN actions which does the lookup in these tables to > handle the > hairpin traffic. > > Signed-off-by: Numan Siddique <num...@ovn.org> > --- > controller/lflow.c | 253 +++++++++++++++++++ > controller/lflow.h | 6 +- > controller/ovn-controller.c | 27 +- > include/ovn/logical-fields.h | 3 + > tests/ovn.at | 469 +++++++++++++++++++++++++++++++++++ > 5 files changed, 756 insertions(+), 2 deletions(-) > > diff --git a/controller/lflow.c b/controller/lflow.c > index f631679c3f..657482626d 100644 > --- a/controller/lflow.c > +++ b/controller/lflow.c > @@ -26,6 +26,7 @@ > #include "ovn-controller.h" > #include "ovn/actions.h" > #include "ovn/expr.h" > +#include "lib/lb.h" > #include "lib/ovn-l7.h" > #include "lib/ovn-sb-idl.h" > #include "lib/extend-table.h" > @@ -1138,6 +1139,213 @@ add_neighbor_flows(struct ovsdb_idl_index > *sbrec_port_binding_by_name, > } > } > > +static void > +add_lb_vip_hairpin_flows(struct ovn_lb *lb, struct lb_vip *lb_vip, > + struct lb_vip_backend *lb_backend, > + uint8_t lb_proto, > + struct ovn_desired_flow_table *flow_table) > +{ > + uint64_t stub[1024 / 8]; > + struct ofpbuf ofpacts = OFPBUF_STUB_INITIALIZER(stub); > + > + uint8_t value = 1; > + put_load(&value, sizeof value, MFF_LOG_FLAGS, > + MLF_LOOKUP_LB_HAIRPIN_BIT, 1, &ofpacts); > + > + ovs_be32 vip4; > + struct in6_addr vip6; > + > + if (lb_vip->addr_family == AF_INET) { > + ovs_assert(ip_parse(lb_vip->vip, &vip4)); > + } else { > + ovs_assert(ipv6_parse(lb_vip->vip, &vip6)); > + } > + > + struct match hairpin_match = MATCH_CATCHALL_INITIALIZER; > + struct match hairpin_reply_match = MATCH_CATCHALL_INITIALIZER; > + > + if (lb_vip->addr_family == AF_INET) { > + ovs_be32 ip4; > + ovs_assert(ip_parse(lb_backend->ip, &ip4)); Because we share the same ovn_lb structure between NB and SB we're now forced to parse the backend IP twice: ovn_sb_lb_create() - ovn_lb_create() -- ip_address_and_port_from_key() --- this parses the string into a sockaddr_storage --- then builds an IP string and extracts the port Here we reparse the IP string into an ovs_be32. If the structs were independent for NB/SB I think we would pottentially have simpler code and we would avoid reparsing IPs. Also, what if the SB DB gets corrupted, do we really want ovn-controller to abort? > + > + match_set_dl_type(&hairpin_match, htons(ETH_TYPE_IP)); > + match_set_nw_src(&hairpin_match, ip4); > + match_set_nw_dst(&hairpin_match, ip4); > + > + match_set_dl_type(&hairpin_reply_match, > + htons(ETH_TYPE_IP)); > + match_set_nw_src(&hairpin_reply_match, ip4); > + match_set_nw_dst(&hairpin_reply_match, vip4); > + } else { > + struct in6_addr ip6; > + ovs_assert(ipv6_parse(lb_backend->ip, &ip6)); Same here. > + > + match_set_dl_type(&hairpin_match, htons(ETH_TYPE_IPV6)); > + match_set_ipv6_src(&hairpin_match, &ip6); > + match_set_ipv6_dst(&hairpin_match, &ip6); > + > + match_set_dl_type(&hairpin_reply_match, > + htons(ETH_TYPE_IPV6)); Indentation is a bit odd. > + match_set_ipv6_src(&hairpin_reply_match, &ip6); > + match_set_ipv6_dst(&hairpin_reply_match, &vip6); > + } > + > + if (lb_backend->port) { > + match_set_nw_proto(&hairpin_match, lb_proto); > + match_set_tp_dst(&hairpin_match, htons(lb_backend->port)); > + > + match_set_nw_proto(&hairpin_reply_match, lb_proto); > + match_set_tp_src(&hairpin_reply_match, > + htons(lb_backend->port)); Same here. > + } > + > + for (size_t i = 0; i < lb->slb->n_datapaths; i++) { > + match_set_metadata(&hairpin_match, > + htonll(lb->slb->datapaths[i]->tunnel_key)); > + match_set_metadata(&hairpin_reply_match, > + htonll(lb->slb->datapaths[i]->tunnel_key)); > + Here too. > + ofctrl_add_flow(flow_table, OFTABLE_CHK_LB_HAIRPIN, 100, > + lb->slb->header_.uuid.parts[0], &hairpin_match, > + &ofpacts, &lb->slb->header_.uuid); > + > + ofctrl_add_flow(flow_table, OFTABLE_CHK_LB_HAIRPIN_REPLY, 100, > + lb->slb->header_.uuid.parts[0], > + &hairpin_reply_match, > + &ofpacts, &lb->slb->header_.uuid); > + } > + > + ofpbuf_uninit(&ofpacts); > +} > + > +static void > +add_lb_ct_snat_vip_flows(struct ovn_lb *lb, struct lb_vip *lb_vip, > + struct ovn_desired_flow_table *flow_table) > +{ > + ovs_be32 vip4; > + struct in6_addr vip6; > + > + if (lb_vip->addr_family == AF_INET) { > + ovs_assert(ip_parse(lb_vip->vip, &vip4)); > + } else { > + ovs_assert(ipv6_parse(lb_vip->vip, &vip6)); > + } As mentioned above for backends, I don't think it's OK to abort here if parsing fails. > + > + uint64_t stub[1024 / 8]; > + struct ofpbuf ofpacts = OFPBUF_STUB_INITIALIZER(stub); > + > + struct ofpact_conntrack *ct = ofpact_put_CT(&ofpacts); > + ct->recirc_table = NX_CT_RECIRC_NONE; > + ct->zone_src.field = mf_from_id(MFF_LOG_SNAT_ZONE); > + ct->zone_src.ofs = 0; > + ct->zone_src.n_bits = 16; > + ct->flags = 0; Shouldn't this be 'ct->flags = NX_CT_F_COMMIT'? > + ct->alg = 0; > + > + size_t nat_offset; > + nat_offset = ofpacts.size; > + ofpbuf_pull(&ofpacts, nat_offset); > + > + struct ofpact_nat *nat = ofpact_put_NAT(&ofpacts); > + nat->flags = NX_NAT_F_SRC; > + nat->range_af = AF_UNSPEC; > + if (lb_vip->addr_family == AF_INET) { > + nat->range_af = AF_INET; > + nat->range.addr.ipv4.min = vip4; > + } else { > + nat->range_af = AF_INET6; > + nat->range.addr.ipv6.min = vip6; > + } > + ofpacts.header = ofpbuf_push_uninit(&ofpacts, nat_offset); > + ct = ofpacts.header; > + ct->flags |= NX_CT_F_COMMIT; Setting the flags here is not needed anymore if we set the value when we ofpact_put_CT(). > + > + ofpact_finish(&ofpacts, &ct->ofpact); > + > + struct match match = MATCH_CATCHALL_INITIALIZER; > + if (lb_vip->addr_family == AF_INET) { > + match_set_dl_type(&match, htons(ETH_TYPE_IP)); > + match_set_ct_nw_dst(&match, vip4); > + } else { > + match_set_dl_type(&match, htons(ETH_TYPE_IPV6)); > + match_set_ct_ipv6_dst(&match, &vip6); > + } > + > + uint32_t ct_state = OVS_CS_F_TRACKED | OVS_CS_F_DST_NAT; Nit: extra space. > + match_set_ct_state_masked(&match, ct_state, ct_state); > + > + for (size_t i = 0; i < lb->slb->n_datapaths; i++) { > + match_set_metadata(&match, > + htonll(lb->slb->datapaths[i]->tunnel_key)); > + > + ofctrl_add_flow(flow_table, OFTABLE_CT_SNAT_FOR_VIP, 100, > + lb->slb->header_.uuid.parts[0], > + &match, &ofpacts, &lb->slb->header_.uuid); > + } > + > + ofpbuf_uninit(&ofpacts); > +} > + > +static void > +consider_lb_hairpin_flows(const struct sbrec_load_balancer *sbrec_lb, > + const struct hmap *local_datapaths, > + struct ovn_desired_flow_table *flow_table) > +{ > + bool consider_lb = false; > + > + /* Check if we need to add flows or not. If there is one datapath > + * in the local_datapaths, it means all the datapaths of the lb > + * will be in the local_datapaths. */ > + for (size_t i = 0; i < sbrec_lb->n_datapaths; i++) { > + if (get_local_datapath(local_datapaths, > + sbrec_lb->datapaths[i]->tunnel_key)) { > + consider_lb = true; > + break; > + } > + } > + > + if (!consider_lb) { > + return; > + } Nit: we could avoid 'consider_lb' by: if (i == sbrec_lb->n_datapaths) { return; } > + > + struct ovn_lb *lb = ovn_sb_lb_create(sbrec_lb); > + uint8_t lb_proto = IPPROTO_TCP; > + if (lb->slb->protocol && lb->slb->protocol[0]) { > + if (!strcmp(lb->slb->protocol, "udp")) { > + lb_proto = IPPROTO_UDP; > + } else if (!strcmp(lb->slb->protocol, "sctp")) { > + lb_proto = IPPROTO_SCTP; > + } > + } > + > + for (size_t i = 0; i < lb->n_vips; i++) { > + struct lb_vip *lb_vip = &lb->vips[i]; > + > + for (size_t j = 0; j < lb_vip->n_backends; j++) { > + struct lb_vip_backend *lb_backend = &lb_vip->backends[j]; > + add_lb_vip_hairpin_flows(lb, lb_vip, lb_backend, lb_proto, > + flow_table); > + } > + > + add_lb_ct_snat_vip_flows(lb, lb_vip, flow_table); > + } > + > + ovn_lb_destroy(lb); > +} > + > +/* Adds OpenFlow flows to flow tables for each Load balancer VIPs and > + * backends to handle the load balanced hairpin traffic. */ > +static void > +add_lb_hairpin_flows(const struct sbrec_load_balancer_table *lb_table, > + const struct hmap *local_datapaths, > + struct ovn_desired_flow_table *flow_table) > +{ > + const struct sbrec_load_balancer *lb; > + SBREC_LOAD_BALANCER_TABLE_FOR_EACH (lb, lb_table) { > + consider_lb_hairpin_flows(lb, local_datapaths, flow_table); > + } > +} > + > /* Handles neighbor changes in mac_binding table. */ > void > lflow_handle_changed_neighbors( > @@ -1197,6 +1405,8 @@ lflow_run(struct lflow_ctx_in *l_ctx_in, struct > lflow_ctx_out *l_ctx_out) > add_neighbor_flows(l_ctx_in->sbrec_port_binding_by_name, > l_ctx_in->mac_binding_table, > l_ctx_in->local_datapaths, > l_ctx_out->flow_table); > + add_lb_hairpin_flows(l_ctx_in->lb_table, l_ctx_in->local_datapaths, > + l_ctx_out->flow_table); > } > > void > @@ -1256,6 +1466,15 @@ lflow_add_flows_for_datapath(const struct > sbrec_datapath_binding *dp, > dhcp_opts_destroy(&dhcpv6_opts); > nd_ra_opts_destroy(&nd_ra_opts); > controller_event_opts_destroy(&controller_event_opts); > + > + /* Add load balancer hairpin flows if the datapath has any load balancers > + * associated. */ > + for (size_t i = 0; i < dp->n_load_balancers; i++) { > + consider_lb_hairpin_flows(dp->load_balancers[i], > + l_ctx_in->local_datapaths, > + l_ctx_out->flow_table); > + } > + > return handled; > } > > @@ -1273,3 +1492,37 @@ lflow_handle_flows_for_lport(const struct > sbrec_port_binding *pb, > return lflow_handle_changed_ref(REF_TYPE_PORTBINDING, pb_ref_name, > l_ctx_in, l_ctx_out, &changed); > } > + > +bool > +lflow_handle_changed_lbs(struct lflow_ctx_in *l_ctx_in, > + struct lflow_ctx_out *l_ctx_out) > +{ > + const struct sbrec_load_balancer *lb; > + > + SBREC_LOAD_BALANCER_TABLE_FOR_EACH_TRACKED (lb, l_ctx_in->lb_table) { > + if (sbrec_load_balancer_is_deleted(lb)) { > + VLOG_DBG("Remove hairpin flows for deleted load balancer > "UUID_FMT, > + UUID_ARGS(&lb->header_.uuid)); > + ofctrl_remove_flows(l_ctx_out->flow_table, &lb->header_.uuid); > + } > + } > + > + SBREC_LOAD_BALANCER_TABLE_FOR_EACH_TRACKED (lb, l_ctx_in->lb_table) { > + if (sbrec_load_balancer_is_deleted(lb)) { > + continue; > + } > + > + if (!sbrec_load_balancer_is_new(lb)) { > + VLOG_DBG("Remove hairpin flows for updated load balancer > "UUID_FMT, > + UUID_ARGS(&lb->header_.uuid)); > + ofctrl_remove_flows(l_ctx_out->flow_table, &lb->header_.uuid); > + } > + > + VLOG_DBG("Add load balancer hairpin flows for "UUID_FMT, > + UUID_ARGS(&lb->header_.uuid)); For consistency, shouldn't we add logs for the non incremental processing case too? Thanks, Dumitru > + consider_lb_hairpin_flows(lb, l_ctx_in->local_datapaths, > + l_ctx_out->flow_table); > + } > + > + return true; > +} > diff --git a/controller/lflow.h b/controller/lflow.h > index 1251fb0f45..1225131deb 100644 > --- a/controller/lflow.h > +++ b/controller/lflow.h > @@ -68,6 +68,9 @@ struct uuid; > #define OFTABLE_LOG_TO_PHY 65 > #define OFTABLE_MAC_BINDING 66 > #define OFTABLE_MAC_LOOKUP 67 > +#define OFTABLE_CHK_LB_HAIRPIN 68 > +#define OFTABLE_CHK_LB_HAIRPIN_REPLY 69 > +#define OFTABLE_CT_SNAT_FOR_VIP 70 > > /* The number of tables for the ingress and egress pipelines. */ > #define LOG_PIPELINE_LEN 24 > @@ -132,6 +135,7 @@ struct lflow_ctx_in { > const struct sbrec_logical_flow_table *logical_flow_table; > const struct sbrec_multicast_group_table *mc_group_table; > const struct sbrec_chassis *chassis; > + const struct sbrec_load_balancer_table *lb_table; > const struct hmap *local_datapaths; > const struct shash *addr_sets; > const struct shash *port_groups; > @@ -160,7 +164,7 @@ void lflow_handle_changed_neighbors( > const struct sbrec_mac_binding_table *, > const struct hmap *local_datapaths, > struct ovn_desired_flow_table *); > - > +bool lflow_handle_changed_lbs(struct lflow_ctx_in *, struct lflow_ctx_out *); > void lflow_destroy(void); > > void lflow_cache_init(struct hmap *); > diff --git a/controller/ovn-controller.c b/controller/ovn-controller.c > index a06cae3ccb..4150b4cb1c 100644 > --- a/controller/ovn-controller.c > +++ b/controller/ovn-controller.c > @@ -790,7 +790,8 @@ ctrl_register_ovs_idl(struct ovsdb_idl *ovs_idl) > SB_NODE(logical_flow, "logical_flow") \ > SB_NODE(dhcp_options, "dhcp_options") \ > SB_NODE(dhcpv6_options, "dhcpv6_options") \ > - SB_NODE(dns, "dns") > + SB_NODE(dns, "dns") \ > + SB_NODE(load_balancer, "load_balancer") > > enum sb_engine_node { > #define SB_NODE(NAME, NAME_STR) SB_##NAME, > @@ -1682,6 +1683,10 @@ static void init_lflow_ctx(struct engine_node *node, > (struct sbrec_multicast_group_table *)EN_OVSDB_GET( > engine_get_input("SB_multicast_group", node)); > > + struct sbrec_load_balancer_table *lb_table = > + (struct sbrec_load_balancer_table *)EN_OVSDB_GET( > + engine_get_input("SB_load_balancer", node)); > + > const char *chassis_id = chassis_get_id(); > const struct sbrec_chassis *chassis = NULL; > struct ovsdb_idl_index *sbrec_chassis_by_name = > @@ -1713,6 +1718,7 @@ static void init_lflow_ctx(struct engine_node *node, > l_ctx_in->logical_flow_table = logical_flow_table; > l_ctx_in->mc_group_table = multicast_group_table; > l_ctx_in->chassis = chassis; > + l_ctx_in->lb_table = lb_table; > l_ctx_in->local_datapaths = &rt_data->local_datapaths; > l_ctx_in->addr_sets = addr_sets; > l_ctx_in->port_groups = port_groups; > @@ -2131,6 +2137,23 @@ flow_output_runtime_data_handler(struct engine_node > *node, > return true; > } > > +static bool > +flow_output_sb_load_balancer_handler(struct engine_node *node, void *data) > +{ > + struct ed_type_runtime_data *rt_data = > + engine_get_input_data("runtime_data", node); > + > + struct ed_type_flow_output *fo = data; > + struct lflow_ctx_in l_ctx_in; > + struct lflow_ctx_out l_ctx_out; > + init_lflow_ctx(node, rt_data, fo, &l_ctx_in, &l_ctx_out); > + > + bool handled = lflow_handle_changed_lbs(&l_ctx_in, &l_ctx_out); > + > + engine_set_node_state(node, EN_UPDATED); > + return handled; > +} > + > struct ovn_controller_exit_args { > bool *exiting; > bool *restart; > @@ -2327,6 +2350,8 @@ main(int argc, char *argv[]) > engine_add_input(&en_flow_output, &en_sb_dhcp_options, NULL); > engine_add_input(&en_flow_output, &en_sb_dhcpv6_options, NULL); > engine_add_input(&en_flow_output, &en_sb_dns, NULL); > + engine_add_input(&en_flow_output, &en_sb_load_balancer, > + flow_output_sb_load_balancer_handler); > > engine_add_input(&en_ct_zones, &en_ovs_open_vswitch, NULL); > engine_add_input(&en_ct_zones, &en_ovs_bridge, NULL); > diff --git a/include/ovn/logical-fields.h b/include/ovn/logical-fields.h > index ac6f2f909b..0fe5bc3bb4 100644 > --- a/include/ovn/logical-fields.h > +++ b/include/ovn/logical-fields.h > @@ -57,6 +57,7 @@ enum mff_log_flags_bits { > MLF_LOCAL_ONLY_BIT = 4, > MLF_NESTED_CONTAINER_BIT = 5, > MLF_LOOKUP_MAC_BIT = 6, > + MLF_LOOKUP_LB_HAIRPIN_BIT = 7, > }; > > /* MFF_LOG_FLAGS_REG flag assignments */ > @@ -88,6 +89,8 @@ enum mff_log_flags { > > /* Indicate that the lookup in the mac binding table was successful. */ > MLF_LOOKUP_MAC = (1 << MLF_LOOKUP_MAC_BIT), > + > + MLF_LOOKUP_LB_HAIRPIN = (1 << MLF_LOOKUP_LB_HAIRPIN_BIT), > }; > > /* OVN logical fields > diff --git a/tests/ovn.at b/tests/ovn.at > index 04b7a3df73..3679978612 100644 > --- a/tests/ovn.at > +++ b/tests/ovn.at > @@ -22800,3 +22800,472 @@ AT_CHECK([test "$encap_rec_mvtep" == > "$encap_rec_mvtep1"], [0], []) > > OVN_CLEANUP([hv1]) > AT_CLEANUP > + > +AT_SETUP([ovn -- Load Balancer LS hairpin OF flows]) > +ovn_start > + > +net_add n1 > + > +sim_add hv1 > +as hv1 > +ovs-vsctl add-br br-phys > +ovn_attach n1 br-phys 192.168.0.1 > +ovs-vsctl -- add-port br-int hv1-vif1 -- \ > + set interface hv1-vif1 external-ids:iface-id=sw0-p1 \ > + options:tx_pcap=hv1/vif1-tx.pcap \ > + options:rxq_pcap=hv1/vif1-rx.pcap \ > + ofport-request=1 > +ovs-vsctl -- add-port br-int hv1-vif2 -- \ > + set interface hv1-vif2 external-ids:iface-id=sw1-p1 \ > + options:tx_pcap=hv1/vif2-tx.pcap \ > + options:rxq_pcap=hv1/vif2-rx.pcap \ > + ofport-request=2 > + > +sim_add hv2 > +as hv2 > +ovs-vsctl add-br br-phys > +ovn_attach n1 br-phys 192.168.0.2 > +ovs-vsctl -- add-port br-int hv2-vif1 -- \ > + set interface hv2-vif1 external-ids:iface-id=sw0-p2 \ > + options:tx_pcap=hv2/vif1-tx.pcap \ > + options:rxq_pcap=hv2/vif1-rx.pcap \ > + ofport-request=1 > +ovs-vsctl -- add-port br-int hv1-vif2 -- \ > + set interface hv1-vif2 external-ids:iface-id=sw1-p2 \ > + options:tx_pcap=hv1/vif2-tx.pcap \ > + options:rxq_pcap=hv1/vif2-rx.pcap \ > + ofport-request=2 > + > +ovn-nbctl --wait=hv ls-add sw0 > +ovn-nbctl lsp-add sw0 sw0-p1 -- lsp-set-addresses sw0-p1 00:00:00:00:00:01 > + > +ovn-nbctl ls-add sw1 > +ovn-nbctl lsp-add sw1 sw1-p1 -- lsp-set-addresses sw1-p1 00:00:00:00:01:01 > + > +OVS_WAIT_UNTIL([test x$(ovn-nbctl lsp-get-up sw0-p1) = xup]) > +OVS_WAIT_UNTIL([test x$(ovn-nbctl lsp-get-up sw1-p1) = xup]) > + > +ovn-nbctl lb-add lb-ipv4-tcp 88.88.88.88:8080 42.42.42.1:4041 tcp > +ovn-nbctl lb-add lb-ipv4-udp 88.88.88.88:4040 42.42.42.1:2021 udp > +ovn-nbctl lb-add lb-ipv6-tcp [[8800::0088]]:8080 [[4200::1]]:4041 tcp > +ovn-nbctl --wait=hv lb-add lb-ipv6-udp [[8800::0088]]:4040 [[4200::1]]:2021 > udp > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=68], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=69], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=70], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=68], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=69], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=70], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +ovn-nbctl --wait=hv ls-lb-add sw0 lb-ipv4-tcp > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 1] > +) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8-], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8-], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8-], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=68], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=69], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=70], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +ovn-nbctl lb-add lb-ipv4-tcp 88.88.88.90:8080 > 42.42.42.42:4041,52.52.52.52:4042 tcp > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 3] > +) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8-], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=68], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=69], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=70], [0], [dnl > +NXST_FLOW reply (xid=0x8): > +]) > + > +ovn-nbctl lsp-add sw0 sw0-p2 > +# hv2 should bind sw0-p2 and it should install the LB hairpin flows. > +OVS_WAIT_UNTIL([test x$(ovn-nbctl lsp-get-up sw0-p2) = xup]) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 3] > +) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8-], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +ovn-nbctl --wait=hv ls-lb-add sw0 lb-ipv4-udp > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 4] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 4] > +) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8-], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8-], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +ovn-nbctl --wait=hv ls-lb-add sw0 lb-ipv6-tcp > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 5] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 5] > +) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +ovn-nbctl --wait=hv ls-lb-add sw0 lb-ipv6-udp > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 6] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 6] > +) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +ovn-nbctl --wait=hv ls-lb-add sw1 lb-ipv6-udp > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 7] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 7] > +) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x2,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x2,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x2 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=42.42.42.42,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=52.52.52.52,tp_dst=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x2,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=42.42.42.42,nw_dst=88.88.88.90,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp,metadata=0x1,nw_src=52.52.52.52,nw_dst=88.88.88.90,tp_src=4042 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x2,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv2 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x2 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.90,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.90)) > +]) > + > +as hv2 ovs-vsctl del-port hv2-vif1 > +OVS_WAIT_UNTIL([test x$(ovn-nbctl lsp-get-up sw0-p2) = xdown]) > + > +# Trigger recompute on hv2 as sw0 will not be cleared from local_datapaths. > +as hv2 ovn-appctl -t ovn-controller recompute > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 7] > +) > + > +ovn-nbctl --wait=hv lb-del lb-ipv4-tcp > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 4] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 0] > +) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=42.42.42.1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x2,ipv6_src=4200::1,ipv6_dst=4200::1,tp_dst=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,tcp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=4041 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp,metadata=0x1,nw_src=42.42.42.1,nw_dst=88.88.88.88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x1,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +priority=100,udp6,metadata=0x2,ipv6_src=4200::1,ipv6_dst=8800::88,tp_src=2021 > actions=load:0x1->NXM_NX_REG10[[7]] > +]) > + > +AT_CHECK([as hv1 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | cut > -d ' ' -f8- | sort], [0], [dnl > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_ipv6_dst=8800::88,ipv6,metadata=0x2 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=8800::88)) > +priority=100,ct_state=+trk+dnat,ct_nw_dst=88.88.88.88,ip,metadata=0x1 > actions=ct(commit,zone=NXM_NX_REG12[[0..15]],nat(src=88.88.88.88)) > +]) > + > +ovn-nbctl --wait=hv ls-del sw0 > +ovn-nbctl --wait=hv ls-del sw1 > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv1 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=68 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=69 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVS_WAIT_UNTIL( > + [test $(as hv2 ovs-ofctl dump-flows br-int table=70 | grep -v NXST | wc > -l) -eq 0] > +) > + > +OVN_CLEANUP([hv1], [hv2]) > +AT_CLEANUP > _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev