Last Call Review of draft-ietf-bess-mvpn-extranet-04
review-ietf-bess-mvpn-extranet-04-genart-lc-holmberg-2015-12-01-00
Request | Review of | draft-ietf-bess-mvpn-extranet |
---|---|---|
Requested revision | No specific revision (document currently at 07) | |
Type | Last Call Review | |
Team | General Area Review Team (Gen-ART) (genart) | |
Deadline | 2015-12-04 | |
Requested | 2015-11-19 | |
Authors | Yakov Rekhter , Eric C. Rosen , Rahul Aggarwal , Yiqun Cai , Thomas Morin | |
I-D last updated | 2015-12-01 | |
Completed reviews |
Opsdir Last Call review of -04
by Susan Hares
(diff)
Genart Last Call review of -04 by Christer Holmberg (diff) |
|
Assignment | Reviewer | Christer Holmberg |
State | Completed | |
Request | Last Call review on draft-ietf-bess-mvpn-extranet by General Area Review Team (Gen-ART) Assigned | |
Reviewed revision | 04 (document currently at 07) | |
Result | Ready w/nits | |
Completed | 2015-12-01 |
review-ietf-bess-mvpn-extranet-04-genart-lc-holmberg-2015-12-01-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> Document: draft-ietf-bess-mvpn-extranet-04.txt Reviewer: Christer Holmberg Review Date: 1 December 2015 IETF LC End Date: 4 December 2015 IETF Telechat Date: 17 December 2015 Summary: The document is well written, and almost ready for publication. However, I have some general editorial comments that I’d like the authors to address. Major Issues: None Minor Issues: None Editorial Issues: General: QG_1: Throughout the document, there are places where the text say “We follow”, “We say”, etc. I suggest to talk about the “document” instead of “We”. I also wonder whether all the “We says are needed. QG_2: The Abstract says that the document updates RFCs 6513, 6514, and 6625. However, there is no dedicated section(s) which defines the updates. I also think the Introduction should contain some general overview text on what is updated. If possible, it would also be good to have dedicated “Updates to RFC XXXX” chapters, so that people can easily find what exactly has been updated. QG_3: The Abstract and Introduction say “Previous RFCs”. I suggest to list the relevant RFCs instead. Something like: “ RFC 6513 and RFC6514 specify the procedures…”