[bess] Genart last call review of draft-ietf-bess-mvpn-evpn-aggregation-label-10

Russ Housley via Datatracker <noreply@ietf.org> Fri, 30 June 2023 15:33 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: bess@ietf.org
Delivered-To: bess@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 27674C151989; Fri, 30 Jun 2023 08:33:38 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Russ Housley via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: bess@ietf.org, draft-ietf-bess-mvpn-evpn-aggregation-label.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 11.3.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <168813921814.23690.1583965201300307474@ietfa.amsl.com>
Reply-To: Russ Housley <housley@vigilsec.com>
Date: Fri, 30 Jun 2023 08:33:38 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/EFpgkPDLXJmuNm_0ZoL48inEtJc>
Subject: [bess] Genart last call review of draft-ietf-bess-mvpn-evpn-aggregation-label-10
X-BeenThere: bess@ietf.org
X-Mailman-Version: 2.1.39
List-Id: BGP-Enabled ServiceS working group discussion list <bess.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bess>, <mailto:bess-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/bess/>
List-Post: <mailto:bess@ietf.org>
List-Help: <mailto:bess-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bess>, <mailto:bess-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 30 Jun 2023 15:33:38 -0000

Reviewer: Russ Housley
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
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-bess-mvpn-evpn-aggregation-label-10
Reviewer: Russ Housley
Review Date: 2023-06-30
IETF LC End Date: 2023-07-12
IESG Telechat date: unknown


Summary: Almost Ready


Major Concerns: None.


Minor Concerns:

Section 2.2: Please provide more discussion of "central authority".  I
do not think that a Internet-wide authority is required to meet this
requirement.  I think that the authority needs to be recognized
throughout a provider network.


Nits:

Section 1: please avoid splitting "I/S-PMSI" across multiple lines.

Section 2: s/ingress PE's default label space/default label space/

Section 2: s/packet's ingress PE/ingress PE of the packet/

Section 2.2.2.1: s/Route-2 respectively/Route-2, respectively/

Section 3.2: s/PTA's Label field/label field of the PTA/

Section 3.2: s/source PE's label space/label space of the source PE/

Section 4: s/third party's label space/label space of a third party/ (2 places)

Section 4: please spell out "LFIBs"; this term is not in the definitions.