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 rev. 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, Joseph Touch
Draft 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 rev. 02 (document currently at 05)
Review result Has Nits
Review completed: 2015-05-08

Review
review-ietf-intarea-gre-mtu-02-opsdir-lc-taylor-2015-05-08

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/