Skip to main content

Last Call Review of draft-ietf-intarea-gre-mtu-02

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
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 and 

as follows:

   If the delivery packet is fragmented, it is reassembled by the GRE

   If the delivery packet is fragmented, it is reassembled by the GRE
   egress if the latter is configured to do so.

2) Minor issue:,, final paragraph:
   s/delivery header/delivery packet/


Last paragraph before Sec. 3, second line: s/lager/larger/ second paragraph, last line on page 5:
    s/an Next-hop MTU/a Next-Hop MTU value/ first line: s/send/sends/

Sec. 5 last paragraph, fourth last line: s/includes/include/