Shraddha -

Thanx for the responses.

Please see inline.

From: Shraddha Hegde <shraddha=40juniper....@dmarc.ietf.org>
Sent: Wednesday, December 02, 2020 9:30 AM
To: Les Ginsberg (ginsberg) <ginsb...@cisco.com>; lsr@ietf.org
Subject: RE: New Version Notification for draft-white-lsr-distoptflood-00.txt

Hi Les,

Thanks for the review and comments.
Pls see inline for replies.



Juniper Business Use Only
From: Les Ginsberg (ginsberg) 
<ginsberg=40cisco....@dmarc.ietf.org<mailto:ginsberg=40cisco....@dmarc.ietf.org>>
Sent: Monday, November 30, 2020 11:44 AM
To: Shraddha Hegde <shrad...@juniper.net<mailto:shrad...@juniper.net>>; 
lsr@ietf.org<mailto:lsr@ietf.org>
Subject: RE: New Version Notification for draft-white-lsr-distoptflood-00.txt

[External Email. Be cautious of content]


Draft authors -



I note that as the draft has evolved over the years a number of mechanisms have 
been removed (revised adjacency formation and auto tier detection) and the 
draft now focuses exclusively on flooding optimizations.

The draft now also references the recent work done in 
draft-ietf-lsr-dynamic-flooding.



A couple of things are not clear to me.



1)Is it your intent to use the mechanisms defined in 
draft-ietf-lsr-dynamic-flooding? I am specifically - though not exclusively - 
referring to how flooding adapts to topology changes.

It isn't clear from Section 3 whether you are actually intending to use the 
mechanisms defined in 
https://tools.ietf.org/html/draft-ietf-lsr-dynamic-flooding-07#section-6.8<https://urldefense.com/v3/__https:/tools.ietf.org/html/draft-ietf-lsr-dynamic-flooding-07*section-6.8__;Iw!!NEt6yMaO-gk!VproGis8uUGmcl-cCplFMwuNceAo-3roUZT7ZdTlwvY7nJSszUo9AeWXymgguZIt$>
 or whether you simply want to use the Flooding sub-TLV defined in 
draft-ietf-lsr-dynamic-flooding to (optionally??) advertise your algorithm and 
all other procedures are specific to your algorithm.

<Shraddha> We want to use the " IS-IS Dynamic Flooding Sub-TLV" defined in sec 
5.1.2 of the lsr-dynamic-flooding draft just to flood the information
that nodes support the algorithm specified in draft-white-lsr-distoptflood. I 
agree it's not writen very well. I'll update that section in next version.





2)The text at the end of 
https://tools.ietf.org/html/draft-white-lsr-distoptflood-00#section-2.1<https://urldefense.com/v3/__https:/tools.ietf.org/html/draft-white-lsr-distoptflood-00*section-2.1__;Iw!!NEt6yMaO-gk!VproGis8uUGmcl-cCplFMwuNceAo-3roUZT7ZdTlwvY7nJSszUo9AeWXyucq9YRN$>
 states:



" LSPs transmitted to adjacent

   neighbors on the DNR list, however, MUST be transmitted using a

   circuit scope PDU as described in [RFC7356]."



Although this text is not new to this revision, it raises a number of 
questions/concerns - some of which relate to how this draft fits with 
draft-ietf-lsr-dynamic-flooding.



For "DNR neighbors", although a circuit scoped LSP was received, the actual 
content is a traditional area scoped LSP.

<Shraddha> yes.

This presumably needs to be used internally (e.g., when running the Decision 
Process) in the same way as an area scoped LSP.

<Shraddha> yes.

So there are now two internal databases (traditional area LSPs and Circuit 
Scoped LSPs) which have to be used as if they are a single database?

<Shraddha> Yes that is correct

When a node sends a traditional CSNP does it include the LSPs it received as 
Circuit-Scoped LSPs?

If not, how is resynchronization achieved when necessary?

<Shraddha> yes. CSNPs includes circuit scoped LSPs.

Since you raised this point, I am thinking that we need a protocol extension to 
indicate that "this circuit scoped LSP" should be treated

In this special way.



Given the base work defined in draft-ietf-lsr-dynamic-flooding why is the use 
of Circuit Scoped LSPs required/useful?

<Shraddha> I am not sure which base work you are suggesting to use. Could you 
pls provide details?

The flooding algorithm as written in the draft breaks if circuit scoped 
flooding is not done.

Because, the distributed algorithm does not flood the LSP back in the direction 
from where it was originated.



[Les:] If you use the mechanisms defined in draft-ietf-lsr-dynamic-flooding 
there would be no need to use Circuit Scoped Flooding to send normal area 
scoped LSPs.

Each node in the topology would know to which neighbors it should have flooding 
enabled - either because you operate in centralized mode and the Area Leader 
distributes the flooding topology or because you operate in distributed mode 
and all nodes employ the same algorithm. (The latter assumes the algorithm 
supports deterministic choices for each node.) And when topology changes occur 
the mechanisms defined in 
https://tools.ietf.org/html/draft-ietf-lsr-dynamic-flooding-07#section-6.8<https://urldefense.com/v3/__https:/tools.ietf.org/html/draft-ietf-lsr-dynamic-flooding-07*section-6.8__;Iw!!NEt6yMaO-gk!VproGis8uUGmcl-cCplFMwuNceAo-3roUZT7ZdTlwvY7nJSszUo9AeWXymgguZIt$>
 assure that flooding continues to be reliable during the transition period 
(i.e., until a new flooding topology is calculated).



The fact that you apparently do NOT want to use the extensions defined in the 
dynamic-flooding draft means that (as you agree above) you have to introduce 
additional protocol extensions which aren't really necessary. This makes the 
solution much less appealing to me - independent of the merits/demerits of the 
algorithm itself.



The use of Circuit Scoped LSPs to send traditional area scoped LSPs is not at 
all what RFC 7356 intends. And it causes difficulties (as you have noted above) 
in what the content of CSNPs should be and how the different LSPDBs are used 
internally.

I really do not like these aspects.



The dynamic flooding draft wasn't in existence when Russ wrote the early 
versions of this draft. I am suggesting you might want to rethink your approach 
to take advantage of what dynamic flooding draft has defined.



   Les





In general, it would be helpful to more completely define the relationship 
between this draft and draft-ietf-lsr-dynamic-flooding.

<Shraddha> yes. I'll produce next version soon.



Thanx.



   Les





> -----Original Message-----

> From: Lsr <lsr-boun...@ietf.org<mailto:lsr-boun...@ietf.org>> On Behalf Of 
> Shraddha Hegde

> Sent: Sunday, November 29, 2020 8:36 PM

> To: lsr@ietf.org<mailto:lsr@ietf.org>

> Subject: [Lsr] FW: New Version Notification for draft-white-lsr-distoptflood-

> 00.txt

>

> WG members,

>

>

> We have posted new version of the draft-white-lsr-distoptflood.

> This draft has been around for sometime by name openfabric and

> draft-white-distoptflood. The current revision has the name changed

> to reflect LSR WG.

> This draft describes a flood reduction mechanism in ISIS which is based on

> similar mechanisms implemented in OSPF for mobile-ad-hoc Networks

> ([RFC5449], [RFC5614], and [RFC7182]).

>

> Request working group to review and provide inputs.

>

> Rgds

> Shraddha

>

>

> Juniper Business Use Only

>

> -----Original Message-----

> From: internet-dra...@ietf.org<mailto:internet-dra...@ietf.org> 
> <internet-dra...@ietf.org<mailto:internet-dra...@ietf.org>>

> Sent: Monday, November 30, 2020 9:59 AM

> To: Shraddha Hegde <shrad...@juniper.net<mailto:shrad...@juniper.net>>; Russ 
> White <r...@riw.us<mailto:r...@riw.us>>;

> Shawn Zandi <sza...@linkedin.com<mailto:sza...@linkedin.com>>

> Subject: New Version Notification for draft-white-lsr-distoptflood-00.txt

>

> [External Email. Be cautious of content]

>

>

> A new version of I-D, draft-white-lsr-distoptflood-00.txt

> has been successfully submitted by Shraddha Hegde and posted to the IETF

> repository.

>

> Name:           draft-white-lsr-distoptflood

> Revision:       00

> Title:          IS-IS Optimal Distributed Flooding for Dense Topologies

> Document date:  2020-11-29

> Group:          Individual Submission

> Pages:          12

> URL:

> https://urldefense.com/v3/__https://www.ietf.org/archive/id/draft-white-<https://urldefense.com/v3/__https:/www.ietf.org/archive/id/draft-white-lsr-distoptflood-00.txt__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSEm2FfjO$>

> lsr-distoptflood-00.txt__;!!NEt6yMaO-<https://urldefense.com/v3/__https:/www.ietf.org/archive/id/draft-white-lsr-distoptflood-00.txt__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSEm2FfjO$>

> gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSEm2FfjO<https://urldefense.com/v3/__https:/www.ietf.org/archive/id/draft-white-lsr-distoptflood-00.txt__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSEm2FfjO$>

> $<https://urldefense.com/v3/__https:/www.ietf.org/archive/id/draft-white-lsr-distoptflood-00.txt__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSEm2FfjO$>

> Status:

> https://urldefense.com/v3/__https://datatracker.ietf.org/doc/draft-white-<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/draft-white-lsr-distoptflood/__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSC0pwavY$>

> lsr-distoptflood/__;!!NEt6yMaO-<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/draft-white-lsr-distoptflood/__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSC0pwavY$>

> gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSC0pwav<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/draft-white-lsr-distoptflood/__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSC0pwavY$>

> Y$<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/draft-white-lsr-distoptflood/__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSC0pwavY$>

> Htmlized:

> https://urldefense.com/v3/__https://datatracker.ietf.org/doc/html/draft-<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/html/draft-white-lsr-distoptflood__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSFYebZSl$>

> white-lsr-distoptflood__;!!NEt6yMaO-<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/html/draft-white-lsr-distoptflood__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSFYebZSl$>

> gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSFYebZSl<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/html/draft-white-lsr-distoptflood__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSFYebZSl$>

> $<https://urldefense.com/v3/__https:/datatracker.ietf.org/doc/html/draft-white-lsr-distoptflood__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSFYebZSl$>

> Htmlized:       
> https://urldefense.com/v3/__https://tools.ietf.org/html/draft-<https://urldefense.com/v3/__https:/tools.ietf.org/html/draft-white-lsr-distoptflood-00__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSANTt6Q2$>

> white-lsr-distoptflood-00__;!!NEt6yMaO-<https://urldefense.com/v3/__https:/tools.ietf.org/html/draft-white-lsr-distoptflood-00__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSANTt6Q2$>

> gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSANTt6Q<https://urldefense.com/v3/__https:/tools.ietf.org/html/draft-white-lsr-distoptflood-00__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSANTt6Q2$>

> 2$<https://urldefense.com/v3/__https:/tools.ietf.org/html/draft-white-lsr-distoptflood-00__;!!NEt6yMaO-gk!T5HX4Enn7jt3zwKoZBiA85YlysK0tdcELjk7Fy829PRSockmshBtqxzkSANTt6Q2$>

>

>

> Abstract:

>    In dense topologies, such as data center fabrics based on the Clos

>    and butterfly fabric topologies, flooding mechanisms designed for

>    sparse topologies, when used in these dense topologies, can

>    "overflood," or carry too many copies of topology and reachability to

>    fabric devices.  This results in slower convergence times and higher

>    resource utilization.  The modifications to the flooding mechanism in

>    the Intermediate System to Intermediate System (IS-IS) link state

>    protocol described in this document reduce resource utilization to a

>    minimum, while increaseing convergence performance in dense

>    topologies.

>

>    Note that a Clos fabric is used as the primary example of a desne

>    flooding topology throughout this document.  However, the flooding

>    optimizations described in this document apply to any dense topology.

>

>

>

>

> Please note that it may take a couple of minutes from the time of submission

> until the htmlized version and diff are available at tools.ietf.org.

>

> The IETF Secretariat

>

> _______________________________________________

> Lsr mailing list

> Lsr@ietf.org<mailto:Lsr@ietf.org>

> https://www.ietf.org/mailman/listinfo/lsr<https://urldefense.com/v3/__https:/www.ietf.org/mailman/listinfo/lsr__;!!NEt6yMaO-gk!VproGis8uUGmcl-cCplFMwuNceAo-3roUZT7ZdTlwvY7nJSszUo9AeWXylitwgOR$>
_______________________________________________
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr

Reply via email to