Last Call Review of draft-ietf-idr-tunnel-encaps-19
review-ietf-idr-tunnel-encaps-19-opsdir-lc-korhonen-2020-10-04-00
| Request | Review of | draft-ietf-idr-tunnel-encaps |
|---|---|---|
| Requested revision | No specific revision (document currently at 22) | |
| Type | Last Call Review | |
| Team | Ops Directorate (opsdir) | |
| Deadline | 2020-10-01 | |
| Requested | 2020-09-17 | |
| Authors | Keyur Patel , Gunter Van de Velde , Srihari R. Sangli , John Scudder | |
| I-D last updated | 2020-10-04 | |
| Completed reviews |
Rtgdir Last Call review of -19
by Harish Sitaraman
(diff)
Genart Last Call review of -19 by Gyan Mishra (diff) Tsvart Last Call review of -19 by Brian Trammell (diff) Opsdir Last Call review of -19 by Jouni Korhonen (diff) Secdir Last Call review of -20 by Scott G. Kelly (diff) |
|
| Assignment | Reviewer | Jouni Korhonen |
| State | Completed | |
| Request | Last Call review on draft-ietf-idr-tunnel-encaps by Ops Directorate Assigned | |
| Posted at | https://mailarchive.ietf.org/arch/msg/ops-dir/nOvu_nwGcCh5-Zm2saNEbSqHvk8 | |
| Reviewed revision | 19 (document currently at 22) | |
| Result | Has nits | |
| Completed | 2020-10-04 |
review-ietf-idr-tunnel-encaps-19-opsdir-lc-korhonen-2020-10-04-00
I find this document ready for publication. Some editorial nits and generic comments below: Editorials: * Lines 522-525: Using parenthesis around references is a bit odd looking. However, the RFC editor can decide what to do with this kind of style. The same style is used also in other places of the document, while the usual without parenthesis is also used. * There are several cases where an acronym is used before expanding it or never expanding it (such as AS). However, the RFC editor will likely take care of these. * Line 1060 opens a parenthesis and text in it but never closes it. * Line 1533 s/section/Section * Section 13 IANA considerations: - There is a separate sub-Section for RFC556 and RFC5640. However, there is no sub-Section for RFC5512, which is handles in the abstract of Section 13. Consider adding an equivalent sub-Section for RFC5512 as well. Comments: * I believe the three downrefs reported by IDnits are required for this document.