Re: [mpls] New Version Notification for draft-qu-mpls-mpls-msd-yang-01.txt

Yingzhen Qu <yingzhen.ietf@gmail.com> Wed, 04 August 2021 05:49 UTC

Return-Path: <yingzhen.ietf@gmail.com>
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 6F71E3A0D1A; Tue, 3 Aug 2021 22:49:45 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.097
X-Spam-Level:
X-Spam-Status: No, score=-2.097 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_BLOCKED=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=gmail.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 R3-XLkZA9GIQ; Tue, 3 Aug 2021 22:49:40 -0700 (PDT)
Received: from mail-pl1-x635.google.com (mail-pl1-x635.google.com [IPv6:2607:f8b0:4864:20::635]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 6A5B63A0D17; Tue, 3 Aug 2021 22:49:40 -0700 (PDT)
Received: by mail-pl1-x635.google.com with SMTP id k2so1779498plk.13; Tue, 03 Aug 2021 22:49:40 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=O7exMkPx1UOCZK2DzRLcEXhDm2brxhQtNraEEwpr5rI=; b=oWrty8cZo00vxKoug1rCqX7knfRyz4vXEcZk+iovbTxnVpQC7KLiK87STos0a+9UFk jeh4QEDu4oCRLxUaKd1zR0zOUHtIYiFJbi46xZJI2vntsQk343SZHj7u3v7ZhrOpG7Zp /oQtEuUPh+8YbbvL7dlDDAl0YQHDumFlLjDBajnA7UsBV77V6BO/+wLwM4swFH+j49b0 GTpsgK55tcqkaVHX3C8G7Orux+8EjA6QyUz4lI/76qL7VL+mU1vYd/+lyGWdl0oPXz02 g5CpDvZqGmtmzdyW8zXnea7PQNqGvy51UK6e9m2aPKkkoQY5MXPhr/mw7aVT21IpbhL5 m1pQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=O7exMkPx1UOCZK2DzRLcEXhDm2brxhQtNraEEwpr5rI=; b=KkuDZoH2s7YBo+m7zNva4I5zWW5EYDf+vyEFHciXq/aGbdC0tFioReTOy3FOY+RjBO pqH42T+BcwU5YY35yl9ILyFkcreLWsF0a9IZl06VQAk/A1hhLuWs5752Q8WaAPgjY9L5 ENHm+22EsZtrQbbNGQpZn4NELA8Rfr65Ns2pAnkTp3GSStphVkUiXYJHQi9K66TlTTrp QljJ1Nuh0m8bbWC8A29kt1oKuLdsFiyIiDDp6Ql76QgCGp573P2S/zlJnDd75Ru1BJjs +QDec196eOOowvywWTvmJTZa8yKGmMJAHC0I9onWx0lYUvQRm65aGwbNIw4cZRGlLE9N ia+A==
X-Gm-Message-State: AOAM533xYoyRtYrmvDJ0ti8WQFB9sdyKQgpb0iZ+ECJjPaFAcOYekZf6 z81ws4vqAIzuwuJ32V5Gaw==
X-Google-Smtp-Source: ABdhPJw8cL/LlFiD1NIxebQEknuS612HxdtF2V6w2puCRF4kkpINv5XosXXTURW5w1pHVBoPErb8Rw==
X-Received: by 2002:a17:902:da89:b029:12c:d398:64e8 with SMTP id j9-20020a170902da89b029012cd39864e8mr2288251plx.56.1628056179187; Tue, 03 Aug 2021 22:49:39 -0700 (PDT)
Received: from ?IPv6:2601:646:9702:c61:4d64:ee5a:8f69:e9b9? ([2601:646:9702:c61:4d64:ee5a:8f69:e9b9]) by smtp.gmail.com with ESMTPSA id o22sm1098358pfu.87.2021.08.03.22.49.38 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Tue, 03 Aug 2021 22:49:38 -0700 (PDT)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 14.0 \(3654.40.0.2.32\))
From: Yingzhen Qu <yingzhen.ietf@gmail.com>
In-Reply-To: <AM7PR07MB6248B6EC151AA6AAEA1FD7C3A0F09@AM7PR07MB6248.eurprd07.prod.outlook.com>
Date: Tue, 03 Aug 2021 22:49:37 -0700
Cc: "mpls@ietf.org" <mpls@ietf.org>, "mpls-chairs@ietf.org" <mpls-chairs@ietf.org>, "draft-qu-mpls-mpls-msd-yang@ietf.org" <draft-qu-mpls-mpls-msd-yang@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <CE6588CE-767E-424C-B4FC-C538EE6E9B65@gmail.com>
References: <162784450568.13184.7481638186028270876@ietfa.amsl.com> <BY5PR13MB304858C756AEA5D8AB06A688E1EF9@BY5PR13MB3048.namprd13.prod.outlook.com> <AM7PR07MB6248410CF95AE457D9CB47A2A0EF9@AM7PR07MB6248.eurprd07.prod.outlook.com> <337367E8-EED8-4709-B9D3-43A81A91D02B@gmail.com> <AM7PR07MB6248B6EC151AA6AAEA1FD7C3A0F09@AM7PR07MB6248.eurprd07.prod.outlook.com>
To: tom petch <ietfc@btconnect.com>
X-Mailer: Apple Mail (2.3654.40.0.2.32)
Archived-At: <https://mailarchive.ietf.org/arch/msg/mpls/Tzg90tqjeMhoTI_st3JYxKCQQPc>
Subject: Re: [mpls] New Version Notification for draft-qu-mpls-mpls-msd-yang-01.txt
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: Wed, 04 Aug 2021 05:49:46 -0000

Hi Tom,

Regarding node-msd, you can find its definition in section 2 and section 6 of RFC 8491. Draft-ietf-isis-mpls-elc (https://datatracker.ietf.org/doc/draft-ietf-isis-mpls-elc/) added “type-2” MSD-type. I will add a reference in the next version of the draft. It’s expected that there will be extensions in the future. Hence the node-msd is defined as a list.

The MSD portion was moved out of SR YANG because MSD is a characteristic of MPLS, it’s not limited to SR. 

Thanks,
Yingzhen

> On Aug 3, 2021, at 4:04 AM, tom petch <ietfc@btconnect.com> wrote:
> 
> From: Yingzhen Qu <yingzhen.ietf@gmail.com>
> Sent: 02 August 2021 23:44
> 
> Hi Tom,
> 
> Thank you for your review and comments. I have submitted version -02 to address your comments. Please see my detailed answers below.
> 
> <tp>
> 
> I was looking at draft-ietf-teas-yang-sr-topo and realised how much that had in common with this I-D, down to the same spelling mistakes!  To the TEAS WG list I posted
> 
> ========================================
> Both I-D puzzle me in the same way.  What is node-msd that it is a list keyed on msd-type?  And what is msd-type for that matter, same in both I-D?
> 
> I understand a node having an MSD, the lowest of those on a link but how can that be a list and why key it on msd-type which appears to be the signalling protocol in use?  I do not understand.
> 
> I note that in -08 node-msd was not a list and that made more sense to me.
> 
> If a list is needed, then I think that you need to say why, how it is used - or reference another SR I-D/RFC where it is explained - and why it is keyed on what it is keyed on.
> 
> Equally fundamentally, what has node-msd got to do with te or topo or MPLS or ...  I see it as a property of a node which would ideally be part of a base definition of a node even if SR was not in use or part of a base SR augmentation as opposed to what we seem to be getting of  multiple I-D creating their own definitions of node-msd.
> ===============end of quote=========================
> 
> So is this the right I-D to define node-msd?  Probably not.
> 
> Why is node-msd a list and not a single object for a single node?
> 
> You queried my reference to BGP-LS.  There are three protocols which support SR, three protocols which can signal MSD, BGP-LS is one of them; all three appear in draft-ietf-teas-yang-sr-topo so why not here?
> 
> The editorial changes look good to me.
> 
> Tom Petch
> 
> Thanks,
> - Xufeng
> 
> 
> _______________________________________________
> 
> Thanks,
> Yingzhen
>> *   updated security consideration with detailed node info
>> *   minor editorial changes
>> 
>> The draft was originally part of the Segment Routing model, now RFC 9020. It was suggested during AD review to split the MSD part off the SR model since it's a generic MPLS parameter. The model is small and straightforward, and had been literally reviewed many times while it was part of the SR model. We'd like the MPLS WG review it, and request WG adoption.
>> 
>> If you have any comments, please let us know.
>> 
>> <tp>
>> I started off with some YANG quirks but ended up concluding I really do not understand the purpose of this I-D.
>> 
>> You create a list indexed by interface-ref for MSD.  MPLS YANG already has a list of interfaces.  Why not use it?
> [Yingzhen]: Good suggestion. I’ve made the change.
>> 
>> You create a list for nodes indexed by msd-type.  Why? and what is the point of  msd-type?
>> 
>> More generally I do not understand what the identity are doing.  You have no identity for two of the three signalling protocols.  I do not know what the purpose of msd-type is.  Needs justifying in the body of the I-D IMO.
> [Yingzhen]: the identity of msd-type is for future extension. In case there are new types created later.
>> 
>> You specify two protocols for signalling MSD; why not BGP-LS?
> [Yingzhen]: I didn’t get this comment. Why BGP-LS here?
>> 
>> s.1 usually includes RESTCONF
> [Yingzhen]: added RESTCONF.
>> 
>> s.1.1 out of date; the YANG module has the up to date description
> [Yingzhen]: fixed.
>> 
>> YANG
>> four authors, three are mailto: one not
> [Yingzhen]: fixed.
>> 
>> MSD needs expanding in the YANG module as that will exist independent of the RFC
> [Yingzhen]: fixed. Please let me know if you see places that need expanding.
>> 
>> identity identifiers have a lot of 'base' which I think redundant
>> 'RFC8491: Singling MSD ...'
>> 
>> erld-msd I think back to front; I prefer msd-erld, msd-….
> [Yingzhen]: changed the names.
>> 
>> groupings that are only used once often get criticised by a YANG Doctor.  Quite right too IMO it just adds complexity and makes the I-D harder to read for no benefit that I can see.
>> 
> [Yingzhen]: removed the grouping.
> 
>> Tom Petch
>> 
>> Thanks,
>> Yingzhen (on behalf of the co-authors)
>> 
>> ________________________________
>> From: internet-drafts@ietf.org <internet-drafts@ietf.org>
>> Sent: Sunday, August 1, 2021 12:01 PM
>> To: Acee Lindem <acee@cisco.com>; Jeff Tantsura <jefftant.ietf@gmail.com>; Stephane Litkowski <slitkows.ietf@gmail.com>; Yingzhen Qu <yingzhen.qu@futurewei.com>
>> Subject: New Version Notification for draft-qu-mpls-mpls-msd-yang-01.txt
>> 
>> 
>> A new version of I-D, draft-qu-mpls-mpls-msd-yang-01.txt
>> has been successfully submitted by Yingzhen Qu and posted to the
>> IETF repository.
>> 
>> Name:           draft-qu-mpls-mpls-msd-yang
>> Revision:       01
>> Title:          A YANG Model for MPLS MSD
>> Document date:  2021-08-01
>> Group:          Individual Submission
>> Pages:          10
>> URL:            https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.ietf.org%2Farchive%2Fid%2Fdraft-qu-mpls-mpls-msd-yang-01.txt&amp;data=04%7C01%7Cyingzhen.qu%40futurewei.com%7C2cc7aa1a9ead41568d1108d9551ecf5f%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637634413085880786%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=dxVGZrfMleDR%2B1%2BlkmEXN%2FRJOmL1zXLRD3xKBjNtOzI%3D&amp;reserved=0
>> Status:         https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fdraft-qu-mpls-mpls-msd-yang%2F&amp;data=04%7C01%7Cyingzhen.qu%40futurewei.com%7C2cc7aa1a9ead41568d1108d9551ecf5f%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637634413085880786%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=sOxeuBLa4ZSUV0to9%2F9SMrpQ2xOE%2Fkxq8hBiPeUlEUc%3D&amp;reserved=0
>> Htmlized:       https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fhtml%2Fdraft-qu-mpls-mpls-msd-yang&amp;data=04%7C01%7Cyingzhen.qu%40futurewei.com%7C2cc7aa1a9ead41568d1108d9551ecf5f%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637634413085880786%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=VX8Y40I3tUA%2BKolwyUUNTTV%2BMVt9s5l5MNXiEuIsjA8%3D&amp;reserved=0
>> Diff:           https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.ietf.org%2Frfcdiff%3Furl2%3Ddraft-qu-mpls-mpls-msd-yang-01&amp;data=04%7C01%7Cyingzhen.qu%40futurewei.com%7C2cc7aa1a9ead41568d1108d9551ecf5f%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637634413085890780%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C1000&amp;sdata=SvVf6VTIySZnUBZCgCMfthT%2FSyqkb1gIoES7FYYZg8U%3D&amp;reserved=0
>> 
>> Abstract:
>>  This document defines a YANG data module augmenting the IETF MPLS
>>  YANG model to provide support for MPLS Maximum SID Depths (MSDs) as
>>  defined in RFC 8476 and RFC 8491.
>> 
>> 
>> 
>> 
>> The IETF Secretariat
>> 
>> 
>> 
>> _______________________________________________
>> mpls mailing list
>> mpls@ietf.org
>> https://www.ietf.org/mailman/listinfo/mpls
>