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).