Skip to main content

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 revision No specific revision (document currently at 12)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2019-08-28
Requested 2019-08-14
Authors Dhruv Dhody , Rakesh Gandhi , Udayasree Palle , Ravi Singh , Luyuan Fang
I-D 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 Fox Franke (diff)
Genart Last Call review of -10 by Erik Kline (diff)
Tsvart Last Call review of -10 by David L. Black (diff)
Opsdir Last Call review of -10 by Joe Clarke (diff)
Tsvart Telechat review of -11 by David L. Black (diff)
Assignment Reviewer Joe Clarke
State Completed
Request Last Call review on draft-ietf-pce-stateful-pce-auto-bandwidth by Ops Directorate Assigned
Posted at https://mailarchive.ietf.org/arch/msg/ops-dir/ouA5StWH7dlE2jtDRGlzPeOFn9Q
Reviewed revision 10 (document currently at 12)
Result Ready
Completed 2019-08-26
review-ietf-pce-stateful-pce-auto-bandwidth-10-opsdir-lc-clarke-2019-08-26-00
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.