[ovs-dev] [PATCH ovn] Correctly document maximum tunnel key for port bindings.

Dumitru Ceara dceara at redhat.com
Thu Apr 9 21:36:22 UTC 2020


On 4/9/20 9:10 PM, Ben Pfaff wrote:
> Only the lower half of the 16-bit range is available for individual
> port bindings.  The upper half of the range is reserved for multicast
> groups.
> 
> Signed-off-by: Ben Pfaff <blp at ovn.org>

Looks good to me.

Acked-by: Dumitru Ceara <dceara at redhat.com>

Thanks,
Dumitru

> ---
>  ovn-nb.xml | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/ovn-nb.xml b/ovn-nb.xml
> index 9d6d00c1e365..5f6cd1031672 100644
> --- a/ovn-nb.xml
> +++ b/ovn-nb.xml
> @@ -1259,7 +1259,7 @@
>  
>      <group title="Tunnel Key">
>        <column name="options" key="requested-tnl-key"
> -          type='{"type": "integer", "minInteger": 1, "maxInteger": 65535}'>
> +          type='{"type": "integer", "minInteger": 1, "maxInteger": 32767}'>
>          Configures the port binding tunnel key for the port.  Usually
>          this is not needed because <code>ovn-northd</code> will assign an
>          unique key for each port by itself.  However, if it is configured,
> @@ -2299,7 +2299,7 @@
>        </column>
>  
>        <column name="options" key="requested-tnl-key"
> -          type='{"type": "integer", "minInteger": 1, "maxInteger": 65535}'>
> +          type='{"type": "integer", "minInteger": 1, "maxInteger": 32767}'>
>          Configures the port binding tunnel key for the port.  Usually
>          this is not needed because <code>ovn-northd</code> will assign an
>          unique key for each port by itself.  However, if it is configured,
> 



More information about the dev mailing list