Re: [Teas] Common rules for TE-related YANG modules prefixes (was Yangdoctors last call review of draft-ietf-ccamp-otn-topo-yang-11)

tom petch <ietfc@btconnect.com> Thu, 21 January 2021 12:37 UTC

Return-Path: <ietfc@btconnect.com>
X-Original-To: teas@ietfa.amsl.com
Delivered-To: teas@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 39E173A099F; Thu, 21 Jan 2021 04:37:43 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=btconnect.onmicrosoft.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 uEaYPbZArgmQ; Thu, 21 Jan 2021 04:37:40 -0800 (PST)
Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on2128.outbound.protection.outlook.com [40.107.20.128]) (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 3BF103A0990; Thu, 21 Jan 2021 04:37:36 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NT9xTYNxDfczSPw/SnD9nX6EKZNg2tIsNNbDsiilMbEaBKN3eRcMeTM6OqI9mmcCDBxAa5Vw3lLzhVlnrS7mrTAZCaaBZPdT5c/GgFtm+fw//bBFX52I5kNBl1Y8POLQZ92F5d9uN5C3GeaCgb3cPM0YzbAzre8Rrf3oxjOXiEGhGo+FWRu9m4F2U6Hd2YCWICxurvHbJDVt+pc+c1dahaBqf9JHqBasBf5t9Q7I3GZF2DcKUTiAxQ27xezXpPerMVW5ET6Ajfvh2R0Rf/DKxyn09C8jboo6e+Xpib3iRc3agCdbfusezHKRi1zuCNgNODmebKz0+OEf3zPwslUhXQ==
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=cS1JDLEpaCyWjFS6WgSP0cwslXqf8oJY2cUg3nfu5+s=; b=SZ+j9yEQkl2Nscd5zJxr5/ggURlHG422XykNi0Jtq6Ha4oUv5+vdqfxpe7uGgwBzbgH7nKSCO0vHmzNFbeo1EeQIRv8Z7KjnkMqakrotMywutgrQN2pVSviLxE1bKHth0szXe5t61wktR251Sti08IDh+eMiaoWbwJsls4s1xhfeK2y8HpszIWEyo8kRSaXSuPu9R5mmAuaz5xiyPtGjQR4KAEQ9m0hA1agWj0YV2Oo/L68tPpURjwe+mWjPfaxzSIqshvaxmfDn1NR9hWcGFfjJ/pdRNU7laUGqxqOgFxtPcicfNgdI+xktjMgf1cKHeMr+4UmTvY9XDMVAMh9r7A==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=btconnect.com; dmarc=pass action=none header.from=btconnect.com; dkim=pass header.d=btconnect.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btconnect.onmicrosoft.com; s=selector2-btconnect-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=cS1JDLEpaCyWjFS6WgSP0cwslXqf8oJY2cUg3nfu5+s=; b=PNlr1N0pAAYS+rCT0wrND6PpkpFiaPBY4Xv2ZmLwEi+Qx0REnAKJ1GViEKEBNSsnDqFtCpowpd9IYHRUOyI8rAzs7uv7pSLWtyYEz+XoAXquLrS8sMOcKL3ZrEYDNr5Ik+HgpBRSlurUgYiwIfEOzm/wAeYpP6sM+g9C9joE3lU=
Received: from (2603:10a6:20b:134::11) by AM5PR0701MB2836.eurprd07.prod.outlook.com (2603:10a6:203:46::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3784.5; Thu, 21 Jan 2021 12:37:33 +0000
Received: from AM7PR07MB6248.eurprd07.prod.outlook.com ([fe80::6d46:4f3c:643:4849]) by AM7PR07MB6248.eurprd07.prod.outlook.com ([fe80::6d46:4f3c:643:4849%5]) with mapi id 15.20.3763.014; Thu, 21 Jan 2021 12:37:33 +0000
From: tom petch <ietfc@btconnect.com>
To: Italo Busi <Italo.Busi@huawei.com>, Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>, "'teas@ietf.org'" <teas@ietf.org>, "ccamp@ietf.org" <ccamp@ietf.org>
CC: "draft-ietf-ccamp-otn-topo-yang.all@ietf.org" <draft-ietf-ccamp-otn-topo-yang.all@ietf.org>, "yang-doctors@ietf.org" <yang-doctors@ietf.org>, "last-call@ietf.org" <last-call@ietf.org>
Thread-Topic: Common rules for TE-related YANG modules prefixes (was Yangdoctors last call review of draft-ietf-ccamp-otn-topo-yang-11)
Thread-Index: Adang/ShaCXr97p0RfGEvolIEdqtpAABzVwuAAHOM+ACWuGjJACn9dNgAVD61VENvdpQ0AAFOw5q
Date: Thu, 21 Jan 2021 12:37:33 +0000
Message-ID: <AM7PR07MB6248AD09582960B6A833399AA0A10@AM7PR07MB6248.eurprd07.prod.outlook.com>
References: <2f249df343f14c0799cccb38574914fe@huawei.com> <AM7PR07MB62483BD8298654CA1B04BC59A01C0@AM7PR07MB6248.eurprd07.prod.outlook.com>, <HE1PR07MB4156E24671500EDE9A577C0DF01C0@HE1PR07MB4156.eurprd07.prod.outlook.com> <AM7PR07MB6248AD722A6AEE1478EA8D81A0100@AM7PR07MB6248.eurprd07.prod.outlook.com>, <54da8864271146b1acd857af12e07c22@huawei.com> <AM7PR07MB6248B35601A230875C19E3F7A0E70@AM7PR07MB6248.eurprd07.prod.outlook.com>, <271de72243bb4103a29357d7025a719b@huawei.com>
In-Reply-To: <271de72243bb4103a29357d7025a719b@huawei.com>
Accept-Language: en-GB, en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: huawei.com; dkim=none (message not signed) header.d=none;huawei.com; dmarc=none action=none header.from=btconnect.com;
x-originating-ip: [86.146.121.140]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 511a6c21-a26c-4488-6723-08d8be095350
x-ms-traffictypediagnostic: AM5PR0701MB2836:
x-microsoft-antispam-prvs: <AM5PR0701MB2836C77A190602A862BE679FA0A10@AM5PR0701MB2836.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: kFmvhucoduHGvcBJr7546KUYFE4OFcEjjJyUSQ22Ds5n/I41ikjJPr35egxqhlByctd5D1qGvH2qWMh5Ph2udWzK6HIXjL7PexelVCHFi5Ar8YQl0WAxKBxkdfv7zyfou+XNSzWtv2P122kEsV2TjB5NUg2rzW7+HN4liYQ4tuPtV1f5n39znrrI6Lqeh7IypvFlLzsGXy6m9+L2u3+XOdT0iI592/bGU6s8vIAUo6j5K6toxtEpVRUxUdiRKLJAwMoTWuTCF63heOV87kYjpJHCvke3pN8YPcmwCf4H42BWb7u9e26C1aeI5mxDCfB+MCLftGF+5fSkPP/3FuUhsgXHIByQYNtKAmRUy3i5BPF7aAQgwKnjxVdjDBVLnlPkbd5Kx482QsnNfPlYTgnwZXubnhT/e8IJMRVsVOEwjEaAfJZxiY9KOpAvRleWIgnLtXsA0VLMW1RDjB2R0C8dkA==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM7PR07MB6248.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(346002)(39860400002)(366004)(136003)(376002)(396003)(316002)(64756008)(66446008)(66556008)(5660300002)(8936002)(110136005)(91956017)(76116006)(7696005)(66946007)(30864003)(54906003)(52536014)(66476007)(71200400001)(478600001)(33656002)(186003)(9686003)(26005)(55016002)(4326008)(8676002)(2906002)(6506007)(86362001)(53546011)(83380400001)(491001)(357404004); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata: 4IsWJi7uAeAcMg0zbDIwkiKZnk7FuHCgMTynltzB3n/rbzR9EUhQoUH3hMG8j+SAnOs1mR4//dNwIcBpAYYWiwFuUUkBPRdd+oXyBpAosrrNf7PrsgywTTqzxm1sTnLmq1942bSGsdrppmspbUiZc/7C3UGvUShxgVnlpN+4rgBkSD2w7MZDA+dgtbpCTfPQAlXf2w7+pZnd/jwqHfvcAOS2e+u40CCSsrIPBbeGk4wuLXm8FK39vTXGzqBTJisM7fzXEEKlL/GnNq2XDoog46YqeHOZmt7yLxd/24fApdl/N4pb4CHdaEQS0aNo5jReuMOhY9YvsZwlxz+FyPsH+8oiHC0L8i3pgGfVtphg5oDkR51Qg5sFGug39ULjxyZ+s4jp1neZHXVNM0bSEcfo6tUdM/DpFUTN+E9poUzNIQyd/LtewljzCuRWcl1MjavkcAFTybRDM0gkHgTvcnqWcW2CbEIMczYA3SlbMA6DTccIV8CM5u1tMvoFvjxg0YzuO8GKI9Btcc6R7dBWdb1bBXVjxKp6oM981KG5DtiLkDsZ3Xw1oVqMAgH+nEm9bqWHzvsFZinbvKU8cfB9rQSuWP3zXj9axgPRu5DsFEb/FZVguvVMaTVUr2Y8NP2JOgD4Vzlw6AMmSwIGLqOpjIIcCHtIcN2QrsD2puRFlsScTHgAJ7UVRUIMATd156z0pBTMZRevXHl4T1CpA5CnXTguPCeagkmxv2WG9eE71mdqAcor4vLqOiAGZn/5jzv6s45Cbp1Ibxmxy2Yzb5macDDq/a/0drADs9BCpSQZgZ4Q9X8ouRWlC9otLQz8ArJ3LVOZ2kD0VKWkflnylZimndJAEw0MQxayYnNlptxb0mURK3rq1Wo2lMvWU38OapoKu4Wy1hwmhsKZy432m81vT7RwuPpI/5rTPIniu8BR6yQSjwQdoaLFm5z9D8R+t1drO7PePBpMOhZl5F8ygyJxUSnUZkGqezAKc9+Wbh3Fxg8gUeWZa26xdhf9NY5U5kXdFLWDnaT0mff8ndafjTZ/8zyZMnQ1rmUU3riPDXC2Eh5kfzT8mzyq4oqlWIiEAjFrrHOK
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM7PR07MB6248.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 511a6c21-a26c-4488-6723-08d8be095350
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Jan 2021 12:37:33.3531 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: cf8853ed-96e5-465b-9185-806bfe185e30
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: BTPN05jqlijsaCfEpfIXLVJqURO3c2CgtnaxsNF8ARCAmVxwUyFFgzza+pOu6ob0qdsmaiPsfjRpwgHUKx6rTQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM5PR0701MB2836
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/2b11AzR8HZ3PuUu8_dfQq13ZrrA>
Subject: Re: [Teas] Common rules for TE-related YANG modules prefixes (was Yangdoctors last call review of draft-ietf-ccamp-otn-topo-yang-11)
X-BeenThere: teas@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Traffic Engineering Architecture and Signaling working group discussion list <teas.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/teas>, <mailto:teas-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/teas/>
List-Post: <mailto:teas@ietf.org>
List-Help: <mailto:teas-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/teas>, <mailto:teas-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 21 Jan 2021 12:37:43 -0000

From: Italo Busi <Italo.Busi@huawei.com>
Sent: 21 January 2021 09:56

Hi Tom,

Sorry for being late but I have been struggling to understand what is your concern and therefore to understand how to move forward on this issue.

Since I have not seen further comments, I am trying a second attempt.

It looks like you do not like using the term te at least for OTN, WSON and Flexi-grid modules' prefixes. I am not sure we can remove the it from the MPLS-TE and ETH-TE to avoid potential conflicts with MPLS and ETH non-TE cases.

Could you please help me to understand which is your preferred option to move forward among the four below?

<tp>
Italo

I think that te: is a fine start to a prefix for yang-te, yang-te-types, yang-te-topo.  

But when the model is for MPLS and te, or WSON and te, or OTN and te, or RSVP and te and so on, then - for me - it is the first attribute that is more significant, the direction in which I want the prefix to point me in a leafref, an augment and so on.  Hence I would like the various WSON e.g. modules to have prefix starting with something in common that reflects WSON; ditto, RSVP etc.  What I have seen here is a tendency to view the world through the prism of te and see everything as an adjunct to te whereas for me the fact that it is te is the more obvious attribute and it is the WSON etc that I would like help with from the prefix.

I have lost track of the current prefixes; Option 2) is the one I find least helpful

Option 3 is fine for WSON, flexigrid, OTN. I dislike te-mpls, prefer mpls-te, but think that that discussion belongs on the MPLS WG list, not here. I note that yang-te-mpls has a diagram with ietf-rsvp-te and ietf-te-mpls in it which to me looks wrong

eth I am unsure about.  The IETF does not do much with eth - some work in the netmod WG - that I see as IEEE territory which has prefix such as dot1q!  My experience with SMI was that the IETF needed input from IEEE to get it right but for work in the IETF, my inclination would be to not start with te.

Tom Petch.


1) keep the current prefixes

2) change the prefixes as per my initial proposal

TE      OTN     WSON    Flexi-Grid      ETH-TE  MPLS-TE
Topology        tet     tet-otn tet-wson        tet-flexig      tet-eth tet-mpls
Tunnel  te      te-otn  te-wson te-flexig       te-eth  te-mpls
Path Computation        tep     tep-otn tep-wson        tep-flexig      tep-eth tep-mpls

3) change as per this updated proposal

TE      OTN     WSON    Flexi-Grid      ETH-TE  MPLS-TE
Topology        tet     otnt    wsont   flexigt tet-eth tet-mpls
Tunnel  tetu    otntu   wsontu  flexigtu        tetu-eth        tetu-mpls
Path Computation        tep     otnp    wsonp   flexigp tep-eth tep-mpls

4) change as per your proposal (please fill in the table above)

TE      OTN     WSON    Flexi-Grid      ETH-TE  MPLS-TE
Topology        tet
Tunnel
Path Computation

Thanks, Italo

> -----Original Message-----
> From: tom petch [mailto:ietfc@btconnect.com]
> Sent: giovedì 12 novembre 2020 12:38
> To: Italo Busi <Italo.Busi@huawei.com>; Daniele Ceccarelli
> <daniele.ceccarelli@ericsson.com>; 'teas@ietf.org' <teas@ietf.org>;
> ccamp@ietf.org
> Cc: draft-ietf-ccamp-otn-topo-yang.all@ietf.org; yang-doctors@ietf.org; last-
> call@ietf.org
> Subject: Re: Common rules for TE-related YANG modules prefixes (was
> Yangdoctors last call review of draft-ietf-ccamp-otn-topo-yang-11)
>
> From: Italo Busi <Italo.Busi@huawei.com<mailto:Italo.Busi@huawei.com>>
> Sent: 05 November 2020 18:07
>
> Hi Tom,
>
> I am not sure whether you are saying:
>
> 1) having a common rule for TE-related YANG modules prefixes DOES NOT
> make sense unless we define common rule for all YANG modules prefixes
>
> 2) having a common rule for TE-related YANG modules prefixes make sense,
> but it would be even better if we define common rules for all YANG modules
> prefixes
>
> <tp>
> None of the above.  I am saying that the world does not revolve around TE for
> many, perhaps most, people so classifying everything as an adjunct to TE is
> unhelpful.
>
> CCAMP deals in mw, WSON, flexigrid, otn, dwdm etc. and I see prefixes derived
> from them as being more helpful.
>
> I also see repeated efforts to use an identifier to encode semantic information
> thus rendering it a poor choice as an identifier.  Identifiers need to be unique,
> easy to use not the source of semantics about what identity they are referring
> to.  YANG Guidelines gets it right but might have gone a bit further.
>
> WSON gets it wrong IMHO.  wson-yang uses wson as if there will never be
> anything else wson.  wson-tunnel fails to register a prefix but does use wson-
> tunnel which is too long. wsont would be a poor choice as ...t is widely used for
> types and other common definitions.  wson-iv has yet to get a YANG module
> but wsoniv might make sense.  In 10 years time, how many WSON modules
> will there be?  I do not know but suspect that there will be several and would
> like them to be wson...  not viewed through the prism of a different axis.
>
> I  think that the approach often adopted in IETF WG militates against good
> choices, in identifiers and in protocols, a tendency to get something out of the
> door after which it is on to the next step and oh dear, if only we had thought
> of the next step sooner then we could have allowed for it in the base instead
> of having to create a hack.  (Amongst protocols MPLS is perhaps among the
> worse for failing to consider encoding the next protocol in its structures. but
> there are plenty of instances where no consideration is given to the probability
> of there being a second version and so no way of telling which is which
> although NETCONF showed a lack of foresight in putting supported modules in
> the initial exchange).
>
> So think beyond the next IESG review, where will we be in five or ten years,
> what will be helpful in reminding users what this is about, and keep it short.
>
> Tom Petch
>
> I am not sure whether it would be really feasible to come up with common
> rules for ALL YANG modules prefixes (at least I am not able to think about a
> proposal, at least because I do not have full visibility on all the YANG models to
> be considered). What would be your proposal?
>
> Thanks, Italo
>
> -----Original Message-----
> From: tom petch [mailto:ietfc@btconnect.com]
> Sent: lunedì 2 novembre 2020 10:56
>
> From: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com<mailto:daniele.ceccarelli@ericsson.com>>
> Sent: 21 October 2020 11:15
>
> Hi,
>
> I think it's a good idea.
> Changing the naming for the WSON documents is not a problem, that can be
> done at the RFC editor stage.
>
> Tom, regarding you comment I would say that all the modules that the
> authors are referring to are TE based .Even if it's a multidimensional issue I
> would say we can use TE as the leading dimension.
>
> <tp>
> We can and for those whose world revolves around TE it might be useful but
> there is more to running a network than TE (shock, horror) and so an
> alternative dimension would be more useful - well it would for me.  Should
> RSVP-TE be renamed TE-RSVP in all our documents?
>
> Tom Petch
>
> BR
> Daniele
>
>
>
> -----Original Message-----
> From: tom petch <ietfc@btconnect.com<mailto:ietfc@btconnect.com>>
> Sent: den 21 oktober 2020 11:33
> To: Italo Busi <Italo.Busi@huawei.com<mailto:Italo.Busi@huawei.com>>; 'teas@ietf.org' <teas@ietf.org<mailto:teas@ietf.org>>;
> ccamp@ietf.org<mailto:ccamp@ietf.org>
> Cc: draft-ietf-ccamp-otn-topo-yang.all@ietf.org<mailto:draft-ietf-ccamp-otn-topo-yang.all@ietf.org>; yang-doctors@ietf.org<mailto:yang-doctors@ietf.org>; last-<mailto:last-call@ietf.org>
> call@ietf.org
> Subject: Re: Common rules for TE-related YANG modules prefixes (was
> Yangdoctors last call review of draft-ietf-ccamp-otn-topo-yang-11)
>
> From: CCAMP <ccamp-bounces@ietf.org<mailto:ccamp-bounces@ietf.org>> on behalf of Italo Busi
> <Italo.Busi@huawei.com<mailto:Italo.Busi@huawei.com>>
> Sent: 21 October 2020 09:26
> To: 'teas@ietf.org'; ccamp@ietf.org<mailto:ccamp@ietf.org>
>
> Hi all,
>
> We have got a YANG doctor review comment on OTN topology YANG model
> advocating that "modules from a common group could use some common and
> obvious rules for prefixes" (see mail below).
>
> While the comment seems reasonable to us, we have noted that, up to now,
> there are no such rules, as summarized in this table:
>
> <tp>
> There are no such rules, as in YANG Guidelines, but it is a good idea, probably
> taken as read by those who have been around the block a few times and have
> had to live with the (lack of) naming conventions in older management
> software, and incorporated automatically by such; and it has been regularly
> commented on by those reviewing YANG modules.  I have made such
> comments recently on BGP and on NSF modules.
>
> But it should be noted that this is a multidimensional issue whereas the
> suggested changes below only take traffic engineering into account, as if that
> was the only attribute that matters.  I think this  wrong.  Thus which matters
> more, that modules have traffic engineering in common or that they have e.g.
> WSON in common?  I think that latter matters more in making it easier for
> users to understand so no, nothing else should start with te.   Note too that
> there is often a separate types module and many such use a suffix of the letter
> 't'  for this so having 'tet' somewhere in the string I also think likely to confuse.
>
> Tom Petch
>
> TE      OTN     WSON    Flexi-Grid      ETH-TE  MPLS-TE
> Topology        tet     otntopo wson    flexi-grid      ethtetopo       tet-mpls
> Tunnel  te      otn-tunnel      wson-tunnel     flexi-grid-media-channel        eth-
> tunnel      te-mpls
> Path Computation        te-pc
>
> In order to have such common rules, the prefixes could be changed as:
>
> TE      OTN     WSON    Flexi-Grid      ETH-TE  MPLS-TE
> Topology        tet     tet-otn tet-wson        tet-flexig      tet-eth tet-mpls
> Tunnel  te      te-otn  te-wson te-flexig       te-eth  te-mpls
> Path Computation        tep     tep-otn tep-wson        tep-flexig      tep-eth tep-
> mpls
>
> It is worth noting that
>
>   *   the prefix used by TE topology cannot be changed since it has been
> already published as RFC8795
>   *   we do not know whether we can change the prefix for the WSON topology
> since the draft has already passed IETF Last Call (our assumption is that this
> would still be possible)
>
>
> We would like to gather CCAMP and TEAS WGs opinion about whether:
>
>
>   1.  Having common rules for TE YANG modules is valuable
>   2.  The proposed prefixes are acceptable
>
>
> Aihua, Haomian and Italo (on behalf of co-authors)
>
> -----Original Message-----
> From: Radek Krejčí via Datatracker [mailto:noreply@ietf.org]
> Sent: venerdì 16 ottobre 2020 15:33
> To: yang-doctors@ietf.org<mailto:yang-doctors@ietf.org>
> Cc: ccamp@ietf.org<mailto:ccamp@ietf.org>; draft-ietf-ccamp-otn-topo-yang.all@ietf.org<mailto:draft-ietf-ccamp-otn-topo-yang.all@ietf.org>; last-<mailto:last-call@ietf.org>
> call@ietf.org
> Subject: Yangdoctors last call review of draft-ietf-ccamp-otn-topo-yang-11
>
> Reviewer: Radek Krejčí
> Review result: Ready with Issues
>
> This is my yang doctor review of draft draft-ietf-ccamp-otn-topo-yang-11 with
> the ietf-otn-topology@2020-09-21 YANG module.
>
> Despite the size of the module, its structure is very simple repeatedly following
> a pattern of augmenting ietf-te-topology by groupings defined in ietf-layer1-
> types module.
>
> Datatracker's validation with yanglint reports a number of warnings, but they
> are false positive (fixed in yanglint 1.9.16 - the fixed version still reports
> warnings, but they are all from the imported ietf-layer1-type module).
>
> My only note to the module itself is about the two defined groupings - I'm not
> sure about the reusability of the groupings in other modules. If the reusability
> is not the concern, I don't see any reason to define them.
>
> Regarding the draft, as a reader, I would appreciate a more targeted
> description in section 3. Instead of just dumping the tree diagram in section
> 3.2, it would be useful to split it into several areas with some brief descriptions
> and examples.
>
> The list of paths is introduced in Section 6 as "the subtrees and data nodes and
> their sensitivity/vulnerability", but I don't see explained/described the
> mentioned sensitivity/vulnerability of those paths.
>
> The prefix of the YANG module (also referred to in Section 7 ) - 'otntopo' -
> seems inconsistent to me. The relevant ietf-te-topology has 'tet' (so I would
> expect 'otnt' here), on the other hand, the ietf-otn-tunnel has 'otn-tunnel'
> prefix (then I would expect 'otn-topo' prefix here). The 'otntopo' seems to
> introduce just another format. As a reader/user, I would prefer if the modules
> from a common group could use some common and obvious rules for prefixes.
>
>
>