On Tue, Mar 07, 2023 at 02:19:40PM +0100, Lorenzo Bianconi wrote:
> > On Tue, Mar 07, 2023 at 11:57:16AM +0100, Lorenzo Bianconi wrote:
> > > Drop ip packets with ct status set to invalid in post snat and
> > > lb_aff_learn router stages.
> > > Skip ICMPv{4,6} error messages packet in ct.inv rules in order to avoid
> > > to introduce too complicated code.
> > > 
> > > Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2160685
> > > Signed-off-by: Lorenzo Bianconi <[email protected]>
> > 
> > ...
> > 
> > > diff --git a/northd/ovn-northd.8.xml b/northd/ovn-northd.8.xml
> > > index 2eab2c4ae..cdf9373a6 100644
> > > --- a/northd/ovn-northd.8.xml
> > > +++ b/northd/ovn-northd.8.xml
> > 
> > ...
> > 
> > > @@ -5070,7 +5109,7 @@ clone {
> > >        </li>
> > >      </ul>
> > >  
> > > -    <h3>Egress Table 5: Delivery</h3>
> > > +    <h3>Egress Table 6: Delivery</h3>
> > >  
> > >      <p>
> > >        Packets that reach this table are ready for delivery.  It contains:
> > 
> > Can I clarify that the change above is intentional?
> 
> yep, because we are missing "Post SNAT" egress table here, so we need to shift
> the following ones.

Thanks, I see that now.

Reviewed-by: Simon Horman <[email protected]>


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to