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