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

Daniele Ceccarelli <daniele.ceccarelli@ericsson.com> Wed, 21 October 2020 10:15 UTC

Return-Path: <daniele.ceccarelli@ericsson.com>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 4816F3A15A9; Wed, 21 Oct 2020 03:15:13 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.101
X-Spam-Level:
X-Spam-Status: No, score=-2.101 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, RCVD_IN_DNSWL_BLOCKED=0.001, 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=ericsson.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 mom0Sl4DtF89; Wed, 21 Oct 2020 03:15:11 -0700 (PDT)
Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-eopbgr30054.outbound.protection.outlook.com [40.107.3.54]) (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 3C0153A15A8; Wed, 21 Oct 2020 03:15:10 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=grFXuFthtf7gVmbrpinhYsRqUA3zAz6AlNjev9m/2XZHyeZMTxbqjMU6+Wx35vlrGdh5clOCAeMCVTTYxNRPJ6XatTFQNA+V6hk9TUDj1MbH1QbRqkmuUcIK4WLJnej5Pk16oEmhjHFyDVU97NVsvlSzeL/lIQGWOh3CAyLTBdybJ+8/F743tsDlWCUZTbJpA13L4A+irkKTLxtQz8Cw18SRf/J/J/Fip8p2J2lV2y+DVUAVfwwi0DxsyBlq6YtprpWjKOFrSIYekrsfPW2TuhpeJ9lwqZV5FhR/dY2xhn7RcmIug7h7nu8vPMJiNOrxUUMOFEzOv8tyStr9EoYtzQ==
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=cd4rFVy1xJmA4JzyBLdBeBH6l0zuo3B0cM4zV1gbXSo=; b=bWkpazLa81dUXJRUf4Kp2+YMcBTSf9vyIM9N1alcA1e6IagdnynPuP0XA8j71TZ84mifkwAq08N0xJX+xf0kMBcWgWyAhNF292OtGLYbjEJBOzPl1YLxCx88zT32H8YyhPnZfRaULC5KAtF1ZpJyJ58NFfew0EZkmxvoYWHs0Gt4C92Ohxsq7NkyWwzeLbW5DoyGRZmXtxaWm93onOSzhp4xlmhxZYioHA7n7LRkLEkUAX47imggZjO5YRcODsIlavYy+fS5Ql84azrJa9wzeGRbDQru3+hPNgHa4L9bpwQF+jzVrDZH1QwsUfVi6lKJe7WZvH2VskfILSeoKl/1eA==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=cd4rFVy1xJmA4JzyBLdBeBH6l0zuo3B0cM4zV1gbXSo=; b=JkR3pSZ3tVowDe6d/JYrxY9iBBGXzeuEqivcy1QXB3se/Jw+dSnFP+qXX1DofU6U4rdd0I61+y6HxdH9MCoZYrHD5Vddl87O7unaBgENsRH65zOXQ3SVnAP0mPeTP7Us2ZjEWpo6eydpOPyULQsTG/FW2U+VCKvL9HcfCM9MHIE=
Received: from HE1PR07MB4156.eurprd07.prod.outlook.com (2603:10a6:7:9b::16) by HE1PR0701MB2665.eurprd07.prod.outlook.com (2603:10a6:3:97::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3499.9; Wed, 21 Oct 2020 10:15:07 +0000
Received: from HE1PR07MB4156.eurprd07.prod.outlook.com ([fe80::8d9b:9ab1:1a37:6e93]) by HE1PR07MB4156.eurprd07.prod.outlook.com ([fe80::8d9b:9ab1:1a37:6e93%7]) with mapi id 15.20.3499.018; Wed, 21 Oct 2020 10:15:07 +0000
From: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>
To: tom petch <ietfc@btconnect.com>, Italo Busi <Italo.Busi@huawei.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+A=
Date: Wed, 21 Oct 2020 10:15:07 +0000
Message-ID: <HE1PR07MB4156E24671500EDE9A577C0DF01C0@HE1PR07MB4156.eurprd07.prod.outlook.com>
References: <2f249df343f14c0799cccb38574914fe@huawei.com> <AM7PR07MB62483BD8298654CA1B04BC59A01C0@AM7PR07MB6248.eurprd07.prod.outlook.com>
In-Reply-To: <AM7PR07MB62483BD8298654CA1B04BC59A01C0@AM7PR07MB6248.eurprd07.prod.outlook.com>
Accept-Language: it-IT, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: btconnect.com; dkim=none (message not signed) header.d=none; btconnect.com; dmarc=none action=none header.from=ericsson.com;
x-originating-ip: [93.38.67.165]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: b66eda4e-1f62-420f-b020-08d875aa2fa2
x-ms-traffictypediagnostic: HE1PR0701MB2665:
x-microsoft-antispam-prvs: <HE1PR0701MB26651CBF59CE597DDE21F49FF01C0@HE1PR0701MB2665.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: d8BthkY7JD7AUy1WNxcRhr5QeKLwj/AUV5so0eOgPml0Poz8mEzkHG2VbEZUm37aEbUgmUymXBRe0enmcED3COw4TwIGqvWIf2zOb7aUlSYc7227Y6xWtOcd2S/8Qirr1U70nu29x8fsEnTH5HI/f1aV8bDK3+rueK6mnmEPjQHG3/10flUHTXUFQXZ1cnftvOU8tD5L8C+0yOuDapTtieslcEDuDSoF8j470YeX0RtuELrmVsqTcs3erMHE4NiFaSGXWYJDtRmjw3MAW4ET2dggYl+Z1Y+MNzNwCiCS/mxbCvJkKVb5f0Q4QsHr2RV8KUTQdIyUxUeq3zxo69xmCHK6dFCbKMJ0cEaTb1hb2d9GopbVnkMYCMYCGEckx+Vl
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:HE1PR07MB4156.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(39860400002)(136003)(346002)(396003)(376002)(366004)(83380400001)(8676002)(54906003)(4326008)(7696005)(53546011)(186003)(2906002)(8936002)(26005)(110136005)(6506007)(55016002)(316002)(296002)(52536014)(5660300002)(71200400001)(66446008)(33656002)(9686003)(86362001)(76116006)(66476007)(66556008)(64756008)(66946007)(44832011)(478600001)(491001); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: 1ZNC9oqZ5tOb1HXi1UDdMi0SQ8nIAJqSl/ZEQSTu56Ji/tGITw+X+UMtHwX3mDZwAeF65Ehk8t/u1Rc87M7jAPTvX0TO5H7gT5hQD3ryicUbLw27cDHdnb/RLGwkqMzJxAFZqj13xvTsMZuKl4Axxbi9G7HeHw0Do3L+dzzPoZDKRJmnyIjitGKiTgRcOkj3EcT5oE34koUAIUcDt+QdIUeNyYa804MMMHVh7a1nDIaD8lj5pQskQqUPpeyOR8IhObF/R+CMS5bzq9pE1xGRjQpEzx9d2x68i3cVZ2T8hFoclT5jTbo3wPh+0/sJ/TmzwirZjSYOm+aUawQ0qx7eEN19STizjSudODJwsSAP8KXzR5cMPx0uto5aHOKDrWQbMQPGVgd2wmVfHPiVXn/dGI/ZghN8zEGfl/HSLwqLIgUWbkhX+ivSpCDQSOo9SSNkw82yS9emweikX3aWC/tGZLzSONLgwz6aHOE8fKHrWXAVg6vytU0/IIyKoWYJUT89peZDjT6E+ogkzRODJe3fgK3lDTHRsZ/kbfegFRjxmMIwJiqUqOZZapCgfbfGEt3YctOrew0E4ucT9guR3HurrsgawZSqkZLaAuj1uWeZTY0090Wk7ji7p8GJZgsDsAxGYx6kjABTwkre1ojojiw3CA==
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: HE1PR07MB4156.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b66eda4e-1f62-420f-b020-08d875aa2fa2
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Oct 2020 10:15:07.5797 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Z3AVa4gJ4EcCpBfpqUAChWhICMpDwkLybmK216BqUUw89ZUkJY8maS5JGd97f5esLx1SysFHPTaxpD2U3RzmPv5cthlqXPwwn/Quf8mYRLA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: HE1PR0701MB2665
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/teRZdl8p-i7Tl8q71mMohgR11Xs>
Subject: Re: [CCAMP] Common rules for TE-related YANG modules prefixes (was Yangdoctors last call review of draft-ietf-ccamp-otn-topo-yang-11)
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 21 Oct 2020 10:15:14 -0000

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.

BR
Daniele  



-----Original Message-----
From: tom petch <ietfc@btconnect.com> 
Sent: den 21 oktober 2020 11:33
To: Italo Busi <Italo.Busi@huawei.com>om>; 'teas@ietf.org' <teas@ietf.org>rg>; 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: CCAMP <ccamp-bounces@ietf.org> on behalf of Italo Busi <Italo.Busi@huawei.com>
Sent: 21 October 2020 09:26
To: 'teas@ietf.org'org'; 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
Cc: ccamp@ietf.org; draft-ietf-ccamp-otn-topo-yang.all@ietf.org; last-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.