Re: [bmwg] bmwg Digest, Vol 200, Issue 6

Hi Joel,

Thank you for the comments. Appreciate it. Kindly find the answers inline and the modification will be done shortly, I will update the new version in a couple of days.

Regards,
Sudhin

From: bmwg-request@ietf.org
Sent: Thu, 17 Jun 2021 00:31:11
To: bmwg@ietf.org
Subject: bmwg Digest, Vol 200, Issue 6

Send bmwg mailing list submissions to
&nbsp;&nbsp;&nbsp;bmwg@ietf.org

To subscribe or unsubscribe via the World Wide Web, visit
&nbsp;&nbsp;&nbsp;https://www.ietf.org/mailman/listinfo/bmwg
or, via email, send a message with subject or body &#39;help&#39; to
&nbsp;&nbsp;&nbsp;bmwg-request@ietf.org

You can reach the person managing the list at
&nbsp;&nbsp;&nbsp;bmwg-owner@ietf.org

When replying, please edit your Subject line so it is more specific
than &quot;Re: Contents of bmwg digest...&quot;

Today&#39;s Topics:

&nbsp; 1. Rtgdir last call review of draft-ietf-bmwg-evpntest-08
&nbsp; &nbsp; &nbsp;(Joel Halpern via Datatracker)

----------------------------------------------------------------------

Message: 1
Date: Wed, 16 Jun 2021 11:59:12 -0700
From: Joel Halpern via Datatracker &lt;noreply@ietf.org&gt;
To: &lt;rtg-dir@ietf.org&gt;
Cc: bmwg@ietf.org, draft-ietf-bmwg-evpntest.all@ietf.org,
&nbsp;&nbsp;&nbsp;last-call@ietf.org
Subject: [bmwg] Rtgdir last call review of draft-ietf-bmwg-evpntest-08
Message-ID: &lt;162386995210.13675.8048702439794738383@ietfa.amsl.com&gt;
Content-Type: text/plain; charset=&quot;utf-8&quot;

Reviewer: Joel Halpern
Review result: Has Issues

I have been selected as the Routing Directorate reviewer for this draft. The
Routing Directorate seeks to review all routing or routing-related drafts as
they pass through IETF last call and IESG review, and sometimes on special
request. The purpose of the review is to provide assistance to the Routing ADs.
For more information about the Routing Directorate, please see
?http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Although these comments are primarily for the use of the Routing ADs, it would
be helpful if you could consider them along with any other IETF Last Call
comments that you receive, and strive to resolve them through discussion or by
updating the draft.

Document: draft-name-version.txt
Reviewer: your-name
Review Date: date
IETF LC End Date: date-if-known
Intended Status: copy-from-I-D

Summary:
I have some minor concerns about this document that I think should be resolved
before publication.

Major Comments: N/A

Minor Comments:
&nbsp; &nbsp;Reading section 3.1 on the rate for learning addresses, I am left guessing
&nbsp; &nbsp;how the procedure is to be performed. &nbsp;The text references RFC 2889 section
&nbsp; &nbsp;5.8, but only for the meaning for the terms. &nbsp;The test procedure is clearly
&nbsp; &nbsp;different since that test relies on observing flooding. &nbsp;As best I can
&nbsp; &nbsp;guess, the test assumes that there is an observable (Netconf ? YANG ?)
&nbsp; &nbsp;variable that reports how many local MAC addresses the device under test
&nbsp; &nbsp;has learned. &nbsp;It would be good to be more explicit about that, if possible
&nbsp; &nbsp;pointing to the YANG module that defines the parameter to be observed.
&nbsp;A similar clarification would be helpful on section 3.2 (on control plane MAC
&nbsp;learning). It probably would be helpful if sections 3.3 and 3.4 then
&nbsp;referenced sections 3.1 and 3.2 respectively for what is being observed. This
&nbsp;probably applies to section 4 as well. &nbsp;If the same variables are to be used,
&nbsp;then a simple reference to the earlier description would seem to suffice.

Sudhin&gt;&gt;&gt;&gt; it must be generic because not all the evpn features are supported in Yang models and it is up to the user choice that is reason any specific ways to capture the data is omitted.

&nbsp;I believe that section 3.8 on high availability is intended to cause a switch
&nbsp;of traffic path from DUT to MHPE2. &nbsp; However, the text in that section never
&nbsp;refers to MHPE 2. &nbsp;It refers to switch of routing processor. &nbsp;It is possible
&nbsp;that this is intended to be a redundancy test within DUT. &nbsp;If so, it would
&nbsp;help to be more explicit, since as far as I know we do not standardize that
&nbsp;behavior.

I am left puzzled as to the need for MHPE2 in these tests. &nbsp;I assume there is
some obvious and simple reason for including it that I missed. &nbsp;Could you add
an explanation?

Sudhin&gt;&gt;&gt; MHPE2 is very much needed in multi homing scenario, it plays as a standby role in testing, sure will add an explnation in the section&nbsp; &quot;Test Setup Configuration&quot;.
Nits:
&nbsp; &nbsp;At the end of section 2, the text reads &quot;The X is used as variable...&quot;
&nbsp; &nbsp;Could you change that to &quot;The X below is used as a variable ...&quot;? &nbsp;I spent
&nbsp; &nbsp;some time looking backwards for the X.

Sudhin &gt;&gt;&gt;&gt; Sure

&nbsp;The equation at the end of section 3.9 (ARD / ND scaling) is somewhat
&nbsp;misleading. &nbsp;It uses the same v1, v2, ..vn in successive lines to implicitly
&nbsp;refer to the IPv4 measurements and the IPv6 measurements. &nbsp;It would be good
&nbsp;to name these separately, as obviously the same calculation can not produce
&nbsp;two different results.

Sudhin&gt;&gt;&gt; sure will change.

------------------------------

Subject: Digest Footer

_______________________________________________
bmwg mailing list
bmwg@ietf.org
https://www.ietf.org/mailman/listinfo/bmwg

------------------------------

End of bmwg Digest, Vol 200, Issue 6
************************************