[mpls] Genart last call review of draft-ietf-mpls-ldp-mrt-06

Peter Yee <peter@akayla.com> Sat, 02 September 2017 01:14 UTC

Return-Path: <peter@akayla.com>
X-Original-To: mpls@ietf.org
Delivered-To: mpls@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 39065134633; Fri, 1 Sep 2017 18:14:51 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Peter Yee <peter@akayla.com>
To: gen-art@ietf.org
Cc: mpls@ietf.org, ietf@ietf.org, draft-ietf-mpls-ldp-mrt.all@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.59.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <150431489114.6649.9807443646105756121@ietfa.amsl.com>
Date: Fri, 01 Sep 2017 18:14:51 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/aPine0n1FLqoA60Ri6XPLbXimlk>
Subject: [mpls] Genart last call review of draft-ietf-mpls-ldp-mrt-06
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 02 Sep 2017 01:14:51 -0000

Reviewer: Peter Yee
Review result: Ready with Issues

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-mpls-ldp-mrt-06
Reviewer: Peter Yee
Review Date: 2017-09-01
IETF LC End Date: 2017-09-01
IESG Telechat date: 2017-09-14

Summary:  The draft is ready for publication after correcting some minor issues.

Major issues:  None

Minor issues:

Page 3, section 3, IBR definition: the definition give here appears to be a
duplicate of the one given for IN.  Copy the correct definition from RFC 7812
instead.

Nits/editorial comments:

Page 3, section 1, 2nd paragraph, 1st sentence: append a comma after "e.g.".

Page 8, section 4.4., 1st paragraph, 2nd sentence: delete duplicate "on".

Page 8, section 4.4, 2nd paragraph, 4th sentence: change comma after
"computations" to a period.

Page 9, section 5.1.1, 4th paragraph, 2nd sentence: the construction
"red(blue)" is used without defining what it means exactly.  From the context,
it isn't quite "red or blue".  An explanation of the meaning of this
construction should be given.  I'm assuming it means something like use "red"
in the sentence in all cases or use use "blue", but don't mix them.  There's
also red/blue used later in the document which muddies the point.

Page 10, 1st full paragraph, 2nd sentence: change "foll" to "follows:".

Page 11, section 5.2.2, 2nd paragraph, 1st sentence: delete the period after
"A.1.7".

Page 12, 1st paragraph, 1st sentence: consider deleting "how".

Page 13, 2nd paragraph after numbered list, 1st sentence: append "to" after
"respect".

Section titles for 5.2, 5.3, and 7: append a space after "RFC" to separate it
from the number.  The spaceless form should only be used for references
enclosed in brackets.