Re: [mpls] MPLS-RT review of draft-hegde-mpls-spring-epe-oam

Shraddha Hegde <shraddha@juniper.net> Fri, 06 March 2020 10:42 UTC

Return-Path: <shraddha@juniper.net>
X-Original-To: mpls@ietfa.amsl.com
Delivered-To: mpls@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 5D2263A0C63; Fri, 6 Mar 2020 02:42:49 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.099
X-Spam-Level:
X-Spam-Status: No, score=-2.099 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, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=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=PkB+KtU5; dkim=pass (1024-bit key) header.d=juniper.net header.b=d+povC2w
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 DYX-fVIvXVtL; Fri, 6 Mar 2020 02:42:46 -0800 (PST)
Received: from mx0b-00273201.pphosted.com (mx0b-00273201.pphosted.com [67.231.152.164]) (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 960683A0C5E; Fri, 6 Mar 2020 02:42:46 -0800 (PST)
Received: from pps.filterd (m0108163.ppops.net [127.0.0.1]) by mx0b-00273201.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id 026Ab95I006376; Fri, 6 Mar 2020 02:42:31 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=juniper.net; h=from : to : cc : subject : date : message-id : references : in-reply-to : content-type : mime-version; s=PPS1017; bh=XUqFibq0TJWV9KQ0gqyyajeMin/oM91ZepM76jTSzoc=; b=PkB+KtU56ycuP4234w9yvm0SeGBBLXDhho2uy1pEO1+6LA39DcDRX8wyJW1ykFAzn/3V stDVgnTU/2L1T8llLxsOuwSq3ro+rGJ5x8w7jnOWUox3beaKYuqnU0dUIkW8C56vm0d/ PTym8w2riWuWxbO/bQ7us9gQN0ED7hpiat8ZPPi+wv+J+SKOrfNlL7w5Xe2pMje2PtA8 v8zm2ULfe8S7jhOt9llCAyB9EcZRnMgoBI3HsnjxXmLlYAXmDE9GL2q+GVvvVQGvx7m2 oLq7nGxGWNPMK8VBvjXFq6EqkX73DDKckexXAj03ECmxH1S/L4doEjn52C9JYwI0y1X+ fA==
Received: from nam12-mw2-obe.outbound.protection.outlook.com (mail-mw2nam12lp2040.outbound.protection.outlook.com [104.47.66.40]) by mx0b-00273201.pphosted.com with ESMTP id 2ykcss0rvx-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Fri, 06 Mar 2020 02:42:31 -0800
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; =?utf-8?q?b=3DReGB3v6Bzj4Lh/znq8pIG925amhduNUOG6wnhMn+aaeq5Bq6GHHK78i1jvgUO?= =?utf-8?q?NDgBm16yqnkVNjvkihCwkvvwnTd2sxXoSkXH7VvAZOSX0rDK+A2c1ER1IflUA4kIT?= =?utf-8?q?AqQ4yozhSAlWYjBUkmpIkPtcdyMHOWW2wjaPET/6swTINiIf2UTvQ93bDQ5hiA2jz?= =?utf-8?q?yK4zQOrbIZ8gbEnBeK2kaMoW2njZiQVVnyVFEDcNsEJZofQtcAh6CTR6sCFlTFEVm?= =?utf-8?q?hZl0TYWOFWn4b/X2r5dpB6Uao/gM2S+2tQfTcEwhOSdpqhsPJDeTOQpjyKr5MpE9V?= =?utf-8?q?ckmlsLiRNbPioG5fH/bpg=3D=3D?=
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; =?utf-8?q?h=3DFrom=3ADate=3ASubject=3AMessage-ID=3ACont?= =?utf-8?q?ent-Type=3AMIME-Version=3AX-MS-Exchange-SenderADCheck=3B?= =?utf-8?q?bh=3DXUqFibq0TJWV9KQ0gqyyajeMin/oM91ZepM76jTSzoc=3D=3B_b=3DUyMrXo?= =?utf-8?q?h9Og2a5Yl0LuzQUipbNZx9KH5W6v3fT27FdHBxlypkT4vTRyaFmRSBMwg8vPkZxZv?= =?utf-8?q?g5y0gteXH5BxKhy0BoVoIT+reBq/XQbqFcRaSB0Jsd9AznHsucY33b8mvoDg7ikxT?= =?utf-8?q?8nGEucswPq1mzTrI/bmbLsnj9HePLMGCe0fHC6Ys952xGuvd/CE/czKncIytFAiI+?= =?utf-8?q?ODnRIkHohxQsV92RD+HjIC3PYIvuOxI+OD2w9WcVl0XQ0oJcjyaAss2UOuq3Yk4dh?= =?utf-8?q?mLe5ZYSk73H0/3NioDs9JZsLZunLuHlrcZlVePt3rcDWF3Zx8RxKbqEx6/tGvPqNM?= =?utf-8?q?MRjKmvgguFw=3D=3D?=
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; =?utf-8?q?h=3DFrom=3ADate=3ASubject=3AMessage-ID=3AContent-Typ?= =?utf-8?q?e=3AMIME-Version=3AX-MS-Exchange-SenderADCheck=3B?= =?utf-8?q?bh=3DXUqFibq0TJWV9KQ0gqyyajeMin/oM91ZepM76jTSzoc=3D=3B_b=3Dd+povC?= =?utf-8?q?2wqKTx4cASVdnbw1T3p6vFh1BSTG80bW1NJ5e9TUkW/XV0sDWkXc+QgJrfBBnUdk9?= =?utf-8?q?VjvTIsf/F5IKfLr+loGpPBgP3oGQYRbPIceugngaAxNUNYb/UAnnROx2UF/jIlS52?= =?utf-8?q?sRmRkxifrTST/sZT4Pda/r8XITyahkxjQSs=3D?=
Received: from BYAPR05MB3943.namprd05.prod.outlook.com (2603:10b6:a02:8b::18) by BYAPR05MB4871.namprd05.prod.outlook.com (2603:10b6:a03:41::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2793.11; Fri, 6 Mar 2020 10:42:28 +0000
Received: from BYAPR05MB3943.namprd05.prod.outlook.com ([fe80::9415:edbd:33a5:5708]) by BYAPR05MB3943.namprd05.prod.outlook.com ([fe80::9415:edbd:33a5:5708%3]) with mapi id 15.20.2814.007; Fri, 6 Mar 2020 10:42:28 +0000
From: Shraddha Hegde <shraddha@juniper.net>
To: Italo Busi <Italo.Busi@huawei.com>, "draft-hegde-mpls-spring-epe-oam@ietf.org" <draft-hegde-mpls-spring-epe-oam@ietf.org>, "mpls-chairs@ietf.org" <mpls-chairs@ietf.org>, Mach Chen <mach.chen@huawei.com>
CC: "mpls@ietf.org" <mpls@ietf.org>, "Bocci, Matthew (Nokia - GB)" <matthew.bocci@nokia.com>, Alexander Vainshtein <Alexander.Vainshtein@ecitele.com>, Sam Aldrin <aldrin.ietf@gmail.com>
Thread-Topic: MPLS-RT review of draft-hegde-mpls-spring-epe-oam
Thread-Index: AQHV4vKl4WCDGjdC3kq5Rs08STypkqgtxmYAgA2xXwA=
Date: Fri, 6 Mar 2020 10:42:27 +0000
Message-ID: =?utf-8?q?=3CBYAPR05MB3943F01C099FF220BA553958D5E30=40BYAPR05MB3?= =?utf-8?q?943=2Enamprd05=2Eprod=2Eoutlook=2Ecom=3E?=
References: <2141e262-752f-0c7e-fdcb-03aea45e9aa1@pi.nu> <5aa73bfd247040d4bae8131f4eb43b5f@huawei.com>
In-Reply-To: <5aa73bfd247040d4bae8131f4eb43b5f@huawei.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
msip_labels: =?utf-8?q?MSIP=5FLabel=5F9784d817-3396-4a4f-b60c-3ef6b345fe55?= =?utf-8?q?=5FActionId=3D5057a3a5-fb0b-4960-b047-00000b9a32ab=3B?= MSIP_Label_9784d817-3396-4a4f-b60c-3ef6b345fe55_ContentBits=0; MSIP_Label_9784d817-3396-4a4f-b60c-3ef6b345fe55_Enabled=true; MSIP_Label_9784d817-3396-4a4f-b60c-3ef6b345fe55_Method=Standard; MSIP_Label_9784d817-3396-4a4f-b60c-3ef6b345fe55_Name=Juniper Business Use Only; MSIP_Label_9784d817-3396-4a4f-b60c-3ef6b345fe55_SetDate=2020-03-06T10:06:00Z; =?utf-8?q?MSIP=5FLabel=5F9784d817-3396-4a4f-b60c-3ef6b345fe55=5FSiteId=3Db?= =?utf-8?q?ea78b3c-4cdb-4130-854a-1d193232e5f4=3B?=
dlp-product: dlpe-windows
dlp-version: 11.2.0.14
dlp-reaction: no-action
x-originating-ip: [116.197.184.12]
x-ms-publictraffictype: Email
x-ms-office365-filtering-ht: Tenant
x-ms-office365-filtering-correlation-id: 2a72797e-4040-4e94-a5cf-08d7c1bb10c6
x-ms-traffictypediagnostic: BYAPR05MB4871:
x-microsoft-antispam-prvs: =?utf-8?q?=3CBYAPR05MB48712B3964C687AF28CBA6F8D5E?= =?utf-8?q?30=40BYAPR05MB4871=2Enamprd05=2Eprod=2Eoutlook=2Ecom=3E?=
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 0334223192
x-forefront-antispam-report: SFV:NSPM; =?utf-8?q?SFS=3A=2810019020=29=284636?= =?utf-8?b?MDA5KSgzOTg2MDQwMDAwMikoMTM2MDAzKSgzNjYwMDQpKDM5NjAwMykoMzc2?= =?utf-8?b?MDAyKSgzNDYwMDIpKDE4OTAwMykoMTk5MDA0KSg4OTM2MDAyKSg2NjQ0NjAwOCko?= =?utf-8?q?66556008=29=2852536014=29=2876116006=29=289326002=29=2866476007?= =?utf-8?b?KSg2Njk0NjAwNykoNjQ3NTYwMDgpKDc2OTYwMDUpKDgxMTY2MDA2KSg4MTE1?= =?utf-8?b?NjAxNCkoNTQ5MDYwMDMpKDExMDEzNjAwNSkoODY3NjAwMikoMzE2MDAyKSg0?= =?utf-8?q?326008=29=2833656002=29=286506007=29=2853546011=29=2871200400001?= =?utf-8?b?KSgxODYwMDMpKDU2NjAzMDAwMDIpKDk2ODYwMDMpKDU1MDE2MDAyKSgyNjAwNSko?= =?utf-8?q?66574012=29=2886362001=29=282906002=29=28478600001=29=3B?= DIR:OUT; SFP:1102; SCL:1; SRVR:BYAPR05MB4871; H:BYAPR05MB3943.namprd05.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A: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: =?utf-8?q?R+6IsUpCgJp3krt1yUxlLrcqa6VuYh9?= =?utf-8?q?ighnALht0aRrYqhiHxtsHa5GVr2Tzx4umCjgg46R/nWirJy6YB8EVWq80UKfR42ur?= =?utf-8?q?HEsjxpxRn2/2uVVa9XUUtFMqA/ihYdOE7SGjJYT+kCig5Qe45j8qfN1KNlrMoqKN1?= =?utf-8?q?S1sDniNWW2fezEM6jI8ReIS+JZ0s9MBXDZdGXJ/v1DnDJUO28x2zxRHCeFIFb5rUp?= =?utf-8?q?iXwXFnxixExh/eQCyh34p0Vxg6tTUPJLr1exqia4CELaPYY/KlXx5f54qAgEW2N4k?= =?utf-8?q?X7KgHoHzM24INuGGHpFumphHpm9SZpXja/MALZ9IBZMCezxLLfJ+oYX2/oRElkW1C?= =?utf-8?q?Htq/diBdAVnK/5OvIanEcnN2Rc51LKeXO/Sm9BXkuL9XBadevKXLgEzEzkjpJEyZo?= =?utf-8?q?0g77hionB+suOw/WSO8I6Q93Bkk?=
x-ms-exchange-antispam-messagedata: =?utf-8?q?FT1uaX3d+U6jT3t6krL/kCdDSS5b/F?= =?utf-8?q?T/rUa6qxrTfsZ7C8UlnSHcx4CJMtz6KmfdXzs5uEJpB5RGJZbjXUWI2uyhMpHD3pd?= =?utf-8?q?ZGcbmoKWaP+NJa4Vb5UkGqKzTjdsgRWYkzANetoZLcb6NpNpw9vrmkw=3D=3D?=
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_BYAPR05MB3943F01C099FF220BA553958D5E30BYAPR05MB3943namp_"
MIME-Version: 1.0
X-OriginatorOrg: juniper.net
X-MS-Exchange-CrossTenant-Network-Message-Id: 2a72797e-4040-4e94-a5cf-08d7c1bb10c6
X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Mar 2020 10:42:27.9674 (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: =?utf-8?q?xE/Bf7O8NEz7hjcAPHyaX?= =?utf-8?q?mTGj8nh35pALbKU3KcRQ8PVE2L0rfWL93i2nAK0NiYBBvjJHUg1m0sU6iQsWbLRlw?= =?utf-8?q?=3D=3D?=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR05MB4871
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.138, 18.0.572 definitions=2020-03-06_03:2020-03-05, 2020-03-06 signatures=0
X-Proofpoint-Spam-Details: rule=outbound_spam_notspam policy=outbound_spam score=0 mlxscore=0 spamscore=0 clxscore=1011 malwarescore=0 impostorscore=0 lowpriorityscore=0 mlxlogscore=913 priorityscore=1501 suspectscore=0 bulkscore=0 adultscore=0 phishscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2001150001 definitions=main-2003060076
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/J7FwFSjY0S7KUglREtFj5KovNso>
Subject: Re: [mpls] MPLS-RT review of draft-hegde-mpls-spring-epe-oam
X-BeenThere: mpls@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Multi-Protocol Label Switching WG <mpls.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/mpls>, <mailto:mpls-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/mpls/>
List-Post: <mailto:mpls@ietf.org>
List-Help: <mailto:mpls-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/mpls>, <mailto:mpls-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 06 Mar 2020 10:42:50 -0000

Hi Italo,

Thanks for review and comments. Pls see inline…

From: Italo Busi <Italo.Busi@huawei.com>
Sent: Wednesday, February 26, 2020 10:30 PM
To: draft-hegde-mpls-spring-epe-oam@ietf.org; mpls-chairs@ietf.org; Mach Chen <mach.chen@huawei.com>
Cc: mpls@ietf.org; Bocci, Matthew (Nokia - GB) <matthew.bocci@nokia.com>om>; Alexander Vainshtein <Alexander.Vainshtein@ecitele.com>om>; Sam Aldrin <aldrin.ietf@gmail.com>
Subject: RE: MPLS-RT review of draft-hegde-mpls-spring-epe-oam

Hi all,

I have been selected as one of the  MPLS-RT reviewers of draft-hegde-mpls-spring-epe-oam.

I have reviewed the latest version of the draft which has been published two days ago (draft-hegde-mpls-spring-epe-oam-05).

I think that the document is coherent, is it useful (i.e., it addresses a real need for operational networks), and it is technically sound.

Therefore, I think that the draft is ready to be adopted as a WG document.

I have few comments that can be addressed either before or after WG adoption.

1.       The Introduction mentions the procedures defined in section 7 of RFC8287 and clarified by RFC8690.

My understanding is that RFC8690 clarifies how to use the length field with the sub-TLVs defined in section 5 of RFC8287 and therefore it is not strictly applicable to this draft.
<shraddha> yes. You are right. I updated this sentence and referred 8690 based on comment that 8287
Has an important clarification in RFC 8690 so it should be reference.

However, it would be worthwhile clarifying in section 4 of this draft how the length field should be set for the EPE SID sub-TLVs. An example would also be useful.
<shraddha> yes updated the draft.

2.       I guess that all the information elements used in the sub-TLVs defined in section 4 of this draft are those defined in draft-ietf-idr-bgpls-segment-routing-epe.
<Shraddha> Yes.

It would be worthwhile to reference draft-ietf-idr-bgpls-segment-routing-epe for these definitions.
<Shraddha> yes, normative reference added.

I guess this would also help resolving comment #1 from Sasha on link-local IPv6 addresses and unnumbered interfaces: whatever is supported by draft-ietf-idr-bgpls-segment-routing-epe should be also supported by this draft (and vice versa)
<shraddha> updated with link-local addresses FFS.

3.       In Figure 2, the “No.of IPv6 interface pairs” field seems a bit redundant since the information can be inferred from the “Length” and “No.of IPv4 interface pairs ” fields
<shraddha> Since IPv4 and ipv6 address pair may be present simultaneously, we need this field.

This is ok for me but please add some text to clarify whether the receiver should/shall perform some validation check
 <shraddha> sure will add new section in next revision.
4.       Figure 3 is not fully clear

If I understand well, the “Remote As Number”, “Remote BGP Router ID”, “No.of IPv4 interface pairs”, … fields are repeated for each element in the set.
<Shraddha> yes, these elements are repeated for each set.

It may be worthwhile to split the figure into two: one describing the TLV containing one or more “elements” and one describing the fields used in each element.
<Shraddha> Yes. updated

5.       In Figure 6, the “No.of elements in set” field as well as the “No.of IPv6 interface pairs” field of the last element in the set seem a bit redundant since the information can be inferred from the “Length” and “No.of IPv4 interface pairs ” of the last element in the set fields
<Shraddha>Since both IPv4 and IPv6 addresses pairs may be present simultaneously, number of pairs field is required.

As for comment #3 above, this is ok for me but please add some text to clarify whether the receiver should/shall perform some validation check
 <shraddha> sure will add new section in next revision.
Italo

> -----Original Message-----
> From: Loa Andersson [mailto:loa@pi.nu]
> Sent: venerdì 14 febbraio 2020 05:53
> To: Bocci, Matthew (Nokia - GB) <matthew.bocci@nokia.com<mailto:matthew.bocci@nokia.com>>; Alexander
> Vainshtein <Alexander.Vainshtein@ecitele.com<mailto:Alexander.Vainshtein@ecitele.com>>; Sam Aldrin
> <aldrin.ietf@gmail.com<mailto:aldrin.ietf@gmail.com>>; Italo Busi <Italo.Busi@huawei.com<mailto:Italo.Busi@huawei.com>>
> Cc: draft-hegde-mpls-spring-epe-oam@ietf.org<mailto:draft-hegde-mpls-spring-epe-oam@ietf.org>; mpls-chairs@ietf.org<mailto:mpls-chairs@ietf.org>
> Subject: MPLS-RT review of draft-hegde-mpls-spring-epe-oam
>
> Mathew, Sam, Sasha and Italo,
>
> You have be selected as MPLS-RT reviewers for draft-hegde-mpls-spring-epe-
> oam.
>
> Note to authors: You have been CC'd on this email so that you can know that
> this review is going on. However, please do not review your own document.
>
> Reviews should comment on whether the document is coherent, is it useful (ie,
> is it likely to be actually useful in operational networks), and is the document
> technically sound?  We are interested in knowing whether the document is
> ready to be considered for WG adoption (ie, it doesn't have to be perfect at
> this point, but should be a good start).
>
> Reviews should be sent to the document authors, WG co-chairs and WG
> secretary, and CC'd to the MPLS WG email list. If necessary, comments may be
> sent privately to only the WG chairs.
>
> If you have technical comments you should try to be explicit about what
> *really* need to be resolved before adopting it as a working group document,
> and what can wait until the document is a working group document and the
> working group has the revision control.
>
> Are you able to review this draft by Feb 28, 2020? Please respond in a timely
> fashion.
>
>
> Thanks, Loa
> (as MPLS WG chair)
> --
> --
>
>
> Loa Andersson                        email: loa@pi.nu<mailto:loa@pi.nu>
> Senior MPLS Expert
> Bronze Dragon Consulting             phone: +46 739 81 21 64