Re: [GROW] I-D Action: draft-ietf-grow-bmp-tlv-ebit-04.txt

Hi Paolo, all,

Thanks for the update. I have a question about something that was actually
already part of the previous version:

Sec 3.2:

In a non-indexed enterprise-specific TLV, the Length includes the PEN, which
I guess helps in making it backward compatible.
For indexed TLVs, the Length does not include the Index (Sec3 in bmp-tlv-13),
but what about indexed enterprise-specific TLVs? Does the length include the
length of the PEN, but not the Index?

If indeed the PEN is included and the Index is not, would it make sense to swap
the Index and PEN fields? Feels more consistent to interpret/process to me, but
perhaps I'm missing any downsides of such a reordering.

Either way, I think the text should clarify what the Length is comprised of in
indexed enterprise TLVs.

cheers,
 luuk

On Mon 23 Oct 2023, 06:38, internet-drafts@ietf.org wrote:
> Internet-Draft draft-ietf-grow-bmp-tlv-ebit-04.txt is now available. It is a
> work item of the Global Routing Operations (GROW) WG of the IETF.
> 
>    Title:   Support for Enterprise-specific TLVs in the BGP Monitoring Protocol
>    Authors: Paolo Lucente
>             Yunan Gu
>    Name:    draft-ietf-grow-bmp-tlv-ebit-04.txt
>    Pages:   7
>    Dates:   2023-10-23
> 
> Abstract:
> 
>    Message types defined by the BGP Monitoring Protocol (BMP) do
>    provision for data in TLV - Type, Length, Value - format, either in
>    the shape of a TLV message body, ie.  Route Mirroring and Stats
>    Reports, or optional TLVs at the end of a BMP message, ie.  Peer Up
>    and Peer Down.  However the space for Type value is unique and
>    governed by IANA.  To allow the usage of vendor-specific TLVs, a
>    mechanism to define per-vendor Type values is required.  In this
>    document we introduce an Enterprise Bit, or E-bit, for such purpose.
> 
> The IETF datatracker status page for this Internet-Draft is:
> https://datatracker.ietf.org/doc/draft-ietf-grow-bmp-tlv-ebit/
> 
> There is also an HTMLized version available at:
> https://datatracker.ietf.org/doc/html/draft-ietf-grow-bmp-tlv-ebit-04
> 
> A diff from the previous version is available at:
> https://author-tools.ietf.org/iddiff?url2=draft-ietf-grow-bmp-tlv-ebit-04
> 
> Internet-Drafts are also available by rsync at:
> rsync.ietf.org::internet-drafts
> 
> 
> _______________________________________________
> GROW mailing list
> GROW@ietf.org
> https://www.ietf.org/mailman/listinfo/grow