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

Menachem Dodge via Datatracker <noreply@ietf.org> Sun, 02 July 2023 08:58 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 F03F9C1524B4; Sun, 2 Jul 2023 01:58:09 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Menachem Dodge via Datatracker <noreply@ietf.org>
To: ops-dir@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: <168828828996.27966.17731070778084914470@ietfa.amsl.com>
Reply-To: Menachem Dodge <menachemdodge1@gmail.com>
Date: Sun, 02 Jul 2023 01:58:09 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/3GdsKuy0oSswvWSnprYSeA4mHYc>
Subject: [bess] Opsdir 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: Sun, 02 Jul 2023 08:58:10 -0000

Reviewer: Menachem Dodge
Review result: Has Nits

Hello,
I have reviewed this document as part of the Operational directorate's ongoing
effort to review all IETF documents being processed by the IESG.  These
comments were written with the intent of improving the operational aspects of
the IETF drafts. Comments that are not addressed in last call may be included
in AD reviews during the IESG review.  Document editors and WG chairs should
treat these comments just like any other last call comments.

The document is well written.

Summary:
For MVPN and EVPN networks where P2MP or MP2MP tunnels are used to carry
traffic, the ingress routers allocate an upstream label for each VPN or for
each BD. This can lead to the egress routers needing to keep track of large
numbers of labels that can be greatly reduced if the association between a
label and a VPN or BD is made by provisioning, so that all ingress routers
assign the same label to a particular VPN or BD.

The document is for the standards track.

Nits
====
1. Section 2.2, 5th paragraph - missing word:
OLD --> However, that is not necessary as the labels used by PEs for the
purposes defined in this document will only rise to the top of the label stack
when traffic arrives the PEs. SUGGEST --> However, that is not necessary as the
labels used by PEs for the purposes defined in this document will only rise to
the top of the label stack when traffic arrives at the PEs.

 2. Section 2.2, Last Paragraph - sentence not clear:
OLD --> Allocating a label from the DCB or from those a few context-specific
label spaces and communicating them to all PEs is not different from allocating
VNIs, and is feasible in today's networks since controllers are used more and
more widely SUGGEST --> Allocating a label from the DCB or from a
context-specific label space and communicating them to all PEs is not different
from allocating VNIs, and is feasible in today's networks since controllers are
used more and more widely

3. Section 2.2.3, first sentence:
OLD --> In summary, labels can be allocated and advertised the following ways:
SUGGEST --> In summary, labels can be allocated and advertised in the following
ways:

4. Section 2.2.3, point 3 - sentence is unclear.
"A central authority assigns disjoint label blocks from those a few
context-specific label spaces to each PE, and allocate labels from the DCB to
identify the context-specific label spaces."

Thank you kindly.

Best Regards,
Menachem Dodge