Last Call Review of draft-ietf-intarea-gre-mtu-02
review-ietf-intarea-gre-mtu-02-opsdir-lc-taylor-2015-05-08-00
Request | Review of | draft-ietf-intarea-gre-mtu |
---|---|---|
Requested revision | No specific revision (document currently at 05) | |
Type | Last Call Review | |
Team | Ops Directorate (opsdir) | |
Deadline | 2015-05-12 | |
Requested | 2015-04-19 | |
Authors | Ron Bonica , Carlos Pignataro , Dr. Joseph D. Touch | |
I-D last updated | 2015-05-08 | |
Completed reviews |
Secdir Last Call review of -02
by Taylor Yu
(diff)
Opsdir Last Call review of -02 by Tom Taylor (diff) |
|
Assignment | Reviewer | Tom Taylor |
State | Completed | |
Review |
review-ietf-intarea-gre-mtu-02-opsdir-lc-taylor-2015-05-08
|
|
Reviewed revision | 02 (document currently at 05) | |
Result | Has Nits | |
Completed | 2015-05-08 |
review-ietf-intarea-gre-mtu-02-opsdir-lc-taylor-2015-05-08-00
My apologies -- I let this slip way past due date. This is a review of operational aspects of this document, primarily for use by the OPS area ADs in their evaluation of the document. Summary: this document describes a commonly encountered set of implemented procedures for handling fragmentation of GRE packets. The described procedures include configuration options. The document is well-written and ready to go subject to the following observations, all of which are trivial except for the second minor issue noted below. Tom Taylor 1) Very minor issue: there is no advice to the operator on coordinating the configuration of the ingress and egress nodes. Section 3.3.2 assumes that configuration is coordinated (i.e., fragmented GRE delivery packets are reassembled at egress). Section 3.4 simply presents the option. This could be fixed by changing the relevant sentence of 3.3.2.1 and 3.3.2.2 as follows: OLD If the delivery packet is fragmented, it is reassembled by the GRE egress. NEW If the delivery packet is fragmented, it is reassembled by the GRE egress if the latter is configured to do so. 2) Minor issue: 3.3.1.1, 3.3.1.2, 3.3.1.3 final paragraph: s/delivery header/delivery packet/ Typos: Last paragraph before Sec. 3, second line: s/lager/larger/ 3.3.1.1 second paragraph, last line on page 5: s/an Next-hop MTU/a Next-Hop MTU value/ 3.3.1.2 first line: s/send/sends/ Sec. 5 last paragraph, fourth last line: s/includes/include/