On Thu, May 16, 2019 at 6:42 PM Lorenzo Bianconi < [email protected]> wrote:
> Run local logical flows first if the gw router port is scheduled > on the local chassis in order to properly manage snat traffic > > Tested-by: Eran Kuris <[email protected]> > Signed-off-by: Lorenzo Bianconi <[email protected]> > Can you please update the ovn-northd documentation as well ? Otherwise the patch LGTM. Thanks Numan > --- > ovn/northd/ovn-northd.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/ovn/northd/ovn-northd.c b/ovn/northd/ovn-northd.c > index de0c06d4b..4348eb281 100644 > --- a/ovn/northd/ovn-northd.c > +++ b/ovn/northd/ovn-northd.c > @@ -6566,6 +6566,7 @@ build_lrouter_flows(struct hmap *datapaths, struct > hmap *ports, > count_1bits(ntohl(mask)) + 1, > ds_cstr(&match), ds_cstr(&actions)); > } else { > + uint16_t priority = count_1bits(ntohl(mask)) + 1; > /* Distributed router. */ > ds_clear(&match); > ds_put_format(&match, "ip && ip4.src == %s" > @@ -6575,6 +6576,7 @@ build_lrouter_flows(struct hmap *datapaths, struct > hmap *ports, > if (!distributed && od->l3redirect_port) { > /* Flows for NAT rules that are centralized are > only > * programmed on the "redirect-chassis". */ > + priority += 128; > ds_put_format(&match, " && > is_chassis_resident(%s)", > od->l3redirect_port->json_key); > } > @@ -6589,8 +6591,8 @@ build_lrouter_flows(struct hmap *datapaths, struct > hmap *ports, > * nat->logical_ip with the longest mask gets a higher > * priority. */ > ovn_lflow_add(lflows, od, S_ROUTER_OUT_SNAT, > - count_1bits(ntohl(mask)) + 1, > - ds_cstr(&match), ds_cstr(&actions)); > + priority, ds_cstr(&match), > + ds_cstr(&actions)); > } > } > > -- > 2.20.1 > > _______________________________________________ > dev mailing list > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
