[Teas] teas-yang-te invalid was Re: Yangdoctors early review of draft-ietf-teas-yang-path-computation-11

tom petch <ietfa@btconnect.com> Fri, 18 December 2020 12:41 UTC

Return-Path: <ietfa@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 502E33A02BB; Fri, 18 Dec 2020 04:41:58 -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 QsqPqQmy9mjR; Fri, 18 Dec 2020 04:41:56 -0800 (PST)
Received: from EUR03-DB5-obe.outbound.protection.outlook.com (mail-eopbgr40094.outbound.protection.outlook.com [40.107.4.94]) (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 D802F3A0140; Fri, 18 Dec 2020 04:41:52 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=byTZFLXObR7F1m+W25CtmTsyYbtbTGfpZiQuAC8O6HZyRNJMII4WIWTCSxV0o3YP5YW4iiyrskz1OuWHysaAI4VGQTrhxds2A6aORwxtfhDya2Xou7D0U3n8oFat6JFvaDHxml5TY3NIVRMJ3DfPb9VzDVWxWTWBBifSl/mIOCNQfcH1yB+NxnAQxKuqomUjuk5NwApNU/cOXvgB/4X1u6A1j+mbKQP7akPKPQp8cbiudzXRgUMfzCcL+YCNPdHWFZXLoISN+8oXKWgr/Dj8bNv2+k3cWrqb9g33llqOk3i8boZyhZgVqG2q5R8wKTGTZdy/4zAwLpdmlVnLdbCQ6g==
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=gojOMqU/uSsB3BYwEdNzBvSm4Bz59ChYG6fUyXNbF6M=; b=hSZyPj0b94X1FOc2dJJSTyoo+8gQOdkf1bbFcIvRghV1rglUoNJND7fzcRbkHVVQikGgPqXJtq7vPi8CS43u0BRpBA4KQLKy3AsJRqHs2arJcD5bcXe3OHHzaAnmib1PV3r3UCpmHzLR6TLhZNFxJvEDiUVUJhHJeBB5cm2YnPf0AqJcIM5HcK3epHHPAoz/HQc1PxsM22J+T5coJo85SiBu39cpHR3d/nxqx3lpQpkqpv7lpg0Ny3TZW4E6HKQt/cJymtbK6NfFxzW8+QWN0v8mwvcgAl03uYL2JaZ4ldukLaLa91u/YHcA1sAI0Dbric3xwsfvxeTpMa7+UUSngQ==
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=gojOMqU/uSsB3BYwEdNzBvSm4Bz59ChYG6fUyXNbF6M=; b=euBNSxBPnIpXFkGcNRPyq9xf+n+uHBM9XExanuzbYq5g2MaMKOkpT94KhymK/YSEJrmIcyWmKtrmGE0owQKsD9F5MCNHKCRYxP5os6X9H/mjl4BOddVh6tCjiQ+9KbsJNkUilAmBgtJF1oLfS9QKH4fxivl8+VTS3yNfU+p0M9E=
Received: from AM6PR07MB5784.eurprd07.prod.outlook.com (2603:10a6:20b:95::29) by AS8PR07MB7463.eurprd07.prod.outlook.com (2603:10a6:20b:2a5::7) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3654.7; Fri, 18 Dec 2020 12:41:50 +0000
Received: from AM6PR07MB5784.eurprd07.prod.outlook.com ([fe80::2c1e:7c35:41ed:9e88]) by AM6PR07MB5784.eurprd07.prod.outlook.com ([fe80::2c1e:7c35:41ed:9e88%7]) with mapi id 15.20.3700.021; Fri, 18 Dec 2020 12:41:50 +0000
From: tom petch <ietfa@btconnect.com>
To: =?iso-8859-1?Q?Martin_Bj=F6rklund?= <mbj+ietf@4668.se>
CC: "draft-ietf-teas-yang-path-computation.all@ietf.org" <draft-ietf-teas-yang-path-computation.all@ietf.org>, "teas@ietf.org" <teas@ietf.org>
Thread-Topic: teas-yang-te invalid was Re: [Teas] Yangdoctors early review of draft-ietf-teas-yang-path-computation-11
Thread-Index: AQHW1KanJVtlWmVVc0Sw2jlX0eRzR6n8y0op
Date: Fri, 18 Dec 2020 12:41:50 +0000
Message-ID: <AM6PR07MB57846938F8645091624D434FA2C30@AM6PR07MB5784.eurprd07.prod.outlook.com>
References: <160823152245.15025.13731491639863650578@ietfa.amsl.com>
In-Reply-To: <160823152245.15025.13731491639863650578@ietfa.amsl.com>
Accept-Language: en-GB, en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: 4668.se; dkim=none (message not signed) header.d=none;4668.se; 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: 4e6f8b45-ac4e-48bc-1d82-08d8a3524a5d
x-ms-traffictypediagnostic: AS8PR07MB7463:
x-microsoft-antispam-prvs: <AS8PR07MB74633C70702C0211CEBBBB2AA2C30@AS8PR07MB7463.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: 3IfXKeWlUzdcx0+BkHqZVdGMT1gz85dvB6anW7rvWDj2CgGEBhmm80R4qJ6Zp+6tNCn3zvDKd3JDvwHj9b1JefffL9UjiXNUeHkhshyJzYiAjUuh7E0jSSo1wBLS6rLPMTsd2vsgw6XaIsv2s6JC707o9ctyvph2ptOonhkqMM0EnUcQolzGHpbc3JaHrkVcvQr2swea94zyNHEX1lfjiskZSKszMByf35V3KJfk3x7GWi9FrhSyLMHhLUYYyV4Yv2v25LNY5IgKp9kCOLtV1KM8bc/yqX2Waeyip76UeGWoWDae54/jAWoXKihOiEIUreBw4BsIbcMlGVRWptSiVEBWcTYZ7dqeNz9j2Kqo9XhTwzFgzEhoLCcWdlwp1bEnvjPBe414zG0LezTbUmPpwQ==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM6PR07MB5784.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(39860400002)(136003)(376002)(366004)(396003)(346002)(966005)(52536014)(5660300002)(7696005)(66556008)(76116006)(54906003)(8936002)(66446008)(71200400001)(66476007)(6506007)(316002)(33656002)(26005)(66574015)(66946007)(91956017)(9686003)(8676002)(86362001)(186003)(83380400001)(53546011)(64756008)(478600001)(4326008)(2906002)(55016002); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata: =?iso-8859-1?Q?+SnLzu9owZd9BHrffvGZYhBA28mhnL929lTVJKpbUgIi4h0/RnRSJ4To9Y?= =?iso-8859-1?Q?2uWriqHhbA8eo4anaA0cwe82oPl4vKesamY8knAVvXg4HulBSy/ntoBm4X?= =?iso-8859-1?Q?1YLpU268PhF+KstVkSmqbFxplKl6LQaCOxzplU4o+dHKT3M1Oefjc3PDDZ?= =?iso-8859-1?Q?5uMuv3tx319NHR1yxydWGGCV8Bdh1UsE4PSH2mswS39tP7Pewm+7dPbovl?= =?iso-8859-1?Q?jotuMk9fi7j+OczBQuvry95PTtTGoxNuBKK7Qy4AjpeQXha6bciXmlXHEr?= =?iso-8859-1?Q?jcXs+oV2w/vQ8hxaa9EntOhiOCKpspOWnD51ACriSyMueRbqAL8vCuOKyJ?= =?iso-8859-1?Q?iWdS1hBOu5nFb7R4TOAeRRwxWoNgwSqR41Mpr3AvrzTtf/s9BZf9nsH4uh?= =?iso-8859-1?Q?9Nue/JFDiV/H3qBiXkDG4zrsfkh26xXHZXxD37roHDGVF/cAz4WxtkGuZc?= =?iso-8859-1?Q?7RwFY0O7jBJUqFxIfGo/lpOpI8VtQ1MgGpx6qP/BiFOewGXgNbPpF51Qfz?= =?iso-8859-1?Q?V6/mVZ9Pp196ZoFSCoGTthCyXWfI3nEgPC1G+JHmNcYLRSPW8x1F8z7LCW?= =?iso-8859-1?Q?IWmIigZZHgugVxHDop/IS6/ejpbITWHy+4WCASNMU+D9EBpjFGfhX4CDlg?= =?iso-8859-1?Q?RX7fUXDh2JzmoI4aeiMFVLva/9lZtP/4TjZFczaWIW3/hoE0L1nAsDIrIw?= =?iso-8859-1?Q?3AAZlCUSJrFiGuGImLrHt/9G/EAzvo6wapvHwUIfgwZw12spR2JI3Kr0Y7?= =?iso-8859-1?Q?5c4+8Yov7c6xEHSGsT7u7HZHvRx9XDIDKrcZWTGkiz6UV0vInMGXBQNrem?= =?iso-8859-1?Q?jK48bdWh4zKCYGxSCkNm4aGoaXmAEiAOAu0BYuY4KAZlbOJA5SlvywgCYw?= =?iso-8859-1?Q?1yovYWjfdAetXHJCUb43f7Y7he1UNhsv7pPKDSa4XOe2FiKCFteJCfP9aS?= =?iso-8859-1?Q?stomD/Q2gONnh8QJZRDDZttNpt5YJRJno7NLk9k2VHxVEVgpME0AuR/N0L?= =?iso-8859-1?Q?MPBdmlmdIFeoIZ5Jc=3D?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM6PR07MB5784.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4e6f8b45-ac4e-48bc-1d82-08d8a3524a5d
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Dec 2020 12:41:50.1708 (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: O2HFhs4znDXInMeHt644NGB0e4/t+yTAM7DX7RBwWVsedBnjJLvHoysMjIJDD01nqeOOYmnFiYIUfF68E4XUiQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AS8PR07MB7463
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/arGC1NdMTE2rBJrtXx-X8TmpUiw>
Subject: [Teas] teas-yang-te invalid was Re: Yangdoctors early review of draft-ietf-teas-yang-path-computation-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: Fri, 18 Dec 2020 12:41:58 -0000

From: Teas <teas-bounces@ietf.org> on behalf of Martin Björklund via Datatracker <noreply@ietf.org>
Sent: 17 December 2020 18:58
Subject: [Teas] Yangdoctors early review of draft-ietf-teas-yang-path-computation-11

Just to make it more obvious what Martin is saying, teas-yang-te-25 is invalid.  I think that the invalid grouping that Martin mentions that causes validation to fail  first appeared in -23 (along with so much else that I have yet to catch up with it:-(

Tom Petch

Reviewer: Martin Björklund
Review result: Ready with Nits


o  General

  The language is called "YANG", not "Yang".


o  1.2. Tree Diagram

  The text says:

    A simplified graphical representation of the data model is used in
    section 6.1 of this this document.  The meaning of the symbols in
    these diagrams is defined in [RFC8340].

  Tree diagrams are used also in chapter 5.  I suggest:

    Tree diagrams used in this document follow the notation defined in
    [RFC8340].


o  Tree diagrams in general

   You can use pyang -f tree --tree-line-length 68 ... in order to
   avoid long lines in the RFC.


o  6.1

  This section presents a fully expanded tree diagram of the module.
  Tree diagrams are mainly used to give an overview of a module's
  structure.  The tree diagram in this section spans 13 pages and is
  quite hard to read.

  I also note that a majority of the nodes in this tree diagram come
  from the expansion of groupings that aren't defined in this
  document.  Hence, I suggest that you might want to run:

    pyang -f tree --tree-line-length 68 \
      --tree-print-groupings --tree-no-expand-uses


o  There are a number of groupings that are used only once, and do not
   seem to be defined to be reused by other modules, e.g.,
   "requested-info", "requested-state", "svec-metrics-bounds" and more.

   If they are intended to be reused, it should be made clear in their
   description statements.  If not, I think they should be inlined and
   removed.


o  grouping svec-exclude

   This grouping has an ordered-by user list.  Why is this list user
   ordered?  If the order matters, it should be explained how it matters.

   Also, the index leaf has this description:

     "XRO subobject index"

   What is "XRO"?  Is this description sufficiently clear?


o  path-request

   In the path-request, there is construct for path-refs:

                   list primary-reverse-path-ref {
                     key index;
                     min-elements 1;
                     description
                       "The list of primary reverse paths that
                        reference this path as a candidate
                        secondary reverse path";
                     leaf index {
                       type uint32;
                       description
                         "The index used by the
                          primary-reverse-path-ref list";
                     }


  What is this index?  Is it only used as an arbitrary index, or
  something else?  If it is an arbitraty index, it should be explained
  in the descriptions.

  Also note that lists in rpc input don't need an index.


o  Validation

   The module fails YANG validation, but that is really due to errors
   in ietf-te@2020-07-12.yang.  Specifially, the leafref in the
   grouping "path-compute-info" must have prefixes in its path.
   Without prefixes, the path refers to nodes in the module that uses
   the grouping.  (same for other groupings in that module).


o  Layout

  I suggest you run the module through

    pyang -f yang --yang-canonical --yang-line-length 68

  in order to have the module indented and formatted consistently.

  This will make the RFC editor's job easier.



/martin



_______________________________________________
Teas mailing list
Teas@ietf.org
https://www.ietf.org/mailman/listinfo/teas