Re: AD Review of draft-ietf-bfd-seamless-base

"Carlos Pignataro (cpignata)" <cpignata@cisco.com> Sun, 03 January 2016 10:12 UTC

Return-Path: <cpignata@cisco.com>
X-Original-To: rtg-bfd@ietfa.amsl.com
Delivered-To: rtg-bfd@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 1CFD01A916D; Sun, 3 Jan 2016 02:12:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -12.612
X-Spam-Level:
X-Spam-Status: No, score=-12.612 tagged_above=-999 required=5 tests=[BAYES_40=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ralxt29rAPDw; Sun, 3 Jan 2016 02:12:46 -0800 (PST)
Received: from alln-iport-1.cisco.com (alln-iport-1.cisco.com [173.37.142.88]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 5B4E41A916C; Sun, 3 Jan 2016 02:12:46 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=9296; q=dns/txt; s=iport; t=1451815966; x=1453025566; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=GuQQ74bJjkPXXl8vVbMMHu3iPwrcIU+N2NCykdZfhKE=; b=XwOLYtcUpeBbS6X2LDBZHpNTOFwJ3MStPNtNUnFF+AfU2Yig9uzfTvst WJi58Ab/0wVQt9xRHhKVvbdWUmNhSqrGx/FcMpdffZ7Dfnwp9Kcdb/+L+ 25XuhDGYbvidNlv/+SiGDj5sR7NdyTyLlYO/fgb2gi371G+cVrV1zCVzj M=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0AlAgAK84hW/4kNJK1egzpSXg+IWbNwAQ2BZCKFbQKBEjgUAQEBAQEBAYEKhDQBAQEDATo/BQcEAgEIEQQBAQEeCQcyFAkIAgQOBYgnCA6/TgEBAQEBAQEBAQEBAQEBAQEBAQEBARQEhlaCD4JwhCYRAR2DTIEbBZMKg3wBhT+CLIVlgVyNH4VYiGEBIAEBQoIRHIFdcoNOgUIBAQE
X-IronPort-AV: E=Sophos;i="5.20,515,1444694400"; d="scan'208";a="223880034"
Received: from alln-core-4.cisco.com ([173.36.13.137]) by alln-iport-1.cisco.com with ESMTP/TLS/DHE-RSA-AES256-SHA; 03 Jan 2016 10:12:45 +0000
Received: from XCH-RTP-003.cisco.com (xch-rtp-003.cisco.com [64.101.220.143]) by alln-core-4.cisco.com (8.14.5/8.14.5) with ESMTP id u03ACixF007779 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Sun, 3 Jan 2016 10:12:44 GMT
Received: from xch-rtp-020.cisco.com (64.101.220.160) by XCH-RTP-003.cisco.com (64.101.220.143) with Microsoft SMTP Server (TLS) id 15.0.1104.5; Sun, 3 Jan 2016 05:12:44 -0500
Received: from xch-rtp-020.cisco.com ([64.101.220.160]) by XCH-RTP-020.cisco.com ([64.101.220.160]) with mapi id 15.00.1104.009; Sun, 3 Jan 2016 05:12:43 -0500
From: "Carlos Pignataro (cpignata)" <cpignata@cisco.com>
To: "Les Ginsberg (ginsberg)" <ginsberg@cisco.com>
Subject: Re: AD Review of draft-ietf-bfd-seamless-base
Thread-Topic: AD Review of draft-ietf-bfd-seamless-base
Thread-Index: AQHQ97IkT4lFfvKt5Uasfc9mskhFeZ6+GxXwgAOR6gCAACS1gIAIyoiAgAZgJDCAAF3TgIABitaAgAKLpgCAAD2ZAIABxXiAgABo1oCAElKYEg==
Date: Sun, 03 Jan 2016 10:12:43 +0000
Message-ID: <2B252CB3-02C7-40C4-A9CF-38BBE3DD946D@cisco.com>
References: <D22876B0.D338D%aretana@cisco.com> <SN1PR0501MB21420F68EA29F1FA425AB295B30A0@SN1PR0501MB2142.namprd05.prod.outlook.com> <D28B4E76.ED8A5%aretana@cisco.com> <D28C6D0D.EDC23%aretana@cisco.com> <20151214000245520882.14fa350b@sniff.de> <SN1PR0501MB2142004E7430F3A5F64AC202B3E10@SN1PR0501MB2142.namprd05.prod.outlook.com> <D29978F4.F453D%aretana@cisco.com> <20151219013323973354.b44d7a1b@sniff.de> <b61f5ea7dbd94badac7544c07543d1ba@XCH-ALN-001.cisco.com> <CAG1kdoikns-v9dSLTQdPdpdQY99on+6vhpE0+GeJ5hgvoO_=0g@mail.gmail.com> <20151221230913162917.3e88c932@sniff.de>, <f46e3858dfef412d99dfd223f0840e9a@XCH-ALN-001.cisco.com>
In-Reply-To: <f46e3858dfef412d99dfd223f0840e9a@XCH-ALN-001.cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-ms-exchange-transport-fromentityheader: Hosted
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <http://mailarchive.ietf.org/arch/msg/rtg-bfd/pRMELF8Xit_d5usYeJM_ox8tm2c>
Cc: "rtg-bfd@ietf.org" <rtg-bfd@ietf.org>, "draft-ietf-bfd-seamless-base@ietf.org" <draft-ietf-bfd-seamless-base@ietf.org>, "bfd-chairs@ietf.org" <bfd-chairs@ietf.org>
X-BeenThere: rtg-bfd@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "RTG Area: Bidirectional Forwarding Detection DT" <rtg-bfd.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-bfd/>
List-Post: <mailto:rtg-bfd@ietf.org>
List-Help: <mailto:rtg-bfd-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 03 Jan 2016 10:12:49 -0000

Hi!

Happy New Year!

I strongly agree with Les' perspective here, as that was also the way I was trying to write the spec.  

As we design a spec, it is important that we, simultaneously:
1. Provide future growth capability and forward compatibility, and not corner ourselves being too stringent in design considerations.
2. Specify clearly the current set of interoperable options. 

The healthy tension between these two calls for:
1. Let the transport/discovery drafts (IGPs, L2TP) have the option of carrying multiple disc
2. Expand S3.8 in the base document to say that, today, the app is to have a single disc and if in the future multiple are needed, then the procedures need to update this spec clearly scoping both endpoints' behavior. 

Concerns with this approach? It allows for well defined current behavior and maximum forward compatibility and growth potential. 

Thumb typed by Carlos Pignataro.
Excuze typofraphicak errows

> On Dec 22, 2015, at 08:24, Les Ginsberg (ginsberg) <ginsberg@cisco.com> wrote:
> 
> Marc -
> 
>> -----Original Message-----
>> From: Marc Binderberger [mailto:marc@sniff.de]
>> Sent: Monday, December 21, 2015 11:09 PM
>> To: Manav Bhatia; Les Ginsberg (ginsberg)
>> Cc: Alvaro Retana (aretana); Santosh P K; rtg-bfd@ietf.org; draft-ietf-bfd-
>> seamless-base@ietf.org; bfd-chairs@ietf.org
>> Subject: Re: AD Review of draft-ietf-bfd-seamless-base
>> 
>> Hello Manav!
>> 
>>> S-BFD draft. You can look at Sec 3.8 of
>>> https://tools.ietf.org/html/draft-ietf-bfd-seamless-use-case-03#page-7
>>> to understand why we may want to support multiple discriminators per
>> node.
>> 
>> ah, that problem :-)
>> 
>>> My considered opinion is to strike that off from the base draft and
>>> move on, since S-BFD solves a real problem and should not be stalled
>>> for something that may never end up getting implemented.
>> 
>> So OSPF, IS-IS, L2TP could transport a single discriminator instead of a list?
> 
> [Les:] Perhaps - or we could leave these drafts as is - allowing the possibility of sending multiple discriminators in the future. The key would be for the base S-BFD draft to say something like "currently only support for a single discriminator per node is defined".
> 
> If in the future support for multiple discriminators is required and defined then the IGP/L2TP drafts could either:
> 
>   o Be left alone - a simple list is all that is required
>   o Be revised to carry whatever additional info S-BFD requires
> 
> My point is that since we have no idea what additional info might be required in the future leaving the IGP/L2TP drafts in their current state does no harm - and restricting them to one discriminator only provides no benefit.
> 
> That said, if folks feel strongly that we should restrict the IGP/L2TP advertisement format to one discriminator I would find that acceptable.
> 
>   Les
> 
>> 
>> 
>> Regards, Marc
>> 
>> 
>> 
>> 
>>> On Mon, 21 Dec 2015 09:36:12 +0530, Manav Bhatia wrote:
>>> Hi Les,
>>> 
>>> I had asked the exact same question in an offline email that i did not
>>> get a reply for.
>>> 
>>> I can say, as the primary co-author of the base S-BFD draft that the
>>> case for multiple SBFD discriminators stands on very tenuous grounds.
>>> The idea was very weird and i had argued that it really was an
>>> architectural/implementation limitation that was being addressed by
>>> way of supporting multiple discriminators per node. Given that there
>>> are others that share this concern I would recommend striking that off
>>> from the base S-BFD draft. You can look at Sec 3.8 of
>>> https://tools.ietf.org/html/draft-ietf-bfd-seamless-use-case-03#page-7
>>> to understand why we may want to support multiple discriminators per
>> node.
>>> 
>>> I had conceded to that being added since i did not want to preclude
>>> the possibility of adding that mechanism in the future. And it was
>>> felt that this would get debated in the WG and we would go based on the
>> consensus.
>>> 
>>> My considered opinion is to strike that off from the base draft and
>>> move on, since S-BFD solves a real problem and should not be stalled
>>> for something that may never end up getting implemented.
>>> 
>>> Cheers, Manav
>>> 
>>> 
>>> On Mon, Dec 21, 2015 at 5:55 AM, Les Ginsberg (ginsberg)
>>> <ginsberg@cisco.com> wrote:
>>>> I certainly agree with everyone that the IGPs are merely a transport
>>>> and do not "allocate" reflector discriminators nor - for the purposes
>>>> of advertising S-BFD discriminators - do they have any understanding
>>>> of how S-BFD discriminators are to be used.
>>>> 
>>>> However, before we rush off in a direction which will invalidate any
>>>> early implementations of the IGP drafts, I would like to see a
>>>> justification of the need for a given node to require multiple
>>>> reflector S-BFD discriminators and an explanation of what criteria
>>>> would be used to determine whether the reflector should/should not
>>>> respond to an Initiator S-BFD packet to a particular S-BFD reflector
>>>> discriminator. Perhaps I have missed it, but to date I am not aware
>>>> of any cogent explanation of this capability. The desire for multiple
>>>> S-BFD discriminators seems to be made out of either:
>>>> 
>>>>   o An abundance of caution ("We don't know why we would need them -
>>>> but if we come up with something in the future it would be nice if we
>>>> didn't preclude it.")
>>>> 
>>>>   o Use cases which no one knows how to support (e.g. mapping a
>>>> particular discriminator to a particular incoming interface or line
>>>> card)
>>>> 
>>>> What are the requirements and what about them necessitates multiple
>>>> S-BFD discriminators?
>>>> 
>>>>   Les
>>>> 
>>>> 
>>>>> -----Original Message-----
>>>>> From: Rtg-bfd [mailto:rtg-bfd-bounces@ietf.org] On Behalf Of Marc
>>>>> Binderberger
>>>>> Sent: Saturday, December 19, 2015 1:33 AM
>>>>> To: Alvaro Retana (aretana); Santosh P K
>>>>> Cc: rtg-bfd@ietf.org; draft-ietf-bfd-seamless-base@ietf.org; bfd-
>>>>> chairs@ietf.org
>>>>> Subject: Re: AD Review of draft-ietf-bfd-seamless-base
>>>>> 
>>>>> Hello Santosh, Alvaro et al.,
>>>>> 
>>>>>>> [SPK] This is implementation specific right? Do we need this to
>>>>>>> be captured in document?
>>>>> 
>>>>> we could make it "just a TLV" which the IGP/L2TP transports to other
>>>> S-BFD
>>>>> modules. The transport mechanism then would not need to know the
>>>>> inner structure, e.g. [type, discriminator], to function correctly.
>>>>> 
>>>>> But for S-BFD modules to interoperate we would need to define the
>>>>> inner structure of the "V" in the TLV.
>>>>> 
>>>>> Implementation specific could be if you want to have awareness of
>>>>> the
>>>> inner
>>>>> structure in the IGP/L2TP code already, e.g. when the IGP wants to
>>>>> make
>>>> use
>>>>> of S-BFD information it transports, for it's own purpose
>>>>> (shortcutting
>>>> some
>>>>> API calls).
>>>>> 
>>>>> 
>>>>> We have to ask the L2TP, OSPF, IS-IS authors if they would be fine
>>>>> with
>>>> this
>>>>> change :-)
>>>>> 
>>>>> 
>>>>> Regards, Marc
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>> 
>>>>>> On Fri, 18 Dec 2015 14:00:16 +0000, Alvaro Retana (aretana) wrote:
>>>>>> On 12/18/15, 4:30 AM, "Santosh P K" <santoshpk@juniper.net> wrote:
>>>>>> 
>>>>>> Santosh:
>>>>>> 
>>>>>> Hi!
>>>>>> 
>>>>>>>> There is another aspect: the protocols (OSPF, IS-IS, L2TP) plan
>>>>>>>> to transport a list of discriminators. Okay ... but how is the
>>>>>>>> receiver S-BFD
>>>> module
>>>>>>>> making sense out of this list?  Would have expected something
>>>>>>>> like
>>>>> (type,
>>>>>>>> discriminator). The protocols don't need to understand the
>>>>>>>> details,
>>>> only
>>>>>>>> that
>>>>>>>> the API transports one or more of these tuples in/out of the
>>>>>>>> protocol module.
>>>>>>>> S-BFD would know/define what a particular type means.
>>>>>>>> Just asking before we send OSPF, IS-IS, L2TP into the wrong
>>>> direction :-)
>>>>>>> 
>>>>>>> [SPK] This is implementation specific right? Do we need this to
>>>>>>> be captured in document?
>>>>>> 
>>>>>> What is implementation specific?
>>>>>> 
>>>>>> Right now the IGPs (generalizing: ISIS, OSPF, L2TP, etc.) are
>>>> developing
>>>>>> drafts to only carry the discriminators.  If, as Mark suggests,
>>>>>> the
>>>> IGPs
>>>>>> also transport something like "type", then S-BFD would know what
>>>>>> each discriminator is for.
>>>>>> 
>>>>>> Several questions:  Is this (transporting [type, discriminator])
>>>>>> what
>>>> is
>>>>>> expected from the IGPs?  If so, I assume the S-BFD module on the
>>>>>> nodes assigns those values for transportation, right?  How does a
>>>>>> receiver
>>>> know
>>>>>> what a particular type means?
>>>>>> 
>>>>>> Maybe the expectation from S-BFD is different...??  That is
>>>>>> something
>>>> that
>>>>>> needs to be clarified so the IGP work can proceed.
>>>>>> 
>>>>>> Thanks!
>>>>>> 
>>>>>> Alvaro.
>>>