Last Call Review of draft-ietf-bess-mvpn-bidir-ingress-replication-02
review-ietf-bess-mvpn-bidir-ingress-replication-02-opsdir-lc-kumari-2015-10-19-00
Request | Review of | draft-ietf-bess-mvpn-bidir-ingress-replication |
---|---|---|
Requested revision | No specific revision (document currently at 04) | |
Type | IETF Last Call Review | |
Team | Ops Directorate (opsdir) | |
Deadline | 2015-10-13 | |
Requested | 2015-09-28 | |
Authors | Zhaohui (Jeffrey) Zhang , Yakov Rekhter , Andrew Dolganow | |
I-D last updated | 2016-01-22 (Latest revision 2015-10-16) | |
Completed reviews |
Genart IETF Last Call review of -02
by Brian E. Carpenter
(diff)
Genart Telechat review of -03 by Brian E. Carpenter (diff) Opsdir IETF Last Call review of -02 by Warren Kumari (diff) |
|
Assignment | Reviewer | Warren Kumari |
State | Completed | |
Request | IETF Last Call review on draft-ietf-bess-mvpn-bidir-ingress-replication by Ops Directorate Assigned | |
Reviewed revision | 02 (document currently at 04) | |
Result | Has nits | |
Completed | 2015-10-19 |
review-ietf-bess-mvpn-bidir-ingress-replication-02-opsdir-lc-kumari-2015-10-19-00
Be ye not afraid. I have reviewed this document as part of the Operational directorate's ongoing effort to review all IETF documents being processed by the IESG. These comments were written with the intent of improving the operational aspects of the IETF drafts. Comments that are not addressed in last call may be included in AD reviews during the IESG review. Document editors and WG chairs should treat these comments just like any other last call comments. Document reviewed: draft-ietf-bess-mvpn-bidir-ingress-replication-03 Summary: Ready, typo / grammar nits. General: I found this document to be very dense. At some point my brain started dribbling out my ears and I largely gave up on trying to understand the mechanism. This seems to be very much a niche application, and my IP multicast and deep MPLS knowledge isn't up to the level of finding issues with the logic. Nits: Abstract: This document specifiess how [O] specifiess [P] specifies [R] spelling 1: Introduction: With these two methods, all PEs of a particular VPN are separated [O] all PEs [P] all PEs (Provider Edge router) [R] first use of acronym. The document does say it assumed familiarity with terminology from [RFC5015], [RFC6513], [RFC6514], and [RFC7582], but this is before that, and also expanding PE here will help people understand if they want to continue reading... W -- I don't think the execution is relevant when it was obviously a bad idea in the first place. This is like putting rabid weasels in your pants, and later expressing regret at having chosen those particular rabid weasels and that pair of pants. ---maf