Genart last call review of draft-ietf-grow-large-communities-usage-06
Stewart Bryant <stewart.bryant@gmail.com> Tue, 18 April 2017 13:41 UTC
Return-Path: <stewart.bryant@gmail.com>
X-Original-To: ietf@ietf.org
Delivered-To: ietf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 8E99D12EC51; Tue, 18 Apr 2017 06:41:15 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Stewart Bryant <stewart.bryant@gmail.com>
To: gen-art@ietf.org
Cc: grow@ietf.org, draft-ietf-grow-large-communities-usage.all@ietf.org, ietf@ietf.org
Subject: Genart last call review of draft-ietf-grow-large-communities-usage-06
X-Test-IDTracker: no
X-IETF-IDTracker: 6.49.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <149252287543.16134.18005737444773296286@ietfa.amsl.com>
Date: Tue, 18 Apr 2017 06:41:15 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/ietf/kxD0XmWLvkkL1LoEdJXCu3DmwJM>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.22
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 18 Apr 2017 13:41:16 -0000
Reviewer: Stewart Bryant Review result: Almost Ready 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-grow-large-communities-usage-?? Reviewer: Stewart Bryant Review Date: 2017-04-18 IETF LC End Date: 2017-04-21 IESG Telechat date: Not scheduled for a telechat Summary: This is mostly a well written document, although unfortunately there are a number of matters that really need to be looked at. Major issues: Abstract Examples and inspiration for operators to use BGP Large Communities. SB> Even if you just copy the Introduction, the Abstract should really be expanded SB> to help the reader understand whether or not they want read the RFC SB> or if they had read it what it was about. ============ 5. Security Considerations Operators should note the recommendations in Section 11 of BGP Operations and Security [RFC7454]. SB> You do not address the question of whether there are new considerations, or considerations SB> that are of increased importance? Is there is text somewhere SB> that discusses the integrity and synchronization of the parameters SB> and any consequences that arise? =========== Minor issues: 2.2. Action Communities Action Communities are added as a label to request that a route be treated in a particular way within an AS. The operator of the AS defines a routing policy that adjusts path attributes based on the community. For example, the route's propagation characteristics, the LOCAL_PREF (local preference), the next-hop, or the number of AS_PATH prepends to be added when it is received or propagated can be changed. SB> Although these are well known to the target audience, I think you SB> need some references in the above para. Nits/editorial comments: 6. IANA Considerations None. SB> A little briefer than normal.
- Genart last call review of draft-ietf-grow-large-… Stewart Bryant
- Re: Genart last call review of draft-ietf-grow-la… heasley
- Re: [GROW] Genart last call review of draft-ietf-… Randy Bush
- Re: Genart last call review of draft-ietf-grow-la… Job Snijders
- Re: [GROW] Genart last call review of draft-ietf-… Nick Hilliard
- Re: [GROW] Genart last call review of draft-ietf-… Randy Bush
- Re: [GROW] Genart last call review of draft-ietf-… Stewart Bryant
- Re: Genart last call review of draft-ietf-grow-la… Stewart Bryant
- Re: Genart last call review of draft-ietf-grow-la… Job Snijders
- Re: Genart last call review of draft-ietf-grow-la… Stewart Bryant
- Re: Genart last call review of draft-ietf-grow-la… Randy Bush
- Re: Genart last call review of draft-ietf-grow-la… Job Snijders
- Re: [GROW] Genart last call review of draft-ietf-… joel jaeggli