Re: [Lsvr] WGLC for draft-ietf-lsvr-applicability-09 (to end May 4, 2023)

[Ketan Talaulikar <[e-mail]>]

Relaying some comments from Jim that he has shared from his initial/quick
review


On Thu, Apr 20, 2023 at 7:28 PM James Guichard <
[e-mail]> wrote:

>
>
> Section 1:
>
>
>
> Really picky comment but i find the introduction awkward as it talks in
> the future tense. For example, the text "Section 5 will describe ..." would
> read
>
> better as "Section 5 describes ..".Also, CLOS here appears to be a name
> rather than a reference so should be Clos (
> https://en.wikipedia.org/wiki/Clos).
>
> See also my section 4 comments.
>
>
>
> Section 4:
>
>
>
> First sentence is not complete. Perhaps "Within a Data Center, servers are
> commonly interconnected using a [CLOS] topology". However, shouldnt CLOS be
>
> "Clos" (https://en.wikipedia.org/wiki/Clos) as used in RFC 7938 (when not
> used as a reference) for example? See section 3.2 of that RFC. Also, you
>
> use CLOS and [CLOS] which is confusing - the former see my "Clos" comment
> and for the latter it is an informative reference which is also used by
>
> RFC 7938?
>
>
>
> Section 5:
>
>
>
> The [RFC7752] reference needs to change to [draft-ietf-idr-rfc7752bis] or
> whatever you want to call the reference.
>
>
>
> Section 6:
>
>
>
>    With the BGP SPF extensions [I-D.ietf-lsvr-bgp-spf], the BGP best-
>
>    path computation and route computation are replaced with OSPF-like
>
>    algorithms [RFC2328]
>
>
>
> It might be better to say "replaced with link-state algorithms such as
> those used by [RFC2328]".
>
>
>
> Section 6.2.1:
>
>
>
> - Need a reference for IEBGP.
>
>
>
> - For example,
>
>    in a Spine-Leaf topology, each leaf switch would only peer with a
>
>    subset of the spines dependent on the flooding redundancy required to
>
>    be reasonably certain that every node within the BGP-LS SPF routing
>
>    domain has the complete topology (refer to Section 6.2.1).
>
>
>
> I do not understand this reference as the text is part of the same section
> that is being referenced ?
>
>
>
> Jim
>
>
>