[Idr] Benjamin Kaduk's Discuss on draft-ietf-idr-te-pm-bgp-17: (with DISCUSS and COMMENT)

[Benjamin Kaduk <[e-mail]>]

Benjamin Kaduk has entered the following ballot position for
draft-ietf-idr-te-pm-bgp-17: Discuss

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-idr-te-pm-bgp/



----------------------------------------------------------------------
DISCUSS:
----------------------------------------------------------------------

How are the measurement interval(s) for these TLVs chosen?  I note that RFC
7810 (and 7810bis), as well as RFC 7471, include some text about
measurement intervals, in particular, a default value of 30 seconds and in
at least one case a requirement for configurability at sub-TLV granularity
(thus, TLV granularity for us here).  That said, it's not entirely clear to me whether
I'm supposed to treat the measurement intervals as also inherited from 7810bis/7471
as part of the "semantics of the value field".  It may be worth a brief clarifying note
in the top-level Section 2.


----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

I will be interested to see what the RFC Editor thinks about "the semantics
[...] are described in [I-D.ietf-lsr-isis-rfc7810bis] and [RFC7471]", i.e.,
with both references containing the same content.  It's unclear that we
need to have a conversation about the apparent duplication at this point,
though.


This is pretty clearly looking at the barn door when the horse is long
gone, but the referenced semantics talk about setting the A bit for when
one or more measured values "exceed a configiured maximum threshold", text
that seems to apply even to the field "min delay".  Is the "exceeds"
supposed to interpreted as "outside the expected range, whether that is
numerically larger or smaller than the threshold value"?

Section 2.8

Would it help anyone to have the TLV tag values defined in this document
in the table as well?

Section 3

Thank you for adding the extra text recommended by the secdir reviewer!