Last Call Review of draft-ietf-pce-stateful-pce-auto-bandwidth-10
review-ietf-pce-stateful-pce-auto-bandwidth-10-opsdir-lc-clarke-2019-08-26-00

Request Review of draft-ietf-pce-stateful-pce-auto-bandwidth
Requested rev. no specific revision (document currently at 11)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2019-08-28
Requested 2019-08-14
Draft last updated 2019-08-26
Completed reviews Rtgdir Last Call review of -09 by Jonathan Hardwick (diff)
Secdir Last Call review of -10 by Daniel Franke (diff)
Genart Last Call review of -10 by Erik Kline (diff)
Tsvart Last Call review of -10 by David Black (diff)
Opsdir Last Call review of -10 by Joe Clarke (diff)
Tsvart Telechat review of -11 by David Black
Assignment Reviewer Joe Clarke
State Completed
Review review-ietf-pce-stateful-pce-auto-bandwidth-10-opsdir-lc-clarke-2019-08-26
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/ouA5StWH7dlE2jtDRGlzPeOFn9Q
Reviewed rev. 10 (document currently at 11)
Review result Ready
Review completed: 2019-08-26

Review
review-ietf-pce-stateful-pce-auto-bandwidth-10-opsdir-lc-clarke-2019-08-26

I have been requested to review this draft on behalf of the OPS Directorate.  This draft describes PCEP extensions for MPLS-TE LSP automatic bandwidth adjustment with stateful PCE.  In general, I think the draft is well-written, and I appreciate the addition of the operational considerations section.  It is in that section I have a couple of comments.

In section 6.2 you say: 

A Management Information Base (MIB) module for modeling PCEP is
described in [RFC7420].  However, one may prefer the mechanism for
configuration using YANG data model [I-D.ietf-pce-pcep-yang].  

However, when you look at that MIB module, there is only one read-write object.  On top of that, the IESG has mandated that new MIB modules should not have read-write objects.  I think your language pointing one to the YANG module should be stronger.  Perhaps:

A MIB module for gathering operational information about PCEP is defined in [RFC7420].  Additionally, the YANG module defined in [I-D.ietf-pce-pcep-yang]  provides for both configuration of PCEP as well as operational management.

===

In section 6.6, do you have any more concrete recommendations on a reasonable limit of LSPs with auto-bandwidth that you have discovered from testing or operational experience?  Providing some data here may prove useful, even if it is somewhat anecdotal.