Hi Paolo, all, Thanks for the updates. Things that caught my eye while reading:
Sec 1: Nit: in the last of the newly added bullet points, 'BGPU PDU' is the only occurence where 'BGP Update' is (in my opinion unnecessarily) abbreviated this way. Sec 4.2.3: Type TBD4 regarding ADD-PATH: with values 0/1 representing false/true, what would it mean if this TLV is not present in the message? Note that in the implementation notes added in path-marking-tlv-00 , one of the mentioned options to implicitly signal 'no path status' is to omit the marking TLV altogether. If we opt for that, it might be nice to be consistent with it here and only attach a TBD4 of length 0 (so no value) if a route is ADD-PATH. I guess the same goes for TBD5. Cheers, luuk On Mon 23 Oct 2023, 06:38, [email protected] wrote: > Internet-Draft draft-ietf-grow-bmp-tlv-13.txt is now available. It is a work > item of the Global Routing Operations (GROW) WG of the IETF. > > Title: BMP v4: TLV support for BMP Route Monitoring and Peer Down > Messages > Authors: Paolo Lucente > Yunan Gu > Name: draft-ietf-grow-bmp-tlv-13.txt > Pages: 11 > Dates: 2023-10-23 > > Abstract: > > Most of the message types defined by the BGP Monitoring Protocol > (BMP) make provision for data in TLV format. However, Route > Monitoring messages (which provide a snapshot of the monitored > Routing Information Base) and Peer Down messages (which indicate that > a peering session was terminated) do not. Supporting (optional) data > in TLV format across all BMP message types allows for a homogeneous > and extensible surface that would be useful for the most different > use-cases that need to convey additional data to a BMP station. > While it is not intended for this document to cover any specific > utilization scenario, it defines a simple way to support TLV data in > all message types. > > The IETF datatracker status page for this Internet-Draft is: > https://datatracker.ietf.org/doc/draft-ietf-grow-bmp-tlv/ > > There is also an HTMLized version available at: > https://datatracker.ietf.org/doc/html/draft-ietf-grow-bmp-tlv-13 > > A diff from the previous version is available at: > https://author-tools.ietf.org/iddiff?url2=draft-ietf-grow-bmp-tlv-13 > > Internet-Drafts are also available by rsync at: > rsync.ietf.org::internet-drafts > > > _______________________________________________ > GROW mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/grow _______________________________________________ GROW mailing list [email protected] https://www.ietf.org/mailman/listinfo/grow
