On Fri, Nov 26, 2021 at 8:11 AM Dumitru Ceara <dce...@redhat.com> wrote:
>
> On 11/9/21 11:11, Frode Nordahl wrote:
> > The Logical Router Port gateway_mtu option is currently only
> > documented together with implementation details in the ovn-northd
> > documentation.
> >
> > As this is CMS API we ought to document this as part of the OVN
> > Northbound Database as well.
> >
> > Reported-at: https://github.com/ovn-org/ovn/issues/78
> > Fixes: 7d42c14 ("ovn: Generate ICMPv4 packet in router pipeline for larger 
> > packets")
> > Signed-off-by: Frode Nordahl <frode.nord...@canonical.com>
> > ---
> >  ovn-nb.xml | 13 +++++++++++++
> >  1 file changed, 13 insertions(+)
> >
> > diff --git a/ovn-nb.xml b/ovn-nb.xml
> > index 036ffa64f..bb0970e94 100644
> > --- a/ovn-nb.xml
> > +++ b/ovn-nb.xml
> > @@ -2820,6 +2820,19 @@
> >            prefix according to RFC3663
> >          </p>
> >        </column>
> > +
> > +      <column name="options" key="gateway_mtu"
> > +          type='{"type": "integer", "minInteger": 68, "maxInteger": 
> > 65535}'>
> > +        <p>
> > +          If set, logical flows will be added to router pipeline to check
> > +          packet length.  If packet length is greater than the value set,
> > +          ICMPv4 type 3 (Destination Unreachable) code 4 (Fragmentation 
> > Needed
> > +          and Don't Fragment was Set) or ICMPv6 type 2 (Packet Too Big)
> > +          code 0 (no route to destination) packets will be generated.
> > +
> > +          This allows for Path MTU Discovery.
> > +        </p>
> > +      </column>
> >      </group>
> >
> >      <group title="Attachment">
> >
>
> I had also posted a patch that was also adding this missing doc section
> [0].  But your doc text looks better in my opinion, so I'll rebase my
> patch when needed.
>
> Acked-by: Dumitru Ceara <dce...@redhat.com>

Thanks.   I applied this patch to the main branch.

Numan

>
> Thanks,
> Dumitru
>
> [0]
> http://patchwork.ozlabs.org/project/ovn/patch/20211105092049.17353-1-dce...@redhat.com/
>
> _______________________________________________
> dev mailing list
> d...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to