Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-25 Thread Christian Hopps


"Les Ginsberg (ginsberg)"  writes:


Chris -

With respect, if you want to move this discussion forward then please make a 
proposal.


Please see below..


If you read the thread (which is by now quite long) you will see that whenever 
a proposal is made I do offer my thoughts about it - and that includes 
proposals to use a capability advertisement.

If you are telling me that I do not need to restate that no encoding changes 
are required - point taken.


I wanted to make sure that I wasn't misunderstanding you, Having an 
understanding is the only way to move forward productively.

Thanks,
Chris.



   Les


-Original Message-
From: Christian Hopps 
Sent: Tuesday, October 25, 2022 2:29 PM
To: Les Ginsberg (ginsberg) 
Cc: bruno.decra...@orange.com; Christian Hopps ;
Peter Psenak (ppsenak) ; Tony Li ;
Robert Raszuk ; Henk Smit ;
lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
01.txt

[as wg-member]

Hi Les,

I want to highlight a particular thing your saying below b/c I'm worried I don't
understand it.

> That does not mean, however, that we need to alter the encoding of
> existing TLVs to support MP.

I could be mistaken here, but I don't think altering the TLVs is a solution that
is on the table, or that anyone is supporting that idea. Is there a particular
proposal you're worried about?

AFAICT what *has* been discussed is using a capability bit to somehow
manage the unfortunate situation created by vendor[s] shipping software
that does things which are incompatible with the deployed software that
follows published standards.

Thanks,
Chris.

"Les Ginsberg (ginsberg)"  writes:

> Bruno -
>
>
>
> I think people have posted to this thread with different intentions.
>
>
>
> Everyone agrees (I think...) that when MP-TLVs are sent for a given
> TLV and not all routers in the network support MP for that TLV that
> bad things will happen.
>
> My posts have been to emphasize that the method of encoding MP-TLVs
> requires no protocol extensions. This in no way conflicts with the
> previous sentence.
>
>
>
> Please see inline.
>
>
>
>> -Original Message-
>
>> From: bruno.decra...@orange.com 
>
>> Sent: Monday, October 24, 2022 6:22 AM
>
>> To: Les Ginsberg (ginsberg) ; Christian Hopps
>
>> 
>
>> Cc: Peter Psenak (ppsenak) ; Tony Li
>
>> ; Robert Raszuk ; Henk Smit
>
>> ; lsr@ietf.org
>
>> Subject: RE: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-
>
>> 01.txt
>
>>
>
>> Les, all
>
>>
>
>> Please see inline
>
>>
>
>> > From: Lsr  On Behalf Of Les Ginsberg
> (ginsberg)
>
>> > Sent: Friday, October 7, 2022 1:35 AM
>
>>
>
>> A bit late in the game. At least I've read all subsequent emails.
>
>>
>
>> > To: Christian Hopps 
>
>> > Cc: Peter Psenak (ppsenak) ; Tony Li
>
>> ; Robert Raszuk ; Henk Smit
>
>> ; lsr@ietf.org
>
>> > Subject: Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-
>
>> 01.txt
>
>> >
>
>> > Chris -
>
>> >
>
>> > Not sure what SE means but...one more significant point.
>
>> >
>
>> > Multiple implementations have successfully deployed MP-TLV
> without any
>
>> protocol extensions. We did not require a new sub-TLV, a new flag,
> a
>
>> sequence number...we simply send additional information encoded
>
>> according to existing standards. This isn't "luck" - it is
> following existing
>
>> standards.
>
>> >
>
>> > For implementations which do not process MP-TLVs correctly - why
> does
>
>> this happen?
>
>> > On the receive side, they do not have the intelligence in their
>
>> implementation to do a merge.
>
>> > On the transmit side they do not have the intelligence to
> generate multiple
>
>> TLVs.
>
>>
>
>> - That protocol does not disallow the sending of multi-part (sub)
> -TLVs is one
>
>> (good) thing.
>
>> - The encoding of MP-TLVs is another thing. (encoding detail IMHO,
> although
>
>> I understand and agree that this is not one for existing
> implementations)
>
>> - Another aspect is the behavior expected on the receiving side. If
> that
>
>> behavior is not specified, that's out of spec/standard in the
> general case,
>
>> especially as in this case different spec specified different
> behaviors.
>
>>
>
>> The following example (for sub-TLVs) show that there is at least
> two
>
>> behaviors: "undefined", "merge"

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-25 Thread Les Ginsberg (ginsberg)
Chris -

With respect, if you want to move this discussion forward then please make a 
proposal.
If you read the thread (which is by now quite long) you will see that whenever 
a proposal is made I do offer my thoughts about it - and that includes 
proposals to use a capability advertisement.

If you are telling me that I do not need to restate that no encoding changes 
are required - point taken.

   Les

> -Original Message-
> From: Christian Hopps 
> Sent: Tuesday, October 25, 2022 2:29 PM
> To: Les Ginsberg (ginsberg) 
> Cc: bruno.decra...@orange.com; Christian Hopps ;
> Peter Psenak (ppsenak) ; Tony Li ;
> Robert Raszuk ; Henk Smit ;
> lsr@ietf.org
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
> 
> [as wg-member]
> 
> Hi Les,
> 
> I want to highlight a particular thing your saying below b/c I'm worried I 
> don't
> understand it.
> 
> > That does not mean, however, that we need to alter the encoding of
> > existing TLVs to support MP.
> 
> I could be mistaken here, but I don't think altering the TLVs is a solution 
> that
> is on the table, or that anyone is supporting that idea. Is there a particular
> proposal you're worried about?
> 
> AFAICT what *has* been discussed is using a capability bit to somehow
> manage the unfortunate situation created by vendor[s] shipping software
> that does things which are incompatible with the deployed software that
> follows published standards.
> 
> Thanks,
> Chris.
> 
> "Les Ginsberg (ginsberg)"  writes:
> 
> > Bruno -
> >
> >
> >
> > I think people have posted to this thread with different intentions.
> >
> >
> >
> > Everyone agrees (I think...) that when MP-TLVs are sent for a given
> > TLV and not all routers in the network support MP for that TLV that
> > bad things will happen.
> >
> > My posts have been to emphasize that the method of encoding MP-TLVs
> > requires no protocol extensions. This in no way conflicts with the
> > previous sentence.
> >
> >
> >
> > Please see inline.
> >
> >
> >
> >> -Original Message-
> >
> >> From: bruno.decra...@orange.com 
> >
> >> Sent: Monday, October 24, 2022 6:22 AM
> >
> >> To: Les Ginsberg (ginsberg) ; Christian Hopps
> >
> >> 
> >
> >> Cc: Peter Psenak (ppsenak) ; Tony Li
> >
> >> ; Robert Raszuk ; Henk Smit
> >
> >> ; lsr@ietf.org
> >
> >> Subject: RE: [Lsr] New Version Notification for
> > draft-pkaneria-lsr-multi-tlv-
> >
> >> 01.txt
> >
> >>
> >
> >> Les, all
> >
> >>
> >
> >> Please see inline
> >
> >>
> >
> >> > From: Lsr  On Behalf Of Les Ginsberg
> > (ginsberg)
> >
> >> > Sent: Friday, October 7, 2022 1:35 AM
> >
> >>
> >
> >> A bit late in the game. At least I've read all subsequent emails.
> >
> >>
> >
> >> > To: Christian Hopps 
> >
> >> > Cc: Peter Psenak (ppsenak) ; Tony Li
> >
> >> ; Robert Raszuk ; Henk Smit
> >
> >> ; lsr@ietf.org
> >
> >> > Subject: Re: [Lsr] New Version Notification for
> > draft-pkaneria-lsr-multi-tlv-
> >
> >> 01.txt
> >
> >> >
> >
> >> > Chris -
> >
> >> >
> >
> >> > Not sure what SE means but...one more significant point.
> >
> >> >
> >
> >> > Multiple implementations have successfully deployed MP-TLV
> > without any
> >
> >> protocol extensions. We did not require a new sub-TLV, a new flag,
> > a
> >
> >> sequence number...we simply send additional information encoded
> >
> >> according to existing standards. This isn't "luck" - it is
> > following existing
> >
> >> standards.
> >
> >> >
> >
> >> > For implementations which do not process MP-TLVs correctly - why
> > does
> >
> >> this happen?
> >
> >> > On the receive side, they do not have the intelligence in their
> >
> >> implementation to do a merge.
> >
> >> > On the transmit side they do not have the intelligence to
> > generate multiple
> >
> >> TLVs.
> >
> >>
> >
> >> - That protocol does not disallow the sending of multi-part (sub)
> > -TLVs is one
> >
> >> (good) thing.
> >
> >> - The encoding of MP-TLVs is an

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-25 Thread Christian Hopps

[as wg-member]

Hi Les,

I want to highlight a particular thing your saying below b/c I'm worried I 
don't understand it.


That does not mean, however, that we need to alter the encoding of
existing TLVs to support MP.


I could be mistaken here, but I don't think altering the TLVs is a solution 
that is on the table, or that anyone is supporting that idea. Is there a 
particular proposal you're worried about?

AFAICT what *has* been discussed is using a capability bit to somehow manage 
the unfortunate situation created by vendor[s] shipping software that does 
things which are incompatible with the deployed software that follows published 
standards.

Thanks,
Chris.

"Les Ginsberg (ginsberg)"  writes:


Bruno -



I think people have posted to this thread with different intentions.



Everyone agrees (I think...) that when MP-TLVs are sent for a given
TLV and not all routers in the network support MP for that TLV that
bad things will happen.

My posts have been to emphasize that the method of encoding MP-TLVs
requires no protocol extensions. This in no way conflicts with the
previous sentence.



Please see inline.




-Original Message-



From: bruno.decra...@orange.com 



Sent: Monday, October 24, 2022 6:22 AM



To: Les Ginsberg (ginsberg) ; Christian Hopps







Cc: Peter Psenak (ppsenak) ; Tony Li



; Robert Raszuk ; Henk Smit



; lsr@ietf.org



Subject: RE: [Lsr] New Version Notification for

draft-pkaneria-lsr-multi-tlv-


01.txt







Les, all







Please see inline







> From: Lsr  On Behalf Of Les Ginsberg

(ginsberg)


> Sent: Friday, October 7, 2022 1:35 AM







A bit late in the game. At least I've read all subsequent emails.







> To: Christian Hopps 



> Cc: Peter Psenak (ppsenak) ; Tony Li



; Robert Raszuk ; Henk Smit



; lsr@ietf.org



> Subject: Re: [Lsr] New Version Notification for

draft-pkaneria-lsr-multi-tlv-


01.txt



>



> Chris -



>



> Not sure what SE means but...one more significant point.



>



> Multiple implementations have successfully deployed MP-TLV

without any


protocol extensions. We did not require a new sub-TLV, a new flag,

a


sequence number...we simply send additional information encoded



according to existing standards. This isn't "luck" - it is

following existing


standards.



>



> For implementations which do not process MP-TLVs correctly - why

does


this happen?



> On the receive side, they do not have the intelligence in their



implementation to do a merge.



> On the transmit side they do not have the intelligence to

generate multiple


TLVs.







- That protocol does not disallow the sending of multi-part (sub)

-TLVs is one


(good) thing.



- The encoding of MP-TLVs is another thing. (encoding detail IMHO,

although


I understand and agree that this is not one for existing

implementations)


- Another aspect is the behavior expected on the receiving side. If

that


behavior is not specified, that's out of spec/standard in the

general case,


especially as in this case different spec specified different

behaviors.






The following example (for sub-TLVs) show that there is at least

two


behaviors: "undefined", "merge"







"Where a receiving system has two copies of a capabilities TLV from



   the same system that have different settings for a given

attribute,


   the procedure used to choose which copy shall be used is

undefined."


https://datatracker.ietf.org/doc/html/rfc4971#section-3







"undefined" is different from "merge".



(Also note the terms "to choose which copy shall be used" could

even be


seen as excluding the possibility for a merge, at least in the mind

of the


authors)






[LES:] You have misinterpreted this excerpt from RFC 4971 (now RFC
7981).



Where a receiving system has two copies of an IS-IS Router CAPABILITY

   TLV from the same system that have conflicting information for a

   given sub-TLV, the procedure used to choose which copy shall be
used

   is undefined.



The text is talking about what an implementation should do when the
same sub-TLV is received with two different values.

It is not talking about what should be done when MP is received for a
given TLV.






Then FlexAlgo defined "merge"  for its FAD with the spec for it







"In such cases, the



   FAD MAY be split into multiple such sub-TLVs and the content of

the


   multiple FAD sub-TLVs combined to provide a complete FAD for the



   Flex-Algorithm.  In such a case, the fixed portion of the FAD

(see


   Section 5.1) MUST be identical in all FAD sub-TLVs for a given

Flex-


   Algorithm from a given IS."







And also raised the question for sub-sub-TLVs, explicitly allowing

for any


behavior







"   Any specification that introduces a new IS-IS FAD sub-sub-TLV

MUST


   specify whether the FAD sub-TLV may 

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-24 Thread Les Ginsberg (ginsberg)
Bruno -



I think people have posted to this thread with different intentions.



Everyone agrees (I think...) that when MP-TLVs are sent for a given TLV and not 
all routers in the network support MP for that TLV that bad things will happen.

My posts have been to emphasize that the method of encoding MP-TLVs requires no 
protocol extensions. This in no way conflicts with the previous sentence.



Please see inline.



> -Original Message-

> From: bruno.decra...@orange.com 

> Sent: Monday, October 24, 2022 6:22 AM

> To: Les Ginsberg (ginsberg) ; Christian Hopps

> 

> Cc: Peter Psenak (ppsenak) ; Tony Li

> ; Robert Raszuk ; Henk Smit

> ; lsr@ietf.org

> Subject: RE: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-

> 01.txt

>

> Les, all

>

> Please see inline

>

> > From: Lsr mailto:lsr-boun...@ietf.org>> On Behalf Of 
> > Les Ginsberg (ginsberg)

> > Sent: Friday, October 7, 2022 1:35 AM

>

> A bit late in the game. At least I've read all subsequent emails.

>

> > To: Christian Hopps mailto:cho...@chopps.org>>

> > Cc: Peter Psenak (ppsenak) mailto:ppse...@cisco.com>>; 
> > Tony Li

> mailto:tony...@tony.li>>; Robert Raszuk 
> mailto:rob...@raszuk.net>>; Henk Smit

> mailto:henk.i...@xs4all.nl>>; 
> lsr@ietf.org<mailto:lsr@ietf.org>

> > Subject: Re: [Lsr] New Version Notification for 
> > draft-pkaneria-lsr-multi-tlv-

> 01.txt

> >

> > Chris -

> >

> > Not sure what SE means but...one more significant point.

> >

> > Multiple implementations have successfully deployed MP-TLV without any

> protocol extensions. We did not require a new sub-TLV, a new flag, a

> sequence number...we simply send additional information encoded

> according to existing standards. This isn't "luck" - it is following existing

> standards.

> >

> > For implementations which do not process MP-TLVs correctly - why does

> this happen?

> > On the receive side, they do not have the intelligence in their

> implementation to do a merge.

> > On the transmit side they do not have the intelligence to generate multiple

> TLVs.

>

> - That protocol does not disallow the sending of multi-part (sub)-TLVs is one

> (good) thing.

> - The encoding of MP-TLVs is another thing. (encoding detail IMHO, although

> I understand and agree that this is not one for existing implementations)

> - Another aspect is the behavior expected on the receiving side. If that

> behavior is not specified, that's out of spec/standard in the general case,

> especially as in this case different spec specified different behaviors.

>

> The following example (for sub-TLVs) show that there is at least two

> behaviors: "undefined", "merge"

>

> "Where a receiving system has two copies of a capabilities TLV from

>the same system that have different settings for a given attribute,

>the procedure used to choose which copy shall be used is undefined."

> https://datatracker.ietf.org/doc/html/rfc4971#section-3

>

> "undefined" is different from "merge".

> (Also note the terms "to choose which copy shall be used" could even be

> seen as excluding the possibility for a merge, at least in the mind of the

> authors)

>

[LES:] You have misinterpreted this excerpt from RFC 4971 (now RFC 7981).


Where a receiving system has two copies of an IS-IS Router CAPABILITY
   TLV from the same system that have conflicting information for a
   given sub-TLV, the procedure used to choose which copy shall be used
   is undefined.



The text is talking about what an implementation should do when the same 
sub-TLV is received with two different values.

It is not talking about what should be done when MP is received for a given TLV.





> Then FlexAlgo defined "merge"  for its FAD with the spec for it

>

> "In such cases, the

>FAD MAY be split into multiple such sub-TLVs and the content of the

>multiple FAD sub-TLVs combined to provide a complete FAD for the

>Flex-Algorithm.  In such a case, the fixed portion of the FAD (see

>Section 5.1) MUST be identical in all FAD sub-TLVs for a given Flex-

>Algorithm from a given IS."

>

> And also raised the question for sub-sub-TLVs, explicitly allowing for any

> behavior

>

> "   Any specification that introduces a new IS-IS FAD sub-sub-TLV MUST

>specify whether the FAD sub-TLV may appear multiple times in the set

>of FAD sub-TLVs for a given Flex-Algorithm from a given IS and how to

>handle them if multiple are allowed."



[LES:] I certainly agree that it is “better” if a specification is expli

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-24 Thread bruno.decraene
Les, all

Please see inline 

> From: Lsr  On Behalf Of Les Ginsberg (ginsberg)
> Sent: Friday, October 7, 2022 1:35 AM

A bit late in the game. At least I've read all subsequent emails.

> To: Christian Hopps 
> Cc: Peter Psenak (ppsenak) ; Tony Li ; 
> Robert Raszuk ; Henk Smit ; 
> lsr@ietf.org
> Subject: Re: [Lsr] New Version Notification for 
> draft-pkaneria-lsr-multi-tlv-01.txt
> 
> Chris -
> 
> Not sure what SE means but...one more significant point.
> 
> Multiple implementations have successfully deployed MP-TLV without any 
> protocol extensions. We did not require a new sub-TLV, a new flag, a sequence 
> number...we simply send additional information encoded according to existing 
> standards. This isn't "luck" - it is following existing standards.
> 
> For implementations which do not process MP-TLVs correctly - why does this 
> happen?
> On the receive side, they do not have the intelligence in their 
> implementation to do a merge.
> On the transmit side they do not have the intelligence to generate multiple 
> TLVs.

- That protocol does not disallow the sending of multi-part (sub)-TLVs is one 
(good) thing.
- The encoding of MP-TLVs is another thing. (encoding detail IMHO, although I 
understand and agree that this is not one for existing implementations)
- Another aspect is the behavior expected on the receiving side. If that 
behavior is not specified, that's out of spec/standard in the general case, 
especially as in this case different spec specified different behaviors.

The following example (for sub-TLVs) show that there is at least two behaviors: 
"undefined", "merge"

"Where a receiving system has two copies of a capabilities TLV from
   the same system that have different settings for a given attribute,
   the procedure used to choose which copy shall be used is undefined."
https://datatracker.ietf.org/doc/html/rfc4971#section-3

"undefined" is different from "merge".
(Also note the terms "to choose which copy shall be used" could even be seen as 
excluding the possibility for a merge, at least in the mind of the authors)

Then FlexAlgo defined "merge"  for its FAD with the spec for it

"In such cases, the
   FAD MAY be split into multiple such sub-TLVs and the content of the
   multiple FAD sub-TLVs combined to provide a complete FAD for the
   Flex-Algorithm.  In such a case, the fixed portion of the FAD (see
   Section 5.1) MUST be identical in all FAD sub-TLVs for a given Flex-
   Algorithm from a given IS."

And also raised the question for sub-sub-TLVs, explicitly allowing for any 
behavior 

"   Any specification that introduces a new IS-IS FAD sub-sub-TLV MUST
   specify whether the FAD sub-TLV may appear multiple times in the set
   of FAD sub-TLVs for a given Flex-Algorithm from a given IS and how to
   handle them if multiple are allowed."

https://datatracker.ietf.org/doc/html/draft-ietf-lsr-flex-algo-25#section-6

Bottom lines:
- if the spec does not define the MP-TLV behavior on the receiving side by the 
spec, then -unless this is obvious and everyone agrees- this is out of spec. 
Let's not blame the receiver.
- I think we should distinguish the different aspects: allowed on the sending 
side, the encoding, the behavior on the receiving side, the need to signal the 
MP capability or not, the reaction to this signal (which can simply be an alarm 
message in log, with no IS-IS actions). I suspect that different persons have 
different sensibilities on each of those points and a sensibility with one does 
not equal to a sensibility with all. (let's not make any disagreement bigger 
than it is)

Thanks,
--Bruno


> You can propose protocol extensions (such as you have done) - but it will not 
> change the need for implementations to enhance their receive/generation logic 
> - and it will not make it any easier for implementations to do so. What it 
> will do is to introduce(sic) an interoperability problem because you will be 
> requiring implementations to understand some new advertisement in order to 
> send/receive MP-TLVs successfully. This is what Peter's point is about i.e., 
> we MUST NOT break existing working MP-TLV implementations by requiring 
> protocol extensions in order to send MP-TLVs.
> 
> As regards deployment controls, I have no problem with recommending that 
> implementations provide ways to control the enablement of the sending of 
> MP-TLVs. 
> 
   > Les
> 
> > -Original Message-
> > From: Christian Hopps 
> > Sent: Thursday, October 6, 2022 3:28 PM
> > To: Les Ginsberg (ginsberg) 
> > Cc: Christian Hopps ; Peter Psenak (ppsenak)
> > ; Tony Li ; Robert Raszuk
> > ; Henk Smit ; lsr@ietf.org
> > Subject: Re: [Lsr] New Version Notification for 
> > draft-pkaneria-lsr-mul

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-09 Thread Aijun Wang
>From my understanding, to introduce the Multi-part TLV into the network, the
following two things should be done:
1) The capability negotiation. Unless all of nodes support such
capabilities, the advertisement of Multi-part TLV should not be initiated,
or else, lack of the correct parsing of Mult-part TLV by one of the nodes
will result in the inconsistence of LSDB, which may result in the forwarding
loop.
2) The indication of the Multi-part TLV is present, similar with the
fragment flag for IP packet encapsulation. Such indication can distinct the
Multi-part TLV from the repeated advertisements of the same TLV.

Is there any other difficult points to be solved?


Best Regards

Aijun Wang
China Telecom

-Original Message-
From: lsr-boun...@ietf.org  On Behalf Of Christian
Hopps
Sent: Sunday, October 9, 2022 8:49 AM
To: Les Ginsberg (ginsberg) 
Cc: Christian Hopps ; Tony Li ; Peter
Psenak (ppsenak) ; Robert Raszuk ;
Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for
draft-pkaneria-lsr-multi-tlv-01.txt


Christian Hopps  writes:
>>> I simply turned your question around and asked: should conforming
>>
>>> implementations be penalized?
>>
>> [LES:] Are you claiming that the absence of an explicit statement 
>> regarding support of MP for a given TLV  is equivalent to a 
>> prohibition against sending them (which fails basic logic)?
>
> Why did we explicitly define multi-part TLVs?
>
> I agree we appear to not making any progress here. Perhaps it would be
more productive to have a discussion in a different forum like an interim or
something similar.
>
> Thanks,
> Chris.
> [as wg-chair]

Gah, I meant "as wg-member"! However, the suggestion of a different forum
could probably come from the chair hat as well. :)

Thanks,
Chris.

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-08 Thread Christian Hopps



Christian Hopps  writes:

I simply turned your question around and asked: should conforming



implementations be penalized?


[LES:] Are you claiming that the absence of an explicit statement
regarding support of MP for a given TLV  is equivalent to a
prohibition against sending them (which fails basic logic)?


Why did we explicitly define multi-part TLVs?

I agree we appear to not making any progress here. Perhaps it would be more 
productive to have a discussion in a different forum like an interim or 
something similar.

Thanks,
Chris.
[as wg-chair]


Gah, I meant "as wg-member"! However, the suggestion of a different forum could 
probably come from the chair hat as well. :)

Thanks,
Chris.

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-08 Thread Christian Hopps



"Les Ginsberg (ginsberg)"  writes:


Chris -



I am a bit concerned that this is degenerating into a
non-constructive argument. We can disagee but hopefully each post
helps move the discussion along in some way.

Please see inline.




-Original Message-



From: Christian Hopps 



Sent: Saturday, October 8, 2022 2:48 PM



To: Les Ginsberg (ginsberg) 



Cc: Christian Hopps ; Tony Li ;

Peter


Psenak (ppsenak) ; Robert Raszuk



; Henk Smit ; lsr@ietf.org



Subject: Re: [Lsr] New Version Notification for

draft-pkaneria-lsr-multi-tlv-


01.txt











I simply turned your question around and asked: should conforming



implementations be penalized?




[LES:] Are you claiming that the absence of an explicit statement
regarding support of MP for a given TLV  is equivalent to a
prohibition against sending them (which fails basic logic)?


Why did we explicitly define multi-part TLVs?

I agree we appear to not making any progress here. Perhaps it would be more 
productive to have a discussion in a different forum like an interim or 
something similar.

Thanks,
Chris.
[as wg-chair]


If not, I do not understand what it is that you think implementations
which are NOT supporting MP for a given TLV are conforming to.








I can't imagine why you are explaining IS-IS TLVs to me, and I have

never


advocated for nor even mentioned changing the encoding of TLVs so I

don't


know what you're talking about with the rest of it. I found this

very


distracting.








[LES:] A couple of quotes from previous emails you have sent:





... I think a stronger case might be made for actually having the
router capability be used *operationally* (i.e., if you don't see the
capability advertised then that router in fact doesn't send multi-tlv
tlvs and they should be seen as replacements of each other)...





This proposes making it illegal to send MP-TLVs unless a capability
is advertised network-wide.





If things would have been done correctly (i.e., documented) we also
would have had the option to add a sub-tlv to the TLVs in question
that indicated they were part of a set rather than replacements.





This proposes adding a sub-TLV into existing TLVs to indicate they
are part of a set of MP-TLVs.



??



   Les






Chris.



[as wg-member]







"Les Ginsberg (ginsberg)"  writes:







> Chris -



>



> There are two different topics under discussion - one has to do

with TLV


encoding - the other has to do with partial deployment issues. We

need to


be careful that we keep the two distinct.



>



> TLVs necessarily have to unambiguously define the object followed

by


attributes of that object.



> If multiple TLVs are required to advertise all the attributes of

an object, the


encoding requirements are the same: unambiguously define the object

in


each TLV and then include attributes.



> IS-IS advertisements have never had any "ordering' requirements

i.e.,


attributes can be advertised in any order within a given TLV and it

makes no


difference whether a TLV is advertised in LSP#2 or LSP#200.



> This means that when MP is used, there is no notion as to whether

a TLV is


the first in a set or the last in a set - they are simply

complementary.


>



> When I say that implementations which have added support for

MP-TLVs


for



> neighbor/prefix advertisement have done the "right thing" I am

simply


saying



> they are using base protocol encoding rules as discussed above.



>



> There is another discussion that has to do with partial

deployment issues


and



> legitimate concerns about how a network might behave when not all



routers



> support MP for a given TLV. But altering TLV encoding to address

that issue


does



> not solve a problem - it simply creates a new one.



> Further, introducing a new rule that prohibits advertisement of

existing


TLVs



> unless some yet to be defined feature support advertisement is

present


> network-wide also does not solve a problem - it introduces a new

one.


>



> Whether advertising features supported is useful and how it

should be


done is a discussion that should continue - but it MUST NOT alter

the format


of TLVs or the rules as to when TLVs can be advertised.



>



>Les



>



>> -Original Message-



>> From: Lsr  On Behalf Of Christian Hopps



>> Sent: Friday, October 7, 2022 4:17 PM



>> To: Les Ginsberg (ginsberg) 



>> Cc: Tony Li ; Christian Hopps <

cho...@chopps.org>;


Peter



>> Psenak (ppsenak) ; Robert Raszuk



>> ; Henk Smit ;

lsr@ietf.org


>> Subject: Re: [Lsr] New Version Notification for

draft-pkaneria-lsr-multi-tlv-


>> 01.txt



>>



>>



>> "Les Ginsberg (ginsberg)"  writes:



>>



>> > Tony -



>> >



>> >



Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-08 Thread Les Ginsberg (ginsberg)
Chris -



I am a bit concerned that this is degenerating into a non-constructive 
argument. We can disagee but hopefully each post helps move the discussion 
along in some way.

Please see inline.



> -Original Message-

> From: Christian Hopps 

> Sent: Saturday, October 8, 2022 2:48 PM

> To: Les Ginsberg (ginsberg) 

> Cc: Christian Hopps ; Tony Li ; Peter

> Psenak (ppsenak) ; Robert Raszuk

> ; Henk Smit ; lsr@ietf.org

> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-

> 01.txt

>

>

> I simply turned your question around and asked: should conforming

> implementations be penalized?



[LES:] Are you claiming that the absence of an explicit statement regarding 
support of MP for a given TLV  is equivalent to a prohibition against sending 
them (which fails basic logic)?

If not, I do not understand what it is that you think implementations which are 
NOT supporting MP for a given TLV are conforming to.



>

> I can't imagine why you are explaining IS-IS TLVs to me, and I have never

> advocated for nor even mentioned changing the encoding of TLVs so I don't

> know what you're talking about with the rest of it. I found this very

> distracting.

>



[LES:] A couple of quotes from previous emails you have sent:





... I think a stronger case might be made for actually having the router 
capability be used *operationally* (i.e., if you don't see the capability 
advertised then that router in fact doesn't send multi-tlv tlvs and they should 
be seen as replacements of each other)...





This proposes making it illegal to send MP-TLVs unless a capability is 
advertised network-wide.





If things would have been done correctly (i.e., documented) we also would have 
had the option to add a sub-tlv to the TLVs in question that indicated they 
were part of a set rather than replacements.





This proposes adding a sub-TLV into existing TLVs to indicate they are part of 
a set of MP-TLVs.



??



   Les





> Chris.

> [as wg-member]

>

> "Les Ginsberg (ginsberg)" mailto:ginsb...@cisco.com>> 
> writes:

>

> > Chris -

> >

> > There are two different topics under discussion - one has to do with TLV

> encoding - the other has to do with partial deployment issues. We need to

> be careful that we keep the two distinct.

> >

> > TLVs necessarily have to unambiguously define the object followed by

> attributes of that object.

> > If multiple TLVs are required to advertise all the attributes of an object, 
> > the

> encoding requirements are the same: unambiguously define the object in

> each TLV and then include attributes.

> > IS-IS advertisements have never had any "ordering' requirements i.e.,

> attributes can be advertised in any order within a given TLV and it makes no

> difference whether a TLV is advertised in LSP#2 or LSP#200.

> > This means that when MP is used, there is no notion as to whether a TLV is

> the first in a set or the last in a set - they are simply complementary.

> >

> > When I say that implementations which have added support for MP-TLVs

> for

> > neighbor/prefix advertisement have done the "right thing" I am simply

> saying

> > they are using base protocol encoding rules as discussed above.

> >

> > There is another discussion that has to do with partial deployment issues

> and

> > legitimate concerns about how a network might behave when not all

> routers

> > support MP for a given TLV. But altering TLV encoding to address that issue

> does

> > not solve a problem - it simply creates a new one.

> > Further, introducing a new rule that prohibits advertisement of existing

> TLVs

> > unless some yet to be defined feature support advertisement is present

> > network-wide also does not solve a problem - it introduces a new one.

> >

> > Whether advertising features supported is useful and how it should be

> done is a discussion that should continue - but it MUST NOT alter the format

> of TLVs or the rules as to when TLVs can be advertised.

> >

> >Les

> >

> >> -Original Message-

> >> From: Lsr mailto:lsr-boun...@ietf.org>> On Behalf Of 
> >> Christian Hopps

> >> Sent: Friday, October 7, 2022 4:17 PM

> >> To: Les Ginsberg (ginsberg) mailto:ginsb...@cisco.com>>

> >> Cc: Tony Li mailto:tony...@tony.li>>; Christian Hopps 
> >> mailto:cho...@chopps.org>>;

> Peter

> >> Psenak (ppsenak) mailto:ppse...@cisco.com>>; Robert 
> >> Raszuk

> >> mailto:rob...@raszuk.net>>; Henk Smit 
> >> mailto:henk.i...@xs4all.nl>>; 
> >> lsr@ietf

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-08 Thread Christian Hopps



I simply turned your question around and asked: should conforming 
implementations be penalized?

I can't imagine why you are explaining IS-IS TLVs to me, and I have never 
advocated for nor even mentioned changing the encoding of TLVs so I don't know 
what you're talking about with the rest of it. I found this very distracting.

Chris.
[as wg-member]

"Les Ginsberg (ginsberg)"  writes:


Chris -

There are two different topics under discussion - one has to do with TLV 
encoding - the other has to do with partial deployment issues. We need to be 
careful that we keep the two distinct.

TLVs necessarily have to unambiguously define the object followed by attributes 
of that object.
If multiple TLVs are required to advertise all the attributes of an object, the 
encoding requirements are the same: unambiguously define the object in each TLV 
and then include attributes.
IS-IS advertisements have never had any "ordering' requirements i.e., 
attributes can be advertised in any order within a given TLV and it makes no 
difference whether a TLV is advertised in LSP#2 or LSP#200.
This means that when MP is used, there is no notion as to whether a TLV is the 
first in a set or the last in a set - they are simply complementary.

When I say that implementations which have added support for MP-TLVs for
neighbor/prefix advertisement have done the "right thing" I am simply saying
they are using base protocol encoding rules as discussed above.

There is another discussion that has to do with partial deployment issues and
legitimate concerns about how a network might behave when not all routers
support MP for a given TLV. But altering TLV encoding to address that issue does
not solve a problem - it simply creates a new one.
Further, introducing a new rule that prohibits advertisement of existing TLVs
unless some yet to be defined feature support advertisement is present
network-wide also does not solve a problem - it introduces a new one.

Whether advertising features supported is useful and how it should be done is a 
discussion that should continue - but it MUST NOT alter the format of TLVs or 
the rules as to when TLVs can be advertised.

   Les


-Original Message-
From: Lsr  On Behalf Of Christian Hopps
Sent: Friday, October 7, 2022 4:17 PM
To: Les Ginsberg (ginsberg) 
Cc: Tony Li ; Christian Hopps ; Peter
Psenak (ppsenak) ; Robert Raszuk
; Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
01.txt


"Les Ginsberg (ginsberg)"  writes:

> Tony -
>
>
>>
>> Your summarization is incorrect.
>>
>> The proposal is to advertise a advisory message that indicates that a node
is
>> ready to receive MP-TLVs. It prohibits nothing.
>
> [LES:] That is what you are proposing - but others in the thread have
proposed other ideas. For example, in an earlier post Chris stated:
>
>>> Once we have this info I think a stronger case might be made for actually
>>> having the router capability be used *operationally* (i.e., if you don't
see the
>>> capability advertised then that router in fact doesn't send multi-tlv tlvs
and
>>> they should be seen as replacements of each other),
>
> What I am trying to highlight is that the existing implementations of MP-
TLVs
> for the "implicit" cases should not be penalized for sending MP-TLVs that
are
> encoded consistent with how MP-TLVs for the "explicit" cases have been
done.
> They are actually doing the right thing.

And I disagree with the word "right" here. They are doing the advantageous
thing as long as one has the correct routers and software that allows you to
advertise more than fits in a single TLV. It certainly won't be the "right" 
thing
if a standards conforming implementation out there doesn't expect or deal
with one of these multi-part "implicit MP-TLV" advertisements.

Are you saying implementations that only handle a single-part TLV are non-
conforming? To turn your question around, why should they be penalized?

Thanks,
Chris.

>
> If we are in agreement on that - great!
>
>Les
>
>>
>> Tony
>>


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-08 Thread Les Ginsberg (ginsberg)
Chris -

There are two different topics under discussion - one has to do with TLV 
encoding - the other has to do with partial deployment issues. We need to be 
careful that we keep the two distinct.

TLVs necessarily have to unambiguously define the object followed by attributes 
of that object.
If multiple TLVs are required to advertise all the attributes of an object, the 
encoding requirements are the same: unambiguously define the object in each TLV 
and then include attributes.
IS-IS advertisements have never had any "ordering' requirements i.e., 
attributes can be advertised in any order within a given TLV and it makes no 
difference whether a TLV is advertised in LSP#2 or LSP#200.
This means that when MP is used, there is no notion as to whether a TLV is the 
first in a set or the last in a set - they are simply complementary.

When I say that implementations which have added support for MP-TLVs for 
neighbor/prefix advertisement have done the "right thing" I am simply saying 
they are using base protocol encoding rules as discussed above.

There is another discussion that has to do with partial deployment issues and 
legitimate concerns about how a network might behave when not all routers 
support MP for a given TLV. But altering TLV encoding to address that issue 
does not solve a problem - it simply creates a new one.
Further, introducing a new rule that prohibits advertisement of existing TLVs 
unless some yet to be defined feature support advertisement is present 
network-wide also does not solve a problem - it introduces a new one.

Whether advertising features supported is useful and how it should be done is a 
discussion that should continue - but it MUST NOT alter the format of TLVs or 
the rules as to when TLVs can be advertised.

   Les

> -Original Message-
> From: Lsr  On Behalf Of Christian Hopps
> Sent: Friday, October 7, 2022 4:17 PM
> To: Les Ginsberg (ginsberg) 
> Cc: Tony Li ; Christian Hopps ; Peter
> Psenak (ppsenak) ; Robert Raszuk
> ; Henk Smit ; lsr@ietf.org
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
> 
> 
> "Les Ginsberg (ginsberg)"  writes:
> 
> > Tony -
> >
> >
> >>
> >> Your summarization is incorrect.
> >>
> >> The proposal is to advertise a advisory message that indicates that a node
> is
> >> ready to receive MP-TLVs. It prohibits nothing.
> >
> > [LES:] That is what you are proposing - but others in the thread have
> proposed other ideas. For example, in an earlier post Chris stated:
> >
> >>> Once we have this info I think a stronger case might be made for actually
> >>> having the router capability be used *operationally* (i.e., if you don't
> see the
> >>> capability advertised then that router in fact doesn't send multi-tlv tlvs
> and
> >>> they should be seen as replacements of each other),
> >
> > What I am trying to highlight is that the existing implementations of MP-
> TLVs
> > for the "implicit" cases should not be penalized for sending MP-TLVs that
> are
> > encoded consistent with how MP-TLVs for the "explicit" cases have been
> done.
> > They are actually doing the right thing.
> 
> And I disagree with the word "right" here. They are doing the advantageous
> thing as long as one has the correct routers and software that allows you to
> advertise more than fits in a single TLV. It certainly won't be the "right" 
> thing
> if a standards conforming implementation out there doesn't expect or deal
> with one of these multi-part "implicit MP-TLV" advertisements.
> 
> Are you saying implementations that only handle a single-part TLV are non-
> conforming? To turn your question around, why should they be penalized?
> 
> Thanks,
> Chris.
> 
> >
> > If we are in agreement on that - great!
> >
> >Les
> >
> >>
> >> Tony
> >>

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-07 Thread Christian Hopps


"Les Ginsberg (ginsberg)"  writes:


Tony -




Your summarization is incorrect.

The proposal is to advertise a advisory message that indicates that a node is
ready to receive MP-TLVs. It prohibits nothing.


[LES:] That is what you are proposing - but others in the thread have proposed 
other ideas. For example, in an earlier post Chris stated:


Once we have this info I think a stronger case might be made for actually
having the router capability be used *operationally* (i.e., if you don't see the
capability advertised then that router in fact doesn't send multi-tlv tlvs and
they should be seen as replacements of each other),


What I am trying to highlight is that the existing implementations of MP-TLVs
for the "implicit" cases should not be penalized for sending MP-TLVs that are
encoded consistent with how MP-TLVs for the "explicit" cases have been done.
They are actually doing the right thing.


And I disagree with the word "right" here. They are doing the advantageous thing as long as one has 
the correct routers and software that allows you to advertise more than fits in a single TLV. It certainly 
won't be the "right" thing if a standards conforming implementation out there doesn't expect or 
deal with one of these multi-part "implicit MP-TLV" advertisements.

Are you saying implementations that only handle a single-part TLV are 
non-conforming? To turn your question around, why should they be penalized?

Thanks,
Chris.



If we are in agreement on that - great!

   Les



Tony





signature.asc
Description: PGP signature
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-07 Thread Tony Li

Hi Acee,

> You realize the latest version still has the statement: 
> 
>If all routers in an area advertise the Multi-part TLV Capability a
>node MAY advertise multi-part TLVs to increase space for payload
>values, unless otherwise specified by the TLV.
>  
> At a minimum, the draft should specify a configuration parameter dictating 
> whether advertisement of the capability by all area IS-IS routers is required 
> for advertisement. With this new parameter, my preference would be to then 
> leave it to implementations as to the default value, i.e., beyond the scope 
> of the document.


Yes, I realize that the current draft is out of date.  The pending version of 
the draft relaxes this.

I cannot publish a new version of the draft until we have consent from all of 
the authors, which we have not achieved.

Tony

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-07 Thread Acee Lindem (acee)
Hi Tony,


From: Lsr  on behalf of Tony Li 
Date: Friday, October 7, 2022 at 11:21 AM
To: "Les Ginsberg (ginsberg)" 
Cc: Christian Hopps , "Peter Psenak (ppsenak)" 
, Robert Raszuk , Henk Smit 
, "lsr@ietf.org" 
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Les,


On Oct 7, 2022, at 8:16 AM, Les Ginsberg (ginsberg) 
mailto:ginsberg=40cisco@dmarc.ietf.org>>
 wrote:

What I am trying to highlight is that the existing implementations of MP-TLVs 
for the "implicit" cases should not be penalized for sending MP-TLVs that are 
encoded consistent with how MP-TLVs for the "explicit" cases have been done. 
They are actually doing the right thing.

If we are in agreement on that - great!


I have no wish to penalize anyone.


You realize the latest version still has the statement:

   If all routers in an area advertise the Multi-part TLV Capability a
   node MAY advertise multi-part TLVs to increase space for payload
   values, unless otherwise specified by the TLV.

At a minimum, the draft should specify a configuration parameter dictating 
whether advertisement of the capability by all area IS-IS routers is required 
for advertisement. With this new parameter, my preference would be to then 
leave it to implementations as to the default value, i.e., beyond the scope of 
the document.

Thanks,
Acee


T

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-07 Thread Tony Li

Les,

> On Oct 7, 2022, at 8:16 AM, Les Ginsberg (ginsberg) 
>  wrote:
> 
> What I am trying to highlight is that the existing implementations of MP-TLVs 
> for the "implicit" cases should not be penalized for sending MP-TLVs that are 
> encoded consistent with how MP-TLVs for the "explicit" cases have been done. 
> They are actually doing the right thing.
> 
> If we are in agreement on that - great!


I have no wish to penalize anyone.

T

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-07 Thread Les Ginsberg (ginsberg)
Tony -


> 
> Your summarization is incorrect.
> 
> The proposal is to advertise a advisory message that indicates that a node is
> ready to receive MP-TLVs. It prohibits nothing.

[LES:] That is what you are proposing - but others in the thread have proposed 
other ideas. For example, in an earlier post Chris stated:

>> Once we have this info I think a stronger case might be made for actually
>> having the router capability be used *operationally* (i.e., if you don't see 
>> the
>> capability advertised then that router in fact doesn't send multi-tlv tlvs 
>> and
>> they should be seen as replacements of each other),

What I am trying to highlight is that the existing implementations of MP-TLVs 
for the "implicit" cases should not be penalized for sending MP-TLVs that are 
encoded consistent with how MP-TLVs for the "explicit" cases have been done. 
They are actually doing the right thing.

If we are in agreement on that - great!

   Les

> 
> Tony
> 

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-07 Thread Tony Li


> On Oct 6, 2022, at 1:56 PM, Christian Hopps  wrote:
> 
> Tony I think you may have interpreted these differently?


Ayup.  As stated, I am human.  I blew it.  Mea culpa.

I will rename the current columns and start over.  Contributors still welcome.

T

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-07 Thread Tony Li


Les,


> On Oct 6, 2022, at 7:22 PM, Les Ginsberg (ginsberg) 
>  wrote:
> 
> Chris -
> 
> Not trying to convince you of anything - just want to step back and summarize 
> where we are.
> 
> MP-TLV support has been explicitly allowed in multiple cases - and in these 
> cases no additional signaling has been specified i.e., all TLVs in the "set" 
> are encoded the same way they would be if the there were only 1 TLV in the 
> set and there is no signaling of MP-TLV support required in order to send 
> MP-TLVs.
> 
> Due to new features/increased scale, we now have a need to send MP-TLVs for 
> some additional TLVs (notably Prefix Reachability and Neighbor TLVs). The 
> RFCs defining these TLVs did not explicitly specify the use of MP-TLVs - but 
> they also did not specify any prohibition against doing so.
> 
> Some implementations have chosen to apply the same MP-TLV model used for the 
> "explicit-MP-TLV allowed" cases to these new cases.
> 
> Two possible protocol changes for these new cases have been suggested in this 
> thread:
> 
> 1)Require some additional encoding in the MP-TLVs to identify the TLV as a 
> member of an MP-TLV set
> 
> This would mean that the encoding of MP-TLVs would be different depending on 
> the TLV type.
> I see no justification for this.
> 
> 2)Prohibit the sending of MP-TLVs unless all nodes advertise support
> 
> Even though the encoding used by these early implementations is correct, they 
> would then be penalized and declared illegal because they did not come to the 
> WG and "ask for permission".
> I find this approach at best very petty.
> 
> I share the concern about how a network might behave when not all nodes 
> support MP-TLVs for a given TLV, but this is best handled by having an 
> implementation knob to disable the sending of MP-TLVs - thus giving the 
> operator control.
> 


Your summarization is incorrect.

The proposal is to advertise a advisory message that indicates that a node is 
ready to receive MP-TLVs. It prohibits nothing.

Tony


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Les Ginsberg (ginsberg)
Chris -

Not trying to convince you of anything - just want to step back and summarize 
where we are.

MP-TLV support has been explicitly allowed in multiple cases - and in these 
cases no additional signaling has been specified i.e., all TLVs in the "set" 
are encoded the same way they would be if the there were only 1 TLV in the set 
and there is no signaling of MP-TLV support required in order to send MP-TLVs.

Due to new features/increased scale, we now have a need to send MP-TLVs for 
some additional TLVs (notably Prefix Reachability and Neighbor TLVs). The RFCs 
defining these TLVs did not explicitly specify the use of MP-TLVs - but they 
also did not specify any prohibition against doing so.

Some implementations have chosen to apply the same MP-TLV model used for the 
"explicit-MP-TLV allowed" cases to these new cases.

Two possible protocol changes for these new cases have been suggested in this 
thread:

1)Require some additional encoding in the MP-TLVs to identify the TLV as a 
member of an MP-TLV set

This would mean that the encoding of MP-TLVs would be different depending on 
the TLV type.
I see no justification for this.

2)Prohibit the sending of MP-TLVs unless all nodes advertise support

Even though the encoding used by these early implementations is correct, they 
would then be penalized and declared illegal because they did not come to the 
WG and "ask for permission".
I find this approach at best very petty.

I share the concern about how a network might behave when not all nodes support 
MP-TLVs for a given TLV, but this is best handled by having an implementation 
knob to disable the sending of MP-TLVs - thus giving the operator control.

   Les



> -Original Message-
> From: Christian Hopps 
> Sent: Thursday, October 6, 2022 4:59 PM
> To: Les Ginsberg (ginsberg) 
> Cc: Christian Hopps ; Peter Psenak (ppsenak)
> ; Tony Li ; Robert Raszuk
> ; Henk Smit ; lsr@ietf.org
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
> 
> 
> Sending multi-part TLVs where the TLV type was not defined or talked about
> in the standard as supporting MP TLV behavior is not "according to existing
> standards". The fact that conforming implementations can be confused by
> this new behavior should be proof enough of that. But, then we have the
> fact that we *were* explicit about defining MP TLV behavior for other TLV
> types in other standards.. We'll have to agree to disagree here I think.
> 
> An SE is a sales-engineer they are the engineers a vendor sends to the
> customer to, among other things, help sell and integrate the product the
> customer is buying with the customers network. They would be privy to, for
> example, internal implementation details of the product and thus be able to
> assure the customer things would "just work", regardless of what a standard
> says.
> 
> Thanks,
> Chris.
> 
> "Les Ginsberg (ginsberg)"  writes:
> 
> > Chris -
> >
> > Not sure what SE means but...one more significant point.
> >
> > Multiple implementations have successfully deployed MP-TLV without any
> protocol
> > extensions. We did not require a new sub-TLV, a new flag, a sequence
> number...we
> > simply send additional information encoded according to existing
> standards. This
> > isn't "luck" - it is following existing standards.
> >
> > For implementations which do not process MP-TLVs correctly - why does
> this happen?
> > On the receive side, they do not have the intelligence in their
> implementation to do a merge.
> > On the transmit side they do not have the intelligence to generate multiple
> TLVs.
> >
> > You can propose protocol extensions (such as you have done) - but it will
> not
> > change the need for implementations to enhance their receive/generation
> logic -
> > and it will not make it any easier for implementations to do so. What it 
> > will
> do
> > is to introduce(sic) an interoperability problem because you will be
> requiring
> > implementations to understand some new advertisement in order to
> send/receive
> > MP-TLVs successfully. This is what Peter's point is about i.e., we MUST NOT
> > break existing working MP-TLV implementations by requiring protocol
> extensions
> > in order to send MP-TLVs.
> >
> > As regards deployment controls, I have no problem with recommending
> that implementations provide ways to control the enablement of the
> sending of MP-TLVs.
> >
> >    Les
> >
> >> -Original Message-
> >> From: Christian Hopps 
> >> Sent: Thursday, October 6, 2022 3:28 PM
> >> To: Les Ginsberg (ginsberg) 
> >> Cc: Chris

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Christian Hopps


Sending multi-part TLVs where the TLV type was not defined or talked about in the 
standard as supporting MP TLV behavior is not "according to existing 
standards". The fact that conforming implementations can be confused by this new 
behavior should be proof enough of that. But, then we have the fact that we *were* 
explicit about defining MP TLV behavior for other TLV types in other standards.. We'll 
have to agree to disagree here I think.

An SE is a sales-engineer they are the engineers a vendor sends to the customer to, among 
other things, help sell and integrate the product the customer is buying with the 
customers network. They would be privy to, for example, internal implementation details 
of the product and thus be able to assure the customer things would "just 
work", regardless of what a standard says.

Thanks,
Chris.

"Les Ginsberg (ginsberg)"  writes:


Chris -

Not sure what SE means but...one more significant point.

Multiple implementations have successfully deployed MP-TLV without any protocol
extensions. We did not require a new sub-TLV, a new flag, a sequence number...we
simply send additional information encoded according to existing standards. This
isn't "luck" - it is following existing standards.

For implementations which do not process MP-TLVs correctly - why does this 
happen?
On the receive side, they do not have the intelligence in their implementation 
to do a merge.
On the transmit side they do not have the intelligence to generate multiple 
TLVs.

You can propose protocol extensions (such as you have done) - but it will not
change the need for implementations to enhance their receive/generation logic -
and it will not make it any easier for implementations to do so. What it will do
is to introduce(sic) an interoperability problem because you will be requiring
implementations to understand some new advertisement in order to send/receive
MP-TLVs successfully. This is what Peter's point is about i.e., we MUST NOT
break existing working MP-TLV implementations by requiring protocol extensions
in order to send MP-TLVs.

As regards deployment controls, I have no problem with recommending that 
implementations provide ways to control the enablement of the sending of 
MP-TLVs.

   Les


-Original Message-
From: Christian Hopps 
Sent: Thursday, October 6, 2022 3:28 PM
To: Les Ginsberg (ginsberg) 
Cc: Christian Hopps ; Peter Psenak (ppsenak)
; Tony Li ; Robert Raszuk
; Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
01.txt


It sounds like you're talking about networks defined to work by SE not by
standards. I don't want to argue about this, so perhaps we can agree to
disagree.

Thanks,
Chris.
[as wg-member]


"Les Ginsberg (ginsberg)"  writes:

> Chris -
>
>> -Original Message-
>> From: Christian Hopps 
>> Sent: Thursday, October 6, 2022 1:36 PM
>> To: Peter Psenak (ppsenak) 
>> Cc: Christian Hopps ; Tony Li ; Les
>> Ginsberg (ginsberg) ; Robert Raszuk
>> ; Henk Smit ; lsr@ietf.org
>> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
>> 01.txt
>>
>>
>> Peter Psenak  writes:
>>
>> > Chris,
>> >
>> > On 06/10/2022 18:34, Christian Hopps wrote:
>> >> Peter Psenak  writes:
>> >>
>> >>> Tony, Les,
>> >>>
>> >>> I believe we can all agree that we do not want to change the behavior
of
>> >>> existing implementations that support MP-TLVs based on the
>> advertisements of the
>> >>> MP-capability from other routers - it would break existing networks.
>> Even the
>> >>> text in the MP-TLV draft does not suggest that to be the case.
>> >> Are people not looking at the spreadsheet Tony put together?
>> >> Which implicit multi-part TLVs are these "existing implementations"
>> >> advertising that keep getting referred to? Please let's work with real
data
>> --
>> >> the spreadsheet shows a grand total of *0* TLVs that could fall in this
>> >> category.
>> >
>> > then the spreadsheet is incorrect.
>> >
>> > I know of implementation that can send and receive Multi part TLVs for
>> IPv4/IPv6
>> > (MT) IP Reach, (MT) Extended IS reachability and IS-IS Router
CAPABILITY
>> TLV to
>> > start with.
>>
>> Do you know all of the implementations, and all of those that don't? If we
>> could publish that list then we presumably could explore more simple
>> solutions. :)
>>
>> People keep talking about breaking deployed networks, but that assumes
>> there are functional networks with implicit MP-TLVs in them. I am not
sure I
>> accep

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Les Ginsberg (ginsberg)
Chris -

Not sure what SE means but...one more significant point.

Multiple implementations have successfully deployed MP-TLV without any protocol 
extensions. We did not require a new sub-TLV, a new flag, a sequence 
number...we simply send additional information encoded according to existing 
standards. This isn't "luck" - it is following existing standards.

For implementations which do not process MP-TLVs correctly - why does this 
happen?
On the receive side, they do not have the intelligence in their implementation 
to do a merge.
On the transmit side they do not have the intelligence to generate multiple 
TLVs.

You can propose protocol extensions (such as you have done) - but it will not 
change the need for implementations to enhance their receive/generation logic - 
and it will not make it any easier for implementations to do so. What it will 
do is to introduce(sic) an interoperability problem because you will be 
requiring implementations to understand some new advertisement in order to 
send/receive MP-TLVs successfully. This is what Peter's point is about i.e., we 
MUST NOT break existing working MP-TLV implementations by requiring protocol 
extensions in order to send MP-TLVs.

As regards deployment controls, I have no problem with recommending that 
implementations provide ways to control the enablement of the sending of 
MP-TLVs. 

   Les

> -Original Message-
> From: Christian Hopps 
> Sent: Thursday, October 6, 2022 3:28 PM
> To: Les Ginsberg (ginsberg) 
> Cc: Christian Hopps ; Peter Psenak (ppsenak)
> ; Tony Li ; Robert Raszuk
> ; Henk Smit ; lsr@ietf.org
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
> 
> 
> It sounds like you're talking about networks defined to work by SE not by
> standards. I don't want to argue about this, so perhaps we can agree to
> disagree.
> 
> Thanks,
> Chris.
> [as wg-member]
> 
> 
> "Les Ginsberg (ginsberg)"  writes:
> 
> > Chris -
> >
> >> -Original Message-
> >> From: Christian Hopps 
> >> Sent: Thursday, October 6, 2022 1:36 PM
> >> To: Peter Psenak (ppsenak) 
> >> Cc: Christian Hopps ; Tony Li ; Les
> >> Ginsberg (ginsberg) ; Robert Raszuk
> >> ; Henk Smit ; lsr@ietf.org
> >> Subject: Re: [Lsr] New Version Notification for 
> >> draft-pkaneria-lsr-multi-tlv-
> >> 01.txt
> >>
> >>
> >> Peter Psenak  writes:
> >>
> >> > Chris,
> >> >
> >> > On 06/10/2022 18:34, Christian Hopps wrote:
> >> >> Peter Psenak  writes:
> >> >>
> >> >>> Tony, Les,
> >> >>>
> >> >>> I believe we can all agree that we do not want to change the behavior
> of
> >> >>> existing implementations that support MP-TLVs based on the
> >> advertisements of the
> >> >>> MP-capability from other routers - it would break existing networks.
> >> Even the
> >> >>> text in the MP-TLV draft does not suggest that to be the case.
> >> >> Are people not looking at the spreadsheet Tony put together?
> >> >> Which implicit multi-part TLVs are these "existing implementations"
> >> >> advertising that keep getting referred to? Please let's work with real
> data
> >> --
> >> >> the spreadsheet shows a grand total of *0* TLVs that could fall in this
> >> >> category.
> >> >
> >> > then the spreadsheet is incorrect.
> >> >
> >> > I know of implementation that can send and receive Multi part TLVs for
> >> IPv4/IPv6
> >> > (MT) IP Reach, (MT) Extended IS reachability and IS-IS Router
> CAPABILITY
> >> TLV to
> >> > start with.
> >>
> >> Do you know all of the implementations, and all of those that don't? If we
> >> could publish that list then we presumably could explore more simple
> >> solutions. :)
> >>
> >> People keep talking about breaking deployed networks, but that assumes
> >> there are functional networks with implicit MP-TLVs in them. I am not
> sure I
> >> accept the assertion that these networks are truly functional.
> >>
> >> AFAICT these networks are *lucky* to be working. There's no document
> to
> >> point at, there's no bit to look at, there's literally nothing to help an
> operator
> >> or their routers know if all the routers correctly receive and process 
> >> these
> >> implicit MP-TLVs. These networks are one network change (even as small
> as
> >> an interface up or down event) awa

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Robert Raszuk
Dear LSR WG,

> MP-TLVs (explicit or implicit) are not an extension of the protocol -
they are completely consistent
> with the base operation of the protocol. I have always viewed lack of
support for MP-TLVs as an
> implementation limitation - not a gap in the protocol.

I don't agree with this assertion. For many of us resending PDU with the
same key is an implicit update.

Is there any text in the base ISIS spec which would suggest otherwise ?

> you're talking about networks defined to work by SE not by standards

It seems like it. The problem with this is that it works well in static
single vendor networks where one can always blame operators for
"provisioning mistakes''.  We can do much better than that. I am with Tony,
Chris, Bruno here. It is cool that the IETF consensus does not have a
"liberum veto" (unanimity <https://en.wikipedia.org/wiki/Unanimity> voting
rule).

Thx,
Robert.

PS. And we all know how splitting the drafts will end up .. so no thank you
Peter & Les for this suggestion.


On Fri, Oct 7, 2022 at 12:20 AM Les Ginsberg (ginsberg) 
wrote:

> Chris -
>
> > -Original Message-
> > From: Christian Hopps 
> > Sent: Thursday, October 6, 2022 1:36 PM
> > To: Peter Psenak (ppsenak) 
> > Cc: Christian Hopps ; Tony Li ; Les
> > Ginsberg (ginsberg) ; Robert Raszuk
> > ; Henk Smit ; lsr@ietf.org
> > Subject: Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-
> > 01.txt
> >
> >
> > Peter Psenak  writes:
> >
> > > Chris,
> > >
> > > On 06/10/2022 18:34, Christian Hopps wrote:
> > >> Peter Psenak  writes:
> > >>
> > >>> Tony, Les,
> > >>>
> > >>> I believe we can all agree that we do not want to change the
> behavior of
> > >>> existing implementations that support MP-TLVs based on the
> > advertisements of the
> > >>> MP-capability from other routers - it would break existing networks.
> > Even the
> > >>> text in the MP-TLV draft does not suggest that to be the case.
> > >> Are people not looking at the spreadsheet Tony put together?
> > >> Which implicit multi-part TLVs are these "existing implementations"
> > >> advertising that keep getting referred to? Please let's work with
> real data
> > --
> > >> the spreadsheet shows a grand total of *0* TLVs that could fall in
> this
> > >> category.
> > >
> > > then the spreadsheet is incorrect.
> > >
> > > I know of implementation that can send and receive Multi part TLVs for
> > IPv4/IPv6
> > > (MT) IP Reach, (MT) Extended IS reachability and IS-IS Router
> CAPABILITY
> > TLV to
> > > start with.
> >
> > Do you know all of the implementations, and all of those that don't? If
> we
> > could publish that list then we presumably could explore more simple
> > solutions. :)
> >
> > People keep talking about breaking deployed networks, but that assumes
> > there are functional networks with implicit MP-TLVs in them. I am not
> sure I
> > accept the assertion that these networks are truly functional.
> >
> > AFAICT these networks are *lucky* to be working. There's no document to
> > point at, there's no bit to look at, there's literally nothing to help
> an operator
> > or their routers know if all the routers correctly receive and process
> these
> > implicit MP-TLVs. These networks are one network change (even as small as
> > an interface up or down event) away from failing, or may even be failing
> > already but not yet in a noticeable way. This is the case regardless of
> what
> > type of bit or functionality this document provides.
>
> [LES:] I don't agree at all with your characterization.
>
> MP-TLVs (explicit or implicit) are not an extension of the protocol - they
> are completely consistent with the base operation of the protocol. I have
> always viewed lack of support for MP-TLVs as an implementation limitation -
> not a gap in the protocol.
> Until relatively recently, there was no need to send MP-TLVs for
> neighbors/prefixes and since it is far from trivial to implement MP-TLV
> support it is understandable why most(all?) implementations did not include
> such support initially.
> But this does not mean that the protocol itself lacks the support.
>
> Would it have been better if all RFCs were explicit about the possibility
> of MP-TLVs? Sure - but hindsight is always easier than foresight.
> And even in those cases where MP-TLV support was explicitly defined, this
> did not guarantee that all implementations had that supp

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Christian Hopps


It sounds like you're talking about networks defined to work by SE not by 
standards. I don't want to argue about this, so perhaps we can agree to 
disagree.

Thanks,
Chris.
[as wg-member]


"Les Ginsberg (ginsberg)"  writes:


Chris -


-Original Message-
From: Christian Hopps 
Sent: Thursday, October 6, 2022 1:36 PM
To: Peter Psenak (ppsenak) 
Cc: Christian Hopps ; Tony Li ; Les
Ginsberg (ginsberg) ; Robert Raszuk
; Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
01.txt


Peter Psenak  writes:

> Chris,
>
> On 06/10/2022 18:34, Christian Hopps wrote:
>> Peter Psenak  writes:
>>
>>> Tony, Les,
>>>
>>> I believe we can all agree that we do not want to change the behavior of
>>> existing implementations that support MP-TLVs based on the
advertisements of the
>>> MP-capability from other routers - it would break existing networks.
Even the
>>> text in the MP-TLV draft does not suggest that to be the case.
>> Are people not looking at the spreadsheet Tony put together?
>> Which implicit multi-part TLVs are these "existing implementations"
>> advertising that keep getting referred to? Please let's work with real data
--
>> the spreadsheet shows a grand total of *0* TLVs that could fall in this
>> category.
>
> then the spreadsheet is incorrect.
>
> I know of implementation that can send and receive Multi part TLVs for
IPv4/IPv6
> (MT) IP Reach, (MT) Extended IS reachability and IS-IS Router CAPABILITY
TLV to
> start with.

Do you know all of the implementations, and all of those that don't? If we
could publish that list then we presumably could explore more simple
solutions. :)

People keep talking about breaking deployed networks, but that assumes
there are functional networks with implicit MP-TLVs in them. I am not sure I
accept the assertion that these networks are truly functional.

AFAICT these networks are *lucky* to be working. There's no document to
point at, there's no bit to look at, there's literally nothing to help an 
operator
or their routers know if all the routers correctly receive and process these
implicit MP-TLVs. These networks are one network change (even as small as
an interface up or down event) away from failing, or may even be failing
already but not yet in a noticeable way. This is the case regardless of what
type of bit or functionality this document provides.


[LES:] I don't agree at all with your characterization.

MP-TLVs (explicit or implicit) are not an extension of the protocol - they are
completely consistent with the base operation of the protocol. I have always
viewed lack of support for MP-TLVs as an implementation limitation - not a gap
in the protocol.
Until relatively recently, there was no need to send MP-TLVs for
neighbors/prefixes and since it is far from trivial to implement MP-TLV support
it is understandable why most(all?) implementations did not include such support
initially.
But this does not mean that the protocol itself lacks the support.

Would it have been better if all RFCs were explicit about the possibility of 
MP-TLVs? Sure - but hindsight is always easier than foresight.
And even in those cases where MP-TLV support was explicitly defined, this did
not guarantee that all implementations had that support. Vendors make decisions
based on business as to how they spend their development budget and I think we
are both familiar with decisions to defer support for some aspects of the
protocol until a stronger business case arises.

Regarding existing networks, MP-TLVs are an aspect of scale and feature support.
If your deployment includes many flex-algos or a large number of TE attributes
or other features which add to the information needing to be advertised, then
MP-TLVs are required.
Implementations which do not support MP-TLVs cannot be deployed in such 
environments - and it isn’t because of interoperability issues - it is because 
they do not support the scale/features required.

As my employer has implementations which do support MP-TLVs, I can say that we
do not depend upon luck - but we do depend upon careful planning. We work with
our customers to ensure that the design of the network - including the
capabilities of the routers deployed - is considered.


   Les



So while looking for a solution here, I think less weight should be placed on
these "lucky networks". I'm not saying we should intentionally break them,
but they should also not count as "fully-functional" either.

Thanks,
Chris.
[as wg-member]


>
> thanks,
> Peter
>> Thanks,
>> Chris.
>>
>>> I find the discussion about advertising supported capabilities for
management
>>> purposes in IGPs interesting, but not specific, nor directly related to the
>>> MP-TLV draft. Keeping the two separate would make a lot o

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Les Ginsberg (ginsberg)
Chris -

> -Original Message-
> From: Christian Hopps 
> Sent: Thursday, October 6, 2022 1:36 PM
> To: Peter Psenak (ppsenak) 
> Cc: Christian Hopps ; Tony Li ; Les
> Ginsberg (ginsberg) ; Robert Raszuk
> ; Henk Smit ; lsr@ietf.org
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
> 
> 
> Peter Psenak  writes:
> 
> > Chris,
> >
> > On 06/10/2022 18:34, Christian Hopps wrote:
> >> Peter Psenak  writes:
> >>
> >>> Tony, Les,
> >>>
> >>> I believe we can all agree that we do not want to change the behavior of
> >>> existing implementations that support MP-TLVs based on the
> advertisements of the
> >>> MP-capability from other routers - it would break existing networks.
> Even the
> >>> text in the MP-TLV draft does not suggest that to be the case.
> >> Are people not looking at the spreadsheet Tony put together?
> >> Which implicit multi-part TLVs are these "existing implementations"
> >> advertising that keep getting referred to? Please let's work with real data
> --
> >> the spreadsheet shows a grand total of *0* TLVs that could fall in this
> >> category.
> >
> > then the spreadsheet is incorrect.
> >
> > I know of implementation that can send and receive Multi part TLVs for
> IPv4/IPv6
> > (MT) IP Reach, (MT) Extended IS reachability and IS-IS Router CAPABILITY
> TLV to
> > start with.
> 
> Do you know all of the implementations, and all of those that don't? If we
> could publish that list then we presumably could explore more simple
> solutions. :)
> 
> People keep talking about breaking deployed networks, but that assumes
> there are functional networks with implicit MP-TLVs in them. I am not sure I
> accept the assertion that these networks are truly functional.
> 
> AFAICT these networks are *lucky* to be working. There's no document to
> point at, there's no bit to look at, there's literally nothing to help an 
> operator
> or their routers know if all the routers correctly receive and process these
> implicit MP-TLVs. These networks are one network change (even as small as
> an interface up or down event) away from failing, or may even be failing
> already but not yet in a noticeable way. This is the case regardless of what
> type of bit or functionality this document provides.

[LES:] I don't agree at all with your characterization.

MP-TLVs (explicit or implicit) are not an extension of the protocol - they are 
completely consistent with the base operation of the protocol. I have always 
viewed lack of support for MP-TLVs as an implementation limitation - not a gap 
in the protocol.
Until relatively recently, there was no need to send MP-TLVs for 
neighbors/prefixes and since it is far from trivial to implement MP-TLV support 
it is understandable why most(all?) implementations did not include such 
support initially.
But this does not mean that the protocol itself lacks the support.

Would it have been better if all RFCs were explicit about the possibility of 
MP-TLVs? Sure - but hindsight is always easier than foresight.
And even in those cases where MP-TLV support was explicitly defined, this did 
not guarantee that all implementations had that support. Vendors make decisions 
based on business as to how they spend their development budget and I think we 
are both familiar with decisions to defer support for some aspects of the 
protocol until a stronger business case arises.

Regarding existing networks, MP-TLVs are an aspect of scale and feature 
support. If your deployment includes many flex-algos or a large number of TE 
attributes or other features which add to the information needing to be 
advertised, then MP-TLVs are required.
Implementations which do not support MP-TLVs cannot be deployed in such 
environments - and it isn’t because of interoperability issues - it is because 
they do not support the scale/features required.

As my employer has implementations which do support MP-TLVs, I can say that we 
do not depend upon luck - but we do depend upon careful planning. We work with 
our customers to ensure that the design of the network - including the 
capabilities of the routers deployed - is considered.


   Les

> 
> So while looking for a solution here, I think less weight should be placed on
> these "lucky networks". I'm not saying we should intentionally break them,
> but they should also not count as "fully-functional" either.
> 
> Thanks,
> Chris.
> [as wg-member]
> 
> 
> >
> > thanks,
> > Peter
> >> Thanks,
> >> Chris.
> >>
> >>> I find the discussion about advertising supported capabilities for
> management
> >>

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Les Ginsberg (ginsberg)
Chris -

An example of what I refer to as "Do Not Apply" is "2   IIS Neighbors".
This is the old style (AKA narrow metric) neighbor advertisement which has a 
fixed format and does not support sub-TLVs. It is impossible to advertise 
information about the same neighbor in multiple TLVs (unless you just want to 
be redundant - which I would consider a bug outside of transient conditions 
where a TLV moves from one LSP to another). The discussion of "multi-part" 
simply has no meaning for this TLV.

Calling this "implicit single-part TLVs" isn’t as attractive to me. It would 
seem to require a definition of what a "single-part TLV" is - whereas saying 
"NA" just indicates "multi-part" isn't applicable and requires no further 
explanation.
But I won’t argue the point as long as we have a common understanding.

The significant issue at the moment is that such TLVs are currently marked as 
"Explicit multi-part TLV allowed" in the spreadsheet and this is wrong.

   Les

> -Original Message-
> From: Christian Hopps 
> Sent: Thursday, October 6, 2022 1:56 PM
> To: Les Ginsberg (ginsberg) 
> Cc: Christian Hopps ; Peter Psenak (ppsenak)
> ; Tony Li ; Robert Raszuk
> ; Henk Smit ; lsr@ietf.org
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
> 
> 
> "Les Ginsberg (ginsberg)"  writes:
> 
> >
> > With this in mind, Tony (as I have commented to you privately) your
> > table needs to be revised as there are some TLVs to which
> > “multi-part” simply doesn’t apply.
> 
> Those are what I believe we should refer to as "implicit single-part TLVs",
> TLVS that it would never make sense to do multi-part for some reason. And
> we probably want to define what those reasons are. It isn't necessarily just
> the lack of keying info. An unordered, unkeyed list of things (e.g., protocols
> supported) has no keying info, but could be sent multi-part and still make
> sense.
> 
> I think the tricky types are the "implicit multi-part TLVs" i.e., the ones 
> like
> Extended Reachability TLV, that were not defined to be sent multi-part but
> could conceivably be advertised that way (and apparently already are but
> some subset of implementations out there).
> 
> Tony I think you may have interpreted these differently?
> 
> Thanks,
> Chris.
> 
> 
> >
> >
> >
> >Les
> >
> >
> >
> >> -Original Message-----
> >
> >> From: Christian Hopps 
> >
> >> Sent: Thursday, October 6, 2022 9:34 AM
> >
> >> To: Peter Psenak (ppsenak) 
> >
> >> Cc: Tony Li ; Les Ginsberg (ginsberg)
> > ;
> >
> >> Christian Hopps ; Robert Raszuk
> >
> >> ; Henk Smit ; lsr@ietf.org
> >
> >> Subject: Re: [Lsr] New Version Notification for
> > draft-pkaneria-lsr-multi-tlv-
> >
> >> 01.txt
> >
> >>
> >
> >>
> >
> >> Peter Psenak  writes:
> >
> >>
> >
> >> > Tony, Les,
> >
> >> >
> >
> >> > I believe we can all agree that we do not want to change the
> > behavior of
> >
> >> > existing implementations that support MP-TLVs based on the
> >
> >> advertisements of the
> >
> >> > MP-capability from other routers - it would break existing
> > networks. Even
> >
> >> the
> >
> >> > text in the MP-TLV draft does not suggest that to be the case.
> >
> >>
> >
> >> Are people not looking at the spreadsheet Tony put together?
> >
> >>
> >
> >> Which implicit multi-part TLVs are these "existing implementations"
> >
> >> advertising that keep getting referred to? Please let's work with
> > real data --
> >
> >> the spreadsheet shows a grand total of *0* TLVs that could fall in
> > this
> >
> >> category.
> >
> >>
> >
> >> Thanks,
> >
> >> Chris.
> >
> >>
> >
> >> > I find the discussion about advertising supported capabilities
> > for
> >
> >> management
> >
> >> > purposes in IGPs interesting, but not specific, nor directly
> > related to the
> >
> >> > MP-TLV draft. Keeping the two separate would make a lot of sense.
> >
> >> >
> >
> >> >
> >
> >> > my 2c,
> >
> >> > Peter
> >
> >> >
> >
> >> >
> >
> >> >
> >
> >

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Christian Hopps


"Les Ginsberg (ginsberg)"  writes:



With this in mind, Tony (as I have commented to you privately) your
table needs to be revised as there are some TLVs to which
“multi-part” simply doesn’t apply.


Those are what I believe we should refer to as "implicit single-part TLVs", 
TLVS that it would never make sense to do multi-part for some reason. And we probably 
want to define what those reasons are. It isn't necessarily just the lack of keying info. 
An unordered, unkeyed list of things (e.g., protocols supported) has no keying info, but 
could be sent multi-part and still make sense.

I think the tricky types are the "implicit multi-part TLVs" i.e., the ones like 
Extended Reachability TLV, that were not defined to be sent multi-part but could 
conceivably be advertised that way (and apparently already are but some subset of 
implementations out there).

Tony I think you may have interpreted these differently?

Thanks,
Chris.






   Les




-Original Message-



From: Christian Hopps 



Sent: Thursday, October 6, 2022 9:34 AM



To: Peter Psenak (ppsenak) 



Cc: Tony Li ; Les Ginsberg (ginsberg)

;


Christian Hopps ; Robert Raszuk



; Henk Smit ; lsr@ietf.org



Subject: Re: [Lsr] New Version Notification for

draft-pkaneria-lsr-multi-tlv-


01.txt











Peter Psenak  writes:







> Tony, Les,



>



> I believe we can all agree that we do not want to change the

behavior of


> existing implementations that support MP-TLVs based on the



advertisements of the



> MP-capability from other routers - it would break existing

networks. Even


the



> text in the MP-TLV draft does not suggest that to be the case.







Are people not looking at the spreadsheet Tony put together?







Which implicit multi-part TLVs are these "existing implementations"



advertising that keep getting referred to? Please let's work with

real data --


the spreadsheet shows a grand total of *0* TLVs that could fall in

this


category.







Thanks,



Chris.







> I find the discussion about advertising supported capabilities

for


management



> purposes in IGPs interesting, but not specific, nor directly

related to the


> MP-TLV draft. Keeping the two separate would make a lot of sense.



>



>



> my 2c,



> Peter



>



>



>



> On 05/10/2022 22:18, Tony Li wrote:



>> Les,



>>



>>> On Oct 5, 2022, at 1:14 PM, Les Ginsberg (ginsberg)



>>> 


>>> <mailto:ginsberg=40cisco@dmarc.ietf.org>> wrote:



>>>



>>> */[LES:] It is clear that we have different opinions on this –

and there are


>>> multiple folks on both sides of this discussion./*



>>> */What I would hope we can agree on is to separate the

discussion of


adding



>>> advertisement of “feature supported” from the MP-TLV draft by

writing


a



>>> separate draft on this proposal./*



>>> */This would allow the two pieces of work to progress

independently –


as they



>>> should./*



>>> *//*



>>> */This makes sense to me since the proposal to advertise

feature


support is



>>> clearly not limited to MP-TLV and has no bearing on how MP-TLVs

are


>>> encoded./*



>>> *//*



>>> */Can we agree on this?/*



>> Sorry, I’m not on board with this.  The two functions would end

up


>> disconnected, all the way to the field.



>> T



>>






___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr




signature.asc
Description: PGP signature
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Christian Hopps


Peter Psenak  writes:


Chris,

On 06/10/2022 18:34, Christian Hopps wrote:

Peter Psenak  writes:


Tony, Les,

I believe we can all agree that we do not want to change the behavior of
existing implementations that support MP-TLVs based on the advertisements of the
MP-capability from other routers - it would break existing networks. Even the
text in the MP-TLV draft does not suggest that to be the case.

Are people not looking at the spreadsheet Tony put together?
Which implicit multi-part TLVs are these "existing implementations"
advertising that keep getting referred to? Please let's work with real data --
the spreadsheet shows a grand total of *0* TLVs that could fall in this
category.


then the spreadsheet is incorrect.

I know of implementation that can send and receive Multi part TLVs for IPv4/IPv6
(MT) IP Reach, (MT) Extended IS reachability and IS-IS Router CAPABILITY TLV to
start with.


Do you know all of the implementations, and all of those that don't? If we 
could publish that list then we presumably could explore more simple solutions. 
:)

People keep talking about breaking deployed networks, but that assumes there 
are functional networks with implicit MP-TLVs in them. I am not sure I accept 
the assertion that these networks are truly functional.

AFAICT these networks are *lucky* to be working. There's no document to point 
at, there's no bit to look at, there's literally nothing to help an operator or 
their routers know if all the routers correctly receive and process these 
implicit MP-TLVs. These networks are one network change (even as small as an 
interface up or down event) away from failing, or may even be failing already 
but not yet in a noticeable way. This is the case regardless of what type of 
bit or functionality this document provides.

So while looking for a solution here, I think less weight should be placed on these "lucky 
networks". I'm not saying we should intentionally break them, but they should also not count 
as "fully-functional" either.

Thanks,
Chris.
[as wg-member]




thanks,
Peter

Thanks,
Chris.


I find the discussion about advertising supported capabilities for management
purposes in IGPs interesting, but not specific, nor directly related to the
MP-TLV draft. Keeping the two separate would make a lot of sense.


my 2c,
Peter



On 05/10/2022 22:18, Tony Li wrote:

Les,


On Oct 5, 2022, at 1:14 PM, Les Ginsberg (ginsberg)
mailto:ginsberg=40cisco@dmarc.ietf.org>> wrote:

*/[LES:] It is clear that we have different opinions on this – and there are
multiple folks on both sides of this discussion./*
*/What I would hope we can agree on is to separate the discussion of adding
advertisement of “feature supported” from the MP-TLV draft by writing a
separate draft on this proposal./*
*/This would allow the two pieces of work to progress independently – as they
should./*
*//*
*/This makes sense to me since the proposal to advertise feature support is
clearly not limited to MP-TLV and has no bearing on how MP-TLVs are
encoded./*
*//*
*/Can we agree on this?/*

Sorry, I’m not on board with this.  The two functions would end up
disconnected, all the way to the field.
T





___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr




signature.asc
Description: PGP signature
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Les Ginsberg (ginsberg)
Sooo, Tony asked that feedback on the table be unicast for now (which is wise) 
- but I think there is one point that needs to be understood publicly - which 
has to do with what is the meaning of "MP-TLV".



As per 
https://www.ietf.org/archive/id/draft-pkaneria-lsr-multi-tlv-01.html#section-3 
(emphasis added):



“If a router advertises multiple TLV tuples with the same Type code in an IS-IS 
IIH packet or in the set of LSPs for a level with the same key value, they are 
considered a multi-part TLV (MP-TLV).”



We are not talking here about cases where a particular TLV type (such as 135 
Extended IP reachability) is sent multiple times where each instance of the TLV 
advertises a disjoint set of prefixes. That clearly is supported by all 
implementations today and is NOT what is meant by “Multi-part”.

A Multi-part TLV (using the example of IP Extended Reachability) is where 
multiple TLVs are advertising information about the same prefix.



If anyone is confused about this or disagrees with this, please speak up now.



With this in mind, Tony (as I have commented to you privately) your table needs 
to be revised as there are some TLVs to which “multi-part” simply doesn’t apply.



   Les



> -Original Message-

> From: Christian Hopps 

> Sent: Thursday, October 6, 2022 9:34 AM

> To: Peter Psenak (ppsenak) 

> Cc: Tony Li ; Les Ginsberg (ginsberg) ;

> Christian Hopps ; Robert Raszuk

> ; Henk Smit ; lsr@ietf.org

> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-

> 01.txt

>

>

> Peter Psenak mailto:ppse...@cisco.com>> writes:

>

> > Tony, Les,

> >

> > I believe we can all agree that we do not want to change the behavior of

> > existing implementations that support MP-TLVs based on the

> advertisements of the

> > MP-capability from other routers - it would break existing networks. Even

> the

> > text in the MP-TLV draft does not suggest that to be the case.

>

> Are people not looking at the spreadsheet Tony put together?

>

> Which implicit multi-part TLVs are these "existing implementations"

> advertising that keep getting referred to? Please let's work with real data --

> the spreadsheet shows a grand total of *0* TLVs that could fall in this

> category.

>

> Thanks,

> Chris.

>

> > I find the discussion about advertising supported capabilities for

> management

> > purposes in IGPs interesting, but not specific, nor directly related to the

> > MP-TLV draft. Keeping the two separate would make a lot of sense.

> >

> >

> > my 2c,

> > Peter

> >

> >

> >

> > On 05/10/2022 22:18, Tony Li wrote:

> >> Les,

> >>

> >>> On Oct 5, 2022, at 1:14 PM, Les Ginsberg (ginsberg)

> >>>  >>> <mailto:ginsberg=40cisco@dmarc.ietf.org>> wrote:

> >>>

> >>> */[LES:] It is clear that we have different opinions on this – and there 
> >>> are

> >>> multiple folks on both sides of this discussion./*

> >>> */What I would hope we can agree on is to separate the discussion of

> adding

> >>> advertisement of “feature supported” from the MP-TLV draft by writing

> a

> >>> separate draft on this proposal./*

> >>> */This would allow the two pieces of work to progress independently –

> as they

> >>> should./*

> >>> *//*

> >>> */This makes sense to me since the proposal to advertise feature

> support is

> >>> clearly not limited to MP-TLV and has no bearing on how MP-TLVs are

> >>> encoded./*

> >>> *//*

> >>> */Can we agree on this?/*

> >> Sorry, I’m not on board with this.  The two functions would end up

> >> disconnected, all the way to the field.

> >> T

> >>


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Tony Li

Gentlebeings,

The spreadsheet is NOT documenting implementations.  It’s documenting what I 
could find in the relevant specifications.

Tony


> On Oct 6, 2022, at 9:51 AM, Peter Psenak  
> wrote:
> 
> Chris,
> 
> On 06/10/2022 18:34, Christian Hopps wrote:
>> Peter Psenak  writes:
>>> Tony, Les,
>>> 
>>> I believe we can all agree that we do not want to change the behavior of
>>> existing implementations that support MP-TLVs based on the advertisements 
>>> of the
>>> MP-capability from other routers - it would break existing networks. Even 
>>> the
>>> text in the MP-TLV draft does not suggest that to be the case.
>> Are people not looking at the spreadsheet Tony put together?
>> Which implicit multi-part TLVs are these "existing implementations" 
>> advertising that keep getting referred to? Please let's work with real data 
>> -- the spreadsheet shows a grand total of *0* TLVs that could fall in this 
>> category.
> 
> then the spreadsheet is incorrect.
> 
> I know of implementation that can send and receive Multi part TLVs for 
> IPv4/IPv6 (MT) IP Reach, (MT) Extended IS reachability and IS-IS Router 
> CAPABILITY TLV to start with.
> 
> thanks,
> Peter
>> Thanks,
>> Chris.
>>> I find the discussion about advertising supported capabilities for 
>>> management
>>> purposes in IGPs interesting, but not specific, nor directly related to the
>>> MP-TLV draft. Keeping the two separate would make a lot of sense.
>>> 
>>> 
>>> my 2c,
>>> Peter
>>> 
>>> 
>>> 
>>> On 05/10/2022 22:18, Tony Li wrote:
 Les,
 
> On Oct 5, 2022, at 1:14 PM, Les Ginsberg (ginsberg)
>  > wrote:
> 
> */[LES:] It is clear that we have different opinions on this – and there 
> are
> multiple folks on both sides of this discussion./*
> */What I would hope we can agree on is to separate the discussion of 
> adding
> advertisement of “feature supported” from the MP-TLV draft by writing a
> separate draft on this proposal./*
> */This would allow the two pieces of work to progress independently – as 
> they
> should./*
> *//*
> */This makes sense to me since the proposal to advertise feature support 
> is
> clearly not limited to MP-TLV and has no bearing on how MP-TLVs are
> encoded./*
> *//*
> */Can we agree on this?/*
 Sorry, I’m not on board with this.  The two functions would end up
 disconnected, all the way to the field.
 T
 
> 
> ___
> Lsr mailing list
> Lsr@ietf.org
> https://www.ietf.org/mailman/listinfo/lsr

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Peter Psenak

Chris,

On 06/10/2022 18:34, Christian Hopps wrote:


Peter Psenak  writes:


Tony, Les,

I believe we can all agree that we do not want to change the behavior of
existing implementations that support MP-TLVs based on the advertisements of the
MP-capability from other routers - it would break existing networks. Even the
text in the MP-TLV draft does not suggest that to be the case.


Are people not looking at the spreadsheet Tony put together?

Which implicit multi-part TLVs are these "existing implementations" advertising 
that keep getting referred to? Please let's work with real data -- the spreadsheet shows 
a grand total of *0* TLVs that could fall in this category.


then the spreadsheet is incorrect.

I know of implementation that can send and receive Multi part TLVs for 
IPv4/IPv6 (MT) IP Reach, (MT) Extended IS reachability and IS-IS Router 
CAPABILITY TLV to start with.


thanks,
Peter


Thanks,
Chris.


I find the discussion about advertising supported capabilities for management
purposes in IGPs interesting, but not specific, nor directly related to the
MP-TLV draft. Keeping the two separate would make a lot of sense.


my 2c,
Peter



On 05/10/2022 22:18, Tony Li wrote:

Les,


On Oct 5, 2022, at 1:14 PM, Les Ginsberg (ginsberg)
mailto:ginsberg=40cisco@dmarc.ietf.org>> wrote:

*/[LES:] It is clear that we have different opinions on this – and there are
multiple folks on both sides of this discussion./*
*/What I would hope we can agree on is to separate the discussion of adding
advertisement of “feature supported” from the MP-TLV draft by writing a
separate draft on this proposal./*
*/This would allow the two pieces of work to progress independently – as they
should./*
*//*
*/This makes sense to me since the proposal to advertise feature support is
clearly not limited to MP-TLV and has no bearing on how MP-TLVs are
encoded./*
*//*
*/Can we agree on this?/*

Sorry, I’m not on board with this.  The two functions would end up
disconnected, all the way to the field.
T






___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Christian Hopps


Peter Psenak  writes:


Tony, Les,

I believe we can all agree that we do not want to change the behavior of
existing implementations that support MP-TLVs based on the advertisements of the
MP-capability from other routers - it would break existing networks. Even the
text in the MP-TLV draft does not suggest that to be the case.


Are people not looking at the spreadsheet Tony put together?

Which implicit multi-part TLVs are these "existing implementations" advertising 
that keep getting referred to? Please let's work with real data -- the spreadsheet shows 
a grand total of *0* TLVs that could fall in this category.

Thanks,
Chris.


I find the discussion about advertising supported capabilities for management
purposes in IGPs interesting, but not specific, nor directly related to the
MP-TLV draft. Keeping the two separate would make a lot of sense.


my 2c,
Peter



On 05/10/2022 22:18, Tony Li wrote:

Les,


On Oct 5, 2022, at 1:14 PM, Les Ginsberg (ginsberg)
mailto:ginsberg=40cisco@dmarc.ietf.org>> wrote:

*/[LES:] It is clear that we have different opinions on this – and there are
multiple folks on both sides of this discussion./*
*/What I would hope we can agree on is to separate the discussion of adding
advertisement of “feature supported” from the MP-TLV draft by writing a
separate draft on this proposal./*
*/This would allow the two pieces of work to progress independently – as they
should./*
*//*
*/This makes sense to me since the proposal to advertise feature support is
clearly not limited to MP-TLV and has no bearing on how MP-TLVs are
encoded./*
*//*
*/Can we agree on this?/*

Sorry, I’m not on board with this.  The two functions would end up
disconnected, all the way to the field.
T





signature.asc
Description: PGP signature
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Tony Li

Hi Chris,

> On Oct 5, 2022, at 1:26 PM, Christian Hopps  wrote:
> 
> That is great news b/c it means we can fix those 3 unpublished TLV to be 
> explicit multi-part. After fixing those 3 we are in a much friendly place of 
> defining only future behavior/standard requirements without concern of 
> impacting current deployments.


Yes, those are easily ‘fixed’, if folks feel that those need fixing.

Please note that the real point of this draft is to allow us to treat TLVs that 
are in the “implicit single part TLV” column as being multi-part.

And the real bone of contention is whether or not we should have a router 
capability as a means of signaling readiness to accept those.

Tony

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-06 Thread Peter Psenak

Tony, Les,

I believe we can all agree that we do not want to change the behavior of 
existing implementations that support MP-TLVs based on the 
advertisements of the MP-capability from other routers - it would break 
existing networks. Even the text in the MP-TLV draft does not suggest 
that to be the case.


I find the discussion about advertising supported capabilities for 
management purposes in IGPs interesting, but not specific, nor directly 
related to the MP-TLV draft. Keeping the two separate would make a lot 
of sense.



my 2c,
Peter



On 05/10/2022 22:18, Tony Li wrote:


Les,

On Oct 5, 2022, at 1:14 PM, Les Ginsberg (ginsberg) 
> wrote:


*/[LES:] It is clear that we have different opinions on this – and 
there are multiple folks on both sides of this discussion./*
*/What I would hope we can agree on is to separate the discussion of 
adding advertisement of “feature supported” from the MP-TLV draft by 
writing a separate draft on this proposal./*
*/This would allow the two pieces of work to progress independently – 
as they should./*

*//*
*/This makes sense to me since the proposal to advertise feature 
support is clearly not limited to MP-TLV and has no bearing on how 
MP-TLVs are encoded./*

*//*
*/Can we agree on this?/*



Sorry, I’m not on board with this.  The two functions would end up 
disconnected, all the way to the field.


T



___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Robert Raszuk
Les,

I agree - the scenario I presented is much bigger than this thread.

But in this thread what is being discussed is one little (arguably very
valuable) piece of it.

Many thx,
R.




On Thu, Oct 6, 2022 at 12:00 AM Les Ginsberg (ginsberg) 
wrote:

> I see – so you want a configuration model that says “Enable X when
> conditions Y and Z are met”.
>
>
>
> There are examples of this already e.g., conditional installation of
> static routes based on reachability/interface state… but they are typically
> quite modest in scope.
>
>
>
> Applying this to all of the potentially applicable configs would be quite
> complex and require lots of extensions to the configuration model supported.
>
> And it still risks oscillation…
>
>
>
> This is a big ask – and I think goes well beyond what has been discussed
> in this thread – so if you want to pursue this I think another thread/draft
> is called for.
>
> Certainly well beyond anything I am thinking about.
>
>
>
>Les
>
>
>
> *From:* Robert Raszuk 
> *Sent:* Wednesday, October 5, 2022 2:41 PM
> *To:* Les Ginsberg (ginsberg) 
> *Cc:* bruno.decra...@orange.com; Tony Li ; Christian
> Hopps ; Henk Smit ; lsr@ietf.org
> *Subject:* Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
>
>
>
> >  where implementations decide when to apply configuration that has been
> accepted
>
>
>
> where implementations decide when to apply *conditional* configuration
> that has been accepted
>
>
>
> Thx,
> R.
>
>
>
> On Wed, Oct 5, 2022 at 11:32 PM Les Ginsberg (ginsberg) <
> ginsb...@cisco.com> wrote:
>
> Robert –
>
>
>
> This has nothing to do with centralized vs distributed.
>
>
>
> You are advocating a model where implementations decide when to apply
> configuration that has been accepted. This fundamentally changes the
> authority from the management tool (be it manual or automated) to the
> implementation.
>
>
>
>Les
>
>
>
> *From:* Robert Raszuk 
> *Sent:* Wednesday, October 5, 2022 1:41 PM
> *To:* Les Ginsberg (ginsberg) 
> *Cc:* bruno.decra...@orange.com; Tony Li ; Christian
> Hopps ; Henk Smit ; lsr@ietf.org
> *Subject:* Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
>
> Hi Les,
>
>
>
> Yes you have correctly understood my last point.
>
>
>
> So your take it to always put a human to operate/run the network.
> Enable statically features and always manually map user traffic to such
> features.
>
>
>
> That's the legacy (old fashioned) network model. Yes it works today
> in a lot of networks, but is it to stay like this forever ?
>
>
>
> My view is that networks should operate by themselves (or as much
> autonomously as possible).
>
>
>
> We can do it via management plane - centralized model
>
> or
>
> We can do it in the network itself - distributed model.
>
>
>
> What are the network's overall transit capabilities should be dynamically
> signalled to applications too.
>
>
>
> It is surprising you are so much endorsing a centralized (oracle like)
> approach. I am even more surprised you are against using IGP to signal
> itself what it is capable of supporting to help today's operators.
>
>
>
> Best,
>
> R.
>
>
>
>
>
>
>
>
>
> On Wed, Oct 5, 2022 at 10:24 PM Les Ginsberg (ginsberg) <
> ginsb...@cisco.com> wrote:
>
> Robert –
>
>
>
> Not certain I completely understand you – but I “think” what you are
> saying is that we could allow configuration independent of what nodes in
> the network currently support, but suppress advertisement/enabling until
> all nodes advertise support.
>
> Am I close??
>
>
>
> If so, this is one of the aspects that adds complexity.
>
> And it still does not result in a working network.
>
> So, no thanks.
>
>
>
>Les
>
>
>
> *From:* Robert Raszuk 
> *Sent:* Wednesday, October 5, 2022 12:40 PM
> *To:* Les Ginsberg (ginsberg) 
> *Cc:* bruno.decra...@orange.com; Les Ginsberg (ginsberg) <
> ginsb...@cisco.com>; Tony Li ; Christian Hopps <
> cho...@chopps.org>; Henk Smit ; lsr@ietf.org
> *Subject:* Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
>
>
>
> Just to add and expand to your point Les ...
>
>
>
> The concept of "forward referencing" used in some OSes explicitly permit
> configuration of elements not even present on the box or in the system
> ahead of time. For example you can configure an interface which will be
> inserted later.
>

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Les Ginsberg (ginsberg)
I see – so you want a configuration model that says “Enable X when conditions Y 
and Z are met”.

There are examples of this already e.g., conditional installation of static 
routes based on reachability/interface state… but they are typically quite 
modest in scope.

Applying this to all of the potentially applicable configs would be quite 
complex and require lots of extensions to the configuration model supported.
And it still risks oscillation…

This is a big ask – and I think goes well beyond what has been discussed in 
this thread – so if you want to pursue this I think another thread/draft is 
called for.
Certainly well beyond anything I am thinking about.

   Les

From: Robert Raszuk 
Sent: Wednesday, October 5, 2022 2:41 PM
To: Les Ginsberg (ginsberg) 
Cc: bruno.decra...@orange.com; Tony Li ; Christian Hopps 
; Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


>  where implementations decide when to apply configuration that has been 
> accepted

where implementations decide when to apply conditional configuration that has 
been accepted

Thx,
R.

On Wed, Oct 5, 2022 at 11:32 PM Les Ginsberg (ginsberg) 
mailto:ginsb...@cisco.com>> wrote:
Robert –

This has nothing to do with centralized vs distributed.

You are advocating a model where implementations decide when to apply 
configuration that has been accepted. This fundamentally changes the authority 
from the management tool (be it manual or automated) to the implementation.

   Les

From: Robert Raszuk mailto:rob...@raszuk.net>>
Sent: Wednesday, October 5, 2022 1:41 PM
To: Les Ginsberg (ginsberg) mailto:ginsb...@cisco.com>>
Cc: bruno.decra...@orange.com<mailto:bruno.decra...@orange.com>; Tony Li 
mailto:tony...@tony.li>>; Christian Hopps 
mailto:cho...@chopps.org>>; Henk Smit 
mailto:henk.i...@xs4all.nl>>; 
lsr@ietf.org<mailto:lsr@ietf.org>
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

Hi Les,

Yes you have correctly understood my last point.

So your take it to always put a human to operate/run the network. Enable 
statically features and always manually map user traffic to such features.

That's the legacy (old fashioned) network model. Yes it works today in a lot of 
networks, but is it to stay like this forever ?

My view is that networks should operate by themselves (or as much autonomously 
as possible).

We can do it via management plane - centralized model
or
We can do it in the network itself - distributed model.

What are the network's overall transit capabilities should be dynamically 
signalled to applications too.

It is surprising you are so much endorsing a centralized (oracle like) 
approach. I am even more surprised you are against using IGP to signal itself 
what it is capable of supporting to help today's operators.

Best,
R.




On Wed, Oct 5, 2022 at 10:24 PM Les Ginsberg (ginsberg) 
mailto:ginsb...@cisco.com>> wrote:
Robert –

Not certain I completely understand you – but I “think” what you are saying is 
that we could allow configuration independent of what nodes in the network 
currently support, but suppress advertisement/enabling until all nodes 
advertise support.
Am I close??

If so, this is one of the aspects that adds complexity.
And it still does not result in a working network.
So, no thanks.

   Les

From: Robert Raszuk mailto:rob...@raszuk.net>>
Sent: Wednesday, October 5, 2022 12:40 PM
To: Les Ginsberg (ginsberg) mailto:ginsb...@cisco.com>>
Cc: bruno.decra...@orange.com<mailto:bruno.decra...@orange.com>; Les Ginsberg 
(ginsberg) mailto:ginsb...@cisco.com>>; Tony Li 
mailto:tony...@tony.li>>; Christian Hopps 
mailto:cho...@chopps.org>>; Henk Smit 
mailto:henk.i...@xs4all.nl>>; 
lsr@ietf.org<mailto:lsr@ietf.org>
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Just to add and expand to your point Les ...

The concept of "forward referencing" used in some OSes explicitly permit 
configuration of elements not even present on the box or in the system ahead of 
time. For example you can configure an interface which will be inserted later.

Same for any network wide feature.

However this means that activation of such network wide features can take place 
under dynamic conditions too. If "all nodes required" support feature X I can 
advertise it. (Let's assume that other nodes which do not require it will not 
crash).

That means that having such dynamic capability may be actually pretty helpful.

Thx,
R.








On Wed, Oct 5, 2022 at 9:33 PM Les Ginsberg (ginsberg) 
mailto:ginsb...@cisco.com>> wrote:
Bruno –

Thanx for commenting – I really want to hear from the operator community.

In regards to using “functionality supported” advertisements to either issue 
warnings or possibly block/disable configuration changes…I think this sounds 
better in 

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Robert Raszuk
>  where implementations decide when to apply configuration that has been
accepted

where implementations decide when to apply *conditional* configuration that
has been accepted

Thx,
R.

On Wed, Oct 5, 2022 at 11:32 PM Les Ginsberg (ginsberg) 
wrote:

> Robert –
>
>
>
> This has nothing to do with centralized vs distributed.
>
>
>
> You are advocating a model where implementations decide when to apply
> configuration that has been accepted. This fundamentally changes the
> authority from the management tool (be it manual or automated) to the
> implementation.
>
>
>
>Les
>
>
>
> *From:* Robert Raszuk 
> *Sent:* Wednesday, October 5, 2022 1:41 PM
> *To:* Les Ginsberg (ginsberg) 
> *Cc:* bruno.decra...@orange.com; Tony Li ; Christian
> Hopps ; Henk Smit ; lsr@ietf.org
> *Subject:* Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
>
> Hi Les,
>
>
>
> Yes you have correctly understood my last point.
>
>
>
> So your take it to always put a human to operate/run the network.
> Enable statically features and always manually map user traffic to such
> features.
>
>
>
> That's the legacy (old fashioned) network model. Yes it works today
> in a lot of networks, but is it to stay like this forever ?
>
>
>
> My view is that networks should operate by themselves (or as much
> autonomously as possible).
>
>
>
> We can do it via management plane - centralized model
>
> or
>
> We can do it in the network itself - distributed model.
>
>
>
> What are the network's overall transit capabilities should be dynamically
> signalled to applications too.
>
>
>
> It is surprising you are so much endorsing a centralized (oracle like)
> approach. I am even more surprised you are against using IGP to signal
> itself what it is capable of supporting to help today's operators.
>
>
>
> Best,
>
> R.
>
>
>
>
>
>
>
>
>
> On Wed, Oct 5, 2022 at 10:24 PM Les Ginsberg (ginsberg) <
> ginsb...@cisco.com> wrote:
>
> Robert –
>
>
>
> Not certain I completely understand you – but I “think” what you are
> saying is that we could allow configuration independent of what nodes in
> the network currently support, but suppress advertisement/enabling until
> all nodes advertise support.
>
> Am I close??
>
>
>
> If so, this is one of the aspects that adds complexity.
>
> And it still does not result in a working network.
>
> So, no thanks.
>
>
>
>    Les
>
>
>
> *From:* Robert Raszuk 
> *Sent:* Wednesday, October 5, 2022 12:40 PM
> *To:* Les Ginsberg (ginsberg) 
> *Cc:* bruno.decra...@orange.com; Les Ginsberg (ginsberg) <
> ginsb...@cisco.com>; Tony Li ; Christian Hopps <
> cho...@chopps.org>; Henk Smit ; lsr@ietf.org
> *Subject:* Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
>
>
>
> Just to add and expand to your point Les ...
>
>
>
> The concept of "forward referencing" used in some OSes explicitly permit
> configuration of elements not even present on the box or in the system
> ahead of time. For example you can configure an interface which will be
> inserted later.
>
>
>
> Same for any network wide feature.
>
>
>
> However this means that activation of such network wide features can take
> place under dynamic conditions too. If "all nodes required" support feature
> X I can advertise it. (Let's assume that other nodes which do not require
> it will not crash).
>
>
>
> That means that having such dynamic capability may be actually pretty
> helpful.
>
>
>
> Thx,
> R.
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> On Wed, Oct 5, 2022 at 9:33 PM Les Ginsberg (ginsberg) 
> wrote:
>
> Bruno –
>
>
>
> Thanx for commenting – I really want to hear from the operator community.
>
>
>
> In regards to using “functionality supported” advertisements to either
> issue warnings or possibly block/disable configuration changes…I think this
> sounds better in theory than it can be realized in practice.
>
>
>
> If you block config when not all nodes advertise support for “feature X”,
> what do you do with existing config which was accepted at a time when all
> nodes that are up/reachable did advertise the needed support, but now a new
> node comes up (or becomes reachable) and it does NOT advertise the support?
>
> What do you do on bootup when the saved config includes configuration that
> should not be accepted under the rules for “feature X”?
>
> Do implementations have to provide a knob

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Les Ginsberg (ginsberg)
Robert –

This has nothing to do with centralized vs distributed.

You are advocating a model where implementations decide when to apply 
configuration that has been accepted. This fundamentally changes the authority 
from the management tool (be it manual or automated) to the implementation.

   Les

From: Robert Raszuk 
Sent: Wednesday, October 5, 2022 1:41 PM
To: Les Ginsberg (ginsberg) 
Cc: bruno.decra...@orange.com; Tony Li ; Christian Hopps 
; Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

Hi Les,

Yes you have correctly understood my last point.

So your take it to always put a human to operate/run the network. Enable 
statically features and always manually map user traffic to such features.

That's the legacy (old fashioned) network model. Yes it works today in a lot of 
networks, but is it to stay like this forever ?

My view is that networks should operate by themselves (or as much autonomously 
as possible).

We can do it via management plane - centralized model
or
We can do it in the network itself - distributed model.

What are the network's overall transit capabilities should be dynamically 
signalled to applications too.

It is surprising you are so much endorsing a centralized (oracle like) 
approach. I am even more surprised you are against using IGP to signal itself 
what it is capable of supporting to help today's operators.

Best,
R.




On Wed, Oct 5, 2022 at 10:24 PM Les Ginsberg (ginsberg) 
mailto:ginsb...@cisco.com>> wrote:
Robert –

Not certain I completely understand you – but I “think” what you are saying is 
that we could allow configuration independent of what nodes in the network 
currently support, but suppress advertisement/enabling until all nodes 
advertise support.
Am I close??

If so, this is one of the aspects that adds complexity.
And it still does not result in a working network.
So, no thanks.

   Les

From: Robert Raszuk mailto:rob...@raszuk.net>>
Sent: Wednesday, October 5, 2022 12:40 PM
To: Les Ginsberg (ginsberg) mailto:ginsb...@cisco.com>>
Cc: bruno.decra...@orange.com<mailto:bruno.decra...@orange.com>; Les Ginsberg 
(ginsberg) mailto:ginsb...@cisco.com>>; Tony Li 
mailto:tony...@tony.li>>; Christian Hopps 
mailto:cho...@chopps.org>>; Henk Smit 
mailto:henk.i...@xs4all.nl>>; 
lsr@ietf.org<mailto:lsr@ietf.org>
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Just to add and expand to your point Les ...

The concept of "forward referencing" used in some OSes explicitly permit 
configuration of elements not even present on the box or in the system ahead of 
time. For example you can configure an interface which will be inserted later.

Same for any network wide feature.

However this means that activation of such network wide features can take place 
under dynamic conditions too. If "all nodes required" support feature X I can 
advertise it. (Let's assume that other nodes which do not require it will not 
crash).

That means that having such dynamic capability may be actually pretty helpful.

Thx,
R.








On Wed, Oct 5, 2022 at 9:33 PM Les Ginsberg (ginsberg) 
mailto:ginsb...@cisco.com>> wrote:
Bruno –

Thanx for commenting – I really want to hear from the operator community.

In regards to using “functionality supported” advertisements to either issue 
warnings or possibly block/disable configuration changes…I think this sounds 
better in theory than it can be realized in practice.

If you block config when not all nodes advertise support for “feature X”, what 
do you do with existing config which was accepted at a time when all nodes that 
are up/reachable did advertise the needed support, but now a new node comes up 
(or becomes reachable) and it does NOT advertise the support?
What do you do on bootup when the saved config includes configuration that 
should not be accepted under the rules for “feature X”?
Do implementations have to provide a knob to control whether configs should be 
rejected or just a warning issued?
Do you think operators will now try to leverage the new “protection” they 
expect from implementations and feel free to relax testing on the assumption 
that a good implementation will prevent a “bad configuration” from being 
enabled?

And what are the candidates for such functionality?
In this thread we talk about MP-TLV support – but this is less than precise. 
What we are really talking about is MP-TLV support for specific TLVs. So this 
now requires per/TLV advertisement.
And what else might be a good candidate? What about individual sub-TLV support 
e.g., for the different link attributes? If a node advertises a link attribute 
that not all nodes support then various forms of TE/flex-algo may not work.
Do we then require each node to advertise every TLV it supports and every 
sub-TLV it supports?
What about flag bits within a given TLV/sub-TLV?

O

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Christian Hopps


"Les Ginsberg (ginsberg)"  writes:



RFC3787 which was published 1 month prior to RFC3784 which RFC5305
replaced.

  "5.  Migration from Narrow Metrics to Wide . . ."

[LES:] Indeed - RFC 3787 - quite a useful document.
Note this section was based on input from actual implementations. So folks implemented 
migration strategies first, then it got "documented".

RFC 3784 was published in 2004 - but V0 of the draft which eventually became 
RFC 3784 was published in 1999 and there were implementations long before the 
RFC was published.


There was a transition path published along with the TLV. That's all that 
matters to make my point.


Part of what the authors of MP-TLV draft have discussed is adding a Deployment 
Considerations section to the draft. But the new version hasn't been published 
yet.


In any case, please look at Tony's spreadsheet b/c I think it changes the 
nature of the problem to be solved.

Thanks,
Chris.



   Les




signature.asc
Description: PGP signature
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Les Ginsberg (ginsberg)
> 
> RFC3787 which was published 1 month prior to RFC3784 which RFC5305
> replaced.
> 
>   "5.  Migration from Narrow Metrics to Wide . . ."
[LES:] Indeed - RFC 3787 - quite a useful document.
Note this section was based on input from actual implementations. So folks 
implemented migration strategies first, then it got "documented".

RFC 3784 was published in 2004 - but V0 of the draft which eventually became 
RFC 3784 was published in 1999 and there were implementations long before the 
RFC was published.

Part of what the authors of MP-TLV draft have discussed is adding a Deployment 
Considerations section to the draft. But the new version hasn't been published 
yet.

   Les

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Christian Hopps


Tony Li  writes:


Hi all,


On Oct 3, 2022, at 8:37 AM, Christian Hopps  wrote:


[As wg-member]

I think the draft should include a table of all top level TLVS as rows and for 
columns we have

- "Implicit Single TLV Only" (e.g., no key info)
- "Implicit Multi-TLV Only"
- "Explicit Single TLV Only"
- "Explicit Multi-TLV-Allowed"

This draft then would *explicitly* cover the existing "implicit" cases and we 
have the table to point at to be precise about what we are talking about.



I have completed this effort.


Wow, thanks for doing this!

This is very interesting and useful data. If I am reading it correctly, we have 
no(*) published TLVs which might implicitly use multi-part TLV encoding.

And, we have only 3 that are not yet published in RFCs that fall into this 
"implicitly multi-part TLV" category.

That is great news b/c it means we can fix those 3 unpublished TLV to be 
explicit multi-part. After fixing those 3 we are in a much friendly place of 
defining only future behavior/standard requirements without concern of 
impacting current deployments.

Thanks again!
Chris.

(*) TRILL TLV has been published, but that is an entirely different protocol.



Rather than burden the draft with this, I have created a spreadsheet and am 
sharing a link to the sheet: 
https://docs.google.com/spreadsheets/d/1VWQHsJTq7JRUVdRVcRd_QYYyY_LiOX7H0ZKn8feRZo4/edit?usp=sharing

There are numerous proprietary TLVs that I cannot evaluate. I have ignored them.

I am human. I expect that I will have missed several things. If you have a 
correction, please unicast me.

Regards,
Tony




signature.asc
Description: PGP signature
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Robert Raszuk
Hi Les,

Yes you have correctly understood my last point.

So your take it to always put a human to operate/run the network.
Enable statically features and always manually map user traffic to such
features.

That's the legacy (old fashioned) network model. Yes it works today
in a lot of networks, but is it to stay like this forever ?

My view is that networks should operate by themselves (or as much
autonomously as possible).

We can do it via management plane - centralized model
or
We can do it in the network itself - distributed model.

What are the network's overall transit capabilities should be dynamically
signalled to applications too.

It is surprising you are so much endorsing a centralized (oracle like)
approach. I am even more surprised you are against using IGP to signal
itself what it is capable of supporting to help today's operators.

Best,
R.




On Wed, Oct 5, 2022 at 10:24 PM Les Ginsberg (ginsberg) 
wrote:

> Robert –
>
>
>
> Not certain I completely understand you – but I “think” what you are
> saying is that we could allow configuration independent of what nodes in
> the network currently support, but suppress advertisement/enabling until
> all nodes advertise support.
>
> Am I close??
>
>
>
> If so, this is one of the aspects that adds complexity.
>
> And it still does not result in a working network.
>
> So, no thanks.
>
>
>
>Les
>
>
>
> *From:* Robert Raszuk 
> *Sent:* Wednesday, October 5, 2022 12:40 PM
> *To:* Les Ginsberg (ginsberg) 
> *Cc:* bruno.decra...@orange.com; Les Ginsberg (ginsberg) <
> ginsb...@cisco.com>; Tony Li ; Christian Hopps <
> cho...@chopps.org>; Henk Smit ; lsr@ietf.org
> *Subject:* Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
>
>
>
> Just to add and expand to your point Les ...
>
>
>
> The concept of "forward referencing" used in some OSes explicitly permit
> configuration of elements not even present on the box or in the system
> ahead of time. For example you can configure an interface which will be
> inserted later.
>
>
>
> Same for any network wide feature.
>
>
>
> However this means that activation of such network wide features can take
> place under dynamic conditions too. If "all nodes required" support feature
> X I can advertise it. (Let's assume that other nodes which do not require
> it will not crash).
>
>
>
> That means that having such dynamic capability may be actually pretty
> helpful.
>
>
>
> Thx,
> R.
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
>
> On Wed, Oct 5, 2022 at 9:33 PM Les Ginsberg (ginsberg) 
> wrote:
>
> Bruno –
>
>
>
> Thanx for commenting – I really want to hear from the operator community.
>
>
>
> In regards to using “functionality supported” advertisements to either
> issue warnings or possibly block/disable configuration changes…I think this
> sounds better in theory than it can be realized in practice.
>
>
>
> If you block config when not all nodes advertise support for “feature X”,
> what do you do with existing config which was accepted at a time when all
> nodes that are up/reachable did advertise the needed support, but now a new
> node comes up (or becomes reachable) and it does NOT advertise the support?
>
> What do you do on bootup when the saved config includes configuration that
> should not be accepted under the rules for “feature X”?
>
> Do implementations have to provide a knob to control whether configs
> should be rejected or just a warning issued?
>
> Do you think operators will now try to leverage the new “protection” they
> expect from implementations and feel free to relax testing on the
> assumption that a good implementation will prevent a “bad configuration”
> from being enabled?
>
>
>
> And what are the candidates for such functionality?
>
> In this thread we talk about MP-TLV support – but this is less than
> precise. What we are really talking about is MP-TLV support for specific
> TLVs. So this now requires per/TLV advertisement.
>
> And what else might be a good candidate? What about individual sub-TLV
> support e.g., for the different link attributes? If a node advertises a
> link attribute that not all nodes support then various forms of
> TE/flex-algo may not work.
>
> Do we then require each node to advertise every TLV it supports and every
> sub-TLV it supports?
>
> What about flag bits within a given TLV/sub-TLV?
>
>
>
> One response to this is to say “we will limit this to ‘important’
> advertisements” – but how are we to agree on what is important and what
> isn’t? Do we have an operational definition of “important”?

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Les Ginsberg (ginsberg)
Robert –

Not certain I completely understand you – but I “think” what you are saying is 
that we could allow configuration independent of what nodes in the network 
currently support, but suppress advertisement/enabling until all nodes 
advertise support.
Am I close??

If so, this is one of the aspects that adds complexity.
And it still does not result in a working network.
So, no thanks.

   Les

From: Robert Raszuk 
Sent: Wednesday, October 5, 2022 12:40 PM
To: Les Ginsberg (ginsberg) 
Cc: bruno.decra...@orange.com; Les Ginsberg (ginsberg) ; 
Tony Li ; Christian Hopps ; Henk Smit 
; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Just to add and expand to your point Les ...

The concept of "forward referencing" used in some OSes explicitly permit 
configuration of elements not even present on the box or in the system ahead of 
time. For example you can configure an interface which will be inserted later.

Same for any network wide feature.

However this means that activation of such network wide features can take place 
under dynamic conditions too. If "all nodes required" support feature X I can 
advertise it. (Let's assume that other nodes which do not require it will not 
crash).

That means that having such dynamic capability may be actually pretty helpful.

Thx,
R.








On Wed, Oct 5, 2022 at 9:33 PM Les Ginsberg (ginsberg) 
mailto:ginsb...@cisco.com>> wrote:
Bruno –

Thanx for commenting – I really want to hear from the operator community.

In regards to using “functionality supported” advertisements to either issue 
warnings or possibly block/disable configuration changes…I think this sounds 
better in theory than it can be realized in practice.

If you block config when not all nodes advertise support for “feature X”, what 
do you do with existing config which was accepted at a time when all nodes that 
are up/reachable did advertise the needed support, but now a new node comes up 
(or becomes reachable) and it does NOT advertise the support?
What do you do on bootup when the saved config includes configuration that 
should not be accepted under the rules for “feature X”?
Do implementations have to provide a knob to control whether configs should be 
rejected or just a warning issued?
Do you think operators will now try to leverage the new “protection” they 
expect from implementations and feel free to relax testing on the assumption 
that a good implementation will prevent a “bad configuration” from being 
enabled?

And what are the candidates for such functionality?
In this thread we talk about MP-TLV support – but this is less than precise. 
What we are really talking about is MP-TLV support for specific TLVs. So this 
now requires per/TLV advertisement.
And what else might be a good candidate? What about individual sub-TLV support 
e.g., for the different link attributes? If a node advertises a link attribute 
that not all nodes support then various forms of TE/flex-algo may not work.
Do we then require each node to advertise every TLV it supports and every 
sub-TLV it supports?
What about flag bits within a given TLV/sub-TLV?

One response to this is to say “we will limit this to ‘important’ 
advertisements” – but how are we to agree on what is important and what isn’t? 
Do we have an operational definition of “important”?

I think this quite easily leads to an explosion of advertisements and a great 
deal of complexity which returns very little.
If an operator configures something it is because they need that in order for 
the network to meet the requirements of the given deployment. 
Blocking/disabling config might prevent some problems – but it won’t make the 
network functional.

I am quite concerned that this is something that “sounds good” but in practice 
adds little value – yet places significant demands on implementations.

   Les


From: Lsr mailto:lsr-boun...@ietf.org>> On Behalf Of 
bruno.decra...@orange.com<mailto:bruno.decra...@orange.com>
Sent: Wednesday, October 5, 2022 2:29 AM
To: Les Ginsberg (ginsberg) 
mailto:40cisco@dmarc.ietf.org>>; Tony 
Li mailto:tony...@tony.li>>; Christian Hopps 
mailto:cho...@chopps.org>>
Cc: Robert Raszuk mailto:rob...@raszuk.net>>; Henk Smit 
mailto:henk.i...@xs4all.nl>>; 
lsr@ietf.org<mailto:lsr@ietf.org>
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

I’ve not followed everything in details so I’ve been reluctant to comment, but 
nonetheless please find below a diverse set of comments.


> From: Christian Hopps mailto:cho...@chopps.org>>
[…]

> AFAICT we now have implementations out there that are sending multiple TLVs 
> which are not documented to be sent that way, that historically were not 
> expected to be received that way, and then we have other implementations that 
> do not expect this new, convenient but rather loose interpretation of the 
&

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Christian Hopps



"Les Ginsberg (ginsberg)"  writes:


Chris -

Please see inline.


[...]


> MP-TLVs are not sent just because an implementation supports them.
> They are sent because the current configuration requires them.

The SW also has the option to alert the operator that their configuration is
not supported, and to revise it, rather than play loose with standards.

The vendors who made these changes should have brought this to the IETF
as a draft that would have clear and deterministic transition mechanism (e.g.,
wide metrics had a clear transition plan documented in it's RFC).


[LES:] Hmmm...RFC 5305 says in the Introduction:

"Mechanisms and procedures to migrate to the new TLVs are not
   discussed in this document."

What are you looking at?


RFC3787 which was published 1 month prior to RFC3784 which RFC5305 replaced.

 "5.  Migration from Narrow Metrics to Wide . . ."


I'm suggesting that the most important thing to come now is to make clear
what operators need to do to have a deterministic functional network. It
doesn't have to be the way I suggested, but we have to get there somehow.


[LES:] If there was a way to do this I would be interested. Nothing proposed 
thus far does this.


What I proposed works, it is just not to your liking.

Thanks,
Chris.

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Tony Li

Les,

> On Oct 5, 2022, at 1:14 PM, Les Ginsberg (ginsberg) 
>  wrote:
> 
> [LES:] It is clear that we have different opinions on this – and there are 
> multiple folks on both sides of this discussion.
> What I would hope we can agree on is to separate the discussion of adding 
> advertisement of “feature supported” from the MP-TLV draft by writing a 
> separate draft on this proposal.
> This would allow the two pieces of work to progress independently – as they 
> should.
>  
> This makes sense to me since the proposal to advertise feature support is 
> clearly not limited to MP-TLV and has no bearing on how MP-TLVs are encoded.
>  
> Can we agree on this?


Sorry, I’m not on board with this.  The two functions would end up 
disconnected, all the way to the field.

T

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Les Ginsberg (ginsberg)
Tony -

From: Tony Li  On Behalf Of Tony Li
Sent: Wednesday, October 5, 2022 7:57 AM
To: Les Ginsberg (ginsberg) 
Cc: Christian Hopps ; Robert Raszuk ; 
Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Hi Les,

Using the protocol to send what is best described as some subset of a PICS 
means that we propose to use the IGP flooding mechanism to send static 
information which the protocol itself cannot (and should not) use in its 
operation. This consumes space, bandwidth, gets periodically refreshed 
unnecessarily, and now a complete copy of the information from every node 
resides on every router in the network when it is only needed by an “NMS”. It 
would be hard to come up with a better example of “IGP isn’t a dump truck” than 
this.


If you can’t beat ‘em, join ‘em.

[LES:] Thanx for the honesty. 
Doesn’t make me like the idea though.


If there is a belief that we can severely limit the amount of information that 
is sent/node, I’d have to say that I am skeptical. Once we allow this into the 
protocol, I don’t see any basis on which to separate what is allowed and what 
is disallowed. It would not be unreasonable for an operator to say that 
everything that is a candidate to be mentioned in a PICS is a legitimate 
candidate for being advertised using this mechanism. Which means the amount of 
information is likely to become very large – especially once it becomes the de 
facto way of providing protocol management information.

The justification seems to be that we don’t have anything better – which 
represents a longstanding failure of the management plane. While I agree with 
you that management plane solutions are not adequate – not least because we 
can’t get the industry to converge on a single solution – this does not mean we 
should invest in the wrong solution.

We would be better served spending time and effort working on the right 
solution - as difficult as that may be.

If we despair of getting a management plane solution, my suggestion would be to 
use RFC 6823/6822 to define an IS-IS protocol management application that could 
support the advertisement of such information. This is technically 
straightforward to define/implement, easily extensible, and it separates the 
management information from the information used by the protocol.  And because 
a separate topology can be used for the “management instance”,  it would be 
possible to reduce the number of copies in the network.


A blue dump truck is not an architectural improvement over a red dump truck and 
definitely not the right solution.

[LES:] I agree – I don’t like the RFC6823/6822 solution – I want a management 
plane solution.
But it is better than carrying the information in the routing instance – for 
all the reasons both of us have mentioned over the years whenever someone 
proposes to use IGP flooding for information not used by the protocol itself.

What we need is a management plane mechanism that can be easily consulted, even 
by nodes not running the IGP.  Or nodes not running any IGP. We should NOT 
require storing the management data on every node. That’s silly.  Rather, we 
should have a set of distributed, synchronized servers that can be easily 
referenced and updated. We need an auto-discovery mechanism so that nodes can 
learn where these servers are. We need a common hierarchical data schema so 
that data is organized consistently. And we needed this in 1992. ;-)

Even if you agree with this, I am not willing to stall the current work the 
decades that it will take for the IETF to make progress on this.

[LES:] It is clear that we have different opinions on this – and there are 
multiple folks on both sides of this discussion.
What I would hope we can agree on is to separate the discussion of adding 
advertisement of “feature supported” from the MP-TLV draft by writing a 
separate draft on this proposal.
This would allow the two pieces of work to progress independently – as they 
should.

This makes sense to me since the proposal to advertise feature support is 
clearly not limited to MP-TLV and has no bearing on how MP-TLVs are encoded.

Can we agree on this?

   Les


Tony

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Les Ginsberg (ginsberg)
Chris -

Please see inline.

> -Original Message-
> From: Christian Hopps 
> Sent: Wednesday, October 5, 2022 7:24 AM
> To: Les Ginsberg (ginsberg) 
> Cc: Christian Hopps ; Robert Raszuk
> ; Tony Li ; Henk Smit
> ; lsr@ietf.org
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
> 
> 
> [as wg-member]
> 
> I don't think what I've written is being read correctly.
> 
> > If a node which supports MP-TLV were to introduce/withdraw MP-TLVs
> > based on the advertised state of support for MP-TLVs by all nodes in
> > the network,  this would introduce flooding storms on the
> > transitions. Not a good thing.
> 
> I didn't suggest this. My suggestion was that a router sending multiple-TLVs
> needs to advertise that it does so, and receivers would interpret what they
> received (is this second TLV a replacement or is it a concatenation?) based on
> that advertisement. The proposal is for a "capability" in this case and that
> could serve as this advertisement.
> 
> I believe you would argue against this as it might impact networks that just
> happen to be lucky and working right now.
>
[LES:] My argument is the presence/use of the "capability" does not result in a 
working network.
As I have stated multiple times, MP-TLVs are sent because the configuration 
requires it - not just because an implementation is capable of doing so.
Preventing advertisement of what is configured does not make the network 
healthy. It simply trades one problem for another.
 
> If things would have been done correctly (i.e., documented) we also would
> have had the option to add a sub-tlv to the TLVs in question that indicated
> they were part of a set rather than replacements.
> 
[LES:] So, this is a new proposal - not one that has been defined or even 
suggested anywhere to date.
And I don’t support it.
It deviates from how existing MP-TLV support that is already explicitly 
specified is done.
It is inefficient in that it requires modification of the original TLV simply 
to advertise additional information about the same object - which means 
additional LSPs will need to be regenerated/flooded/processed.
It would require a flag day (or knobs of some sort) because it would not work 
until all implementations supported the new sub-TLV.

> > MP-TLVs are not sent just because an implementation supports them.
> > They are sent because the current configuration requires them.
> 
> The SW also has the option to alert the operator that their configuration is
> not supported, and to revise it, rather than play loose with standards.
> 
> The vendors who made these changes should have brought this to the IETF
> as a draft that would have clear and deterministic transition mechanism (e.g.,
> wide metrics had a clear transition plan documented in it's RFC).
> 
[LES:] Hmmm...RFC 5305 says in the Introduction:

"Mechanisms and procedures to migrate to the new TLVs are not
   discussed in this document."

What are you looking at?

> I'm suggesting that the most important thing to come now is to make clear
> what operators need to do to have a deterministic functional network. It
> doesn't have to be the way I suggested, but we have to get there somehow.
>
[LES:] If there was a way to do this I would be interested. Nothing proposed 
thus far does this.

   Les
 
> Thanks,
> Chris.
> [as wg-member]
> 
> 
> "Les Ginsberg (ginsberg)"  writes:
> 
> > Chris -
> >
> >
> >
> > Inline.
> >
> >
> >
> >> -Original Message-
> >
> >> From: Christian Hopps 
> >
> >> Sent: Monday, October 3, 2022 5:52 PM
> >
> >> To: Les Ginsberg (ginsberg) 
> >
> >> Cc: Christian Hopps ; Robert Raszuk
> >
> >> ; Tony Li ; Henk Smit
> >
> >> ; lsr@ietf.org
> >
> >> Subject: Re: [Lsr] New Version Notification for
> > draft-pkaneria-lsr-multi-tlv-
> >
> >> 01.txt
> >
> >>
> >
> >>
> >
> >> [As wg-member] --- inline...
> >
> >>
> >
> >> "Les Ginsberg (ginsberg)"  writes:
> >
> >>
> >
> >> > Chris -
> >
> >> >
> >
> >> >> -Original Message-
> >
> >> >
> >
> >> >> From: Christian Hopps 
> >
> >> >> Sent: Monday, October 3, 2022 8:37 AM
> >
> >> >> To: Les Ginsberg (ginsberg) 
> >
> >> >> Cc: Robert Raszuk ; Tony Li  >>; Henk
> >
> >> Smit
> >
> >> >> ; lsr@ietf.org
> >
> >> >> Subject: Re: [Lsr] New Version

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Tony Li

Hi Les,


> In this thread we talk about MP-TLV support – but this is less than precise. 
> What we are really talking about is MP-TLV support for specific TLVs. So this 
> now requires per/TLV advertisement.


No, it does not.  As you yourself verbally agreed, we are referring to the 
class of TLVs that Chris describes as ‘implicit’. They do not explicitly permit 
or reject MP-TLVs.  We need one bit of information to characterize this entire 
class.

Please stop misrepresenting what we are asking for. Please stop backpeddling on 
what you’ve already agreed to.


> One response to this is to say “we will limit this to ‘important’ 
> advertisements” – but how are we to agree on what is important and what 
> isn’t? Do we have an operational definition of “important”?


We might have to apply judgment and common sense.

Tony

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Robert Raszuk
Just to add and expand to your point Les ...

The concept of "forward referencing" used in some OSes explicitly permit
configuration of elements not even present on the box or in the system
ahead of time. For example you can configure an interface which will be
inserted later.

Same for any network wide feature.

However this means that activation of such network wide features can take
place under dynamic conditions too. If "all nodes required" support feature
X I can advertise it. (Let's assume that other nodes which do not require
it will not crash).

That means that having such dynamic capability may be actually pretty
helpful.

Thx,
R.








On Wed, Oct 5, 2022 at 9:33 PM Les Ginsberg (ginsberg) 
wrote:

> Bruno –
>
>
>
> Thanx for commenting – I really want to hear from the operator community.
>
>
>
> In regards to using “functionality supported” advertisements to either
> issue warnings or possibly block/disable configuration changes…I think this
> sounds better in theory than it can be realized in practice.
>
>
>
> If you block config when not all nodes advertise support for “feature X”,
> what do you do with existing config which was accepted at a time when all
> nodes that are up/reachable did advertise the needed support, but now a new
> node comes up (or becomes reachable) and it does NOT advertise the support?
>
> What do you do on bootup when the saved config includes configuration that
> should not be accepted under the rules for “feature X”?
>
> Do implementations have to provide a knob to control whether configs
> should be rejected or just a warning issued?
>
> Do you think operators will now try to leverage the new “protection” they
> expect from implementations and feel free to relax testing on the
> assumption that a good implementation will prevent a “bad configuration”
> from being enabled?
>
>
>
> And what are the candidates for such functionality?
>
> In this thread we talk about MP-TLV support – but this is less than
> precise. What we are really talking about is MP-TLV support for specific
> TLVs. So this now requires per/TLV advertisement.
>
> And what else might be a good candidate? What about individual sub-TLV
> support e.g., for the different link attributes? If a node advertises a
> link attribute that not all nodes support then various forms of
> TE/flex-algo may not work.
>
> Do we then require each node to advertise every TLV it supports and every
> sub-TLV it supports?
>
> What about flag bits within a given TLV/sub-TLV?
>
>
>
> One response to this is to say “we will limit this to ‘important’
> advertisements” – but how are we to agree on what is important and what
> isn’t? Do we have an operational definition of “important”?
>
>
>
> I think this quite easily leads to an explosion of advertisements and a
> great deal of complexity which returns very little.
>
> If an operator configures something it is because they need that in order
> for the network to meet the requirements of the given deployment.
> Blocking/disabling config might prevent some problems – but it won’t make
> the network functional.
>
>
>
> I am quite concerned that this is something that “sounds good” but in
> practice adds little value – yet places significant demands on
> implementations.
>
>
>
>Les
>
>
>
>
>
> *From:* Lsr  *On Behalf Of *
> bruno.decra...@orange.com
> *Sent:* Wednesday, October 5, 2022 2:29 AM
> *To:* Les Ginsberg (ginsberg) ; Tony
> Li ; Christian Hopps 
> *Cc:* Robert Raszuk ; Henk Smit ;
> lsr@ietf.org
> *Subject:* Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
>
> I’ve not followed everything in details so I’ve been reluctant to comment,
> but nonetheless please find below a diverse set of comments.
>
>
>
> > From: Christian Hopps 
>
> […]
>
> > AFAICT we now have implementations out there that are sending multiple
> TLVs which are not documented to be sent that way, that historically were
> not expected to be received that way, and then we have other
> implementations that do not expect this new, convenient but rather loose
> interpretation of the standards. Consider we have documents that explicitly
> state that a TLV can be sent multiple times, it would be completely normal
> for someone to then assume that when that isn't stated explicitly that
> multiple versions of those TLV will not be sent.
>
>
>
> > Thus the need for this document -- in some form.
>
>
>
> Thank you Chris.
>
> Definitely +1
>
>
>
> To clarify, are we talking about:
>
> - different nodes in the flooding domain having a different understanding
> of the LSDB content
>

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Les Ginsberg (ginsberg)
Bruno -

Thanx for commenting - I really want to hear from the operator community.

In regards to using "functionality supported" advertisements to either issue 
warnings or possibly block/disable configuration changes...I think this sounds 
better in theory than it can be realized in practice.

If you block config when not all nodes advertise support for "feature X", what 
do you do with existing config which was accepted at a time when all nodes that 
are up/reachable did advertise the needed support, but now a new node comes up 
(or becomes reachable) and it does NOT advertise the support?
What do you do on bootup when the saved config includes configuration that 
should not be accepted under the rules for "feature X"?
Do implementations have to provide a knob to control whether configs should be 
rejected or just a warning issued?
Do you think operators will now try to leverage the new "protection" they 
expect from implementations and feel free to relax testing on the assumption 
that a good implementation will prevent a "bad configuration" from being 
enabled?

And what are the candidates for such functionality?
In this thread we talk about MP-TLV support - but this is less than precise. 
What we are really talking about is MP-TLV support for specific TLVs. So this 
now requires per/TLV advertisement.
And what else might be a good candidate? What about individual sub-TLV support 
e.g., for the different link attributes? If a node advertises a link attribute 
that not all nodes support then various forms of TE/flex-algo may not work.
Do we then require each node to advertise every TLV it supports and every 
sub-TLV it supports?
What about flag bits within a given TLV/sub-TLV?

One response to this is to say "we will limit this to 'important' 
advertisements" - but how are we to agree on what is important and what isn't? 
Do we have an operational definition of "important"?

I think this quite easily leads to an explosion of advertisements and a great 
deal of complexity which returns very little.
If an operator configures something it is because they need that in order for 
the network to meet the requirements of the given deployment. 
Blocking/disabling config might prevent some problems - but it won't make the 
network functional.

I am quite concerned that this is something that "sounds good" but in practice 
adds little value - yet places significant demands on implementations.

   Les


From: Lsr  On Behalf Of bruno.decra...@orange.com
Sent: Wednesday, October 5, 2022 2:29 AM
To: Les Ginsberg (ginsberg) ; Tony Li 
; Christian Hopps 
Cc: Robert Raszuk ; Henk Smit ; 
lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

I've not followed everything in details so I've been reluctant to comment, but 
nonetheless please find below a diverse set of comments.


> From: Christian Hopps mailto:cho...@chopps.org>>
[...]

> AFAICT we now have implementations out there that are sending multiple TLVs 
> which are not documented to be sent that way, that historically were not 
> expected to be received that way, and then we have other implementations that 
> do not expect this new, convenient but rather loose interpretation of the 
> standards. Consider we have documents that explicitly state that a TLV can be 
> sent multiple times, it would be completely normal for someone to then assume 
> that when that isn't stated explicitly that multiple versions of those TLV 
> will not be sent.



> Thus the need for this document -- in some form.

Thank you Chris.
Definitely +1

To clarify, are we talking about:
- different nodes in the flooding domain having a different understanding of 
the LSDB content
- a (LS) protocol relying on all nodes in the flooding domain to have a 
consistent (view of the) LSDB (especially with FlexAlgo for which the number of 
(sub)TLV requiring a consistent view across the flooding domain has increased 
and may further increase)


And a standardization group defining specifications to allow for interop?

Sure, I'd be interested in an implementation report to at least learn about 
such (sub) TLV and those implementations.

[...]

> From: Lsr lsr-boun...@ietf.org<mailto:lsr-boun...@ietf.org> On Behalf Of Les 
> Ginsberg (ginsberg)


> [LES:] This isn't a new problem for operators. There are many examples of 
> extensions to the protocol which have been introduced which result in a 
> broken network in cases of partial deployment. To list just a few:

>

>  - Wide metrics

> - Cryptographic authentication

> - Support for extended LSP lifetime (> 1200 seconds)



> The consequences of sending MP-TLVs when not all routers support them are no 
> more severe than the consequences of partial support for any of the above 
> features.

Agreed.
However, it's not because this is not a new 

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Tony Li

Hi Les,
 
> Using the protocol to send what is best described as some subset of a PICS 
> means that we propose to use the IGP flooding mechanism to send static 
> information which the protocol itself cannot (and should not) use in its 
> operation. This consumes space, bandwidth, gets periodically refreshed 
> unnecessarily, and now a complete copy of the information from every node 
> resides on every router in the network when it is only needed by an “NMS”. It 
> would be hard to come up with a better example of “IGP isn’t a dump truck” 
> than this.


If you can’t beat ‘em, join ‘em.


> If there is a belief that we can severely limit the amount of information 
> that is sent/node, I’d have to say that I am skeptical. Once we allow this 
> into the protocol, I don’t see any basis on which to separate what is allowed 
> and what is disallowed. It would not be unreasonable for an operator to say 
> that everything that is a candidate to be mentioned in a PICS is a legitimate 
> candidate for being advertised using this mechanism. Which means the amount 
> of information is likely to become very large – especially once it becomes 
> the de facto way of providing protocol management information.
>  
> The justification seems to be that we don’t have anything better – which 
> represents a longstanding failure of the management plane. While I agree with 
> you that management plane solutions are not adequate – not least because we 
> can’t get the industry to converge on a single solution – this does not mean 
> we should invest in the wrong solution.
>  
> We would be better served spending time and effort working on the right 
> solution - as difficult as that may be.
>  
> If we despair of getting a management plane solution, my suggestion would be 
> to use RFC 6823/6822 to define an IS-IS protocol management application that 
> could support the advertisement of such information. This is technically 
> straightforward to define/implement, easily extensible, and it separates the 
> management information from the information used by the protocol.  And 
> because a separate topology can be used for the “management instance”,  it 
> would be possible to reduce the number of copies in the network.


A blue dump truck is not an architectural improvement over a red dump truck and 
definitely not the right solution.

What we need is a management plane mechanism that can be easily consulted, even 
by nodes not running the IGP.  Or nodes not running any IGP. We should NOT 
require storing the management data on every node. That’s silly.  Rather, we 
should have a set of distributed, synchronized servers that can be easily 
referenced and updated. We need an auto-discovery mechanism so that nodes can 
learn where these servers are. We need a common hierarchical data schema so 
that data is organized consistently. And we needed this in 1992. ;-)

Even if you agree with this, I am not willing to stall the current work the 
decades that it will take for the IETF to make progress on this.

Tony

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Tony Li


Hi all,

> On Oct 3, 2022, at 8:37 AM, Christian Hopps  wrote:
> 
> 
> [As wg-member]
> 
> I think the draft should include a table of all top level TLVS as rows and 
> for columns we have
> 
> - "Implicit Single TLV Only" (e.g., no key info)
> - "Implicit Multi-TLV Only"
> - "Explicit Single TLV Only"
> - "Explicit Multi-TLV-Allowed"
> 
> This draft then would *explicitly* cover the existing "implicit" cases and we 
> have the table to point at to be precise about what we are talking about.


I have completed this effort.

Rather than burden the draft with this, I have created a spreadsheet and am 
sharing a link to the sheet: 
https://docs.google.com/spreadsheets/d/1VWQHsJTq7JRUVdRVcRd_QYYyY_LiOX7H0ZKn8feRZo4/edit?usp=sharing

There are numerous proprietary TLVs that I cannot evaluate. I have ignored them.

I am human. I expect that I will have missed several things. If you have a 
correction, please unicast me.

Regards,
Tony

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Christian Hopps


[as wg-member]

I don't think what I've written is being read correctly.


If a node which supports MP-TLV were to introduce/withdraw MP-TLVs
based on the advertised state of support for MP-TLVs by all nodes in
the network,  this would introduce flooding storms on the
transitions. Not a good thing.


I didn't suggest this. My suggestion was that a router sending multiple-TLVs needs to 
advertise that it does so, and receivers would interpret what they received (is this 
second TLV a replacement or is it a concatenation?) based on that advertisement. The 
proposal is for a "capability" in this case and that could serve as this 
advertisement.

I believe you would argue against this as it might impact networks that just 
happen to be lucky and working right now.

If things would have been done correctly (i.e., documented) we also would have 
had the option to add a sub-tlv to the TLVs in question that indicated they 
were part of a set rather than replacements.


MP-TLVs are not sent just because an implementation supports them.
They are sent because the current configuration requires them.


The SW also has the option to alert the operator that their configuration is 
not supported, and to revise it, rather than play loose with standards.

The vendors who made these changes should have brought this to the IETF as a 
draft that would have clear and deterministic transition mechanism (e.g., wide 
metrics had a clear transition plan documented in it's RFC).

I'm suggesting that the most important thing to come now is to make clear what 
operators need to do to have a deterministic functional network. It doesn't 
have to be the way I suggested, but we have to get there somehow.

Thanks,
Chris.
[as wg-member]


"Les Ginsberg (ginsberg)"  writes:


Chris -



Inline.




-Original Message-



From: Christian Hopps 



Sent: Monday, October 3, 2022 5:52 PM



To: Les Ginsberg (ginsberg) 



Cc: Christian Hopps ; Robert Raszuk



; Tony Li ; Henk Smit



; lsr@ietf.org



Subject: Re: [Lsr] New Version Notification for

draft-pkaneria-lsr-multi-tlv-


01.txt











[As wg-member] --- inline...







"Les Ginsberg (ginsberg)"  writes:







> Chris -



>



>> -Original Message-



>



>> From: Christian Hopps 



>> Sent: Monday, October 3, 2022 8:37 AM



>> To: Les Ginsberg (ginsberg) 



>> Cc: Robert Raszuk ; Tony Li 


Smit



>> ; lsr@ietf.org



>> Subject: Re: [Lsr] New Version Notification for

draft-pkaneria-lsr-multi-tlv-


01.txt



>



>> [As wg-member]



>>



>> I think the draft should include a table of all top level TLVS

as rows and for


>> columns we have



>>



>> - "Implicit Single TLV Only" (e.g., no key info)



>> - "Implicit Multi-TLV Only"



>> - "Explicit Single TLV Only"



>> - "Explicit Multi-TLV-Allowed"



>>



>> This draft then would *explicitly* cover the existing "implicit"

cases and


we



>> have the table to point at to be precise about what we are

talking about.






> [LES:] I am not overly enthused about this - but I can see its



> usefulness, so I don’t oppose it.



>



> Probably best realized as an additional column in the existing

IS-IS


> TLV Codepoints registry.



>



> But also realize that in some cases this extends to sub-TLVs (as

one


> example "16 Application-Specific Link Attributes").







>> Now about the capability... There's an argument about including

a router


>> capability and it seems to be that people have agreed that it

should *not*


>> have any operation impact on the protocol.







>> I think this is hasty. I would like to look at the above table

to see what


TLVs



>> we are *exactly* talking about here (the implicit multi-tlv

ones). People


have



>> said that implementations support multi-tlv use of these

implicit cases


(e.g.,



>> the draft talks about extended reachability).



>>



>> Really?







> [LES:] Yes - really. 







>> I could easily believe its not common. I think we should get

specific about


>> vendors and versions (and maybe specific TLVS?) if we are saying

this has


>> been deployed and is in use. I've written a few IS-IS

implementations and


I



>> don't think *any* of them supported multiple tlvs of, for

example,


extended



>> reachability (seeing 2 of the same keyed info would be seen as

one


replacing



>> the other, or a bug if in the same LSP segment).







> [LES:] All of us who have "been around for a while" have worked

on


> implementations that did not support MP-TLVs. Prior to new

features


> (SR, TE Metric Extensions (RFC 8570), flex-algo, more extensive



> deployments of IPv

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread Tony Li

Hi Bruno,

> To clarify, are we talking about:
> - different nodes in the flooding domain having a different understanding of 
> the LSDB content


We are trying to prevent that.  We are trying to figure out how we can relax 
the cases where the specifications imply, but do not clearly state that a TLV 
may occur multiple times. If some nodes are not prepared for that, then they 
would misinterpret the advertisements, perceive different information content, 
and misbehave. What are the necessary and sufficient precautions to prevent 
this?


> And a standardization group defining specifications to allow for interop?


In particular, how do we propagate information to know which nodes are capable 
of correctly interpreting multi-part TLVs?

 
> Sure, I’d be interested in an implementation report to at least learn about 
> such (sub) TLV and those implementations.


Chris is not asking for an implementation report, nor is that what I’m working 
on.

T


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-05 Thread bruno.decraene
I've not followed everything in details so I've been reluctant to comment, but 
nonetheless please find below a diverse set of comments.


> From: Christian Hopps mailto:cho...@chopps.org>>
[...]

> AFAICT we now have implementations out there that are sending multiple TLVs 
> which are not documented to be sent that way, that historically were not 
> expected to be received that way, and then we have other implementations that 
> do not expect this new, convenient but rather loose interpretation of the 
> standards. Consider we have documents that explicitly state that a TLV can be 
> sent multiple times, it would be completely normal for someone to then assume 
> that when that isn't stated explicitly that multiple versions of those TLV 
> will not be sent.



> Thus the need for this document -- in some form.

Thank you Chris.
Definitely +1

To clarify, are we talking about:
- different nodes in the flooding domain having a different understanding of 
the LSDB content
- a (LS) protocol relying on all nodes in the flooding domain to have a 
consistent (view of the) LSDB (especially with FlexAlgo for which the number of 
(sub)TLV requiring a consistent view across the flooding domain has increased 
and may further increase)


And a standardization group defining specifications to allow for interop?

Sure, I'd be interested in an implementation report to at least learn about 
such (sub) TLV and those implementations.

[...]

> From: Lsr lsr-boun...@ietf.org<mailto:lsr-boun...@ietf.org> On Behalf Of Les 
> Ginsberg (ginsberg)


> [LES:] This isn't a new problem for operators. There are many examples of 
> extensions to the protocol which have been introduced which result in a 
> broken network in cases of partial deployment. To list just a few:

>

>  - Wide metrics

> - Cryptographic authentication

> - Support for extended LSP lifetime (> 1200 seconds)



> The consequences of sending MP-TLVs when not all routers support them are no 
> more severe than the consequences of partial support for any of the above 
> features.

Agreed.
However, it's not because this is not a new problem that this is not a problem. 
We don't need to increase the problems and make things worse.
And it's not because this can be made to work (with extra work), that this 
can't also fails. (and this does fail sometimes)


[...]

> From: Lsr lsr-boun...@ietf.org<mailto:lsr-boun...@ietf.org> On Behalf Of Les 
> Ginsberg (ginsberg)
[...]
> Using the protocol to send what is best described as some subset of a PICS 
> means that we propose to use the IGP flooding mechanism to send static 
> information which the protocol itself cannot (and should not) use in its 
> operation.

I'm not sure that I agree with "cannot (and should not) use in its operation".
If the correct understanding of MP TLV is required for correct operation, such 
information can be used by IS-IS to warn the network operator that IS-IS is not 
correctly working anymore. As of today, an operator may not be even aware of 
the problem.
And a so-called "smart" implementation could even forbid a configuration change 
which would translate into sending a MP TLV not understood by all nodes.


So on my side, I would rather welcome a continued discussion on this topic 
which seems important for network operations.

Thank you,
Regards,
--Bruno



Orange Restricted
From: Lsr  On Behalf Of Les Ginsberg (ginsberg)
Sent: Tuesday, October 4, 2022 7:35 PM
To: Tony Li 
Cc: Christian Hopps ; Robert Raszuk ; 
Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

Tony -

We don't agree - but that isn't news.
Let me try to start a meaningful discussion.

Using the protocol to send what is best described as some subset of a PICS 
means that we propose to use the IGP flooding mechanism to send static 
information which the protocol itself cannot (and should not) use in its 
operation. This consumes space, bandwidth, gets periodically refreshed 
unnecessarily, and now a complete copy of the information from every node 
resides on every router in the network when it is only needed by an "NMS". It 
would be hard to come up with a better example of "IGP isn't a dump truck" than 
this.

If there is a belief that we can severely limit the amount of information that 
is sent/node, I'd have to say that I am skeptical. Once we allow this into the 
protocol, I don't see any basis on which to separate what is allowed and what 
is disallowed. It would not be unreasonable for an operator to say that 
everything that is a candidate to be mentioned in a PICS is a legitimate 
candidate for being advertised using this mechanism. Which means the amount of 
information is likely to become very large - especially once it becomes the de 
facto way of providing protocol management information.

The justifi

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-04 Thread Les Ginsberg (ginsberg)
Tony –

We don’t agree – but that isn’t news.
Let me try to start a meaningful discussion.

Using the protocol to send what is best described as some subset of a PICS 
means that we propose to use the IGP flooding mechanism to send static 
information which the protocol itself cannot (and should not) use in its 
operation. This consumes space, bandwidth, gets periodically refreshed 
unnecessarily, and now a complete copy of the information from every node 
resides on every router in the network when it is only needed by an “NMS”. It 
would be hard to come up with a better example of “IGP isn’t a dump truck” than 
this.

If there is a belief that we can severely limit the amount of information that 
is sent/node, I’d have to say that I am skeptical. Once we allow this into the 
protocol, I don’t see any basis on which to separate what is allowed and what 
is disallowed. It would not be unreasonable for an operator to say that 
everything that is a candidate to be mentioned in a PICS is a legitimate 
candidate for being advertised using this mechanism. Which means the amount of 
information is likely to become very large – especially once it becomes the de 
facto way of providing protocol management information.

The justification seems to be that we don’t have anything better – which 
represents a longstanding failure of the management plane. While I agree with 
you that management plane solutions are not adequate – not least because we 
can’t get the industry to converge on a single solution – this does not mean we 
should invest in the wrong solution.

We would be better served spending time and effort working on the right 
solution - as difficult as that may be.

If we despair of getting a management plane solution, my suggestion would be to 
use RFC 6823/6822 to define an IS-IS protocol management application that could 
support the advertisement of such information. This is technically 
straightforward to define/implement, easily extensible, and it separates the 
management information from the information used by the protocol.  And because 
a separate topology can be used for the “management instance”,  it would be 
possible to reduce the number of copies in the network.

Thoughts??

   Les

From: Tony Li  On Behalf Of Tony Li
Sent: Tuesday, October 4, 2022 9:16 AM
To: Les Ginsberg (ginsberg) 
Cc: Christian Hopps ; Robert Raszuk ; 
Henk Smit ; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

Hi Les,

Folks may well complain that management tools are not as good as they need to 
be, but trying to compensate for this by adding management information into the 
protocol itself isn’t a good solution.


It is not a good solution. But it is the only practical solution available. At 
scale, we need automation. We have tried and failed (again) to get broad 
adoption of a management infrastructure. We continue to reject alternative 
approaches. The thought of someone keeping all of this in their heads is simply 
naive.

We have already painted ourselves into this corner. There is no good way out.

Tony


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-04 Thread Tony Li


Hi all,

> On Oct 3, 2022, at 8:37 AM, Christian Hopps  wrote:
> 
> 
> [As wg-member]
> 
> I think the draft should include a table of all top level TLVS as rows and 
> for columns we have
> 
> - "Implicit Single TLV Only" (e.g., no key info)
> - "Implicit Multi-TLV Only"
> - "Explicit Single TLV Only"
> - "Explicit Multi-TLV-Allowed"
> 
> This draft then would *explicitly* cover the existing "implicit" cases and we 
> have the table to point at to be precise about what we are talking about.


I am actively working on this. Collaboration would be most welcome.

T

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-04 Thread Robert Raszuk
Tony & Les,

I do think we should be signalling node's enabled/active capabilities
including the ability to process various types of encoded PDUs in band.

I do not believe in live management planes. Sure getting stats, config push
etc ... it is all doable and deployed. But dynamic configuration based on
nodes activated capabilities via mgmt plane is hard as we keep adding stuff
to protocols. Even if one builds such tool today tomorrow it is obsoleted.

I do however consider Les's points as valid too. Especially insertion of
node anywhere in the area which may not be capable of handling some
encoding (and does not need to operationally) yet his signalling if we act
on it in the protocol operationally would have a potential to break
existing features. So to do this right such capability signalling would
need to contain scope.

But I see no harm in having such info available as mgmt information. After
all this is all pretty static and so little that I am not sure why we are
even spending time to discuss it so long.

> The thought of someone keeping all of this in their heads is simply naive.

Or maybe it is actually smart :) Maybe the goal here is to make networks so
complex that only a few can safely operate them. Otherwise just outsource
it and use "...as-a-service".

Best,
R.

On Tue, Oct 4, 2022 at 6:16 PM Tony Li  wrote:

> Hi Les,
>
> *Folks may well complain that management tools are not as good as they
> need to be, but trying to compensate for this by adding management
> information into the protocol itself isn’t a good solution.*
>
> It is not a good solution. But it is the only practical solution
> available. At scale, we need automation. We have tried and failed (again)
> to get broad adoption of a management infrastructure. We continue to reject
> alternative approaches. The thought of someone keeping all of this in their
> heads is simply naive.
>
> We have already painted ourselves into this corner. There is no good way
> out.
>
> Tony
>
>
>
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-04 Thread Tony Li
Hi Les,

> Folks may well complain that management tools are not as good as they need to 
> be, but trying to compensate for this by adding management information into 
> the protocol itself isn’t a good solution.


It is not a good solution. But it is the only practical solution available. At 
scale, we need automation. We have tried and failed (again) to get broad 
adoption of a management infrastructure. We continue to reject alternative 
approaches. The thought of someone keeping all of this in their heads is simply 
naive.

We have already painted ourselves into this corner. There is no good way out.

Tony


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-04 Thread Les Ginsberg (ginsberg)
Chris -



Inline.



> -Original Message-

> From: Christian Hopps 

> Sent: Monday, October 3, 2022 5:52 PM

> To: Les Ginsberg (ginsberg) 

> Cc: Christian Hopps ; Robert Raszuk

> ; Tony Li ; Henk Smit

> ; lsr@ietf.org

> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-

> 01.txt

>

>

> [As wg-member] --- inline...

>

> "Les Ginsberg (ginsberg)" mailto:ginsb...@cisco.com>> 
> writes:

>

> > Chris -

> >

> >> -Original Message-

> >

> >> From: Christian Hopps mailto:cho...@chopps.org>>

> >> Sent: Monday, October 3, 2022 8:37 AM

> >> To: Les Ginsberg (ginsberg) mailto:ginsb...@cisco.com>>

> >> Cc: Robert Raszuk mailto:rob...@raszuk.net>>; Tony Li 
> >> mailto:tony...@tony.li>>; Henk

> Smit

> >> mailto:henk.i...@xs4all.nl>>; 
> >> lsr@ietf.org<mailto:lsr@ietf.org>

> >> Subject: Re: [Lsr] New Version Notification for 
> >> draft-pkaneria-lsr-multi-tlv-

> 01.txt

> >

> >> [As wg-member]

> >>

> >> I think the draft should include a table of all top level TLVS as rows and 
> >> for

> >> columns we have

> >>

> >> - "Implicit Single TLV Only" (e.g., no key info)

> >> - "Implicit Multi-TLV Only"

> >> - "Explicit Single TLV Only"

> >> - "Explicit Multi-TLV-Allowed"

> >>

> >> This draft then would *explicitly* cover the existing "implicit" cases and

> we

> >> have the table to point at to be precise about what we are talking about.

>

> > [LES:] I am not overly enthused about this - but I can see its

> > usefulness, so I don’t oppose it.

> >

> > Probably best realized as an additional column in the existing IS-IS

> > TLV Codepoints registry.

> >

> > But also realize that in some cases this extends to sub-TLVs (as one

> > example "16 Application-Specific Link Attributes").

>

> >> Now about the capability... There's an argument about including a router

> >> capability and it seems to be that people have agreed that it should *not*

> >> have any operation impact on the protocol.

>

> >> I think this is hasty. I would like to look at the above table to see what

> TLVs

> >> we are *exactly* talking about here (the implicit multi-tlv ones). People

> have

> >> said that implementations support multi-tlv use of these implicit cases

> (e.g.,

> >> the draft talks about extended reachability).

> >>

> >> Really?

>

> > [LES:] Yes - really. 

>

> >> I could easily believe its not common. I think we should get specific about

> >> vendors and versions (and maybe specific TLVS?) if we are saying this has

> >> been deployed and is in use. I've written a few IS-IS implementations and

> I

> >> don't think *any* of them supported multiple tlvs of, for example,

> extended

> >> reachability (seeing 2 of the same keyed info would be seen as one

> replacing

> >> the other, or a bug if in the same LSP segment).

>

> > [LES:] All of us who have "been around for a while" have worked on

> > implementations that did not support MP-TLVs. Prior to new features

> > (SR, TE Metric Extensions (RFC 8570), flex-algo, more extensive

> > deployments of IPv6, etc.) there wasn't a need - at least for

> > Neighbor/Prefix advertisements.

> >

> > But deployment requirements have evolved. We absolutely have cases

> > today where a single TLV is insufficient space-wise for both neighbor

> > and prefix advertisements and there are multiple implementations

> > which already support this.

> >

> > If the WG wants to pursue an Implementation Report on this, I have no

> > objection.

> >

> > BTW, the need for this should not surprise you. Discussion of this

> > problem dates back at least to 2004: https://datatracker.ietf.org/doc

> > /html/draft-shen-isis-extended-tlv-00

>

> The need is not a surprise to me, no.

>

> >> Once we have this info I think a stronger case might be made for actually

> >> having the router capability be used *operationally* (i.e., if you don't 
> >> see

> the

> >> capability advertised then that router in fact doesn't send multi-tlv tlvs

> and

> >> they should be seen as replacements of each other), and the argument

> >> about it's inclusion goes away as it's then required.

>

> > [LES:] I don't a

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-03 Thread Christian Hopps


[As wg-member] --- inline...

"Les Ginsberg (ginsberg)"  writes:


Chris -


-Original Message-



From: Christian Hopps 
Sent: Monday, October 3, 2022 8:37 AM
To: Les Ginsberg (ginsberg) 
Cc: Robert Raszuk ; Tony Li ; Henk Smit
; lsr@ietf.org
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt



[As wg-member]

I think the draft should include a table of all top level TLVS as rows and for
columns we have

- "Implicit Single TLV Only" (e.g., no key info)
- "Implicit Multi-TLV Only"
- "Explicit Single TLV Only"
- "Explicit Multi-TLV-Allowed"

This draft then would *explicitly* cover the existing "implicit" cases and we
have the table to point at to be precise about what we are talking about.



[LES:] I am not overly enthused about this - but I can see its
usefulness, so I don’t oppose it.

Probably best realized as an additional column in the existing IS-IS
TLV Codepoints registry.

But also realize that in some cases this extends to sub-TLVs (as one
example "16 Application-Specific Link Attributes").



Now about the capability... There's an argument about including a router
capability and it seems to be that people have agreed that it should *not*
have any operation impact on the protocol.



I think this is hasty. I would like to look at the above table to see what TLVs
we are *exactly* talking about here (the implicit multi-tlv ones). People have
said that implementations support multi-tlv use of these implicit cases (e.g.,
the draft talks about extended reachability).

Really?



[LES:] Yes - really. 



I could easily believe its not common. I think we should get specific about
vendors and versions (and maybe specific TLVS?) if we are saying this has
been deployed and is in use. I've written a few IS-IS implementations and I
don't think *any* of them supported multiple tlvs of, for example, extended
reachability (seeing 2 of the same keyed info would be seen as one replacing
the other, or a bug if in the same LSP segment).



[LES:] All of us who have "been around for a while" have worked on
implementations that did not support MP-TLVs. Prior to new features
(SR, TE Metric Extensions (RFC 8570), flex-algo, more extensive
deployments of IPv6, etc.) there wasn't a need - at least for
Neighbor/Prefix advertisements.

But deployment requirements have evolved. We absolutely have cases
today where a single TLV is insufficient space-wise for both neighbor
and prefix advertisements and there are multiple implementations
which already support this.

If the WG wants to pursue an Implementation Report on this, I have no
objection.

BTW, the need for this should not surprise you. Discussion of this
problem dates back at least to 2004: https://datatracker.ietf.org/doc
/html/draft-shen-isis-extended-tlv-00


The need is not a surprise to me, no.


Once we have this info I think a stronger case might be made for actually
having the router capability be used *operationally* (i.e., if you don't see the
capability advertised then that router in fact doesn't send multi-tlv tlvs and
they should be seen as replacements of each other), and the argument
about it's inclusion goes away as it's then required.



[LES:] I don't agree with this argument - but I think the discussion
triggered by posts from Gunter and Henk has already covered this well
from both points of view, so I won’t repeat here.


That discussion had to do with whether to include a bit that is for management 
purposes only. What I am seeing here is a need for an operationally relevant 
bit (i.e., determines how the protocol functions).

AFAICT we now have implementations out there that are sending multiple TLVs 
which are not documented to be sent that way, that historically were not 
expected to be received that way, and then we have other implementations that 
do not expect this new, convenient but rather loose interpretation of the 
standards. Consider we have documents that explicitly state that a TLV can be 
sent multiple times, it would be completely normal for someone to then assume 
that when that isn't stated explicitly that multiple versions of those TLV will 
not be sent.

Thus the need for this document -- in some form.

Having all routers work from the same link-state database is basic requirement 
to the correct functioning of the decision process.

Are we just lucky that things haven't really broken yet? How can an operator 
even know what they've got deployed? Before this document there's nothing to 
even refer to to document the possible different behaviors.

If people want to argue that no operationally significant bit is needed then 
how can an operator be expected to get this right? What is the exact process 
they should follow to have interoperating routers?

Thanks,
Chris.
[as wg-member]


   Les



Thanks,
Chris.
[as wg-member]


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-03 Thread Les Ginsberg (ginsberg)
Chris -



> -Original Message-

> From: Christian Hopps 

> Sent: Monday, October 3, 2022 8:37 AM

> To: Les Ginsberg (ginsberg) 

> Cc: Robert Raszuk ; Tony Li ; Henk Smit

> ; lsr@ietf.org

> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-

> 01.txt

>

>

> [As wg-member]

>

> I think the draft should include a table of all top level TLVS as rows and for

> columns we have

>

> - "Implicit Single TLV Only" (e.g., no key info)

> - "Implicit Multi-TLV Only"

> - "Explicit Single TLV Only"

> - "Explicit Multi-TLV-Allowed"

>

> This draft then would *explicitly* cover the existing "implicit" cases and we

> have the table to point at to be precise about what we are talking about.

>



[LES:] I am not overly enthused about this - but I can see its usefulness, so I 
don’t oppose it.

Probably best realized as an additional column in the existing IS-IS TLV 
Codepoints registry.

But also realize that in some cases this extends to sub-TLVs (as one example 
"16 Application-Specific Link Attributes").





> Now about the capability... There's an argument about including a router

> capability and it seems to be that people have agreed that it should *not*

> have any operation impact on the protocol.

>

> I think this is hasty. I would like to look at the above table to see what 
> TLVs

> we are *exactly* talking about here (the implicit multi-tlv ones). People have

> said that implementations support multi-tlv use of these implicit cases (e.g.,

> the draft talks about extended reachability).

>

> Really?



[LES:] Yes - really. 



>

> I could easily believe its not common. I think we should get specific about

> vendors and versions (and maybe specific TLVS?) if we are saying this has

> been deployed and is in use. I've written a few IS-IS implementations and I

> don't think *any* of them supported multiple tlvs of, for example, extended

> reachability (seeing 2 of the same keyed info would be seen as one replacing

> the other, or a bug if in the same LSP segment).



[LES:] All of us who have "been around for a while" have worked on 
implementations that did not support MP-TLVs. Prior to new features (SR, TE 
Metric Extensions (RFC 8570), flex-algo, more extensive deployments of IPv6, 
etc.) there wasn't a need - at least for Neighbor/Prefix advertisements.

But deployment requirements have evolved. We absolutely have cases today where 
a single TLV is insufficient space-wise for both neighbor and prefix 
advertisements and there are multiple implementations which already support 
this.

If the WG wants to pursue an Implementation Report on this, I have no objection.



BTW, the need for this should not surprise you. Discussion of this problem 
dates back at least to 2004: 
https://datatracker.ietf.org/doc/html/draft-shen-isis-extended-tlv-00



>

> Once we have this info I think a stronger case might be made for actually

> having the router capability be used *operationally* (i.e., if you don't see 
> the

> capability advertised then that router in fact doesn't send multi-tlv tlvs and

> they should be seen as replacements of each other), and the argument

> about it's inclusion goes away as it's then required.

>



[LES:] I don't agree with this argument - but I think the discussion triggered 
by posts from Gunter and Henk has already covered this well from both points of 
view, so I won’t repeat here.



   Les



> Thanks,

> Chris.

> [as wg-member]
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-10-03 Thread Christian Hopps


[As wg-member]

I think the draft should include a table of all top level TLVS as rows and for 
columns we have

- "Implicit Single TLV Only" (e.g., no key info)
- "Implicit Multi-TLV Only"
- "Explicit Single TLV Only"
- "Explicit Multi-TLV-Allowed"

This draft then would *explicitly* cover the existing "implicit" cases and we 
have the table to point at to be precise about what we are talking about.

Now about the capability... There's an argument about including a router 
capability and it seems to be that people have agreed that it should *not* have 
any operation impact on the protocol.

I think this is hasty. I would like to look at the above table to see what TLVs 
we are *exactly* talking about here (the implicit multi-tlv ones). People have 
said that implementations support multi-tlv use of these implicit cases (e.g., 
the draft talks about extended reachability).

Really?

I could easily believe its not common. I think we should get specific about 
vendors and versions (and maybe specific TLVS?) if we are saying this has been 
deployed and is in use. I've written a few IS-IS implementations and I don't 
think *any* of them supported multiple tlvs of, for example, extended 
reachability (seeing 2 of the same keyed info would be seen as one replacing 
the other, or a bug if in the same LSP segment).

Once we have this info I think a stronger case might be made for actually 
having the router capability be used *operationally* (i.e., if you don't see 
the capability advertised then that router in fact doesn't send multi-tlv tlvs 
and they should be seen as replacements of each other), and the argument about 
it's inclusion goes away as it's then required.

Thanks,
Chris.
[as wg-member]


signature.asc
Description: PGP signature
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-22 Thread Les Ginsberg (ginsberg)
Robert –

If you are suggesting that – based on the current state of advertised support 
for a given feature (such as Multi-tlv) by all routers in the network - that 
routers should dynamically modify what they send, this is VERY DANGEROUS – and 
should not be done.
It can lead to flooding storms as all routers attempt to enable/disable the 
sending of information which was previously suppressed/unsuppressed.

Let’s not go there please.

Les

From: Robert Raszuk 
Sent: Thursday, September 22, 2022 1:37 PM
To: Les Ginsberg (ginsberg) 
Cc: Tony Li ; Henk Smit ; lsr 

Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

Hi Les,

Ok that helps to clarify the current use case (and name confusion) of RFC7981. 
I did look at some of the drafts defined in the registry of this Capability TLV 
bringing in sub-TLVs and while clearly lots of them are used in a run time I 
did see a few which could be also stated to use mgmt plane instead :).

But back to this topic - do you see that support for Multi-TLV processing could 
not be disabled by a node ? If so, would this information not be useful in run 
time ?

Thx,
R.





On Thu, Sep 22, 2022 at 10:30 PM Les Ginsberg (ginsberg) 
mailto:ginsb...@cisco.com>> wrote:
Robert –

The intent of my response was to get agreement on separating the discussion of 
advertising features supported by an implementation from the content of the 
multi-tlv draft.

Router capabilities TLV (RFC 4971/7981) is something quite different. In every 
case, information advertised in the Router Capability TLV is used at run time 
by the protocol. For example, advertising SR Capability is not there so that 
the operator can know which implementations include SR support. It is there so 
that at run time other routers in the network will know whether a given router 
has SR enabled – which influences how (for example) label stacks are built when 
forwarding via that node.

What is being proposed here is for the protocol to advertise what features it 
supports. This is not information which will be used by the protocol at run 
time – it is there solely to inform the network operator of what support is 
included in a given implementation.
This is not what Router Capability TLV does (please do not argue with me about 
the TLV name – it was chosen long ago…) and if the WG were to decide that 
management information should be sent by the protocol I would certainly argue 
that Router Capability TLV is not the correct TLV to be used.

If you are interested in discussing having the protocol include management 
information in the LSPDB – please discuss this in a separate thread – and note 
that (with the notable exception of “hostname”) this isn’t done today – so it 
is something very new.

   Les



From: Robert Raszuk mailto:rob...@raszuk.net>>
Sent: Thursday, September 22, 2022 12:38 PM
To: Les Ginsberg (ginsberg) mailto:ginsb...@cisco.com>>
Cc: Tony Li mailto:tony...@tony.li>>; Henk Smit 
mailto:henk.i...@xs4all.nl>>; lsr 
mailto:lsr@ietf.org>>
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

Hi Les,

> 2) Applicability of advertising what features an implementation supports 
> extends
> to much more than just multi-tlv support.

Indeed. Spot on !

And wasn't it the reason for rfc4971 then updated by rfc7981 ?

I think having such capabilities flooded via area or entire domain is a very 
useful thing.

It is much better to crash local node then randomly see crashes here and there 
when it comes to handling new feature.

Is the resistance here coming from the fact that multi-part TLVs are used today 
without such capability and introducing it now would cause a mess ? If so maybe 
rewording first sentence from section 5 rather then removing section 4 is 
better solution.

Mgmt plane exists here and there. I am yet to see parity in how routers expose 
information from ISIS and OSPF. So if someone is seriously thinking of self 
driving networks we will see much more management stuff being sent inline other 
then expecting that networks will be driven by magic oracles. That experiment 
of SDN is not going that well I am afraid.

Best,
R.


On Thu, Sep 22, 2022 at 6:40 PM Les Ginsberg (ginsberg) 
mailto:40cisco@dmarc.ietf.org>> wrote:
Tony -

For me, your discussion with Henk highlights two points:

1)There are different POVs on whether advertising management information (like 
multi-tlv support) in the LSPDB is a good idea

2)Applicability of advertising what features an implementation supports extends 
to much more than just multi-tlv support.

Therefore, introduction of such advertisements should be removed from the 
multi-tlv draft. If you and/or others want to pursue this idea, then a new 
draft focused on this new use of the protocol should be written.
In this way the WG will have the opportunity to discuss the merits of such a 
significant protocol extension independently - w

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-22 Thread Robert Raszuk
Hi Les,

Ok that helps to clarify the current use case (and name confusion) of
RFC7981. I did look at some of the drafts defined in the registry of this
Capability TLV bringing in sub-TLVs and while clearly lots of them are used
in a run time I did see a few which could be also stated to use mgmt plane
instead :).

But back to this topic - do you see that support for Multi-TLV processing
could not be disabled by a node ? If so, would this information not be
useful in run time ?

Thx,
R.





On Thu, Sep 22, 2022 at 10:30 PM Les Ginsberg (ginsberg) 
wrote:

> Robert –
>
>
>
> The intent of my response was to get agreement on separating the
> discussion of advertising features supported by an implementation from the
> content of the multi-tlv draft.
>
>
>
> Router capabilities TLV (RFC 4971/7981) is something quite different. In
> every case, information advertised in the Router Capability TLV is used at
> run time by the protocol. For example, advertising SR Capability is not
> there so that the operator can know which implementations include SR
> support. It is there so that at run time other routers in the network will
> know whether a given router has SR enabled – which influences how (for
> example) label stacks are built when forwarding via that node.
>
>
>
> What is being proposed here is for the protocol to advertise what features
> it supports. This is not information which will be used by the protocol at
> run time – it is there solely to inform the network operator of what
> support is included in a given implementation.
>
> This is not what Router Capability TLV does (please do not argue with me
> about the TLV name – it was chosen long ago…) and if the WG were to decide
> that management information should be sent by the protocol I would
> certainly argue that Router Capability TLV is not the correct TLV to be
> used.
>
>
>
> If you are interested in discussing having the protocol include management
> information in the LSPDB – please discuss this in a separate thread – and
> note that (with the notable exception of “hostname”) this isn’t done today
> – so it is something very new.
>
>
>
>Les
>
>
>
>
>
>
>
> *From:* Robert Raszuk 
> *Sent:* Thursday, September 22, 2022 12:38 PM
> *To:* Les Ginsberg (ginsberg) 
> *Cc:* Tony Li ; Henk Smit ; lsr <
> lsr@ietf.org>
> *Subject:* Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
>
> Hi Les,
>
>
>
> > 2) Applicability of advertising what features an implementation supports
> extends
>
> > to much more than just multi-tlv support.
>
>
>
> Indeed. Spot on !
>
>
>
> And wasn't it the reason for rfc4971 then updated by rfc7981 ?
>
>
>
> I think having such capabilities flooded via area or entire domain is a
> very useful thing.
>
>
>
> It is much better to crash local node then randomly see crashes here and
> there when it comes to handling new feature.
>
>
>
> Is the resistance here coming from the fact that multi-part TLVs are used
> today without such capability and introducing it now would cause a mess ?
> If so maybe rewording first sentence from section 5 rather then removing
> section 4 is better solution.
>
>
>
> Mgmt plane exists here and there. I am yet to see parity in how routers
> expose information from ISIS and OSPF. So if someone is seriously thinking
> of self driving networks we will see much more management stuff being sent
> inline other then expecting that networks will be driven by magic oracles.
> That experiment of SDN is not going that well I am afraid.
>
>
>
> Best,
>
> R.
>
>
>
>
>
> On Thu, Sep 22, 2022 at 6:40 PM Les Ginsberg (ginsberg)  40cisco@dmarc.ietf.org> wrote:
>
> Tony -
>
> For me, your discussion with Henk highlights two points:
>
> 1)There are different POVs on whether advertising management information
> (like multi-tlv support) in the LSPDB is a good idea
>
> 2)Applicability of advertising what features an implementation supports
> extends to much more than just multi-tlv support.
>
> Therefore, introduction of such advertisements should be removed from the
> multi-tlv draft. If you and/or others want to pursue this idea, then a new
> draft focused on this new use of the protocol should be written.
> In this way the WG will have the opportunity to discuss the merits of such
> a significant protocol extension independently - which is appropriate.
> And the work on how to support multi-tlv - which I think is both useful
> and non-controversial - can proceed.
>
> I hope this is something on which we can easily agree - even if we do not
> agree on whether featu

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-22 Thread Les Ginsberg (ginsberg)
Robert –

The intent of my response was to get agreement on separating the discussion of 
advertising features supported by an implementation from the content of the 
multi-tlv draft.

Router capabilities TLV (RFC 4971/7981) is something quite different. In every 
case, information advertised in the Router Capability TLV is used at run time 
by the protocol. For example, advertising SR Capability is not there so that 
the operator can know which implementations include SR support. It is there so 
that at run time other routers in the network will know whether a given router 
has SR enabled – which influences how (for example) label stacks are built when 
forwarding via that node.

What is being proposed here is for the protocol to advertise what features it 
supports. This is not information which will be used by the protocol at run 
time – it is there solely to inform the network operator of what support is 
included in a given implementation.
This is not what Router Capability TLV does (please do not argue with me about 
the TLV name – it was chosen long ago…) and if the WG were to decide that 
management information should be sent by the protocol I would certainly argue 
that Router Capability TLV is not the correct TLV to be used.

If you are interested in discussing having the protocol include management 
information in the LSPDB – please discuss this in a separate thread – and note 
that (with the notable exception of “hostname”) this isn’t done today – so it 
is something very new.

   Les



From: Robert Raszuk 
Sent: Thursday, September 22, 2022 12:38 PM
To: Les Ginsberg (ginsberg) 
Cc: Tony Li ; Henk Smit ; lsr 

Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

Hi Les,

> 2) Applicability of advertising what features an implementation supports 
> extends
> to much more than just multi-tlv support.

Indeed. Spot on !

And wasn't it the reason for rfc4971 then updated by rfc7981 ?

I think having such capabilities flooded via area or entire domain is a very 
useful thing.

It is much better to crash local node then randomly see crashes here and there 
when it comes to handling new feature.

Is the resistance here coming from the fact that multi-part TLVs are used today 
without such capability and introducing it now would cause a mess ? If so maybe 
rewording first sentence from section 5 rather then removing section 4 is 
better solution.

Mgmt plane exists here and there. I am yet to see parity in how routers expose 
information from ISIS and OSPF. So if someone is seriously thinking of self 
driving networks we will see much more management stuff being sent inline other 
then expecting that networks will be driven by magic oracles. That experiment 
of SDN is not going that well I am afraid.

Best,
R.


On Thu, Sep 22, 2022 at 6:40 PM Les Ginsberg (ginsberg) 
mailto:40cisco@dmarc.ietf.org>> wrote:
Tony -

For me, your discussion with Henk highlights two points:

1)There are different POVs on whether advertising management information (like 
multi-tlv support) in the LSPDB is a good idea

2)Applicability of advertising what features an implementation supports extends 
to much more than just multi-tlv support.

Therefore, introduction of such advertisements should be removed from the 
multi-tlv draft. If you and/or others want to pursue this idea, then a new 
draft focused on this new use of the protocol should be written.
In this way the WG will have the opportunity to discuss the merits of such a 
significant protocol extension independently - which is appropriate.
And the work on how to support multi-tlv - which I think is both useful and 
non-controversial - can proceed.

I hope this is something on which we can easily agree - even if we do not agree 
on whether feature support should/should not be advertised in the LSPDB.

A few more comments inline.

> -Original Message-
> From: Lsr mailto:lsr-boun...@ietf.org>> On Behalf Of 
> Tony Li
> Sent: Tuesday, September 13, 2022 1:44 PM
> To: Henk Smit mailto:henk.i...@xs4all.nl>>
> Cc: lsr mailto:lsr@ietf.org>>
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
>
>
> Hi Henk,
>
> >> Yes, I'm advocate for putting things elsewhere, but that proposal has
> >> met with crickets.  You don't get it both ways: no capabilities in the
> >> protocol and nowhere else does not work.
> >
> > I'm not sure I know what you are talking about.
> > Did you write a draft?
>
>
> I did.  You don’t remember it. It was that memorable.

[LES:] Sorry, I am not aware that you (or anyone else) has written a draft 
proposing advertisement of feature support in the LSPDB.
Could you provide a reference?

>
>
> >> Because the thought of trying to deploy this capability at scale without
> >> this attribute seems impossible. Consi

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-22 Thread Robert Raszuk
Hi Les,

> 2) Applicability of advertising what features an implementation supports
extends
> to much more than just multi-tlv support.

Indeed. Spot on !

And wasn't it the reason for rfc4971 then updated by rfc7981 ?

I think having such capabilities flooded via area or entire domain is a
very useful thing.

It is much better to crash local node then randomly see crashes here and
there when it comes to handling new feature.

Is the resistance here coming from the fact that multi-part TLVs are used
today without such capability and introducing it now would cause a mess ?
If so maybe rewording first sentence from section 5 rather then removing
section 4 is better solution.

Mgmt plane exists here and there. I am yet to see parity in how routers
expose information from ISIS and OSPF. So if someone is seriously thinking
of self driving networks we will see much more management stuff being sent
inline other then expecting that networks will be driven by magic oracles.
That experiment of SDN is not going that well I am afraid.

Best,
R.


On Thu, Sep 22, 2022 at 6:40 PM Les Ginsberg (ginsberg)  wrote:

> Tony -
>
> For me, your discussion with Henk highlights two points:
>
> 1)There are different POVs on whether advertising management information
> (like multi-tlv support) in the LSPDB is a good idea
>
> 2)Applicability of advertising what features an implementation supports
> extends to much more than just multi-tlv support.
>
> Therefore, introduction of such advertisements should be removed from the
> multi-tlv draft. If you and/or others want to pursue this idea, then a new
> draft focused on this new use of the protocol should be written.
> In this way the WG will have the opportunity to discuss the merits of such
> a significant protocol extension independently - which is appropriate.
> And the work on how to support multi-tlv - which I think is both useful
> and non-controversial - can proceed.
>
> I hope this is something on which we can easily agree - even if we do not
> agree on whether feature support should/should not be advertised in the
> LSPDB.
>
> A few more comments inline.
>
> > -Original Message-
> > From: Lsr  On Behalf Of Tony Li
> > Sent: Tuesday, September 13, 2022 1:44 PM
> > To: Henk Smit 
> > Cc: lsr 
> > Subject: Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-
> > 01.txt
> >
> >
> > Hi Henk,
> >
> > >> Yes, I'm advocate for putting things elsewhere, but that proposal has
> > >> met with crickets.  You don't get it both ways: no capabilities in the
> > >> protocol and nowhere else does not work.
> > >
> > > I'm not sure I know what you are talking about.
> > > Did you write a draft?
> >
> >
> > I did.  You don’t remember it. It was that memorable.
>
> [LES:] Sorry, I am not aware that you (or anyone else) has written a draft
> proposing advertisement of feature support in the LSPDB.
> Could you provide a reference?
>
> >
> >
> > >> Because the thought of trying to deploy this capability at scale
> without
> > >> this attribute seems impossible. Consider the case of Tier 1 providers
> > >> who have large IS-IS deployments. Are you really going to evaluate
> 2000+
> > >> nodes without some kind of help?
> > >
> > > With the help of the management-plane?
> >
> >
> > There is no management plane.  We had the chance at one, but we had the
> > great schism between OpenConfig and the IETF. So now we have nothing
> > that we can rely on.
>
> [LES:] I sympathize with your POV here. From an industry standpoint the
> schism is most unfortunate.
> But making the protocol itself responsible for sending management info is
> not a solution.
>
>Les
>
> >
> >
> > > How did those providers make changes to their
> > configs/features/architecture before?
> > > I would expect them to use the same tools.
> >
> >
> > They have configuration databases, but they do NOT have good tools that
> tell
> > them about router capabilities. They MAY be able to do something ad hoc
> > based on software release numbers, but this is far from a good solution.
> >
> >
> > >> And the routers will do computations based on the multi-part TLVs.
> > >> One level of indirection for a capability does not seem extreme.
> > >
> > > Not extreme, indeed.
> > > But again, I rather not see 20 different minor or irrelevant things
> > > in the router-capability TLV. Certainly not at 2 octets per item.
> > > 1 Bit would already be (16 times) better.
> >
> >
> > 

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-22 Thread Les Ginsberg (ginsberg)
Tony -

For me, your discussion with Henk highlights two points:

1)There are different POVs on whether advertising management information (like 
multi-tlv support) in the LSPDB is a good idea

2)Applicability of advertising what features an implementation supports extends 
to much more than just multi-tlv support.

Therefore, introduction of such advertisements should be removed from the 
multi-tlv draft. If you and/or others want to pursue this idea, then a new 
draft focused on this new use of the protocol should be written.
In this way the WG will have the opportunity to discuss the merits of such a 
significant protocol extension independently - which is appropriate.
And the work on how to support multi-tlv - which I think is both useful and 
non-controversial - can proceed.

I hope this is something on which we can easily agree - even if we do not agree 
on whether feature support should/should not be advertised in the LSPDB.

A few more comments inline.

> -Original Message-
> From: Lsr  On Behalf Of Tony Li
> Sent: Tuesday, September 13, 2022 1:44 PM
> To: Henk Smit 
> Cc: lsr 
> Subject: Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-
> 01.txt
> 
> 
> Hi Henk,
> 
> >> Yes, I'm advocate for putting things elsewhere, but that proposal has
> >> met with crickets.  You don't get it both ways: no capabilities in the
> >> protocol and nowhere else does not work.
> >
> > I'm not sure I know what you are talking about.
> > Did you write a draft?
> 
> 
> I did.  You don’t remember it. It was that memorable.

[LES:] Sorry, I am not aware that you (or anyone else) has written a draft 
proposing advertisement of feature support in the LSPDB.
Could you provide a reference?

> 
> 
> >> Because the thought of trying to deploy this capability at scale without
> >> this attribute seems impossible. Consider the case of Tier 1 providers
> >> who have large IS-IS deployments. Are you really going to evaluate 2000+
> >> nodes without some kind of help?
> >
> > With the help of the management-plane?
> 
> 
> There is no management plane.  We had the chance at one, but we had the
> great schism between OpenConfig and the IETF. So now we have nothing
> that we can rely on.

[LES:] I sympathize with your POV here. From an industry standpoint the schism 
is most unfortunate.
But making the protocol itself responsible for sending management info is not a 
solution.

   Les

> 
> 
> > How did those providers make changes to their
> configs/features/architecture before?
> > I would expect them to use the same tools.
> 
> 
> They have configuration databases, but they do NOT have good tools that tell
> them about router capabilities. They MAY be able to do something ad hoc
> based on software release numbers, but this is far from a good solution.
> 
> 
> >> And the routers will do computations based on the multi-part TLVs.
> >> One level of indirection for a capability does not seem extreme.
> >
> > Not extreme, indeed.
> > But again, I rather not see 20 different minor or irrelevant things
> > in the router-capability TLV. Certainly not at 2 octets per item.
> > 1 Bit would already be (16 times) better.
> 
> 
> I am happy to go with one bit.  However, there is no place to encode that
> single bit today.
> 
> 
> >>> Regardless whether we do that or not, this discussion maybe should be
> done
> >>> outside the multipart TLV  discussion. Maybe another draft should be
> written
> >>> about these software-capabilities in general?
> >>
> >> Please feel free.  My proposal was shot down.
> >
> > Are you talking about a very recent proposal? Linked to the multipart-TLV
> > draft? Or something older? I vaguely remember some idea about
> > "generic transport" in IS-IS (or rather: outside the regular IS-IS 
> > instance).
> 
> 
> This was outside of IS-IS entirely.  Several people disliked it so much that 
> they
> wanted it thrown out of the WG.
> 
> T
> 
> ___
> Lsr mailing list
> Lsr@ietf.org
> https://www.ietf.org/mailman/listinfo/lsr
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-13 Thread Tony Li

Hi Henk,

>> Yes, I'm advocate for putting things elsewhere, but that proposal has
>> met with crickets.  You don't get it both ways: no capabilities in the
>> protocol and nowhere else does not work.
> 
> I'm not sure I know what you are talking about.
> Did you write a draft?


I did.  You don’t remember it. It was that memorable.


>> Because the thought of trying to deploy this capability at scale without
>> this attribute seems impossible. Consider the case of Tier 1 providers
>> who have large IS-IS deployments. Are you really going to evaluate 2000+
>> nodes without some kind of help?
> 
> With the help of the management-plane?


There is no management plane.  We had the chance at one, but we had the great 
schism between OpenConfig and the IETF. So now we have nothing that we can rely 
on.


> How did those providers make changes to their configs/features/architecture 
> before?
> I would expect them to use the same tools.


They have configuration databases, but they do NOT have good tools that tell 
them about router capabilities. They MAY be able to do something ad hoc based 
on software release numbers, but this is far from a good solution.


>> And the routers will do computations based on the multi-part TLVs.
>> One level of indirection for a capability does not seem extreme.
> 
> Not extreme, indeed.
> But again, I rather not see 20 different minor or irrelevant things
> in the router-capability TLV. Certainly not at 2 octets per item.
> 1 Bit would already be (16 times) better.


I am happy to go with one bit.  However, there is no place to encode that 
single bit today.


>>> Regardless whether we do that or not, this discussion maybe should be done
>>> outside the multipart TLV  discussion. Maybe another draft should be written
>>> about these software-capabilities in general?
>> 
>> Please feel free.  My proposal was shot down.
> 
> Are you talking about a very recent proposal? Linked to the multipart-TLV
> draft? Or something older? I vaguely remember some idea about
> "generic transport" in IS-IS (or rather: outside the regular IS-IS instance).


This was outside of IS-IS entirely.  Several people disliked it so much that 
they wanted it thrown out of the WG.

T

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-13 Thread Henk Smit
Hi Tony,


> Yes, I'm advocate for putting things elsewhere, but that proposal has
> met with crickets.  You don't get it both ways: no capabilities in the
> protocol and nowhere else does not work.

I'm not sure I know what you are talking about.
Did you write a draft?

> Because the thought of trying to deploy this capability at scale without
> this attribute seems impossible. Consider the case of Tier 1 providers
> who have large IS-IS deployments. Are you really going to evaluate 2000+
> nodes without some kind of help?

With the help of the management-plane?
How did those providers make changes to their configs/features/architecture 
before?
I would expect them to use the same tools.

> And the routers will do computations based on the multi-part TLVs.
> One level of indirection for a capability does not seem extreme.

Not extreme, indeed.
But again, I rather not see 20 different minor or irrelevant things
in the router-capability TLV. Certainly not at 2 octets per item.
1 Bit would already be (16 times) better.

> > Regardless whether we do that or not, this discussion maybe should be done
> > outside the multipart TLV  discussion. Maybe another draft should be written
> > about these software-capabilities in general?
> 
> Please feel free.  My proposal was shot down.

Are you talking about a very recent proposal? Linked to the multipart-TLV
draft? Or something older? I vaguely remember some idea about
"generic transport" in IS-IS (or rather: outside the regular IS-IS instance).

henk.

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-12 Thread Tony Li

Hi Henk,

> My point was: multipart TLVs exist today, before the introduction of the
> capability advertisement. So when you look at a LSPDB, you still don't know 
> for
> sure which routers support multipart TLVs. Some might, but don't advertise it.
> Because their software was written before the new capability existed.


True, but irrelevant. The point is that if a router does advertise the 
capability, then you can reasonably infer that it is ready to receive 
multi-part TLVs.


> I hope not. And I will oppose those attempts too.


Example: MPLS MNA is coming. It’s a bundle of independent network actions, plus 
user defined actions. The MPLS WG is going to need a capability for each 
action. I expect that they will come here (LSR) and ask.


>> we still do not have an effective management plane and
>> continue to stuff things into the LSDB that belong in the management plane.
> 
> Yes. But that does not make it an excuse to put just anything in the LSPDB.


That’s what we’ve been doing for decades.

Yes, I’m advocate for putting things elsewhere, but that proposal has met with 
crickets.  You don’t get it both ways: no capabilities in the protocol and 
nowhere else does not work.

If the IGP is not a dump truck, then what else is?


> I've seen you in this work-group as someone who tried to keep things out of
> IS-IS that don't belong there. I am surprised to see you want this capability 
> in.


Because the thought of trying to deploy this capability at scale without this 
attribute seems impossible. Consider the case of Tier 1 providers who have 
large IS-IS deployments. Are you really going to evaluate 2000+ nodes without 
some kind of help?


>> The entire definition of a Flex Algo topology constraint should be
>> in the management plane.
> 
> Sure. But at least the routers do make route-calculation decisions based on
> that information.


And the routers will do computations based on the multi-part TLVs.  One level 
of indirection for a capability does not seem extreme.


>> That's not an excuse for not trying to do a good job now.
> 
> That is the whole question. This capability is adding 2 more octets to LSPs..
> Is that worth it?


Yes.


> What if indeed a few dozen drafts will follow to advertise
> more of these capabilities?


They are coming regardless.


> Should we define a new top-level TLV for "feature/software support 
> capability"?


No, since duplicating an existing TLV is wasteful of TLV code points. The 
current capability TLV is both necessary and sufficient.


> Not whether something is configured or not (as does the router-capability 
> TLV),
> but whether a router's software has that capability to begin with.
> Or should we define a new variable-length bitmask sub-TLV for the existing 
> router
> capability TLV. Where every bit indicates another piece of software the router
> supports?


That’s coming. See above.


> Regardless whether we do that or not, this discussion maybe should be done
> outside the multipart TLV  discussion. Maybe another draft should be written
> about these software-capabilities in general?


Please feel free.  My proposal was shot down.

T

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-12 Thread Henk Smit
Hi Tony,

> Some exist today. There are many TLVs where they have never been specified.

My point was: multipart TLVs exist today, before the introduction of the
capability advertisement. So when you look at a LSPDB, you still don't know for
sure which routers support multipart TLVs. Some might, but don't advertise it.
Because their software was written before the new capability existed.

>> In the end, every detail, will get its own router capability.
>  That's correct. It will. That's going to happen independently of this draft.

I hope not. And I will oppose those attempts too.

> we still do not have an effective management plane and
> continue to stuff things into the LSDB that belong in the management plane.

Yes. But that does not make it an excuse to put just anything in the LSPDB.

I've seen you in this work-group as someone who tried to keep things out of
IS-IS that don't belong there. I am surprised to see you want this capability 
in.

> The entire definition of a Flex Algo topology constraint should be
> in the management plane.

Sure. But at least the routers do make route-calculation decisions based on
that information.

> That's not an excuse for not trying to do a good job now.

That is the whole question. This capability is adding 2 more octets to LSPs.
Is that worth it? What if indeed a few dozen drafts will follow to advertise
more of these capabilities?

Should we define a new top-level TLV for "feature/software support capability"?
Not whether something is configured or not (as does the router-capability TLV),
but whether a router's software has that capability to begin with.
Or should we define a new variable-length bitmask sub-TLV for the existing 
router
capability TLV. Where every bit indicates another piece of software the router
supports?

Regardless whether we do that or not, this discussion maybe should be done
outside the multipart TLV  discussion. Maybe another draft should be written
about these software-capabilities in general?

henk.



> Op 11-09-2022 21:32 schreef Tony Li :
> 
>  
> Hi Henk,
> 
> > > If we want to introduce MP-TLVs, that change would warrant the existence 
> > > of the flag.
> > 
> > Multipart TLVs already exist today. 
> 
> 
> Some exist today.  There are many TLVs where they have never been specified.
> 
> 
> > As discussed here, after introducing a "software capability TLV", if a 
> > router doesn't
> > advertise any of those new capabilities, we still don't know whether that 
> > router supports
> > multipart TLVs or not.
> > So the new capability seems to have limited value.
> 
> 
> In particular, the current proposal on the table is to have the capability 
> apply to the TLVs where multi-part has not been previously defined.
> 
> 
> > > I dispute that a binary flag warrants the word 'complexity'.
> > 
> > You might think of a single bit now. 
> > But people might want to add more. What TLVs on a router can and can not be 
> > received
> > multipart? What about sub-TLVs?
> 
> 
> We are not proposing that level of specificity. It’s all or nothing.
> 
> 
> > It seems these days we have more people in the LSR work-group that prefer 
> > to write drafts
> > than that prefer to write code.
> 
> 
> Ok, I’m offended.
> 
> 
> > I fear a large amount of drafts about router capabilities to
> > advertise support for every bit, every TLV and every sub-TLV in LSPs. In 
> > the end, every
> > feature, every detail or option in a feature, will get its own router 
> > capability.
> 
> 
> That’s correct. It will. That’s going to happen independently of this draft.
> 
> 
> > I'm happy nobody wants routers to react to advertised software 
> > capabilities. 
> > But if routers don't react to info in a LSP, I don't think that info 
> > belongs in the control-plane.
> > It belongs in the management-plane.
> 
> 
> Thank you, but we still (40 years in?) do not have an effective management 
> plane and continue to stuff things into the LSDB that belong in the 
> management plane.
> 
> 
> > That's a different thing, imho. It's a single exception. It's very useful 
> > to identify LSPs and routers.
> > I don't see other management information we should put in LSPs.
> 
> 
> There’s tons of stuff.  The entire definition of a Flex Algo topology 
> constraint should be in the management plane. Almost everything that is in 
> the router capability TLV should be in the management plane.
> 
> We stepped down this slippery slope a long, long, time ago.
> 
> 
> > Twenty-five years ago, you and me were the first people to implement 
> > support for wide metrics.
> > I came up with a strategy to migrate a running network.
> > I introduced the "metric-style [wide|narrow]" command. That was supposed to 
> > be a temporary thing.
> > Just for use in the next 1-2 years, when every IS-IS network was migrating 
> > to wide metrics.
> 
> 
> And it’s still there.
> 
> 
> > When I came back to work, in 2015, I saw that the Nokia SR-OS routers still 
> > have narrow metrics as
> > the default 

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-11 Thread Tony Li

Hi Henk,

> > If we want to introduce MP-TLVs, that change would warrant the existence of 
> > the flag.
> 
> Multipart TLVs already exist today. 


Some exist today.  There are many TLVs where they have never been specified.


> As discussed here, after introducing a "software capability TLV", if a router 
> doesn't
> advertise any of those new capabilities, we still don't know whether that 
> router supports
> multipart TLVs or not.
> So the new capability seems to have limited value.


In particular, the current proposal on the table is to have the capability 
apply to the TLVs where multi-part has not been previously defined.


> > I dispute that a binary flag warrants the word 'complexity'.
> 
> You might think of a single bit now. 
> But people might want to add more. What TLVs on a router can and can not be 
> received
> multipart? What about sub-TLVs?


We are not proposing that level of specificity. It’s all or nothing.


> It seems these days we have more people in the LSR work-group that prefer to 
> write drafts
> than that prefer to write code.


Ok, I’m offended.


> I fear a large amount of drafts about router capabilities to
> advertise support for every bit, every TLV and every sub-TLV in LSPs. In the 
> end, every
> feature, every detail or option in a feature, will get its own router 
> capability.


That’s correct. It will. That’s going to happen independently of this draft.


> I'm happy nobody wants routers to react to advertised software capabilities. 
> But if routers don't react to info in a LSP, I don't think that info belongs 
> in the control-plane.
> It belongs in the management-plane.


Thank you, but we still (40 years in?) do not have an effective management 
plane and continue to stuff things into the LSDB that belong in the management 
plane.


> That's a different thing, imho. It's a single exception. It's very useful to 
> identify LSPs and routers.
> I don't see other management information we should put in LSPs.


There’s tons of stuff.  The entire definition of a Flex Algo topology 
constraint should be in the management plane. Almost everything that is in the 
router capability TLV should be in the management plane.

We stepped down this slippery slope a long, long, time ago.


> Twenty-five years ago, you and me were the first people to implement support 
> for wide metrics.
> I came up with a strategy to migrate a running network.
> I introduced the "metric-style [wide|narrow]" command. That was supposed to 
> be a temporary thing.
> Just for use in the next 1-2 years, when every IS-IS network was migrating to 
> wide metrics.


And it’s still there.


> When I came back to work, in 2015, I saw that the Nokia SR-OS routers still 
> have narrow metrics as
> the default setting. I laughed. Now I am back at cisco, and I see that IOS-XR 
> also has narrow metrics
> as the default. I cried. FYI, both OSes had their FCS many years after 1997. 
> If I am not mistaken, JunOS has "metric-style both" as the default. 
> So on all these 3 OSes, you need to explicitly configure "metric-style wide". 
> Twenty five years after the migration …..


Excellent point: things don’t progress. So if you want to move forward, you 
need to be very careful.  That’s all that we’re suggesting.


> I fear that the same will happen with your router-capability. The new 
> capability will have some
> value now. To help migrate to a network where all boxes support multipart 
> TLVs.
> But 1-2 years from now, all (major) IS-IS implementations will support 
> multipart TLVs for all TLVs.
> And then the new router-capability will have no use anymore. But routers all 
> around the world will
> still advertise it. I predict that 25 years from now, 23 years after all 
> IS-IS implementations started
> to support multi-part TLVs, routers will still advertise your capability.
> 
> I don't like that.


I know that it will take more than 2 years.  Deployment alone is another 2 
years. And I’m sorry that you don’t like it.  There are lots of things out 
there that could be better. That’s not an excuse for not trying to do a good 
job now.

Tony


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-09-11 Thread Henk Smit



Hi Tony, 


 > If we want to introduce MP-TLVs, that change would warrant the existence of 
 > the flag. 


 Multipart TLVs already exist today. 
As discussed here, after introducing a "software capability TLV", if a router 
doesn't 
 advertise any of those new capabilities, we still don't know whether that 
router supports 
 multipart TLVs or not. 
 So the new capability seems to have limited value. 


 > I dispute that a binary flag warrants the word 'complexity'. 


 You might think of a single bit now. 
But people might want to add more. What TLVs on a router can and can not be 
received 
 multipart? What about sub-TLVs? 


 > We are not allowing that level of granularity. A system that is 
> going to support MP-TLVs should take care to operate correctly 
> for ALL TLVs before advertising that it supports them. 


 It seems these days we have more people in the LSR work-group that prefer to 
write drafts 
 than that prefer to write code. I fear a large amount of drafts about router 
capabilities to 
 advertise support for every bit, every TLV and every sub-TLV in LSPs. In the 
end, every 
 feature, every detail or option in a feature, will get its own router 
capability. 





 I'm happy nobody wants routers to react to advertised software capabilities. 
But if routers don't react to info in a LSP, I don't think that info belongs in 
the control-plane. 
 It belongs in the management-plane. 


 > We have been sending management information in the LSDB since 
> we introduced the hostname TLV. 


 That's a different thing, imho. It's a single exception. It's very useful to 
identify LSPs and routers. 
 I don't see other management information we should put in LSPs. 





 Twenty-five years ago, you and me were the first people to implement support 
for wide metrics. 
 I came up with a strategy to migrate a running network. 
 I introduced the "metric-style [wide|narrow]" command. That was supposed to be 
a temporary thing. 
 Just for use in the next 1-2 years, when every IS-IS network was migrating to 
wide metrics. 


 When I came back to work, in 2015, I saw that the Nokia SR-OS routers still 
have narrow metrics as 
 the default setting. I laughed. Now I am back at cisco, and I see that IOS-XR 
also has narrow metrics 
 as the default. I cried. FYI, both OSes had their FCS many years after 1997. 
If I am not mistaken, JunOS has "metric-style both" as the default. 
So on all these 3 OSes, you need to explicitly configure "metric-style wide". 
Twenty five years after the migration . 


 I fear that the same will happen with your router-capability. The new 
capability will have some 
 value now. To help migrate to a network where all boxes support multipart 
TLVs. 
 But 1-2 years from now, all (major) IS-IS implementations will support 
multipart TLVs for all TLVs. 
 And then the new router-capability will have no use anymore. But routers all 
around the world will 
 still advertise it. I predict that 25 years from now, 23 years after all IS-IS 
implementations started 
 to support multi-part TLVs, routers will still advertise your capability. 


 I don't like that. 


 henk. 

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-25 Thread Acee Lindem (acee)
I agree that retaining the option and using it for debugging would be a good 
thing. However, given that multi-part TLVs are already in use, the absence of 
the advertisement doesn’t necessarily mean that the IS-IS router doesn’t 
support multi-part TLVs. Rather, it presence would mean that beyond a shadow of 
a doubt, they are supported.

Similarly, for backward compatibility,  an IS-IS router would not be required 
to strictly enforce the requirement that all IS-IS routers within the scope of 
the Multi-Part TLV advertise the capability.

Thanks,
Acee

From: Lsr  on behalf of Robert Raszuk 
Date: Thursday, August 25, 2022 at 12:18 PM
To: "Acee Lindem (acee)" 
Cc: Gunter Van de Velde , Tony Li 
, lsr 
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

All,

I am actually finding this capability useful. If for nothing else then to help 
the operator to see what is going on in the area. On any node simple show 
command will clearly show who is willing and capable to receive MP-TLVs and who 
is not.

Analogy to including hostnames Tony brought here as an example is spot on and 
along the same lines.

Of course how node uses that info could be discussed further. I would also not 
object to put that capability into a separate document.

Thx,
R.





On Thu, Aug 25, 2022 at 6:06 PM Acee Lindem (acee) 
mailto:40cisco@dmarc.ietf.org>> wrote:
Speaking as WG member:

Hi Gunder, Tony, Les,

I'm also not a fan of the Multi-Part TLV Capability flag. While the intent of 
the draft is to encourage multi-part TLV advertisement and usage, the addition 
of this flag and the requirement for advertisement will most likely have the 
opposite effect. Given that IS-IS implementations are already advertising 
multi-part TLVs but none are advertising the proposed capability flag, 
implementation of the draft as currently written would inhibit Multi-Part TLV 
usage and be a step backwards. Of course, we know implementations that are 
already advertising these multi-part TLVs will most likely ignore the 
recommendation and continue to advertise them even when not all IS-IS routers 
within the scope of the Multi-Part TLV advertise the capability.

Rather, I propose that the draft eliminates the capability flag and introduces 
a recommended configuration parameter that would allow Multi-Part TLVs to be 
suppressed. The recommended default would be FALSE. This would provide an out 
if these Multi-Part TLVs did, in fact, have dire consequences.

Thanks,
Acee

On 8/25/22, 6:53 AM, "Lsr on behalf of Van De Velde, Gunter (Nokia - 
BE/Antwerp)" mailto:lsr-boun...@ietf.org> on behalf of 
gunter.van_de_ve...@nokia.com<mailto:gunter.van_de_ve...@nokia.com>> wrote:

Inline: GV>

From: Tony Li mailto:tony1ath...@gmail.com>> On 
Behalf Of Tony Li
Sent: Wednesday, August 24, 2022 5:26 PM
To: Van De Velde, Gunter (Nokia - BE/Antwerp) 
mailto:gunter.van_de_ve...@nokia.com>>
    Cc: lsr mailto:lsr@ietf.org>>
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Hi Gunter,

I am having troubles understanding the value of ‘The Multi-part TLV 
Capability’ flag.
What would break if ‘Multi-part TLV Capability’ flag would not exist?


A system that supported MP-TLVs would not be able to determine that there 
were other systems in the area that did not support MP-TLVs.  The system might 
then advertise MP-TLVs and they would be misinterpreted or cause system crashes 
in the systems that did not support them.

GV> crashes? I really hope that is not happening.
GV> When a legacy router receives MP-TLVs from another system and legacy 
router has no support for handling MP-TLV, then yes, things get misinterpreted. 
There is nothing wrong with that, is there? Do you have an example where things 
go wrong?

If we want to introduce MP-TLVs, that change would warrant the existence of 
the flag.

GV> I am not convinced yet how a MP-TLV catch-all flag would make ISIS 
behave better

I dispute that a binary flag warrants the word ‘complexity’.

GV>  living without binary flag is simpler and less complex then dealing 
with a binary flag. (i.e. what, when, how, why, who sets this flag?)

Note: thoughts about the flag: What if a system by accident sends 
flip-flopping (set/unset/set/unset/etc) of this flag?

Then other systems might misinterpret the results and generate inconsistent 
TLVs.  That would be bad.

GV> correct, no good at all.

What if an advertising system support multi-tlv for TLV ‘A’ but not for TLV 
‘B’?

We are not allowing that level of granularity.  A system that is going to 
support MP-TLVs should take care to operate correctly for ALL TLVs before 
advertising that it supports them.

GV> I suspect that 'ALL TLVs' is a reference to  ALL TLVs supported by the 
local system. This means that e.g. when new TLVs would be sup

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-25 Thread Les Ginsberg (ginsberg)
Having the ability to “see” what a given box supports is certainly useful – but 
the question is whether sending such information in LSPs is the right way to do 
it.
The information cannot be used be used by the routers themselves.

Better ways to make this available to operators include:

On box show commands
Retrieval via management apps/models
Vendor documentation

Sending this in LSPs violates something many of us on this thread have argued 
against for years: “Don’t send information which isn’t used by the protocol.”

   Les


From: Lsr  On Behalf Of Robert Raszuk
Sent: Thursday, August 25, 2022 9:17 AM
To: Acee Lindem (acee) 
Cc: Van De Velde, Gunter (Nokia - BE/Antwerp) ; 
Tony Li ; lsr 
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

All,

I am actually finding this capability useful. If for nothing else then to help 
the operator to see what is going on in the area. On any node simple show 
command will clearly show who is willing and capable to receive MP-TLVs and who 
is not.

Analogy to including hostnames Tony brought here as an example is spot on and 
along the same lines.

Of course how node uses that info could be discussed further. I would also not 
object to put that capability into a separate document.

Thx,
R.





On Thu, Aug 25, 2022 at 6:06 PM Acee Lindem (acee) 
mailto:40cisco@dmarc.ietf.org>> wrote:
Speaking as WG member:

Hi Gunder, Tony, Les,

I'm also not a fan of the Multi-Part TLV Capability flag. While the intent of 
the draft is to encourage multi-part TLV advertisement and usage, the addition 
of this flag and the requirement for advertisement will most likely have the 
opposite effect. Given that IS-IS implementations are already advertising 
multi-part TLVs but none are advertising the proposed capability flag, 
implementation of the draft as currently written would inhibit Multi-Part TLV 
usage and be a step backwards. Of course, we know implementations that are 
already advertising these multi-part TLVs will most likely ignore the 
recommendation and continue to advertise them even when not all IS-IS routers 
within the scope of the Multi-Part TLV advertise the capability.

Rather, I propose that the draft eliminates the capability flag and introduces 
a recommended configuration parameter that would allow Multi-Part TLVs to be 
suppressed. The recommended default would be FALSE. This would provide an out 
if these Multi-Part TLVs did, in fact, have dire consequences.

Thanks,
Acee

On 8/25/22, 6:53 AM, "Lsr on behalf of Van De Velde, Gunter (Nokia - 
BE/Antwerp)" mailto:lsr-boun...@ietf.org> on behalf of 
gunter.van_de_ve...@nokia.com<mailto:gunter.van_de_ve...@nokia.com>> wrote:

Inline: GV>

From: Tony Li mailto:tony1ath...@gmail.com>> On 
Behalf Of Tony Li
Sent: Wednesday, August 24, 2022 5:26 PM
To: Van De Velde, Gunter (Nokia - BE/Antwerp) 
mailto:gunter.van_de_ve...@nokia.com>>
Cc: lsr mailto:lsr@ietf.org>>
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Hi Gunter,

I am having troubles understanding the value of ‘The Multi-part TLV 
Capability’ flag.
What would break if ‘Multi-part TLV Capability’ flag would not exist?


A system that supported MP-TLVs would not be able to determine that there 
were other systems in the area that did not support MP-TLVs.  The system might 
then advertise MP-TLVs and they would be misinterpreted or cause system crashes 
in the systems that did not support them.

GV> crashes? I really hope that is not happening.
GV> When a legacy router receives MP-TLVs from another system and legacy 
router has no support for handling MP-TLV, then yes, things get misinterpreted. 
There is nothing wrong with that, is there? Do you have an example where things 
go wrong?

If we want to introduce MP-TLVs, that change would warrant the existence of 
the flag.

GV> I am not convinced yet how a MP-TLV catch-all flag would make ISIS 
behave better

I dispute that a binary flag warrants the word ‘complexity’.

GV>  living without binary flag is simpler and less complex then dealing 
with a binary flag. (i.e. what, when, how, why, who sets this flag?)

Note: thoughts about the flag: What if a system by accident sends 
flip-flopping (set/unset/set/unset/etc) of this flag?

Then other systems might misinterpret the results and generate inconsistent 
TLVs.  That would be bad.

GV> correct, no good at all.

What if an advertising system support multi-tlv for TLV ‘A’ but not for TLV 
‘B’?

We are not allowing that level of granularity.  A system that is going to 
support MP-TLVs should take care to operate correctly for ALL TLVs before 
advertising that it supports them.

GV> I suspect that 'ALL TLVs' is a reference to  ALL TLVs supported by the 
local system. This means that e.g. when new TLVs would be supported after a 
sys

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-25 Thread Robert Raszuk
All,

I am actually finding this capability useful. If for nothing else then to
help the operator to see what is going on in the area. On any node simple
show command will clearly show who is willing and capable to receive
MP-TLVs and who is not.

Analogy to including hostnames Tony brought here as an example is spot on
and along the same lines.

Of course how node uses that info could be discussed further. I would also
not object to put that capability into a separate document.

Thx,
R.





On Thu, Aug 25, 2022 at 6:06 PM Acee Lindem (acee)  wrote:

> Speaking as WG member:
>
> Hi Gunder, Tony, Les,
>
> I'm also not a fan of the Multi-Part TLV Capability flag. While the intent
> of the draft is to encourage multi-part TLV advertisement and usage, the
> addition of this flag and the requirement for advertisement will most
> likely have the opposite effect. Given that IS-IS implementations are
> already advertising multi-part TLVs but none are advertising the proposed
> capability flag, implementation of the draft as currently written would
> inhibit Multi-Part TLV usage and be a step backwards. Of course, we know
> implementations that are already advertising these multi-part TLVs will
> most likely ignore the recommendation and continue to advertise them even
> when not all IS-IS routers within the scope of the Multi-Part TLV advertise
> the capability.
>
> Rather, I propose that the draft eliminates the capability flag and
> introduces a recommended configuration parameter that would allow
> Multi-Part TLVs to be suppressed. The recommended default would be FALSE.
> This would provide an out if these Multi-Part TLVs did, in fact, have dire
> consequences.
>
> Thanks,
> Acee
>
> On 8/25/22, 6:53 AM, "Lsr on behalf of Van De Velde, Gunter (Nokia -
> BE/Antwerp)"  gunter.van_de_ve...@nokia.com> wrote:
>
> Inline: GV>
>
> From: Tony Li  On Behalf Of Tony Li
> Sent: Wednesday, August 24, 2022 5:26 PM
>     To: Van De Velde, Gunter (Nokia - BE/Antwerp) <
> gunter.van_de_ve...@nokia.com>
> Cc: lsr 
> Subject: Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
> Hi Gunter,
>
> I am having troubles understanding the value of ‘The Multi-part TLV
> Capability’ flag.
> What would break if ‘Multi-part TLV Capability’ flag would not exist?
>
>
> A system that supported MP-TLVs would not be able to determine that
> there were other systems in the area that did not support MP-TLVs.  The
> system might then advertise MP-TLVs and they would be misinterpreted or
> cause system crashes in the systems that did not support them.
>
> GV> crashes? I really hope that is not happening.
> GV> When a legacy router receives MP-TLVs from another system and
> legacy router has no support for handling MP-TLV, then yes, things get
> misinterpreted. There is nothing wrong with that, is there? Do you have an
> example where things go wrong?
>
> If we want to introduce MP-TLVs, that change would warrant the
> existence of the flag.
>
> GV> I am not convinced yet how a MP-TLV catch-all flag would make ISIS
> behave better
>
> I dispute that a binary flag warrants the word ‘complexity’.
>
> GV>  living without binary flag is simpler and less complex then
> dealing with a binary flag. (i.e. what, when, how, why, who sets this flag?)
>
> Note: thoughts about the flag: What if a system by accident sends
> flip-flopping (set/unset/set/unset/etc) of this flag?
>
> Then other systems might misinterpret the results and generate
> inconsistent TLVs.  That would be bad.
>
> GV> correct, no good at all.
>
> What if an advertising system support multi-tlv for TLV ‘A’ but not
> for TLV ‘B’?
>
> We are not allowing that level of granularity.  A system that is going
> to support MP-TLVs should take care to operate correctly for ALL TLVs
> before advertising that it supports them.
>
> GV> I suspect that 'ALL TLVs' is a reference to  ALL TLVs supported by
> the local system. This means that e.g. when new TLVs would be supported
> after a system upgrade, that the operator has to be aware and correct the
> flag during each single upgrade.
>
> GV> Unfortunately I remain to have troubles understanding the value
> "Multi-part TLV Capability’ flag brings to an ISIS network.
>   * Without flag it is indeed uncertain if area wide mp-tlv is
> supported (sub-optimal).
>   * but with catch all MP-TLV flag I am not sure we improve ISIS
> operation:
>   ** Who guarantees that the flag is set correctly on all systems at
> all times
>   ** Maybe all systems falls back to advertise single TLV beca

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-25 Thread Tony Przygienda
Given the realities of deploying something like this I am all for
advertisement of what I'll call here the "multi-TLV-compliance" flag
(assuming we agree that capability implies a change in procedures on
reception from other nodes which this draft does not).  Being able to see
that a customer network deployed something that is _not_ compliant with the
RFC (even potentially) can save a lot of head scratching and ghost chasing
given that nodes receiving multi-part-TLVs and not processing them
correctly will exhibit most vexing behavior that is not observable on LSDB
or any of the other tools we normally use.

=--- tony

On Thu, Aug 25, 2022 at 6:07 PM Acee Lindem (acee)  wrote:

> Speaking as WG member:
>
> Hi Gunder, Tony, Les,
>
> I'm also not a fan of the Multi-Part TLV Capability flag. While the intent
> of the draft is to encourage multi-part TLV advertisement and usage, the
> addition of this flag and the requirement for advertisement will most
> likely have the opposite effect. Given that IS-IS implementations are
> already advertising multi-part TLVs but none are advertising the proposed
> capability flag, implementation of the draft as currently written would
> inhibit Multi-Part TLV usage and be a step backwards. Of course, we know
> implementations that are already advertising these multi-part TLVs will
> most likely ignore the recommendation and continue to advertise them even
> when not all IS-IS routers within the scope of the Multi-Part TLV advertise
> the capability.
>
> Rather, I propose that the draft eliminates the capability flag and
> introduces a recommended configuration parameter that would allow
> Multi-Part TLVs to be suppressed. The recommended default would be FALSE.
> This would provide an out if these Multi-Part TLVs did, in fact, have dire
> consequences.
>
> Thanks,
> Acee
>
> On 8/25/22, 6:53 AM, "Lsr on behalf of Van De Velde, Gunter (Nokia -
> BE/Antwerp)"  gunter.van_de_ve...@nokia.com> wrote:
>
> Inline: GV>
>
> From: Tony Li  On Behalf Of Tony Li
> Sent: Wednesday, August 24, 2022 5:26 PM
>     To: Van De Velde, Gunter (Nokia - BE/Antwerp) <
> gunter.van_de_ve...@nokia.com>
> Cc: lsr 
> Subject: Re: [Lsr] New Version Notification for
> draft-pkaneria-lsr-multi-tlv-01.txt
>
>
> Hi Gunter,
>
> I am having troubles understanding the value of ‘The Multi-part TLV
> Capability’ flag.
> What would break if ‘Multi-part TLV Capability’ flag would not exist?
>
>
> A system that supported MP-TLVs would not be able to determine that
> there were other systems in the area that did not support MP-TLVs.  The
> system might then advertise MP-TLVs and they would be misinterpreted or
> cause system crashes in the systems that did not support them.
>
> GV> crashes? I really hope that is not happening.
> GV> When a legacy router receives MP-TLVs from another system and
> legacy router has no support for handling MP-TLV, then yes, things get
> misinterpreted. There is nothing wrong with that, is there? Do you have an
> example where things go wrong?
>
> If we want to introduce MP-TLVs, that change would warrant the
> existence of the flag.
>
> GV> I am not convinced yet how a MP-TLV catch-all flag would make ISIS
> behave better
>
> I dispute that a binary flag warrants the word ‘complexity’.
>
> GV>  living without binary flag is simpler and less complex then
> dealing with a binary flag. (i.e. what, when, how, why, who sets this flag?)
>
> Note: thoughts about the flag: What if a system by accident sends
> flip-flopping (set/unset/set/unset/etc) of this flag?
>
> Then other systems might misinterpret the results and generate
> inconsistent TLVs.  That would be bad.
>
> GV> correct, no good at all.
>
> What if an advertising system support multi-tlv for TLV ‘A’ but not
> for TLV ‘B’?
>
> We are not allowing that level of granularity.  A system that is going
> to support MP-TLVs should take care to operate correctly for ALL TLVs
> before advertising that it supports them.
>
> GV> I suspect that 'ALL TLVs' is a reference to  ALL TLVs supported by
> the local system. This means that e.g. when new TLVs would be supported
> after a system upgrade, that the operator has to be aware and correct the
> flag during each single upgrade.
>
> GV> Unfortunately I remain to have troubles understanding the value
> "Multi-part TLV Capability’ flag brings to an ISIS network.
>   * Without flag it is indeed uncertain if area wide mp-tlv is
> supported (sub-optimal).
>   * but with catch all MP-TLV flag I am not sure we improve ISIS
> operation:
>   ** Who guarantees

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-25 Thread Acee Lindem (acee)
Speaking as WG member:

Hi Gunder, Tony, Les, 

I'm also not a fan of the Multi-Part TLV Capability flag. While the intent of 
the draft is to encourage multi-part TLV advertisement and usage, the addition 
of this flag and the requirement for advertisement will most likely have the 
opposite effect. Given that IS-IS implementations are already advertising 
multi-part TLVs but none are advertising the proposed capability flag, 
implementation of the draft as currently written would inhibit Multi-Part TLV 
usage and be a step backwards. Of course, we know implementations that are 
already advertising these multi-part TLVs will most likely ignore the 
recommendation and continue to advertise them even when not all IS-IS routers 
within the scope of the Multi-Part TLV advertise the capability. 

Rather, I propose that the draft eliminates the capability flag and introduces 
a recommended configuration parameter that would allow Multi-Part TLVs to be 
suppressed. The recommended default would be FALSE. This would provide an out 
if these Multi-Part TLVs did, in fact, have dire consequences. 

Thanks,
Acee

On 8/25/22, 6:53 AM, "Lsr on behalf of Van De Velde, Gunter (Nokia - 
BE/Antwerp)"  
wrote:

Inline: GV>

From: Tony Li  On Behalf Of Tony Li
Sent: Wednesday, August 24, 2022 5:26 PM
To: Van De Velde, Gunter (Nokia - BE/Antwerp) 

Cc: lsr 
    Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Hi Gunter,

I am having troubles understanding the value of ‘The Multi-part TLV 
Capability’ flag.
What would break if ‘Multi-part TLV Capability’ flag would not exist? 


A system that supported MP-TLVs would not be able to determine that there 
were other systems in the area that did not support MP-TLVs.  The system might 
then advertise MP-TLVs and they would be misinterpreted or cause system crashes 
in the systems that did not support them.

GV> crashes? I really hope that is not happening.
GV> When a legacy router receives MP-TLVs from another system and legacy 
router has no support for handling MP-TLV, then yes, things get misinterpreted. 
There is nothing wrong with that, is there? Do you have an example where things 
go wrong? 

If we want to introduce MP-TLVs, that change would warrant the existence of 
the flag.  

GV> I am not convinced yet how a MP-TLV catch-all flag would make ISIS 
behave better

I dispute that a binary flag warrants the word ‘complexity’.

GV>  living without binary flag is simpler and less complex then dealing 
with a binary flag. (i.e. what, when, how, why, who sets this flag?)

Note: thoughts about the flag: What if a system by accident sends 
flip-flopping (set/unset/set/unset/etc) of this flag? 

Then other systems might misinterpret the results and generate inconsistent 
TLVs.  That would be bad.

GV> correct, no good at all.

What if an advertising system support multi-tlv for TLV ‘A’ but not for TLV 
‘B’?

We are not allowing that level of granularity.  A system that is going to 
support MP-TLVs should take care to operate correctly for ALL TLVs before 
advertising that it supports them.

GV> I suspect that 'ALL TLVs' is a reference to  ALL TLVs supported by the 
local system. This means that e.g. when new TLVs would be supported after a 
system upgrade, that the operator has to be aware and correct the flag during 
each single upgrade. 

GV> Unfortunately I remain to have troubles understanding the value 
"Multi-part TLV Capability’ flag brings to an ISIS network. 
  * Without flag it is indeed uncertain if area wide mp-tlv is supported 
(sub-optimal). 
  * but with catch all MP-TLV flag I am not sure we improve ISIS operation: 
  ** Who guarantees that the flag is set correctly on all systems at all 
times
  ** Maybe all systems falls back to advertise single TLV because another 
(legacy?) system advertise a wrong flag  (sub-optimal)
  ** Legacy system with MP-TLV support gets upgraded and now supports 
additional TLVs but not with MP-TLV...  ?manual intervention? (sub-optimal)
  ** what, when, how, why, who sets the MP-TLV flag? What with flapping of 
MP-TLV flag (undefined)

G/

Tony



___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-25 Thread Van De Velde, Gunter (Nokia - BE/Antwerp)
Inline: GV>

From: Tony Li  On Behalf Of Tony Li
Sent: Wednesday, August 24, 2022 5:26 PM
To: Van De Velde, Gunter (Nokia - BE/Antwerp) 
Cc: lsr 
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Hi Gunter,

I am having troubles understanding the value of ‘The Multi-part TLV Capability’ 
flag.
What would break if ‘Multi-part TLV Capability’ flag would not exist? 


A system that supported MP-TLVs would not be able to determine that there were 
other systems in the area that did not support MP-TLVs.  The system might then 
advertise MP-TLVs and they would be misinterpreted or cause system crashes in 
the systems that did not support them.

GV> crashes? I really hope that is not happening.
GV> When a legacy router receives MP-TLVs from another system and legacy router 
has no support for handling MP-TLV, then yes, things get misinterpreted. There 
is nothing wrong with that, is there? Do you have an example where things go 
wrong? 

If we want to introduce MP-TLVs, that change would warrant the existence of the 
flag.  

GV> I am not convinced yet how a MP-TLV catch-all flag would make ISIS behave 
better

I dispute that a binary flag warrants the word ‘complexity’.

GV>  living without binary flag is simpler and less complex then dealing with a 
binary flag. (i.e. what, when, how, why, who sets this flag?)

Note: thoughts about the flag: What if a system by accident sends flip-flopping 
(set/unset/set/unset/etc) of this flag? 

Then other systems might misinterpret the results and generate inconsistent 
TLVs.  That would be bad.

GV> correct, no good at all.

What if an advertising system support multi-tlv for TLV ‘A’ but not for TLV ‘B’?

We are not allowing that level of granularity.  A system that is going to 
support MP-TLVs should take care to operate correctly for ALL TLVs before 
advertising that it supports them.

GV> I suspect that 'ALL TLVs' is a reference to  ALL TLVs supported by the 
local system. This means that e.g. when new TLVs would be supported after a 
system upgrade, that the operator has to be aware and correct the flag during 
each single upgrade. 

GV> Unfortunately I remain to have troubles understanding the value "Multi-part 
TLV Capability’ flag brings to an ISIS network. 
  * Without flag it is indeed uncertain if area wide mp-tlv is supported 
(sub-optimal). 
  * but with catch all MP-TLV flag I am not sure we improve ISIS operation: 
  ** Who guarantees that the flag is set correctly on all systems at all times
  ** Maybe all systems falls back to advertise single TLV because another 
(legacy?) system advertise a wrong flag  (sub-optimal)
  ** Legacy system with MP-TLV support gets upgraded and now supports 
additional TLVs but not with MP-TLV...  ?manual intervention? (sub-optimal)
  ** what, when, how, why, who sets the MP-TLV flag? What with flapping of 
MP-TLV flag (undefined)

G/

Tony



___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-24 Thread Les Ginsberg (ginsberg)
Tony –

I am only expressing my POV on the proposed capability advertisement– which I 
have shared with you and other authors privately. I was not discussing what 
is/is not in the draft.
A few responses inline to correct misinterpretations on your part.

From: Tony Li  On Behalf Of Tony Li
Sent: Wednesday, August 24, 2022 1:34 PM
To: Les Ginsberg (ginsberg) 
Cc: Van De Velde, Gunter (Nokia - BE/Antwerp) ; 
lsr 
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt

MP-TLVs are sent by routers not simply because they have the ability to do so, 
but because the configuration requires them to send more information about an 
object than will fit in a single TLV. This means that any attempt to suppress 
generation of a MP-TLV based on the current capabilities (advertised or not) of 
the routers in the network will not result in a working network. Some 
information required by the configuration would be missing – behavior of the 
network in such a condition is unpredictable. In addition, suppression of 
MP-TLVs triggered by the introduction of a router which does not have support 
could result in flooding storms when the state of the network transitions from 
“all nodes support” to “some nodes support” (or vice versa).


This is simply dishonest.
[LES:] There is nothing “dishonest” here. It is an accurate description of the 
consequences if an implementation attempted to use the capability advertisement 
to alter what it sends.
I was not commenting on language in the draft – please confine yourself to what 
I said – not what is top of mind for you.

But, since you have brought the draft discussion into the thread, I think it is 
accurate to state that the authors of the draft (including you and I) have 
privately agreed that no such usage would be made of the capability 
advertisement.
My comment here is not about the language in the 
yet-to-be-published-new-version-of-the-draft – it was simply to share with 
folks reading this email thread that IMO it is not safe to make such a usage.

There has never been any text that even remotely suggested that nodes should 
withdraw MP-TLVs if capabilities were not seen.  Even if an implementation 
gated their advertisement of MP-TLVs based on the capability, it would hardly 
be a “flooding storm”.  It is a single update from all relevant nodes.  One 
copy of the full LSDB.  If the network can’t support that, then a partition is 
also going to collapse it.


The only safe use of such an advertisement would be as informational.


Which is exactly the text that we have agreed to, yet, you are the only author 
blocking us from publishing it.

[LES:] My discussion point with you and the other authors is on whether the 
proposed capability advertisement should be part of the MP-TLV draft or another 
draft that is focused solely on the introduction of such capability 
advertisements. I prefer the latter because the introduction of such an 
advertisement is unprecedented (and yes – this is easily proven) and as such 
deserves to be reviewed by the WG on its own merit – not snuck in as a small 
paragraph in the midst of a draft concerned with something else.

Advertisement of “features supported” is unprecedented in IS-IS (existing 
advertisements in Router Capability TLV are not advertising feature support – 
they are advertising feature specific information used by other routers or 
other entities (e.g., controllers ) in performing routing calculations).


This is irrelevant. We tweak semantics all of the time.  There is nothing in 
the definition of the router capability TLV that prevents what we are 
proposing.  You are the only one who objects.

[LES:] This isn’t “semantics”. It is a completely new usage.


It introduces the sending of management information in real time protocol 
updates – something which is more properly provided via management 
applications, on box show commands, and/or vendor documentation.


We have been sending management information in the LSDB since we introduced the 
hostname TLV.

[LES:] Fair enough as regards hostname – but that has direct usage in show 
commands on all routers as an aid to box identification. Quite different than 
advertising a list of “features supported”.


There is also the reality that MP-TLVs are already being sent by multiple 
implementations and that existing protocol specifications have explicitly 
specified the use of MP-TLVs as valid in a number of existing cases including:

  242 Router Capability TLV  [RFC7981)
  138 GMPLS-SRLG [RFC5307]
  139   IPv6 SRLG  [RFC6119]
  238 Application-Specific SRLG [RFC8919]
  Also sub-TLV  16 Application-Specific Link Attributes [RFC8919]


This is also incorrect.  The new text of the document specifically excludes all 
of these cases from MP-TLV.

[LES:] Again – you misinterpret my post. I am not claiming the draft discusses 
these. I am simply discussing the merits (or lack thereof) of introducing the 
proposed capability

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-24 Thread Tony Li
> MP-TLVs are sent by routers not simply because they have the ability to do 
> so, but because the configuration requires them to send more information 
> about an object than will fit in a single TLV. This means that any attempt to 
> suppress generation of a MP-TLV based on the current capabilities (advertised 
> or not) of the routers in the network will not result in a working network. 
> Some information required by the configuration would be missing – behavior of 
> the network in such a condition is unpredictable. In addition, suppression of 
> MP-TLVs triggered by the introduction of a router which does not have support 
> could result in flooding storms when the state of the network transitions 
> from “all nodes support” to “some nodes support” (or vice versa).


This is simply dishonest.

There has never been any text that even remotely suggested that nodes should 
withdraw MP-TLVs if capabilities were not seen.  Even if an implementation 
gated their advertisement of MP-TLVs based on the capability, it would hardly 
be a “flooding storm”.  It is a single update from all relevant nodes.  One 
copy of the full LSDB.  If the network can’t support that, then a partition is 
also going to collapse it.


> The only safe use of such an advertisement would be as informational.


Which is exactly the text that we have agreed to, yet, you are the only author 
blocking us from publishing it.


> Advertisement of “features supported” is unprecedented in IS-IS (existing 
> advertisements in Router Capability TLV are not advertising feature support – 
> they are advertising feature specific information used by other routers or 
> other entities (e.g., controllers ) in performing routing calculations).


This is irrelevant. We tweak semantics all of the time.  There is nothing in 
the definition of the router capability TLV that prevents what we are 
proposing.  You are the only one who objects.


> It introduces the sending of management information in real time protocol 
> updates – something which is more properly provided via management 
> applications, on box show commands, and/or vendor documentation.


We have been sending management information in the LSDB since we introduced the 
hostname TLV.


> There is also the reality that MP-TLVs are already being sent by multiple 
> implementations and that existing protocol specifications have explicitly 
> specified the use of MP-TLVs as valid in a number of existing cases including:
>  
>   242 Router Capability TLV  [RFC7981)
>   138 GMPLS-SRLG [RFC5307]
>   139   IPv6 SRLG  [RFC6119]
>   238 Application-Specific SRLG [RFC8919]
>   Also sub-TLV  16 Application-Specific Link Attributes [RFC8919]


This is also incorrect.  The new text of the document specifically excludes all 
of these cases from MP-TLV.

 
> Therefore, the absence of an MP-TLV supported advertisement cannot be 
> reliably used to indicate lack of support – it could just mean that the 
> feature support advertisement itself is not supported.


The absence of the MP-TLV support advertisement indicates that the node does 
not declare support for all MP-TLVs.  That is both necessary and sufficient.

Tony


___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-24 Thread Les Ginsberg (ginsberg)
NOTE: I am speaking here as a WG member – not as a co-author of the draft.

I am very much in agreement with Gunter’s POV – and I would caution that even 
those who may find the idea of advertising “Multi-part TLV Capability’ 
appealing should temper their expectations.

MP-TLVs are sent by routers not simply because they have the ability to do so, 
but because the configuration requires them to send more information about an 
object than will fit in a single TLV. This means that any attempt to suppress 
generation of a MP-TLV based on the current capabilities (advertised or not) of 
the routers in the network will not result in a working network. Some 
information required by the configuration would be missing – behavior of the 
network in such a condition is unpredictable. In addition, suppression of 
MP-TLVs triggered by the introduction of a router which does not have support 
could result in flooding storms when the state of the network transitions from 
“all nodes support” to “some nodes support” (or vice versa).

The only safe use of such an advertisement would be as informational.

Advertisement of “features supported” is unprecedented in IS-IS (existing 
advertisements in Router Capability TLV are not advertising feature support – 
they are advertising feature specific information used by other routers or 
other entities (e.g., controllers ) in performing routing calculations). It 
introduces the sending of management information in real time protocol updates 
– something which is more properly provided via management applications, on box 
show commands, and/or vendor documentation.

There is also the reality that MP-TLVs are already being sent by multiple 
implementations and that existing protocol specifications have explicitly 
specified the use of MP-TLVs as valid in a number of existing cases including:

  242 Router Capability TLV  [RFC7981)
  138 GMPLS-SRLG [RFC5307]
  139   IPv6 SRLG  [RFC6119]
  238 Application-Specific SRLG [RFC8919]
  Also sub-TLV  16 Application-Specific Link Attributes [RFC8919]

Therefore, the absence of an MP-TLV supported advertisement cannot be reliably 
used to indicate lack of support – it could just mean that the feature support 
advertisement itself is not supported.

I do not see value in introducing such an advertisement.

Les


From: Lsr  On Behalf Of Tony Li
Sent: Wednesday, August 24, 2022 8:26 AM
To: Van De Velde, Gunter (Nokia - BE/Antwerp) 
Cc: lsr 
Subject: Re: [Lsr] New Version Notification for 
draft-pkaneria-lsr-multi-tlv-01.txt


Hi Gunter,

I am having troubles understanding the value of ‘The Multi-part TLV Capability’ 
flag.
What would break if ‘Multi-part TLV Capability’ flag would not exist?


A system that supported MP-TLVs would not be able to determine that there were 
other systems in the area that did not support MP-TLVs.  The system might then 
advertise MP-TLVs and they would be misinterpreted or cause system crashes in 
the systems that did not support them.



IS-IS has been working well for many years. Why would that suddenly change and 
mandate existence and complexity of a ‘Multi-part TLV Capability’ flag?


If we want to introduce MP-TLVs, that change would warrant the existence of the 
flag.  I dispute that a binary flag warrants the word ‘complexity’.



Note: thoughts about the flag: What if a system by accident sends flip-flopping 
(set/unset/set/unset/etc) of this flag?


Then other systems might misinterpret the results and generate inconsistent 
TLVs.  That would be bad.



What if an advertising system support multi-tlv for TLV ‘A’ but not for TLV ‘B’?


We are not allowing that level of granularity.  A system that is going to 
support MP-TLVs should take care to operate correctly for ALL TLVs before 
advertising that it supports them.

Tony



___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-24 Thread Gyan Mishra
Tony

On Wed, Aug 24, 2022 at 11:26 AM Tony Li  wrote:

>
> Hi Gunter,
>
> I am having troubles understanding the value of ‘The Multi-part TLV
> Capability’ flag.
>
> What would break if ‘*Multi-part TLV Capability*’ flag would not exist?
>
>
>
> A system that supported MP-TLVs would not be able to determine that there
> were other systems in the area that did not support MP-TLVs.  The system
> might then advertise MP-TLVs and they would be misinterpreted or cause
> system crashes in the systems that did not support them.
>

   Gyan> That would be bad if the system would misinterpret the MP-TLV
causing it to crash.  So then the non supporting devices could be
vulnerable to crash.

>
>
> IS-IS has been working well for many years. Why would that suddenly change
> and mandate existence and complexity of a ‘Multi-part TLV Capability’ flag?
>
>
>
> If we want to introduce MP-TLVs, that change would warrant the existence
> of the flag.  I dispute that a binary flag warrants the word ‘complexity’.
>
>
> Note: thoughts about the flag: What if a system by accident sends
> flip-flopping (set/unset/set/unset/etc) of this flag?
>
>
>
> Then other systems might misinterpret the results and generate
> inconsistent TLVs.  That would be bad.
>
>
> What if an advertising system support multi-tlv for TLV ‘A’ but not for
> TLV ‘B’?
>
>
>
> We are not allowing that level of granularity.  A system that is going to
> support MP-TLVs should take care to operate correctly for ALL TLVs before
> advertising that it supports them.
>
> Tony
>
>
>
> ___
> Lsr mailing list
> Lsr@ietf.org
> https://www.ietf.org/mailman/listinfo/lsr
>
-- 



*Gyan Mishra*

*Network Solutions A**rchitect *

*Email gyan.s.mis...@verizon.com *



*M 301 502-1347*
___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-08-24 Thread Tony Li

Hi Gunter,

> I am having troubles understanding the value of ‘The Multi-part TLV 
> Capability’ flag.
> What would break if ‘Multi-part TLV Capability’ flag would not exist?


A system that supported MP-TLVs would not be able to determine that there were 
other systems in the area that did not support MP-TLVs.  The system might then 
advertise MP-TLVs and they would be misinterpreted or cause system crashes in 
the systems that did not support them.


> IS-IS has been working well for many years. Why would that suddenly change 
> and mandate existence and complexity of a ‘Multi-part TLV Capability’ flag?


If we want to introduce MP-TLVs, that change would warrant the existence of the 
flag.  I dispute that a binary flag warrants the word ‘complexity’.


> Note: thoughts about the flag: What if a system by accident sends 
> flip-flopping (set/unset/set/unset/etc) of this flag?


Then other systems might misinterpret the results and generate inconsistent 
TLVs.  That would be bad.


> What if an advertising system support multi-tlv for TLV ‘A’ but not for TLV 
> ‘B’?


We are not allowing that level of granularity.  A system that is going to 
support MP-TLVs should take care to operate correctly for ALL TLVs before 
advertising that it supports them.

Tony



___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr


Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-07-05 Thread Tony Li

Hi Hannes,

Thank you.  Not everyone is yet in agreement with this, so we are discussing 
correct wording.

T


> On Jul 5, 2022, at 7:00 AM, Hannes Gredler 
>  wrote:
> 
> Hi Tony, et al,
> 
> minor nit:
> 
> ---
>   As an example, consider the Extended IS Reachability TLV (type 22).
>   A neighbor in this TLV is specified by:
> 
>   *  7 octets of system ID and pseudonode number
>   *  3 octets of default metric
> 
>   This acts as the key for this entry.  The key is followed by up to
>   244 octets of sub-TLV information.
> ---
> 
> I believe that in most implementations the key in the LSDB is the 7 octets of 
> system ID and pseudonode number
> and does not include the metric (it's really an attribute to the key), 
> whereas the current wording might
> wrongly imply that the key is {sysid, PSN and metric}.
> 
> thanks,
> 
> /hannes
> 
> |From: Lsr mailto:lsr-boun...@ietf.org>> On Behalf 
> Of Tony Li
> |Sent: Tuesday, July 5, 2022 3:09 AM
> |To: lsr mailto:lsr@ietf.org>>
> |Subject: [Lsr] Fwd: New Version Notification for
> |draft-pkaneria-lsr-multi-tlv-01.txt
> | 
> | 
> | 
> | 
> | 
> |Hi all,
> | 
> | 
> | 
> |This is an update to reflect some of the discussions to date. A diff is
> |attached.  Most of this is a change to terminology to stop using the word
> |‘instance’ and shift to using ‘multi-part TLV’.
> | 
> | 
> | 
> |We have been having a discussion about adding more discussion of keys in
> |this document.  We have not done that yet.  This is not an indication of
> |refusal, just making one baby step forward.  More to come… Text
> |contributions are more than welcome.
> | 
> | 
> | 
> |Other comments?
> | 
> | 
> | 
> |Regards,
> | 
> |Tony
> | 
> | 
> | 
> | 
> | 
> | 
> | 
> | 
> | 
> |  Begin forwarded message:
> | 
> |   
> | 
> |  From: internet-dra...@ietf.org 
> | 
> |  Subject: New Version Notification for
> |  draft-pkaneria-lsr-multi-tlv-01.txt
> | 
> |  Date: July 4, 2022 at 6:04:37 PM PDT
> | 
> |  To: "Antoni Przygienda" mailto:p...@juniper.net>>, 
> "Chris Bowers"
> |  mailto:cbo...@juniper.net>>, "Les Ginsberg" 
> mailto:ginsb...@cisco.com>>, "Parag
> |  Kaneriya" mailto:pkane...@juniper.net>>, 
> "Shraddha Hegde"
> |  mailto:shrad...@juniper.net>>, "Tony Li" 
> mailto:tony...@tony.li>>, "Tony Przygienda"
> |  mailto:p...@juniper.net>>
> | 
> |   
> | 
> |  A new version of I-D, draft-pkaneria-lsr-multi-tlv-01.txt
> |  has been successfully submitted by Tony Li and posted to the
> |  IETF repository.
> | 
> |  Name:  draft-pkaneria-lsr-multi-tlv
> |  Revision:  01
> |  Title:  Multi-part TLVs in IS-IS
> |  Document date:  2022-07-04
> |  Group:  Individual Submission
> |  Pages:  7
> |  URL:
> | 
> https://www.ietf.org/archive/id/draft-pkaneria-lsr-multi-tlv-01.txt
> |  Status:
> |  https://datatracker.ietf.org/doc/draft-pkaneria-lsr-multi-tlv/
> |  Htmlized:
> |
> https://datatracker.ietf.org/doc/html/draft-pkaneria-lsr-multi-tlv
> |  Diff:
> |
> https://www.ietf.org/rfcdiff?url2=draft-pkaneria-lsr-multi-tlv-01
> | 
> |  Abstract:
> |New technologies are adding new information into IS-IS while
> |deployment scales are simultaneously increasing, causing the contents
> |of many critical TLVs to exceed the currently supported limit of 255
> |octets.  Extensions such as [RFC7356] require significant IS-IS
> |changes that could help address the problem, but a less drastic
> |solution would be beneficial.  This document codifies the common
> |mechanism of extending the TLV content space through multiple TLVs.
> | 
> |  The IETF Secretariat
> | 
> | 
> | 
> |  
> _
> | 
> |  Ce message et ses pieces jointes peuvent contenir des informations 
> confidentielles ou privilegiees et ne doivent donc
> |  pas etre diffuses, exploites ou copies sans autorisation. Si vous avez 
> recu ce message par erreur, veuillez le signaler
> |  a l'expediteur et le detruire ainsi que les pieces jointes. Les messages 
> electroniques etant susceptibles d'alteration,
> |  Orange decline toute responsabilite si ce message a ete altere, deforme ou 
> falsifie. Merci.
> | 
> |  This message and its attachments may contain confidential or privileged 
> information that may be protected by law;
> |  they should not be distributed, used or copied without authorisation.
> |  If you have received this email in error, please notify the sender and 
> delete this message and its attachments.
> |  As emails may be altered, Orange is not liable for messages that have been 
> modified, changed or falsified.
> |  Thank you.
> 
> | 

Re: [Lsr] New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt

2022-07-05 Thread Tony Li

Bruno,

Thank you, the authors are discussing this.

T


> On Jul 5, 2022, at 4:52 AM,  
>  wrote:
> 
> Hi Tony,
>  
> Thanks the update.
> 1 clarification question on §5 (new capability)
> « If all routers in an area advertise the Multi-part TLV Capability a node 
> MAY advertise multi-part TLVs “
>  
> Does this mean that if one router does not advertise the capability, routers 
> MUST NOT advertise multi-part TLVs? (and if necessary readvertises LSPs which 
> contained such multi-part TLV)
> If so, I would favor adding this explicitly.
>  
> Regards,
> --Bruno
>  
>  
> Orange Restricted
> From: Lsr mailto:lsr-boun...@ietf.org>> On Behalf Of 
> Tony Li
> Sent: Tuesday, July 5, 2022 3:09 AM
> To: lsr mailto:lsr@ietf.org>>
> Subject: [Lsr] Fwd: New Version Notification for 
> draft-pkaneria-lsr-multi-tlv-01.txt
>  
>  
> Hi all,
>  
> This is an update to reflect some of the discussions to date. A diff is 
> attached.  Most of this is a change to terminology to stop using the word 
> ‘instance’ and shift to using ‘multi-part TLV’.
>  
> We have been having a discussion about adding more discussion of keys in this 
> document.  We have not done that yet.  This is not an indication of refusal, 
> just making one baby step forward.  More to come… Text contributions are more 
> than welcome.
>  
> Other comments?
>  
> Regards,
> Tony
>  
>  
>  
>  
> 
> 
> Begin forwarded message:
>  
> From: internet-dra...@ietf.org 
> Subject: New Version Notification for draft-pkaneria-lsr-multi-tlv-01.txt
> Date: July 4, 2022 at 6:04:37 PM PDT
> To: "Antoni Przygienda" mailto:p...@juniper.net>>, "Chris 
> Bowers" mailto:cbo...@juniper.net>>, "Les Ginsberg" 
> mailto:ginsb...@cisco.com>>, "Parag Kaneriya" 
> mailto:pkane...@juniper.net>>, "Shraddha Hegde" 
> mailto:shrad...@juniper.net>>, "Tony Li" 
> mailto:tony...@tony.li>>, "Tony Przygienda" 
> mailto:p...@juniper.net>>
>  
> 
> A new version of I-D, draft-pkaneria-lsr-multi-tlv-01.txt
> has been successfully submitted by Tony Li and posted to the
> IETF repository.
> 
> Name: draft-pkaneria-lsr-multi-tlv
> Revision: 01
> Title: Multi-part TLVs in IS-IS
> Document date: 2022-07-04
> Group: Individual Submission
> Pages: 7
> URL:
> https://www.ietf.org/archive/id/draft-pkaneria-lsr-multi-tlv-01.txt 
> 
> Status: 
> https://datatracker.ietf.org/doc/draft-pkaneria-lsr-multi-tlv/ 
> 
> Htmlized:   
> https://datatracker.ietf.org/doc/html/draft-pkaneria-lsr-multi-tlv 
> 
> Diff:   
> https://www.ietf.org/rfcdiff?url2=draft-pkaneria-lsr-multi-tlv-01 
> 
> 
> Abstract:
>   New technologies are adding new information into IS-IS while
>   deployment scales are simultaneously increasing, causing the contents
>   of many critical TLVs to exceed the currently supported limit of 255
>   octets.  Extensions such as [RFC7356] require significant IS-IS
>   changes that could help address the problem, but a less drastic
>   solution would be beneficial.  This document codifies the common
>   mechanism of extending the TLV content space through multiple TLVs.
> 
> 
> 
> 
> The IETF Secretariat
> 
> 
>  
> _
> 
> Ce message et ses pieces jointes peuvent contenir des informations 
> confidentielles ou privilegiees et ne doivent donc
> pas etre diffuses, exploites ou copies sans autorisation. Si vous avez recu 
> ce message par erreur, veuillez le signaler
> a l'expediteur et le detruire ainsi que les pieces jointes. Les messages 
> electroniques etant susceptibles d'alteration,
> Orange decline toute responsabilite si ce message a ete altere, deforme ou 
> falsifie. Merci.
> 
> This message and its attachments may contain confidential or privileged 
> information that may be protected by law;
> they should not be distributed, used or copied without authorisation.
> If you have received this email in error, please notify the sender and delete 
> this message and its attachments.
> As emails may be altered, Orange is not liable for messages that have been 
> modified, changed or falsified.
> Thank you.

___
Lsr mailing list
Lsr@ietf.org
https://www.ietf.org/mailman/listinfo/lsr