Re: [bess] Rtgdir early review of draft-ietf-bess-evpn-unequal-lb-18
Dhruv Dhody <dd@dhruvdhody.com> Wed, 06 December 2023 04:12 UTC
Return-Path: <dd@dhruvdhody.com>
X-Original-To: bess@ietfa.amsl.com
Delivered-To: bess@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 24127C14F5FC for <bess@ietfa.amsl.com>; Tue, 5 Dec 2023 20:12:06 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.904
X-Spam-Level:
X-Spam-Status: No, score=-1.904 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=dhruvdhody-com.20230601.gappssmtp.com
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id j4pZSLPwvePu for <bess@ietfa.amsl.com>; Tue, 5 Dec 2023 20:12:02 -0800 (PST)
Received: from mail-oa1-x2b.google.com (mail-oa1-x2b.google.com [IPv6:2001:4860:4864:20::2b]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id F1FC1C14F5F5 for <bess@ietf.org>; Tue, 5 Dec 2023 20:12:01 -0800 (PST)
Received: by mail-oa1-x2b.google.com with SMTP id 586e51a60fabf-1fa37df6da8so3560224fac.2 for <bess@ietf.org>; Tue, 05 Dec 2023 20:12:01 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dhruvdhody-com.20230601.gappssmtp.com; s=20230601; t=1701835921; x=1702440721; darn=ietf.org; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=GC9IzDgw7MtdKN5BYBUu6nSAW8a4sSz6R6eiqZbbyzg=; b=Z5fJtXR8nKFxhAapZk4DUjx05J7GTdR/Mlv9wxsQ0Mst7WfD9jQqc8vezxqjGDqLZG 9SaP2vXH+Qy2lwk36Jli4fuPZ9zqfBiX3b1SpgCw1JwocTRLJK8baptVONvxgTmjp9oJ /leYd4frGUz5HLczSw8C7TwpxkUc/R0DR8Rn1CMvo1XBuNe5T17KCIyEVzSzeKYg5Awe tDqq2Ge11XxQYvLDB7/C3wdE979O8xRPOErBw8aIbp6equpUBekZ/+d1+7JRUBVRJ5I5 M9Ry52i+JdwEfIh1ArxdqiYAEaWrRKc51TFKwM1DXRaHlOQ3xIqW+zlQlUEIGw9gtlEl VkJA==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1701835921; x=1702440721; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=GC9IzDgw7MtdKN5BYBUu6nSAW8a4sSz6R6eiqZbbyzg=; b=dtb/8vnreJ6J8uZU8nzBCHPkEqCGxxXdYr8ON9FD+1Wkbb5ZjErLKzWIcL7whAJkI6 quhn60j0/KVQIKuFPnSJVWYk4K/R3LOw5nD+cAgG6UheJ9DaA9JsotEXlI5SVLJDRcUy +UOuSuqzqqqm4VoCuSR0uSguG4J5UqARA3ZCdXoZc0gfWQKXDDJ10B91aL6+AmCAFeCU ajYFBvhUjvBJ261Ys3ag/4mIQ+PxXXkjhzd+II95sudgVvXIE3NEo5eqceU1Fbs+M2jK NK2E6mvRkbQGlDBNLfyQRwyY7P49ADj8q+q5DDk9e1g6pGJxWV6aMBSjLjpdFBAaLSOL H5KQ==
X-Gm-Message-State: AOJu0YyEh5E7xnLo4rF4yJdzJRMEJW8yO5mZA/estX7p6l3/XTDjs1io Xu3NXCj4tInva0KtH2B8KXvwoJGSycMRihzKYDiIyg==
X-Google-Smtp-Source: AGHT+IFcDMJ4wOyN+VzZiqMuRpeJw+TNSOpzqNvTRmmtSuPVzhvCdgdYQSlwzbrj8kouyZWwA1CcNooS8N6/RQWxK1s=
X-Received: by 2002:a05:6870:169a:b0:1fa:e851:7204 with SMTP id j26-20020a056870169a00b001fae8517204mr327205oae.43.1701835920980; Tue, 05 Dec 2023 20:12:00 -0800 (PST)
MIME-Version: 1.0
References: <170013249509.30988.13683270341413520261@ietfa.amsl.com> <BY5PR11MB4290C989D8B1ACBD4E0614B8CF86A@BY5PR11MB4290.namprd11.prod.outlook.com> <CAP7zK5asgg3gbRZtdxTMTh3jx09gxEShVSvxA5PzG7f=_6u2Mw@mail.gmail.com> <BY5PR11MB4290D311620D44FB7C1CAF1ACF85A@BY5PR11MB4290.namprd11.prod.outlook.com>
In-Reply-To: <BY5PR11MB4290D311620D44FB7C1CAF1ACF85A@BY5PR11MB4290.namprd11.prod.outlook.com>
From: Dhruv Dhody <dd@dhruvdhody.com>
Date: Wed, 06 Dec 2023 09:41:24 +0530
Message-ID: <CAP7zK5bdUm+EvZTuA4QQTyQ58Jz5afW+3525NLFBH3y+VK3b1w@mail.gmail.com>
To: "Neeraj Malhotra (nmalhotr)" <nmalhotr@cisco.com>
Cc: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "bess@ietf.org" <bess@ietf.org>, "draft-ietf-bess-evpn-unequal-lb.all@ietf.org" <draft-ietf-bess-evpn-unequal-lb.all@ietf.org>
Content-Type: multipart/alternative; boundary="000000000000895788060bcf8d42"
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/EpRMz2JkT7vEa1GZ7mcZdCqc5U0>
Subject: Re: [bess] Rtgdir early review of draft-ietf-bess-evpn-unequal-lb-18
X-BeenThere: bess@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
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: Wed, 06 Dec 2023 04:12:06 -0000
Hi Neeraj, On Wed, Dec 6, 2023 at 2:19 AM Neeraj Malhotra (nmalhotr) < nmalhotr@cisco.com> wrote: > > > Hi Dhruv, > > > > rev20 incorporates all of the additional points below. > > > > Regarding, > > > > “* In cases where allocations are already made under FCFS, please state > that > instead of asking IANA to make the allocation again!” > > > > I am not aware of any allocations that have already been made. Have > updated the text in this section (now section 10) to call out all requested > allocations as “suggested” values. > > > > Please do let me know in case you see anything else missing. > > > Dhruv: See https://www.iana.org/assignments/bgp-extended-communities/bgp-extended-communities.xhtml#evpn 0x10 EVPN Link Bandwidth Extended Community [ draft-ietf-bess-evpn-unequal-lb-15 <https://www.iana.org/go/draft-ietf-bess-evpn-unequal-lb-15>] 2022-03-11 Thus, this text needs to change -- A new EVPN Link Bandwidth extended community is defined to signal local ES > link bandwidth to ingress PEs. This extended community is defined of type > 0x06 (EVPN). IANA is requested to assign a suggested sub-type value of 0x10 > for the EVPN Link bandwidth extended community, of type 0x06 (EVPN). EVPN > Link Bandwidth extended community is defined as transitive.¶ It would also be a good idea to clearly identify the registry here "EVPN Extended Community Sub-Types". Thanks! Dhruv > Thanks, > > Neeraj > > > > *From: *Dhruv Dhody <dd@dhruvdhody.com> > *Date: *Monday, December 4, 2023 at 11:36 PM > *To: *Neeraj Malhotra (nmalhotr) <nmalhotr@cisco.com> > *Cc: *rtg-dir@ietf.org <rtg-dir@ietf.org>, bess@ietf.org <bess@ietf.org>, > draft-ietf-bess-evpn-unequal-lb.all@ietf.org < > draft-ietf-bess-evpn-unequal-lb.all@ietf.org> > *Subject: *Re: Rtgdir early review of draft-ietf-bess-evpn-unequal-lb-18 > > Thanks Neeraj! Thanks for taking my comments into consideration! > > Looking at -19 some additional points! > > - No reference should be added in the abstract > - Note to the IESG in the abstract, can be moved to the shepherd report > and provided the assigned shepherd agrees with your justification. > - s/advertisong/advertising/ > - I am worried about the use of "operators SHOULD" in Section 8 i.e. we > are using SHOULD for how operators need to behave instead of how the > implementations ought to handle these operational issues. > - This is missed: > ### Section 14 > > * In cases where allocations are already made under FCFS, please state that > instead of asking IANA to make the allocation again! > > Thanks! > Dhruv > > > > > > On Tue, Dec 5, 2023 at 8:08 AM Neeraj Malhotra (nmalhotr) < > nmalhotr@cisco.com> wrote: > > > > Hi Dhruv, > > > > Many thanks for a very detailed review and comments. I have just published > version 19 that significantly revises the document to incorporate all of > your comments as well as comments from Genart early review. Please see > additional clarifications inline below. Please do let me know in case you > see anything else outstanding. > > > > Thanks, > > Neeraj > > > > > > ## Comments: > > ### General > > * Request the shepherd to make sure that there is a valid justification > for 6 > authors. Better yet just make it 5 authors (you have 3 authors from the > same > affiliation and one author marked as editor) > > [NM]: added justification for 6 authors. > > * Please follow the RFC style guide. For instance, the Introduction should > be > the first section as per - > https://www.rfc-editor.org/rfc/rfc7322.html#section-4.8.1. The best would > be to > have a new Introduction section that briefly introduces the concept and > change > section 2 to "Motivation" or something like that. > > [NM]: done > > > * Use of some words in all capital letters - OR, ALL, NOT. This should be > avoided so as not to confuse with RFC2119 keywords which have special > meaning > when in capital. > > [NM]: done > > * The documents should add references to relevant RFCs when talking about > existing EVPN features. > * IRB > * > > [NM]: done > > > ### Section 1 > > * Please update the Requirements Language template to - > ```` > The key words "MUST", "MUST NOT", "REQUIRED", "SHALL", "SHALL NOT", > "SHOULD", "SHOULD NOT", "RECOMMENDED", "NOT RECOMMENDED", "MAY", and > "OPTIONAL" in this document are to be interpreted as described in > BCP 14 [RFC2119] [RFC8174] when, and only when, they appear in all > capitals, as shown here. > ```` > > [NM]: done > > > * Please add references for the terms where possible. Definitions such as > "Egress PE" and "Ingress PE" refer to RT-1, RT-2, and RT-5 especially needs > one. Also, can the local PE and Ingress PE be different? > > [NM]: done. Made the terminology consistent to use “Ingress/Egress PE” and > removed “Local/Remote PE” terminology. > > ### Section 4 > > * Why SHOULD and not MUST in - > ```` > Implementations SHOULD support the default units of Mbps > ```` > > [NM]: done. Corrected to MUST. > > > * IMHO section 4.2 is better suited in the appendix > > [NM]: done > > ### Section 5 > > * Section 5.1; Why SHOULD and not MUST? > > [NM]: done. Corrected to MUST. > > > * Section 5.1; Consider adding the reasoning behind > ```` > EVPN link bandwidth extended community SHOULD NOT be attached to per- > EVI RT-1 or to EVPN RT-2. > ```` > > [NM]: done > > > * Section 5.2; If the extended commuity is silently ignored, how would an > operator learn about it? At least a requirement for a log should be added. > * > > [NM]: done > > > Section 5.2; How is the weighted path list computed when the normalized > weight > is in fractions i.e. L(1, 10) = 2500 Mbps and thus W(1, 10) = 2.5? I am > guessing you assume it is an integer (same as BW Increment) but it is not > stated. > > [NM]: The method in this section is only an example. Weighted pathlist > computation is a local implementation choice. That said, divide by HCF > method in this example will always result in integer weights, although the > computed weight values using this example method may not always to be > reasonably programmed in HW. I have added another paragraph to explicitly > clarify this as well as that this is an implementation choice. > > > ### Section 6 > > * The update procedure listed here "updates" other specifications. I > wonder if > this should be captured in metadata, abstract etc. > > [NM]: Added additional text to abstract. > > * Section 6.3.1, > * Change L(min) to Lmin t to not be conffused by L(i) > > [NM]: done. > > > * I am unsure of what you mean by "with PE(1) = 10, PE(2) = 10, PE(3) > = 20" > which later changes to "with PE(1) = 10, PE(2) = 10, PE(3) = 10" * > Other > documents do not use the word affinity, it was difficult for me to > verify > the affinity formula and I left that for the WG to verify for > correctness. > > [NM]: fixed. > > > * Inconsistency between MUST and MAY - > ```` > Depending on the chosen HRW hash function, the affinity function MUST be > extended to include bandwidth increment in the computation. > > affinity function specified in [EVPN-PER-MCAST-FLOW-DF] MAY be > extended as follows to incorporate bandwidth increment j: > > affinity or random function specified in [RFC8584] MAY be extended as > follows to incorporate bandwidth increment j: > ```` > > [NM]: fixed. > > * Section 6.4, asks to add a new bullet (f) in > > https://www.ietf.org/archive/id/draft-ietf-bess-evpn-pref-df-13.html#section-4.1 > ; Note that there is already a bullet f there! > > [NM]: This section updates bullet f in [EVPN-PREF-DF]. I have updated the > text to clearly state that. > > ### Section 9 > > * What should be the value-units in this case to be interpreted as relative > weight? > > [NM]: 0x01. Added text to state that (this is now section 7.2 in rev19). > > ### Section 12 > > * Isn't there operation issues with the correct settings of "value-units" > for > Generalized weight? How does an operator learn about the inconsistency? How > does the operator know this feature is working properly? What fields > should one > monitor? Any changes in the YANG model? > > [NM]: added. > > ### Section 13 > > * Even if your claim that there are no new security concerns could be > true, it > needs to be justified and the relevant security of base EVPN needs to be > referenced. You may also highlight some security concerns most relevant to > this > extension. > > [NM]: added. > > > ### Section 14 > > * Please don't squat on codepoint and allocate them yourself. > * Best to use TBAx > * Or at the very least say that they are suggested values > * In cases where allocations are already made under FCFS, please state that > instead of asking IANA to make the allocation again! > > [NM]: fixed. > > > ## Nits: > > * Expand the abbreviation on first use > * CE (also in abstract) > * PE (also in abstract) > * LAG (also in abstract) > * IRB > * BUM > * HRW > * DP > > [NM]: done. > > * s/detailed in RFC 7432/detailed in [RFC7432]/ > * s/all egress PEs, ALL remote traffic/all egress PEs, all remote traffic/ > * There are various instances where you use"proposed", this should be > changed > to "specified" as we are moving towards RFC publication and it is no longer > just a proposal. > > [NM]: done. > > * Isnt "per-[ES, EVI] RT-1" enough? Why does it say "per-ES > RT-1 and per-[ES, EVI] RT-1" in - ```` > In an unlikely scenario where an EVPN > implementation does not support EVPN aliasing procedures, MAC > forwarding path-list at the ingress PE is computed based on per-ES > RT-1 and RT-2 routes received from egress PEs, instead of per-ES RT-1 > and per-[ES, EVI] RT-1 from egress PEs. > ```` > > [NM]: Both per-[ES] RT-1 and per-[ES, EVI] RT-1 routes are required for > aliasing procedure specified in RFC 7432. > > > * Section 7 should ideally be a subsection of Section 6 as it is related > to the > DF election > > [NM]: done. > > > >
- [bess] Rtgdir early review of draft-ietf-bess-evp… Dhruv Dhody via Datatracker
- Re: [bess] Rtgdir early review of draft-ietf-bess… Neeraj Malhotra
- Re: [bess] Rtgdir early review of draft-ietf-bess… Neeraj Malhotra (nmalhotr)
- Re: [bess] Rtgdir early review of draft-ietf-bess… Dhruv Dhody
- Re: [bess] Rtgdir early review of draft-ietf-bess… Neeraj Malhotra (nmalhotr)
- Re: [bess] Rtgdir early review of draft-ietf-bess… Dhruv Dhody
- Re: [bess] Rtgdir early review of draft-ietf-bess… Neeraj Malhotra (nmalhotr)