[Gen-art] Genart last call review of draft-ietf-mpls-egress-protection-framework-05
Peter Yee via Datatracker <noreply@ietf.org> Fri, 21 June 2019 04:50 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 404401200FF; Thu, 20 Jun 2019 21:50:18 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Peter Yee via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: draft-ietf-mpls-egress-protection-framework.all@ietf.org, ietf@ietf.org, mpls@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.98.1
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Peter Yee <peter@akayla.com>
Message-ID: <156109261821.12295.3031530485744900655@ietfa.amsl.com>
Date: Thu, 20 Jun 2019 21:50:18 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/d9JIFo_E2BQLwUull3cfqgN6mu4>
Subject: [Gen-art] Genart last call review of draft-ietf-mpls-egress-protection-framework-05
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 21 Jun 2019 04:50:18 -0000
Reviewer: Peter Yee Review result: Ready with Nits 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-egress-protection-framework-05 Reviewer: Peter Yee Review Date: 2019-06-20 IETF LC End Date: 2019-06-17 IESG Telechat date: Not scheduled for a telechat Summary: Ready with Nits. I didn't really find any substantial issues with the document, although I admit this one is quite out of my area of expertise. I'm reduced to making sure it sounds logical (it does) and trying to reduce the (future) burden on the RFC Editor. Major issues: Minor issues: Nits/editorial comments: General: For each occurrence of "e.g." and "i.e.", make sure that it is followed by a comma and a single space. There are some occurrences of double spaces and pretty much no commas. Change "aka." to "aka" or to "a.k.a." Specific: Page 1, Abstract, 3rd sentence: change "context based" to "context-based". Page 3, section 1, 2nd paragraph, 1st sentence: change "repair based" to "repair-based". Insert "and" before "[RFC7812]" or put the sequence of references inside of parentheses. Page 3, section 1, 2nd paragraph, 2nd sentence: delete the comma and "i.e." after "PLR". Put "point of local repair" in brackets. Delete the comma and "i.e." after "MP" and place "merge point" inside brackets. Page 7, section 4, 1st bullet item: append a comma after the first occurrence of "MP2P". Page 7, section 4, 4th bullet item, 2nd sentence: insert "on a" before "per-service-destination". Page 9, 1st bullet item: change "Or" to "or". Page 10, 1st partial paragraph, last sentence: change "It" to "it". Page 10, section 5.4, 2nd paragraph, 3rd sentence: delete extraneous space between "{E, P1}" and the following comma. Page 11, section 5.7 title: change "context based" to "context-based". Page 14, item 3, 4th sentence: change "any-cast" to "anycast". Page 14, item 3, last sentence: I can't parse the end of that sentence. Do you want one or the other of "to" and "in", or is there a word or words missing in there? Page 15, 1st paragraph, 1st sentence: append "of" after "AS'". And I'll just say that that's a very labored construction to avoid using "ASs" or "AS's" in the sentence. It took be a while to parse what was meant. Page 16, 1st full paragraph, 1st sentence: insert "the" before "PLR". Page 18, 1st paragraph, last sentence: change the first occurrence of "in" to "if". Page 19, 1st partial paragraph, 1st full sentence: change "Figure-3" to "Figure 3" to match the figure's actual labeling. Page 21, section 8, 1st sentence: append a comma after "far". Page 25, section 11, 2nd paragraph, 2nd sentence: insert "of" before "trust".
- [Gen-art] Genart last call review of draft-ietf-m… Peter Yee via Datatracker
- Re: [Gen-art] Genart last call review of draft-ie… Yimin Shen
- Re: [Gen-art] Genart last call review of draft-ie… Alissa Cooper