Re: [Isis-wg] [Bier] BAR field length in draft-ietf-bier-isis-extensions and draft-ietf-bier-ospf-extensions

IJsbrand Wijnands <ice@cisco.com> Wed, 21 February 2018 00:19 UTC

Return-Path: <ice@cisco.com>
X-Original-To: isis-wg@ietfa.amsl.com
Delivered-To: isis-wg@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 0BB0812E050; Tue, 20 Feb 2018 16:19:42 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.529
X-Spam-Level:
X-Spam-Status: No, score=-14.529 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com
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 5qiae9N5h9O4; Tue, 20 Feb 2018 16:19:39 -0800 (PST)
Received: from rcdn-iport-5.cisco.com (rcdn-iport-5.cisco.com [173.37.86.76]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1A92A126DD9; Tue, 20 Feb 2018 16:19:39 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=83303; q=dns/txt; s=iport; t=1519172379; x=1520381979; h=mime-version:subject:from:in-reply-to:date:cc:message-id: references:to; bh=zgzkgLmsSY4Er7sUKG23PIAXbOifJCJ6oY226M/bgSs=; b=kDEcq0pP0FAVzsSwsfQuBKKfBzbLsppnnpmeqR0v5xsM4yiSzXwb8Eue z0maOw2k0JCwTT0+izk1cKn5/QzvUsICQa21upq/TKMW/mceWoB+Y+c6K GDcfIFAFi5225fg3cmOzhneOiV86f608kNy7konSH6MeIN29/+fMsrOuY Y=;
X-Files: PastedGraphic-6.png : 43631
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0BwCQC0uoxa/4oNJK1TChkBAQEBAQEBA?= =?us-ascii?q?QEBAQEHAQEBAQGDHgQtZnAog2iYLEIBAQEBAQEGgTSBF4d/jkqCEwMHAQIYAQo?= =?us-ascii?q?mNoQ1AoJsVRcBAgEBAQEBAQJrKIUjAQEBAwEBAQMeSwsFCwsRAQMBAQEFAQEBH?= =?us-ascii?q?wMCAgIVAQkFAR8DBggGEgEGigUDCAUIEKtPgieHPw2BMoITAQEBAQEBAQEBAQE?= =?us-ascii?q?BAQEBAQEBAQEBDgoFhQ6CJ4EPgi4BKYMFgmxEAQGBRkIPgnExgjQFklKBFJAaN?= =?us-ascii?q?QmHIYlghQuMOIgKjk+GBIMhAgQLAhkBgTwgATeBUU0jFToqAYIYPoIWHBlxAQ6?= =?us-ascii?q?BDiA3jHsBAQE?=
X-IronPort-AV: E=Sophos;i="5.46,542,1511827200"; d="png'150?scan'150,208,217,150";a="140513194"
Received: from alln-core-5.cisco.com ([173.36.13.138]) by rcdn-iport-5.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 21 Feb 2018 00:19:29 +0000
Received: from [10.154.161.40] ([10.154.161.40]) by alln-core-5.cisco.com (8.14.5/8.14.5) with ESMTP id w1L0JS4j027024 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Wed, 21 Feb 2018 00:19:28 GMT
Content-Type: multipart/alternative; boundary="Apple-Mail=_6F394ED5-027F-4BB6-A9C0-7BE30D219F0C"
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2104\))
From: IJsbrand Wijnands <ice@cisco.com>
In-Reply-To: <CY1PR0501MB1340543164C052E207A44D9DD4CF0@CY1PR0501MB1340.namprd05.prod.outlook.com>
Date: Tue, 20 Feb 2018 16:19:27 -0800
Cc: Eric Rosen <erosen@juniper.net>, "EXT-arkadiy.gulko@thomsonreuters.com" <arkadiy.gulko@thomsonreuters.com>, "akatlas@gmail.com" <akatlas@gmail.com>, "bier@ietf.org" <bier@ietf.org>, "isis-wg@ietf.org" <isis-wg@ietf.org>
Message-Id: <DD0F774D-E132-488E-B75A-A8FFA3B771F6@cisco.com>
References: <CAG4d1remdUKutEdc2DU6Gaan3z63CAZVo1D-L0GXg_=eHJxffw@mail.gmail.com> <9778B23E32FB2745BEA3BE037F185DC4A5BA61A3@BLREML503-MBX.china.huawei.com> <CAG4d1rc8=2gnEj4vTjjAja5SPfezBT+hBKRg219uLgndvA78Kg@mail.gmail.com> <CA+wi2hPoTA0u2rx0f5eoBsoOAH+m1uN0ggr=P7sSYFcX=1qQxw@mail.gmail.com> <CAG4d1rf_mphexVqMv20HQbd=px5koH5c_+VW_5TTfgjWq4EtSA@mail.gmail.com> <B94D11DC-F46A-4F8C-873F-6F4A21BC4071@thomsonreuters.com> <14dca8e9-9afd-b5ff-c753-3554b911d753@juniper.net> <CY1PR0501MB1340543164C052E207A44D9DD4CF0@CY1PR0501MB1340.namprd05.prod.outlook.com>
To: "Jeffrey (Zhaohui) Zhang" <zzhang@juniper.net>
X-Mailer: Apple Mail (2.2104)
Archived-At: <https://mailarchive.ietf.org/arch/msg/isis-wg/LBUZn66NQWIcIspRA0uYqHHgGpU>
Subject: Re: [Isis-wg] [Bier] BAR field length in draft-ietf-bier-isis-extensions and draft-ietf-bier-ospf-extensions
X-BeenThere: isis-wg@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: IETF IS-IS working group <isis-wg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/isis-wg>, <mailto:isis-wg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/isis-wg/>
List-Post: <mailto:isis-wg@ietf.org>
List-Help: <mailto:isis-wg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/isis-wg>, <mailto:isis-wg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 21 Feb 2018 00:19:42 -0000

Hi Jeffrey,

> When I said I prefer Option B earlier, I was actually referring to something similar to what was discussed below between Eric and Arkadiy, though with some differences.

ICE: Yes, I think this proposal has merit and a possible consensus for those who prefer to align with IGP and those who like to have BIER specific algorithms.

>  
> Whether we call it Option F or whatever, I have the following text prepared for everyone to review and comment. Ironically, I started with the OSPF spec J
>  
> 2.1.  BIER Sub-TLV
>  
>    0                   1                   2                   3
>    0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1
>    +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>    |              Type             |             Length            |
>    +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>    | Sub-domain-ID |     MT-ID     |              BFR-id           |
>    +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>    |    BART       |     BARM      |            Reserved           |
>    +-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+-+
>    |                      Sub-TLVs (variable)                      |
>    +-                                                             -+
>    |                                                               |
>  
> BART: A single-octet specifying the Bier AlgoRiThm used to calculate underlay
>       paths to reach BFERs. Values are allocated from the BIER Algorithm Registry.
>      Value 0 indicates the basic Shortest Path First algorithm using IGP metric.
>  
> BARM: A single-octet specifying the BIER AlgoRithm Modififer that can be used
>       to either modify, enhance or replace calculation of underlay paths to
>       reach BFERs as defined by the BART value.

ICE: In write up below its more clear on what BARM is, maybe change the name to make it more explicit this is the IGP Algorithm registry. Maybe call it BIER IGP AlgoriThm (BIAT)?

>  
> In this document version, both the BART and BARM octets MUST be zero. If an
> implementation based on this document receives a non-zero BART or BARM octet,
> it MUST treat it as an error and ignore the received BIER sub-TLV.
>  
> BART and BARM are independent of each other, and the BARM value identify
> an IGP Algorithm (either those 0~128 values registered in the “IGP Algorithm Type”
> registry, or a number that identifies a Flexible Algorithm [<reference here>],
> which is also considered as an IGP Algorithm.

ICE: Ok, I like that. Both values are independent of each other. This means the developments for both BIER Algorithm and IGP Algorithm can move forward without blocking each other.

>  
> Future specifications may specify BART values that change the
> interpretation of the BARM octet. Those specifications must handle backwards

ICE: This creates a potential dependency which I think we should avoid. I think there are possible use-cases where the combination of the two values could be valuable. But since we don’t yet know what that is, lets not speculate on it. Let keep both values as equal importance without interdependency.

> compatibility issues (or simply declare that backwards compatibility is not
> required - the deployment will then require the operator to make sure that all
> routers in the subdomain conform to the same specification).
>  
> [note that will not be added to the spec: if someone wants define a BART value
> in the future that changes the meaning of BARM, he would need
> to write a spec to explicitly state that and get the spec approved. W/o that,
> the BARM field is always interpreted as an IGP Algorithm]
>  
> 4.  IANA Considerations
>  
>   IANA is requested to set up a registry called "BIER Algorithm Registry". 
>   The registration policies for this registry are
>    * "Standards Action" ([RFC5226] and [RFC7120]) for values 0-240
>    * "Experimental Use" for values 240-255
>  
>    The initial values in the BIER Algorithm Registry are:
>  
>    0: Shortest Path First (SPF) algorithm based on IGP link metric
>    1-255: Unassigned

ICE: We should probably create a registry for BIAT, and make clear in the draft the values are identical to the IGP Algorithm registry. In an ideal world we would like to point to it, but that might get the draft stuck. Or, if there are other ideas on how to deal with that, would be good to know...

> Comments?

I think this is good, making the two values of equal importance makes sure nobody is blocked on each other and we can move on.

Thanks Jeffrey,

Ice.

>  
> Jeffrey
>  
> From: BIER [mailto:bier-bounces@ietf.org] On Behalf Of Eric C Rosen
> Sent: Tuesday, February 20, 2018 2:43 PM
> To: EXT-arkadiy.gulko@thomsonreuters.com <arkadiy.gulko@thomsonreuters.com>;; akatlas@gmail.com; bier@ietf.org
> Cc: isis-wg@ietf.org
> Subject: Re: [Bier] BAR field length in draft-ietf-bier-isis-extensions and draft-ietf-bier-ospf-extensions
>  
> On 2/20/2018 12:56 PM, arkadiy.gulko@thomsonreuters.com wrote:
> 
> I personally like Eric proposed option -two independed 1Byte filed one for IGP Algo and another one for BUAM : the "BIER Underlay Algorithm Modifier" registry.  The way the underlay paths are computed for a given BIER sub-domain is determined by the pair of codepoints: <IGP Algorithms codepoint, BIER Underlay Algorithm Modifier codepoint>.
> Not sure why it is not in a list of proposed options since I saw a lot of support for it on the WG.
> It sort of Option-B but allow more independence between BAR(BUAM) from IGP Algo
> 
> I'm also wondering what's wrong with this proposal as a way of moving forward.  The only real change I'd make to it is that the first one-octet field would either be a codepoint from the IGP Algorithms registry or a Flex-Algo codepoint.  Since the former are in the range 1-127 and the latter in the range 128-254, no ambiguity is possible.
> 
> With regard to the question of why it makes sense to use an IGP Algorithms codepoint, I think the argument is the following.  Per the architecture, BIER relies on a routing underlay to tell it the next hop for a given BFER.  Per the architecture, the routing underlay may use the exact same decision procedure applied to the exact same topology as can be applied for unicast routing.  One way of identifying a unicast routing decision procedure  is with codepoints from the IGP Algorithms registry and/or Flex-Algo codepoints.  Thus it makes sense for the IGP signaling to use these codepoints as a way of providing the BIER layer information about the routing underlay.
> 
> With regard to the question of why it makes sense to have a second one-octet BIER-specific field, I think the argument is the following.  The architecture does not require BIER to use a routing underlay that applies a decision procedure that is useful for or even applicable to unicast packets.   In such a case, there might not be a way to identify the decision procedure with a codepoint from the IGP Algorithms registry or even with a Flex-Algo codepoint.   So it's useful to have a codepoint that does not have to hold values from the IGP Algorithms registry and does not need to have Flex-Algo codepoints.
> 
> There is also some worry that there may in the future be a lot of arguments about populating IGP Algorithms registry, and it would be good to have a way to extend BIER by allocating codepoints that help identify the routing underlay, but that might not be useful for unicast applications.
> 
> To some extent, this is all a tempest in a teapot, because the extensible TLV structure can be used, as Alia points out, to work around any codepoint problems.  Of course, continually adding TLVs to modify the interpretation of other TLVs can becomes a problem in itself.
> 
> I think the most compelling argument for adding the second codepoint field is that it provide more options for exploring the issues that might arise as production deployments begin.  
> 
> I don't believe that any field containing a codepoint should ever be created without an association to a registry.  That makes squatting and future codepoint clash inevitable.
> Thus I think the current documents, which have a one-octet field that is not associated with a registry at all, are not really acceptable.  So I don't see any way to move forward now other than with a compromise like the one I suggested.  This is not exactly Alia's option B, because the second codepoint is not properly thought of as a sub-type of the other.
> 
> _______________________________________________
> BIER mailing list
> BIER@ietf.org
> https://www.ietf.org/mailman/listinfo/bier