Re: [bess] draft-ietf-bess-evpn-overlay / section 5.1.3 vs. section 9 (was Re: [Idr] draft-ietf-bess-evpn-overlay vs. draft-ietf-idr-tunnel-encaps)
"Ali Sajassi (sajassi)" <sajassi@cisco.com> Mon, 04 July 2016 04:21 UTC
Return-Path: <sajassi@cisco.com>
X-Original-To: bess@ietfa.amsl.com
Delivered-To: bess@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id E58D112D1E7 for <bess@ietfa.amsl.com>; Sun, 3 Jul 2016 21:21:28 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -15.947
X-Spam-Level:
X-Spam-Status: No, score=-15.947 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, RP_MATCHES_RCVD=-1.426, SPF_PASS=-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 5-aG7SZSeLWb for <bess@ietfa.amsl.com>; Sun, 3 Jul 2016 21:21:26 -0700 (PDT)
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 875E012B01B for <bess@ietf.org>; Sun, 3 Jul 2016 21:21:26 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=9041; q=dns/txt; s=iport; t=1467606086; x=1468815686; h=from:to:subject:date:message-id:references:in-reply-to: content-id:content-transfer-encoding:mime-version; bh=qy4atCy0nfHh0q3pzlEOq9XdqHfuuCKBgVeMFr6RIXE=; b=DBjzFTii0wcUyHgVfTFbDcRwkgQ+adeCT6McQVJQ+9XI4My3fkf9etn/ 3eZDWVQULwALLB881wS3q3xU07vHTw1vgSw52V+SpHHtV/05qThDBSA1Q 9tFoyFo3U7fScs+JFurAw0ptfljSGOpkGARX4cuSRXrey+XVoMZf5xcEr A=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0A9AgAV43lX/5xdJa1bgz5WfAa5LYF5IoV2AoEqOBQBAQEBAQEBZSeETAEBBQ5XAh4EAgEIEQQBAQEjBAcyFAkIAgQBEhSIHA7BOAEBAQEBAQEBAQEBAQEBAQEBAQEZBYlygQOKGwWOPIpXAYYIgnqFRI8qkAkBHjaCCAUXF4E1bogNfwEBAQ
X-IronPort-AV: E=Sophos;i="5.26,573,1459814400"; d="scan'208";a="122002984"
Received: from rcdn-core-5.cisco.com ([173.37.93.156]) by rcdn-iport-5.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 04 Jul 2016 04:21:25 +0000
Received: from XCH-RTP-001.cisco.com (xch-rtp-001.cisco.com [64.101.220.141]) by rcdn-core-5.cisco.com (8.14.5/8.14.5) with ESMTP id u644LPQc019317 (version=TLSv1/SSLv3 cipher=AES256-SHA bits=256 verify=FAIL); Mon, 4 Jul 2016 04:21:25 GMT
Received: from xch-rtp-005.cisco.com (64.101.220.145) by XCH-RTP-001.cisco.com (64.101.220.141) with Microsoft SMTP Server (TLS) id 15.0.1210.3; Mon, 4 Jul 2016 00:21:24 -0400
Received: from xch-rtp-005.cisco.com ([64.101.220.145]) by XCH-RTP-005.cisco.com ([64.101.220.145]) with mapi id 15.00.1210.000; Mon, 4 Jul 2016 00:21:24 -0400
From: "Ali Sajassi (sajassi)" <sajassi@cisco.com>
To: Thomas Morin <thomas.morin@orange.com>, John E Drake <jdrake@juniper.net>, "Rabadan, Jorge (Nokia - US)" <jorge.rabadan@nokia.com>, BESS <bess@ietf.org>, "draft-ietf-bess-evpn-overlay@tools.ietf.org" <draft-ietf-bess-evpn-overlay@tools.ietf.org>
Thread-Topic: draft-ietf-bess-evpn-overlay / section 5.1.3 vs. section 9 (was Re: [Idr] draft-ietf-bess-evpn-overlay vs. draft-ietf-idr-tunnel-encaps)
Thread-Index: AQHRxwQVar89BF9PoE++IfK/IWUITZ/q1UuAgAACPYCAHL8ugA==
Date: Mon, 04 Jul 2016 04:21:24 +0000
Message-ID: <D39F30DF.1B1306%sajassi@cisco.com>
References: <5729F1C3.1030605@orange.com> <5729F7C5.6040604@orange.com> <52D35106-ED5E-4C95-9131-6EA4527370D5@alcatel-lucent.com> <BY2PR0501MB1702CD2423A817F3725CB5DFC77B0@BY2PR0501MB1702.namprd05.prod.outlook.com> <012C176C-A8D6-45AA-BA69-616C0ED7E41E@alcatel-lucent.com> <SN1PR0501MB1709E1AF8C398791421E2123C77B0@SN1PR0501MB1709.namprd05.prod.outlook.com> <31d2c99b-de11-9e3c-fae9-2d60017c3090@orange.com> <BY2PR05MB2310ACC9C44A066EBB615D53C7550@BY2PR05MB2310.namprd05.prod.outlook.com> <5980f27a-32f1-25d5-a64c-3786f88d3f69@orange.com>
In-Reply-To: <5980f27a-32f1-25d5-a64c-3786f88d3f69@orange.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/14.6.4.160422
x-ms-exchange-messagesentrepresentingtype: 1
x-ms-exchange-transport-fromentityheader: Hosted
x-originating-ip: [10.19.76.53]
Content-Type: text/plain; charset="iso-8859-1"
Content-ID: <006CA301A800EE428A6C18149E2AECC9@emea.cisco.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/qROmSxgaSL9oWb2WLCQxfqhmXRM>
Subject: Re: [bess] draft-ietf-bess-evpn-overlay / section 5.1.3 vs. section 9 (was Re: [Idr] draft-ietf-bess-evpn-overlay vs. draft-ietf-idr-tunnel-encaps)
X-BeenThere: bess@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: BGP-Enabled ServiceS working group discussion list <bess.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bess>, <mailto:bess-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/bess/>
List-Post: <mailto:bess@ietf.org>
List-Help: <mailto:bess-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bess>, <mailto:bess-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 04 Jul 2016 04:21:29 -0000
Hi Thomas, Thanks for your comments again. I have incorporated them except the minor comment of removing a sentence in section 9. Previously, section 9 had a repetitive text which I replaced it with a reference to section 5.1.3 which I think should be kept - i.e., if someone is jumping directly to mcast handling section (sec 9), it is useful to have a reference to sec 5.1.3 on how Eth tag field is set. Cheers, Ali On 6/15/16, 7:21 AM, "Thomas Morin" <thomas.morin@orange.com> wrote: >Sounds good. > >Thanks, > >-Thomas > > >2016-06-15, John E Drake: >> Thomas, >> >> Comments inline. >> >> Yours Irrespectively, >> >> John >> >>> -----Original Message----- >>> From: Thomas Morin [mailto:thomas.morin@orange.com] >>> Sent: Wednesday, June 15, 2016 8:47 AM >>> To: John E Drake; Rabadan, Jorge (Nokia - US); BESS; >>>draft-ietf-bess-evpn- >>> overlay@tools.ietf.org; Ali Sajassi (sajassi) >>> Subject: draft-ietf-bess-evpn-overlay / section 5.1.3 vs. section 9 >>>(was Re: [Idr] draft-ietf- >>> bess-evpn-overlay vs. draft-ietf-idr-tunnel-encaps) >>> >>> Hi John, Ali, >>> >>> Through the discussion below it appeared that section 9 and section >>> 5.1.3 needed adjustments to be brought in sync, and indeed there were >>>some changes in >>> last revision. >>> >>> However, I don't think the cleanup/precision is complete yet: >>> - section 5.1.3 says "the MPLS label field in the [...] Inclusive >>>Multicast Ethernet Tag routes is >>> used to carry the VNI" although the "Inclusive Multicast Ethernet Tag >>>Route" has no "MPLS >>> label field" >>> - (directly related to the above) none of these section talks about >>>using the MPLS field of >>> the PMSI Tunnel Attribute as the VNI, although the discussion below >>>concluded that it is >>> what implementations actually do >> >> >> [JD] Accordingly, and specifically to support the option of locally >>assigned VNIs, the MPLS label1 field in the MAC Advertisement route, the >>MPLS label field in the Ethernet AD per EVI route, and the MPLS label >>field in the PMSI Tunnel Attribute of the Inclusive Multicast Ethernet >>Tag route are used to carry the VNI. >> >> >>> - also, section 9 now says "The Ethernet Tag field of this route is >>>set as described in section >>> 5.1.3.", but I find this sentence useless and redundant (precisely >>>because 5.1.3 already says >>> it and nothing would indicate that section 9 would be exempt of what >>>5.1.3 says) >> >> >> [JD] We should strike the sentence. >> >> >>> >>> Additionally, it occurred to me that "the MPLS field" is not, strictly >>>speaking, unambiguous >>> for MAC Advertisement routes, because the route actually has two MPLS >>>fields. The text >>> should just say "MPLS Label1 field" for the MAC/IP advertisement route. >> >> >> [JD] See above. >> >> >>> >>> Best, >>> >>> -Thomas >>> >>> >>> 2016-05-04, John E Drake: >>>> Jorge, >>>> >>>> We put the VNI value in the MPLS label field of the PMSI attribute >>>>for all service types, >>> and we put a value in the Ethernet Tag field following the rules for >>>each service type as >>> described in 5.1.3 >>>(https://tools.ietf.org/html/draft-ietf-bess-evpn-overlay-02#section- >>> 5.1.3). >>>> >>>> You're right that we need to clean up section 9. >>>> >>>> Yours Irrespectively, >>>> >>>> John >>>> >>>>> -----Original Message----- >>>>> From: Rabadan, Jorge (Nokia - US) [mailto:jorge.rabadan@nokia.com] >>>>> Sent: Wednesday, May 04, 2016 3:53 PM >>>>> To: John E Drake; EXT - thomas.morin@orange.com; BESS; IDR; >>>>> draft-ietf-bess-evpn- overlay@tools.ietf.org; Ali Sajassi (sajassi) >>>>> Subject: Re: [Idr] draft-ietf-bess-evpn-overlay vs. >>>>> draft-ietf-idr-tunnel-encaps >>>>> >>>>> Hi John, >>>>> >>>>> About this: >>>>> >>>>> [JD] For the IMET route the MPLS label field is carried in the PMSI >>>>> attribute. I think we need to ask everyone whether they used the >>>>> Ethernet Tag or the PMSI attribute to carry the VNI >>>>> >>>>> >>>>> In case it helps, I¹ve seen a few implementations running and they >>>>> all encode the VNI in the MPLS label field in the PTA. And a couple >>>>> of them, encode the VNI in the ethernet-tag, in addition to the MPLS >>>>> label in the PTA. In any case, I think section 9 contradicts section >>>>>5.1.3 and should be >>> clarified. >>>>> >>>>> "5.1.3 Constructing EVPN BGP Routes >>>>> <snip> >>>>> the MPLS label field in the MAC Advertisement, Ethernet AD per EVI, >>>>> and **Inclusive Multicast Ethernet Tag** routes is used to carry the >>>>>VNI or VSID." >>>>> >>>>> Thanks. >>>>> Jorge >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> On 5/4/16, 8:34 PM, "EXT John E Drake" <jdrake@juniper.net> wrote: >>>>> >>>>>> Thomas and Jorge, >>>>>> >>>>>> Snipped, comments inline. >>>>>> >>>>>> Yours Irrespectively, >>>>>> >>>>>> John >>>>>> >>>>>>>> >>>>>>>> draft-ietf-bess-evpn-overlay (see section 9) relies on the BGP >>>>>>>> Encapsulation extended to encode the tunnel encap to use for BUM >>>>>>>> traffic, but contrary to other E-VPN routes, relies on the >>>>>>>> Ethernet Tag field of the NLRI to encode the VNI/VSID. >>>>>>> >>>>>>> [JORGE] This is certainly a leftover from an old version where the >>>>>>> VNI/VSID was encoded in the ethernet tag for all the routes. The >>>>>>> VNI should be encoded in the Label field in all the routes. This >>>>>>>has to be corrected. >>>>>>> >>>>>>> In fact, section 5.1.3 says: >>>>>>> >>>>>>> 5.1.3 Constructing EVPN BGP Routes >>>>>>> >>>>>>> <snip> >>>>>>> >>>>>>> Accordingly, and >>>>>>> specifically to support the option of locally assigned VNIs, >>>>>>>the MPLS >>>>>>> label field in the MAC Advertisement, Ethernet AD per EVI, and >>>>>>> Inclusive Multicast Ethernet Tag routes is used to carry the >>>>>>>VNI or >>>>>>> VSID. For the balance of this memo, the MPLS label field will >>>>>>>be >>>>>>> referred to as the VNI/VSID field. The VNI/VSID field is used >>>>>>>for >>>>>>> both local and global VNIs/VSIDs, and for either case the >>>>>>>entire 24- >>>>>>> bit field is used to encode the VNI/VSID value. >>>>>>> >>>>>>> <snip> >>>>>> >>>>>> >>>>>> [JD] For the IMET route the MPLS label field is carried in the PMSI >>>>>> attribute. I think we >>>>> need to ask everyone whether they >>>>>> used the Ethernet Tag or the PMSI attribute to carry the VNI >>>>>> >>>>>> >>>>>>>>> >>>>>>>>> There are minor things that could be improved in >>>>>>>>> draft-ietf-bess-evpn-overlay wrt. consistency with >>>>>>>>> draft-ietf-idr-tunnel-encaps : >>>>>>>>> >>>>>>>>> * since draft-ietf-idr-tunnel-encaps will deprecate RFC5512, it >>>>>>>>> would be better that draft-ietf-bess-evpn-overlay refers to >>>>>>>>> draft-ietf-idr-tunnel-encaps and not anymore to RFC5512. >>>>>>> >>>>>>> [JORGE] I agree, as long as draft-ietf-idr-tunnel-encaps keeps the >>>>>>> encapsulation extended community. There are a few implementations >>>>>>> using this community and it is enough when only the encapsulation >>>>>>>type is needed. >>>>>> >>>>>> >>>>>> [JD] I agree and the tunnel encaps draft does keep the EC >>>>>> >>>>>> >>>>>>> >>>>>>>>> >>>>>>>>> * I think it would be better to avoid the explicit list of encap >>>>>>>>> types in section 5.1.3, and rather refer to >>>>>>>>> draft-ietf-idr-tunnel-encaps instead >>>>>>> >>>>>>> [JORGE] I agree. >>>>>> >>>>>> >>>>>> [JD] According to IANA, it allocated the five tunnels types to the >>>>>> overlay draft so I think we need to keep them >>>>>> >>>>>> >>>>>>> >>>>>>>>> * the following minor modification was proposed, but not yet >>>>>>>>>incorporated: >>>>>>>>> >>>>>>>>> John Drake, 2015-11-13 (to BESS ML): >>>>>>>>>> For the overlay draft, replace this text in section 5.1.3: >>>>>>>>>> >>>>>>>>>> "If the BGP Encapsulation extended community is not present, >>>>>>>>>> then the default MPLS encapsulation or a statically configured >>>>>>>>>> encapsulation is assumed." >>>>>>>>>> >>>>>>>>>> With the following: >>>>>>>>>> >>>>>>>>>> "Note that the MPLS encapsulation tunnel type is needed in >>>>>>>>>> order to distinguish between an advertising node that only >>>>>>>>>> supports non-MPLS encapsulations and one that supports MPLS and >>>>>>>>>> non-MPLS encapsulations. An advertising node that only >>>>>>>>>> supports MPLS encapsulation does not need to advertise any >>>>>>>>>> encapsulation tunnel types; i.e., if the BGP Encapsulation >>>>>>>>>> extended community is not present, then either MPLS >>>>>>>>>> encapsulation or a statically configured encapsulation is >>>>>>>>>>assumed." >>>>>>>>> >>>>>>>>> I think this change is useful and should be incorporated, >>>>>>>>> although skipping the last sentence would be wise if the full >>>>>>>>> list of tunnel types is removed. >>>>>> >>>>>> >>>>>> [JD] Fine with me either w/ or w/o the last sentence >>>>>> >>>>>> >> >
- Re: [bess] draft-ietf-bess-evpn-overlay / section… John E Drake
- Re: [bess] draft-ietf-bess-evpn-overlay / section… Thomas Morin
- [bess] draft-ietf-bess-evpn-overlay / section 5.1… Thomas Morin
- [bess] draft-ietf-bess-evpn-overlay vs. draft-iet… Thomas Morin
- Re: [bess] draft-ietf-bess-evpn-overlay vs. draft… Thomas Morin
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Rabadan, Jorge (Nokia - US)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Rabadan, Jorge (Nokia - US)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Ravi Shekhar
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Rabadan, Jorge (Nokia - US)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Ravi Shekhar
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Rabadan, Jorge (Nokia - US)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… thomas.morin
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Rabadan, Jorge (Nokia - US)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… thomas.morin
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Ali Sajassi (sajassi)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Ali Sajassi (sajassi)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… thomas.morin
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Thomas Morin
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Thomas Morin
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Ali Sajassi (sajassi)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] draft-ietf-bess-evpn-overlay / section… Ali Sajassi (sajassi)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Ali Sajassi (sajassi)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… thomas.morin
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Ali Sajassi (sajassi)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Martin Vigoureux
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Ali Sajassi (sajassi)
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… John E Drake
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Martin Vigoureux
- Re: [bess] [Idr] draft-ietf-bess-evpn-overlay vs.… Thomas Morin