Skip to main content

Last Call Review of draft-ietf-pim-yang-12
review-ietf-pim-yang-12-opsdir-lc-schoenwaelder-2017-12-20-00

Request Review of draft-ietf-pim-yang
Requested revision No specific revision (document currently at 17)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2017-12-15
Requested 2017-12-01
Authors Xufeng Liu , Pete McAllister , Anish Peter , Mahesh Sivakumar , Yisong Liu , fangwei hu
I-D last updated 2017-12-20
Completed reviews Yangdoctors Early review of -00 by Dean Bogdanović (diff)
Yangdoctors Last Call review of -12 by Jürgen Schönwälder (diff)
Rtgdir Telechat review of -12 by Adrian Farrel (diff)
Opsdir Last Call review of -12 by Jürgen Schönwälder (diff)
Genart Last Call review of -12 by Roni Even (diff)
Secdir Last Call review of -12 by Melinda Shore (diff)
Assignment Reviewer Jürgen Schönwälder
State Completed
Request Last Call review on draft-ietf-pim-yang by Ops Directorate Assigned
Reviewed revision 12 (document currently at 17)
Result Has issues
Completed 2017-12-20
review-ietf-pim-yang-12-opsdir-lc-schoenwaelder-2017-12-20-00
I have reviewed this document both as ops-dir reviewer and as yang doctor. A
more detailed review has been submitted as part of the yang doctor review. Here
I am focusing on more general questions from an operational perspective.

- There are a number of parameters without defined defaults. Is the
  idea that every vendor augments in their defaults? Would it not
  overall be simpler if the PIM WG can find agreement on common
  defaults? (Vendors can still publish deviations I think.)

- I wonder how these YANG modules relate to the PIM MIB modules. Are
  for example counters the same or different? I think it would be good
  if the text would discuss relationship of the YANG modules relate to
  corresponding MIB modules.

-  There are no example configurations provided, demonstration how, for
  example, a simple PIM installation would be configured is not
  present in the document (e.g., as an appendix).