Last Call Review of draft-ietf-l3vpn-mvpn-mldp-nlri-06
review-ietf-l3vpn-mvpn-mldp-nlri-06-genart-lc-shirazipour-2014-10-28-00
| Request | Review of | draft-ietf-l3vpn-mvpn-mldp-nlri |
|---|---|---|
| Requested revision | No specific revision (document currently at 10) | |
| Type | Last Call Review | |
| Team | General Area Review Team (Gen-ART) (genart) | |
| Deadline | 2014-10-27 | |
| Requested | 2014-10-16 | |
| Authors | IJsbrand Wijnands , Eric C. Rosen , Uwe Joorde | |
| I-D last updated | 2014-10-28 | |
| Completed reviews |
Genart Last Call review of -06
by Meral Shirazipour
(diff)
Genart Telechat review of -07 by Meral Shirazipour (diff) Secdir Last Call review of -06 by Shawn M Emery (diff) Opsdir Last Call review of -06 by Susan Hares (diff) |
|
| Assignment | Reviewer | Meral Shirazipour |
| State | Completed | |
| Request | Last Call review on draft-ietf-l3vpn-mvpn-mldp-nlri by General Area Review Team (Gen-ART) Assigned | |
| Reviewed revision | 06 (document currently at 10) | |
| Result | Ready w/nits | |
| Completed | 2014-10-28 |
review-ietf-l3vpn-mvpn-mldp-nlri-06-genart-lc-shirazipour-2014-10-28-00
I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq . Please resolve these comments along with any other Last Call comments you may receive. Document: draft-ietf-l3vpn-mvpn-mldp-nlri-06 Reviewer: Meral Shirazipour Review Date: 2014-10-27 IETF LC End Date: 2014-10-27 IESG Telechat date: NA Summary: This draft is ready to be published as Standards Track RFC but I have some comments . Nits/editorial comments: Nits: -[Page 3] Section 1, "an SP may"---->"a SP may" -[Page 3] Section 1," "Subsequent Address Family" (SAFI)"---->" "Subsequent Address Family Identifier" (SAFI)" -[Page 3] Section 1, "Forwarding Equivalance"---->"Forwarding Equivalence" -[Page 5] PMSI first use not spelled out (P-Multicast Service Interface) Tunnel -[Page 6], "of these route"---->"of these routes" or "of these route types" -[Page 8], "the the NLRI"---->"the NLRI" -[Page 8] Range 0x41-0x7f, why not Range 0x40-0x7f ? 0x40-0x42 is specified as reserved. (should it be Range 0x43-0x7f ?) -[Page 11] References, [BGP-ERR] has a new revision (rfc editor should review the work in progress versions) Best Regards, Meral --- Meral Shirazipour Ericsson Research www.ericsson.com