On Thu, Mar 09, 2023 at 03:11:33PM +0100, Lorenzo Bianconi wrote: > Fix non-tcp hairpin use-case if the load-balancer is configured without > ports. > > Tested-by: Ying Xu <[email protected]> > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2157846
Is a fixes tag appropriate here? > Signed-off-by: Lorenzo Bianconi <[email protected]> Reviewed-by: Simon Horman <[email protected]> > --- > controller/lflow.c | 19 ++++++-- > tests/ovn.at | 115 +++++++++++++++++++++++++++++++++++++++++++++ > 2 files changed, 129 insertions(+), 5 deletions(-) > > diff --git a/controller/lflow.c b/controller/lflow.c > index 003195ae4..023a30860 100644 > --- a/controller/lflow.c > +++ b/controller/lflow.c > @@ -1729,6 +1729,7 @@ add_lb_vip_hairpin_flows(const struct ovn_controller_lb > *lb, > > static void > add_lb_ct_snat_hairpin_for_dp(const struct ovn_controller_lb *lb, > + const struct ovn_lb_vip *lb_vip, nit: I suppose vip_port could be passed rather than all of lb_vip. > const struct sbrec_datapath_binding *datapath, > const struct hmap *local_datapaths, > struct match *dp_match, ... _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
