Re: [bess] Chair review of draft-ietf-bess-evpn-bum-procedure-updates

"Jeffrey (Zhaohui) Zhang" <zzhang@juniper.net> Mon, 18 November 2019 08:19 UTC

Return-Path: <zzhang@juniper.net>
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 AF3DA1200A1; Mon, 18 Nov 2019 00:19:48 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.7
X-Spam-Level:
X-Spam-Status: No, score=-2.7 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=juniper.net header.b=d49m/3de; dkim=pass (1024-bit key) header.d=juniper.net header.b=biCWlb7g
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 W98W4ld5-_0I; Mon, 18 Nov 2019 00:19:46 -0800 (PST)
Received: from mx0a-00273201.pphosted.com (mx0a-00273201.pphosted.com [208.84.65.16]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 974C11200B6; Mon, 18 Nov 2019 00:19:46 -0800 (PST)
Received: from pps.filterd (m0108158.ppops.net [127.0.0.1]) by mx0a-00273201.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id xAI8HvnU023850; Mon, 18 Nov 2019 00:19:45 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=juniper.net; h=from : to : subject : date : message-id : references : in-reply-to : content-type : mime-version; s=PPS1017; bh=KqQWd1MM0M7brzqDMh6VvI+dSe7DfsEszZH5TXBJ8g0=; b=d49m/3de7RxVG2UTOBeKT4OjNqtVp/Khv/vVNwXECkufuM5zbh1ctnSUn4QqxZIh2Lfr qF/Lx8/ne8a7PGHkKR2XOLvxbZjZEZhBJsl5FthEKwmq7HDFay0iiA6CVVLXssMBDQQQ frd5pdNPMiC8kdZTTHTGSDxt/Zg4zt33xuwU/B4939o8R9VuDMJEFWyHaaHG1N6d6rvK MmQzPmXMHHf397Ok7SXp+46oe9QXSmuUigdoAn+8NOd5wPIq0q5vhhBsssaKTQx7Tp5z oJZ4QMMCcT0B9VrpKrUiIH/VxceJPdeXzU/RaRHYWiNp5ShakgEXY1DlVSA4sYiL4Twa pw==
Received: from nam05-co1-obe.outbound.protection.outlook.com (mail-co1nam05lp2058.outbound.protection.outlook.com [104.47.48.58]) by mx0a-00273201.pphosted.com with ESMTP id 2wadpsabns-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 18 Nov 2019 00:19:45 -0800
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=G/uZcI+41R+1NgH9y2OZZNv//dzEV76gbHNUWUM4Qb5q7srlujxP/CMIjHtN8UYr4zYbEWawsME3Al/aM4e4w3hcoNAUDL8ZGmSkTwXgjNdxCvaYwXnePt8venBw4ZPhG3GyrhEe2Fmgsh0D1vuKoXpapFbBiTs7CytBuyNluOPlcO7yeL7HVfcIVYKENDNB3BraTbPdNXnKjrb5abbjH1wwJiAZ0HAaKjA+R9TitYhsWHTKEqQJZuYm1w+9dZR7txK73mCfsGL6Anw97LKAC0BZ1pcYByxTgfA77GB+8q//RVQKhFxlWO8SxRDTosW4cS87ouICa6r8LPoIQA6guQ==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=KqQWd1MM0M7brzqDMh6VvI+dSe7DfsEszZH5TXBJ8g0=; b=i8DjwFGiOOCb4fQG87wVGL3y+RvaWJIsmFJzOHHo+twhz3ehLiesJMeU03m4NC7seJej/3w3mU3+fWBeixL/kejMlQepqbmhKHw/MjAc8WdN2gP3bz/bIktl7KvNLsUR6MvLEfn+AT4FpKlsZ4XyOqJl2hW/QVtNoiw4ZD2OsVdQXnCqKXsMhwow6DmtyVT0IyMwt254SC81Xv3f3zrj9aHE4i/XnocdLBbV/VYh6X3o5M3XS5B1c0wp1cC+fqVVRas+pLCbw50Py1+cQEBtE6Ox6yduhs1vI+HzuHp/OqIdUWgAnWNiCUBy+xUXwjS/GujcvTqu4yEKHuQ1oEzUtQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=juniper.net; dmarc=pass action=none header.from=juniper.net; dkim=pass header.d=juniper.net; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=juniper.net; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=KqQWd1MM0M7brzqDMh6VvI+dSe7DfsEszZH5TXBJ8g0=; b=biCWlb7glyGucDxJTv7xOazqg8VNbrrlopXN9GYCNPHYizc9T00Nh+n2rq1CzaegqqXg7QBfyXI2cfIClHxGuoOf5CL3/btHXYB5TIkkbSemXuaxHdbkXywpSelxyjRckSED4KeJZVXcdBI+W6b9O/T8TCZAlubEpZDFufFaGSE=
Received: from CY4PR05MB3637.namprd05.prod.outlook.com (10.171.248.25) by CY4PR05MB2821.namprd05.prod.outlook.com (10.169.187.11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2474.10; Mon, 18 Nov 2019 08:19:43 +0000
Received: from CY4PR05MB3637.namprd05.prod.outlook.com ([fe80::e17b:1a81:f64c:f03d]) by CY4PR05MB3637.namprd05.prod.outlook.com ([fe80::e17b:1a81:f64c:f03d%4]) with mapi id 15.20.2474.012; Mon, 18 Nov 2019 08:19:43 +0000
From: "Jeffrey (Zhaohui) Zhang" <zzhang@juniper.net>
To: "slitkows.ietf@gmail.com" <slitkows.ietf@gmail.com>, "bess@ietf.org" <bess@ietf.org>, "draft-ietf-bess-evpn-bum-procedure-updates@ietf.org" <draft-ietf-bess-evpn-bum-procedure-updates@ietf.org>
Thread-Topic: Chair review of draft-ietf-bess-evpn-bum-procedure-updates
Thread-Index: AdWaMbQQDiEFR7jeRl6keTOttTevBgDl0ClQ
Date: Mon, 18 Nov 2019 08:19:42 +0000
Message-ID: <CY4PR05MB3637F13CBEAD9D3A0190E455D44D0@CY4PR05MB3637.namprd05.prod.outlook.com>
References: <018e01d59a38$becade70$3c609b50$@gmail.com>
In-Reply-To: <018e01d59a38$becade70$3c609b50$@gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
dlp-product: dlpe-windows
dlp-version: 11.2.0.14
dlp-reaction: no-action
x-originating-ip: [66.129.242.12]
x-ms-publictraffictype: Email
x-ms-office365-filtering-ht: Tenant
x-ms-office365-filtering-correlation-id: e15295a0-e248-4b63-a410-08d76c0010b1
x-ms-traffictypediagnostic: CY4PR05MB2821:
x-ms-exchange-purlcount: 3
x-microsoft-antispam-prvs: <CY4PR05MB2821885A9A43CF2B02C2C65AD44D0@CY4PR05MB2821.namprd05.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 0225B0D5BC
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(366004)(346002)(136003)(376002)(39860400002)(396003)(199004)(189003)(9326002)(236005)(54896002)(5660300002)(55016002)(15650500001)(81156014)(86362001)(66066001)(2501003)(25786009)(6436002)(6306002)(110136005)(9686003)(316002)(52536014)(81166006)(2201001)(99286004)(229853002)(8676002)(14444005)(256004)(6506007)(53546011)(66946007)(478600001)(71190400001)(71200400001)(102836004)(561944003)(66446008)(606006)(8936002)(66476007)(66556008)(64756008)(14454004)(476003)(2906002)(966005)(26005)(6246003)(2420400007)(3846002)(7736002)(74316002)(11346002)(446003)(33656002)(186003)(76116006)(76176011)(7110500001)(6116002)(790700001)(7696005)(486006); DIR:OUT; SFP:1102; SCL:1; SRVR:CY4PR05MB2821; H:CY4PR05MB3637.namprd05.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: juniper.net does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: pv4XC4NgYFB6zkL+oejCdhU6NGqBa/X/Rl8K9etuv2cg7kTWlLDDUnIFLcNcnfiU3u6eDrytJ0QyC6U+7Sf2Ys0F9xfv7oIbiV+BJIZBd6z0sBwbZW1W7SkAVBf7ihWvOoauHgKoZr71o79X71o+1kIlxhsvj0pGYHgP2+sr2kVDVlEQrXDK+RlD0CKgumRI/6YwxuQZm5KfJ7OBUzCoOb/1Wey1FmHCTwQxMHIoXS+pm5mgDbuDLtgG2e1i0X+WpebzyaejwY3aSNPMbeQKQ5ZHTecoc6Lkz5iX+Ro7tRnGXwMmZGyv3j75XMdAvqqJ+eihmhDzdg9h+o98uS/hA/zR/A4VLmD1hrbFmg3AirdP8CMxH0/CBcJWk86WWgp0EJKjiqBCKXFEIOs+vmb7ZczPfURoHVT8KRMdeVSNlrMa1J9yACWQBTTWWits1JHRK0+/ByRLbSJZVIsRC640nKH/CMtdfMNVIvuSreCqHcQ=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_CY4PR05MB3637F13CBEAD9D3A0190E455D44D0CY4PR05MB3637namp_"
MIME-Version: 1.0
X-OriginatorOrg: juniper.net
X-MS-Exchange-CrossTenant-Network-Message-Id: e15295a0-e248-4b63-a410-08d76c0010b1
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Nov 2019 08:19:42.9443 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: bea78b3c-4cdb-4130-854a-1d193232e5f4
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: n9phQd2xSKYCFQ4fKFRhyRiFncgFJ/0bIy3hoMK9bH3IpJps1wdF1JomXMCaMtiTdoYz1QcFuNdFFTPNocGyNA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR05MB2821
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.95,18.0.572 definitions=2019-11-18_01:2019-11-15,2019-11-17 signatures=0
X-Proofpoint-Spam-Details: rule=outbound_spam_notspam policy=outbound_spam score=0 lowpriorityscore=0 suspectscore=0 priorityscore=1501 phishscore=0 adultscore=0 spamscore=0 mlxscore=0 bulkscore=0 clxscore=1011 mlxlogscore=999 impostorscore=0 malwarescore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-1910280000 definitions=main-1911180074
Archived-At: <https://mailarchive.ietf.org/arch/msg/bess/eI8fhrD8bq-psJtp8pGaf3Oz9Ow>
Subject: Re: [bess] Chair review of draft-ietf-bess-evpn-bum-procedure-updates
X-BeenThere: bess@ietf.org
X-Mailman-Version: 2.1.29
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, 18 Nov 2019 08:19:49 -0000

Hi Stephane,

Thanks for your review and comments/suggestions. Please see zzh> below (and diff in https://www.ietf.org/rfcdiff?url2=draft-ietf-bess-evpn-bum-procedure-updates-08).

From: slitkows.ietf@gmail.com <slitkows.ietf@gmail.com>
Sent: Wednesday, November 13, 2019 9:11 PM
To: bess@ietf.org; draft-ietf-bess-evpn-bum-procedure-updates@ietf.org
Subject: Chair review of draft-ietf-bess-evpn-bum-procedure-updates

Hi,

Before moving forward to IESG, here is my review of the document:

Section 2:

"Note that these are to be applied

   to EVPN only, even though sometimes they may sound to be updates to

   [RFC7117<https://urldefense.com/v3/__https:/tools.ietf.org/html/rfc7117__;!8WoA6RjC81c!Rt3VAuyRgtWfhTdkP8k2Y7tvamLYm-MqQ2q6tqzgSHpBFeYkTX7kbAqUhNbNeio1$>] or [RFC7524<https://urldefense.com/v3/__https:/tools.ietf.org/html/rfc7524__;!8WoA6RjC81c!Rt3VAuyRgtWfhTdkP8k2Y7tvamLYm-MqQ2q6tqzgSHpBFeYkTX7kbAqUhENk13cl$>]$>]. >
The second part of the sentence is not really appropriate for a standard document. The text should always be crystal clear that it applies to EVPN only, reuse the procedures from other RFCs with the associated modifications. But you must never talk about updating VPLS RFCs, if you don't.

Zzh> The second part of that sentence is meant to say that they MIGHT SOUND like updates to 7117/7524 but they're not. I changed it to "and not updates to [RFC7117] or [RFC7524]".

Section 3:
Not all the route types are coming from RFC7432, could you provide a reference for route-types that are not defined in RFC7432 ?

Zzh> Done.

Section 3.1:
As you say in the text, the extended community is not an attribute here. Wouldn't it be better to rename it as Region ID, telling then in the text that it is encoded similarly to an extended community using type/subtype/...
In case you agree, update of 6.2 is necessary.


Zzh> Done.


Section 4:
s/and an receiving NVE/and a receiving NVE
s/In a nut shell/In a nutshell
s/S-SPMSI/S-PMSI

s/an egress NVE may omit the Leaf/an egress NVE MAY omit the Leaf
s/if it already advertises/if it has already advertised
s/and the source NVE will use that/and the source NVE MUST use that

zzh> Fixed.

Section 5.1
In the proposal text changes for 7.2.2.4 in RFC7117, please use normative language s/must/MUST

Zzh> Done.

Section 5.2:
I don't understand this sentence (ps: I'm not telling the sentence is wrong) :

"Note

   that in case of Ingress Replication, when an ASBR re-advertises IBGP

   I-PMSI A-D routes, it MUST advertise the same label for all those for

   the same Ethernet Tag ID and the same EVI. "
Could you please explain me ? Do you mean that ASBR allocate the same label for different routes with same ETAG/EVI ?

Zzh> Right - as you understood that ASBR1/3 below will advertise the same label for IMET routes from PE2/PE3.
Zzh> I did change "re-advertise IBGP I-PMSI A-D routes" to "re-advertise IMET A-D routes to IBGP peers".

Consider the following setup

PE1 -(CORE)-- ASBR1 ---- ASBR2 ---(CORE)  ---PE2
                      --ASBR3 --- ASBR4 ---         ------- PE3

PE1 being the source of BUM.

PE2 and PE3 send IMET route for EVI1/ETAG1 respectively with label 20 and 30 (IR assumed).
ASBR2 and ASBR4 sends the route as Inter-AS A-D routes setting themselves as NH and using PTA type set to IR. Does it set a different label value for each Inter-AS A-D route ?

Zzh> ASBR2/4 does not need to modify the label in PE2/3's IMET routes when they're re-advertised to ASBR1/3. The reason is that for traffic from beyond ASRBR1/3, the PE2/PE3's IMET routes will not be used by ASBR1/3. Rather, the Leaf A-D routes from ASBR2/4 (triggered by PE1's IMET route) will be used. I did realize that additional changes need to be made in section 5.1 (see diffs for the new revision).

I understand that the behavior described above (from your draft) applies to ASBR1 and ASBR3 which will readvertise the same label for both routes but why not doing the label aggregation at ASBR2/4 ?

Zzh> As mentioned above, there is no need on ASBR2/4.

The text talks about I-PMSI A-D routes, but do you confirm that you talk about Inter-AS I-PMSI A-D routes ?

Zzh> When I-PMSI A-D route is used w/o "Inter-AS", it means "Intra-AS I-PMSI" which is IMET route in case of EVPN and type 1 route in case of MVPN. However, when not doing per-AS/region aggregation, an "Intra-AS I-PMSI" route is also re-advertised across AS boundaries, while RFC7117 refers any across-AS routes as "Inter-AS AD routes".
Zzh> That's why I mentioned in this document that "Intra-AS I-PMSI" routes are better called per-PE I-PMSI routes while MVPN's "Inter-AS I-PMSI" routes are called per-AS I-PMSI routes and the name "per-region I-PMSI" route is introduced in this document (since we're talking about aggregation at "region" level).
Zzh> Given all these, in this section I changed to using IMET route instead, which is a per-PE I-PMSI route for EVPN (and this section's context is w/o per-region aggregation).

In addition, you should use normative language in :

"When an ingress PE builds

   its flooding list, multiple routes may have the same (nexthop, label)

   tuple and they will only be added as a single branch in the flooding

   list. >

zzh> The "may" is changed to "might" (it should not change to "MAY"). The "will" is changed to "MUST".
Zzh> Thanks!
Zzh> Jeffrey