[bess] Lars Eggert's No Objection on draft-ietf-bess-evpn-pref-df-11: (with COMMENT)
Lars Eggert via Datatracker <noreply@ietf.org> Fri, 04 August 2023 11:53 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 617E0C16B5A9; Fri, 4 Aug 2023 04:53:41 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Lars Eggert via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-bess-evpn-pref-df@ietf.org, bess-chairs@ietf.org, bess@ietf.org, Stephane Litkowski <slitkows.ietf@gmail.com>, slitkows.ietf@gmail.com
X-Test-IDTracker: no
X-IETF-IDTracker: 11.5.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Lars Eggert <lars@eggert.org>
Message-ID: <169115002138.60533.14192234626058102309@ietfa.amsl.com>
Date: Fri, 04 Aug 2023 04:53:41 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/gDCEchYJDV49TygN59LoqK2wpIQ>
Subject: [bess] Lars Eggert's No Objection on draft-ietf-bess-evpn-pref-df-11: (with COMMENT)
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, 04 Aug 2023 11:53:41 -0000
Lars Eggert has entered the following ballot position for draft-ietf-bess-evpn-pref-df-11: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/ for more information about how to handle DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-bess-evpn-pref-df/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- # GEN AD review of draft-ietf-bess-evpn-pref-df-11 CC @larseggert Thanks to Vijay Gurbani for the General Area Review Team (Gen-ART) review (https://mailarchive.ietf.org/arch/msg/gen-art/yyKue3u0e4F2LuAMSzjlxaEKHrQ). ## Comments ### Section 1.1, paragraph 2 ``` While the Default Designated Forwarder Algorithm [RFC7432] or the Highest Random Weight algorithm (HRW) [RFC8584] provide an efficient and automated way of selecting the Designated Forwarder across different Ethernet Tags in the Ethernet Segment, there are some use- cases where a more 'deterministic' and user-controlled method is required. At the same time, Service Providers require an easy way to ``` Why is "deterministic" in quotes here? Is this algorithm not (always) in fact deterministic? Could a more accurate term be chosen? ### Section 4.1, paragraph 1 ``` Assuming the operator wants to control - in a flexible way - what PE becomes the Designated Forwarder for a given virtual Ethernet Segment and the order in which the PEs become Designated Forwarder in case of multiple failures, the following procedure may be used: ``` It's not clear what kind of procedure the list items a-f describe. The individual list items read more like standalone considerations than any kind of procedure. ## Nits All comments below are about very minor potential issues that you may choose to address in some way - or ignore - as you see fit. Some were flagged by automated tools (via https://github.com/larseggert/ietf-reviewtool), so there will likely be some false positives. There is no need to let me know what you did with these suggestions. ### Typos #### Section 4.1, paragraph 2 ``` - (100,0,Highest-Preferance), (200,0,Highest-Preference) and - ^ + (100,0,Highest-Preference), (200,0,Highest-Preference) and + ^ ``` #### Section 4.1, paragraph 11 ``` - if PE2's IP addres is lower than PE1's. Same example applies + if PE2's IP address is lower than PE1's. Same example applies + + ``` #### Section 4.2, paragraph 1 ``` - Segment. A potential way to achive a more granular load balancing is - decribed below. + Segment. A potential way to achieve a more granular load balancing is + + + described below. + + ``` ### Outdated references Document references `draft-ietf-bess-evpn-virtual-eth-segment-11`, but `-12` is the latest available revision. ### Grammar/style #### Section 4.1, paragraph 3 ``` nce is more preferred than PE2's). Hence PE1 becomes the Designated Forwarde ^^^^^ ``` A comma may be missing after the conjunctive/linking adverb "Hence". #### Section 4.1, paragraph 4 ``` dered list for vES1 is <PE2, PE1>. Hence PE2 becomes the Designated Forwarde ^^^^^ ``` A comma may be missing after the conjunctive/linking adverb "Hence". #### Section 4.1, paragraph 6 ``` s are used as tie-breakers. If more that one PE is advertising itself as the ^^^^ ``` Did you mean "than"? #### Section 4.3, paragraph 11 ``` with DP=1, that is, PE2 (Pref=200). Hence PE3 will inherit PE2's preference a ^^^^^ ``` A comma may be missing after the conjunctive/linking adverb "Hence". #### Section 4.3, paragraph 19 ``` warder Election Algorithm different than the one configured in the rest of t ^^^^ ``` Did you mean "different from"? "Different than" is often considered colloquial style. ## Notes This review is in the ["IETF Comments" Markdown format][ICMF], You can use the [`ietf-comments` tool][ICT] to automatically convert this review into individual GitHub issues. Review generated by the [`ietf-reviewtool`][IRT]. [ICMF]: https://github.com/mnot/ietf-comments/blob/main/format.md [ICT]: https://github.com/mnot/ietf-comments [IRT]: https://github.com/larseggert/ietf-reviewtool
- [bess] Lars Eggert's No Objection on draft-ietf-b… Lars Eggert via Datatracker
- Re: [bess] Lars Eggert's No Objection on draft-ie… Jorge Rabadan (Nokia)