Last Call Review of draft-ietf-pce-pcep-yang-26
review-ietf-pce-pcep-yang-26-tsvart-lc-scharf-2024-11-12-00

This document has been reviewed as part of the transport area review team's
ongoing effort to review key IETF documents. These comments were written
primarily for the transport area directors, but are copied to the document's
authors and WG to allow them to address any issues raised and also to the IETF
discussion list for information.

When done at the time of IETF Last Call, the authors should consider this
review as part of the last-call comments they receive. Please always CC
tsv-art@ietf.org if you reply to or forward this review.

This document specifies a YANG data model for PCEP as application protocol.
This document is well-written and I have not found any specific issues related
to transport protocols.

However, I'd like to note that it is quite hard to review in detail a complex
YANG data model that has apparently not been implemented and tested at all.

From a high-level point of view, there may be two nits:

1. It is not clear to me why Section 3.3 is needed.

2. Some design choices for data types in the YANG model are not really obvious.
For instance, "init-back-off-time" is uint16, but "max-back-off-timer" is
uint32. There are also quite a number of timer limites defined only as unit8.
In some cases, the chosen value ranges seem to originate from PCEP standard,
but I haven't been able to track down some other ones (well, as a
non-PCEP-expert). Similarly, it is hard to know if "counter32" will be
sufficient for some stat values.

Michael