[ovs-dev] [PATCH ovn v2 3/7] controller: Add load balancer hairpin OF flows.
Numan Siddique
numans at ovn.org
Wed Nov 4 13:51:14 UTC 2020
On Fri, Oct 30, 2020 at 2:48 AM Dumitru Ceara <dceara at redhat.com> wrote:
>
> On 10/27/20 6:18 PM, numans at ovn.org wrote:
> > From: Numan Siddique <numans at 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]?
No. The same register is used for hairpinning the traffic and for the
reverse hairpin since we have a different
table for each.
>
> > 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 <numans at 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
The output of the function - ip_address_and_port_from_key() is ip
address as char * and port as uint16_t.
We do need to anyway call ip_parse/ipv6_parse whether we has same
'struct lb_vip' for both NB and SB
or different structures.
Otherwise we should not use ip_address_and_port_from_key() at all.
Also I don't see any issues with using ip_parse here.
> --- 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?
If suppose, lb vip is invalid, then the lib/lb.c util function
ovn_lb_create() will not even
create the 'struct lb_vip' instance. Same for the backend ip.
So I think it's ok to assert here.
>
> > +
> > + 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.
Ack.
>
> > + 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.
Ack.
>
> > + }
> > +
> > + 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.
Ack.
>
> > + 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.
Same explanation as above.
>
> > +
> > + 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'?
Ack.
>
> > + 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().
Ack.
>
> > +
> > + 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.
Ack.
>
> > + 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;
> }
Ack.
>
> > +
> > + 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?
During full recompute, we don't log for consider_logical_flow() and
consider_neighbor_flow().
So I think it's ok not to log and be consistent with the other non
incremental processing scenarios.
Also it would create a lot of noise.
Thanks
Numan
>
> 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
> dev at openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
More information about the dev
mailing list