[Lsvr] Rtgdir early review of draft-ietf-lsvr-applicability-09

Stig Venaas via Datatracker <noreply@ietf.org> Fri, 14 April 2023 17:08 UTC

MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Stig Venaas via Datatracker <noreply@ietf.org>
To: rtg-dir@ietf.org
Cc: draft-ietf-lsvr-applicability.all@ietf.org, lsvr@ietf.org
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <168149208987.47600.2673051175895320022@ietfa.amsl.com>
Reply-To: Stig Venaas <stig@venaas.com>
Date: Fri, 14 Apr 2023 10:08:09 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsvr/Bft33rNCSuB9R91-nYqbZO286ek>
Subject: [Lsvr] Rtgdir early review of draft-ietf-lsvr-applicability-09

Reviewer: Stig Venaas
Review result: Ready

The draft is in good shape. It is easy to read. There is one technical point I
want to raise, but not sure if anything needs to change. There are a couple of
minor grammatical issues.

In section 4 it says

   Within a Data Center, servers are commonly interconnected the CLOS
   topology [CLOS].  The CLOS topology is fully non-blocking and the
   topology is realized using Equal Cost Multi-Path (ECMP).  In a CLOS
   topology, the minimum number of parallel paths between two servers is
   determined by the width of a tier-1 stage as shown in the figure 1.

Regarding number of parallel paths, isn't the the width of tier-2 a factor as
well?

I found two grammar issues. One is in this first sentence. A word or two need
to be added to make it a complete sentence.

In section 5 it says:

   In order to simplify layer-3 routing and operations [RFC7938], many
   data centers use BGP as a routing protocol to create both an underlay
   and overlay network for their CLOS Topologies.

"an" should be added in the last sentence so that it says "both an underlay and
an overlay".

[Lsvr] Rtgdir early review of draft-ietf-lsvr-app… Stig Venaas via Datatracker