Re: [yang-doctors] [netmod] grouping prefix Re: [Teas] Yangdoctors early review of draft-ietf-teas-yang-path-computation-11

tom petch <ietfc@btconnect.com> Wed, 23 December 2020 16:40 UTC

Return-Path: <ietfc@btconnect.com>
X-Original-To: yang-doctors@ietfa.amsl.com
Delivered-To: yang-doctors@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2CD553A0C35; Wed, 23 Dec 2020 08:40:03 -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 N_HpD2MqHrQZ; Wed, 23 Dec 2020 08:40:00 -0800 (PST)
Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-eopbgr80121.outbound.protection.outlook.com [40.107.8.121]) (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 687A83A0BFF; Wed, 23 Dec 2020 08:39:59 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EFBFImOJhnWS8NrQuwMr1wiCA27yodTiIqmn0G4Celb5UDd92ypUTzM6zji930sX4dBsFdU3u2wDdagONV9FT98ycPRH24YrVOUpkEBKwVQ9ESC4bSeWFL/FAvDC7t0+5g2oVPLVRxQEKeJmmmOFw+pLM3XL2gPMZbNmyxGVHQrsV/SaAQIV3pyjryLyBUREOkboyF8BHIrvy31+ui+lj21pPB+rnbc73y4Fr4CZ/1HDd7+rRzp3sI8sMYARLdKqbzlM28YsMIiT32R+Xpd9tV5lbtWqy0l+4XQdIiz8luU4Jacg8fzb4p0iUse8XyJCKVwL0cmRq5Qq3mB/tyGBnQ==
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=zOjL60beA4gpTG3yns346T2iBQBOuf2C9wkSlIQh3xU=; b=O3bRQUqNUUVBdNgz8H4eB/78hc0sm0GhjWKxyZ2t4aHfxp0WFOwnKoxkDIsCWRs2O00Ng4ssJ9I9btiwS+pcjD8/9fm8lScDk9H0twyOT82nrzgbYcd+rdQuwPQDG3jo6F87eAXwbEmH9s4B9Jrf80Ft1bSJVoQsrSU6R4NlzqZ8Cu/ChND0d7UlTuzS+D7zv/VxtoByaUxK6R30W5ULBonEhB/T2UHKrE4nzjEPWR12bkw4vJ4K171ZfdKYRzEhxwiDFX301QwL8Z42tb3bs20eh66Vz7yR9vF2AWdb8n9r1CjuuqeCcCpyUU08hziV7BDNcZgVsbMtAHbBnD6zwA==
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=zOjL60beA4gpTG3yns346T2iBQBOuf2C9wkSlIQh3xU=; b=RRENuJcuYrpLpqsFWDiopXGpAQMsc61g3EQKTUSY69RjwgFhlWOdRWEKhGRWqeQki9xcqeU6Xd94KK841xWz6HcUD5dtVMeDBzJP2yom+ObZfsamWRiswUezemOxDZke7LNEa4Z81kp37w4PbgwRly4nyEdNKR5F5JdyP5gSEJU=
Received: from AM7PR07MB6248.eurprd07.prod.outlook.com (2603:10a6:20b:134::11) by AM6PR07MB4789.eurprd07.prod.outlook.com (2603:10a6:20b:19::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3700.21; Wed, 23 Dec 2020 16:39:57 +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.3700.026; Wed, 23 Dec 2020 16:39:57 +0000
From: tom petch <ietfc@btconnect.com>
To: Martin Björklund <mbj+ietf@4668.se>
CC: "netmod@ietf.org" <netmod@ietf.org>, "yang-doctors@ietf.org" <yang-doctors@ietf.org>
Thread-Topic: [netmod] grouping prefix Re: [Teas] Yangdoctors early review of draft-ietf-teas-yang-path-computation-11
Thread-Index: AQHW1KanJVtlWmVVc0Sw2jlX0eRzR6oEnDzGgAAAV7aAAAlfO4AAGGCAgAAqrh8=
Date: Wed, 23 Dec 2020 16:39:57 +0000
Message-ID: <AM7PR07MB6248813DD45BF3136C9218F6A0DE0@AM7PR07MB6248.eurprd07.prod.outlook.com>
References: <AM6PR07MB5784208BE5EDE515021B8FA7A2DE0@AM6PR07MB5784.eurprd07.prod.outlook.com> <AM7PR07MB6248D4AA3256DC3BABAD0E42A0DE0@AM7PR07MB6248.eurprd07.prod.outlook.com> <AM7PR07MB6248589F990C977364396E79A0DE0@AM7PR07MB6248.eurprd07.prod.outlook.com>, <20201223.145921.437198295219717483.id@4668.se>
In-Reply-To: <20201223.145921.437198295219717483.id@4668.se>
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: 37fe68c6-013b-4cc0-d9bc-08d8a7616220
x-ms-traffictypediagnostic: AM6PR07MB4789:
x-microsoft-antispam-prvs: <AM6PR07MB4789E9D0883E7DA0F09BE853A0DE0@AM6PR07MB4789.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: 9GkEGQMp+69mV4+efLibC/e0l7Ejm0imiiGYiskTUYYXizcbdVPxKGXkRc+YQ/8Ds8tF2UhkD5ct+ATykLr4M8h4W4N0L2QQeA+NhpzzdEpXELJe3oqb1TjRK3O/gmYU2uxBhEOPhXnIIbqlrmBI9VR9ZgmTJ7S5nV5lWrPznBbEDXaUTTNgmeY7pCaVBR7g0wN75jkm98/72yEl1ASRkx+UD0JpVhoKs+2dD9dp5iKQ6upuMy6ACfFS8SmELcqeo43EffAIypVOMN8IFTVEsjXvemXyJNZAeyGzGHy1dqZRUQ0uud7ERbXP6Yu6Gh99leWpq7mU1Bseu4CeTLTq3fKM4lmE63Fvyv1hPbyLFIes1ZbacY4aXzOh9u8PNTEAQlHudOr7lWAi5B6oIfOq9aAuCzSKfgQ2EMc38dMbKvoEkbLT+HmXrMrStuXTjxWLL3e3tqvAfHM4VfNiUxQOMg==
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)(376002)(136003)(396003)(366004)(39860400002)(5660300002)(478600001)(54906003)(966005)(33656002)(8936002)(316002)(76116006)(186003)(4326008)(26005)(66556008)(66946007)(55016002)(83380400001)(71200400001)(91956017)(6506007)(8676002)(2906002)(9686003)(52536014)(64756008)(86362001)(66476007)(7696005)(66574015)(66446008); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata: kILwgQVz2wz7+IwKy9sBpTlsgq4unmpO1XeeDCRuChsJSXjHNAUXAc9F8vk2k0FKHTVXnHOegIZj9Nlt/9bsc3SO7S3tPgXzqOOpK5wdE0ODyCUx/EgAidKzAvJZm21+Kh3cMpRpDiy4DJ1EFLN0CbBj8l/tpPs6FgV2TGdWBX8GqLaRiBr+Ub1EOghCNXgrSA0yGNYqlcCaoj6xzPXQ1rvplRh/Fx1YQV1u5ymUiyrbdeku5SODvMnWNFsiqgKZnsaracmpK+zZ+auY+V+ASP3Fm6S3i/BmISPQH6wvAQdIevcvb+DGtu7NhLJKyiD9E33LlV76WaWDpLehp7kEawhy1bhOaaJSJH0+R6s8dlsynATihuOW7KiyYsAwvAzTqMp9dRRHCkGNHbJSUyeJWK0L9fOdd6RjMI7M8fLLBtInf38JTlrGWFnzJUsAWS8o3TunSY3upv+EL9+qusf+XC3bjGQ8M0f9JhPkY/CVdTO2OS/p75hD/cSsYl3+auUTkIFTnY5VS+Z13P+5aX0NCoBxVVJ0XJVB5DOEN0r+cewWxC2vhSMdIiq33sATazDlFPZJnKn/wS2j3r15NvShTOF97SOD0/1AgNsZaBNx/gau/JKnOCet+j1XqkUrW3sVKEeOSoZt3fQFY5wfbmrmizz/QvJK+K/mlV0ids8hNJiZBLEy6dRWxmAr7Vn/pu7CV9wFIVfSdKm80O7j08YCeiWw29tzvfxDbStEEPkh0DHf4jqPw2g+hJKjanOflfC1XLepQpfM2ExToZ1Ci8/0JMzZdB/1A5r3fQp8vI/OLoz/W0bn3LvMtmX66RxiCV9MOxXJx19l4LgzV0BrPeEATDA7E80QuSMrbW+QAYX5vgVUc7XlWAbnRrCRBJ0aQWLDoNURZKDsOO9D34ra3oZbGvjXYeAUnuIVHPTUo8jxrBzBkEYkCAiSj2ynDK+Dj0o15U3KMkXnWiKe1ZCRc3fBD/xzFV7hZvnYiXAaDdMz+qw=
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: AM7PR07MB6248.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 37fe68c6-013b-4cc0-d9bc-08d8a7616220
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Dec 2020 16:39:57.1338 (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: 0F2H5ZTOXuToW3447aPag7BNcrmSPiEw8vVQEKqJmzw3SOHRxFp0vI6c7mixgqUnmSRn05yiYNe0QkwDxgvxSw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR07MB4789
Archived-At: <https://mailarchive.ietf.org/arch/msg/yang-doctors/83HFwg7fRIJ3qC4AuY_VINv8mWM>
Subject: Re: [yang-doctors] [netmod] grouping prefix Re: [Teas] Yangdoctors early review of draft-ietf-teas-yang-path-computation-11
X-BeenThere: yang-doctors@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Email list of the yang-doctors directorate <yang-doctors.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/yang-doctors/>
List-Post: <mailto:yang-doctors@ietf.org>
List-Help: <mailto:yang-doctors-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 23 Dec 2020 16:40:03 -0000

<inline>

From: Martin Björklund <mbj+ietf@4668.se>
Sent: 23 December 2020 13:59

tom petch <ietfc@btconnect.com> wrote:
> Sorry about the incomplete e-mail.  Try again
>
> From: Teas <teas-bounces@ietf.org> on behalf of Martin Björklund via
> Datatracker <noreply@ietf.org>
> Sent: 17 December 2020 18:58
>
> Reviewer: Martin Björklund
> Review result: Ready with Nits
>
> <snip>
> 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).
>
> <tp>
> Martin,
>
> I am confused (which is why I will never be a YANG doctor:-(.
>
> RFC7950 p.104 says
>       Identifiers appearing inside
>    the grouping are resolved relative to the scope in which the grouping
>    is defined, not where it is used.
> I recall much debate about paths and their resolution but I cannot
> find a statement that paths are resolved where they are used in
> RFC7950, nor in the many Errata.

p. 104 further says:

   Prefix mappings, type names,
   grouping names, and extension usage are evaluated in the hierarchy
   where the "grouping" statement appears.


But in this case we have a leafref path w/o prefixes.

Section 6.4.1 says:

   o  Names without a namespace prefix belong to the same namespace as
      the identifier of the current node.  Inside a grouping, that
      namespace is affected by where the grouping is used (see
      Section 7.13).

And 7.13 says:

   The effect of a "uses" reference to a grouping is that the nodes
   defined by the grouping are copied into the current schema tree and
   are then updated according to the "refine" and "augment" statements.

   The identifiers defined in the grouping are not bound to a namespace
   until the contents of the grouping are added to the schema tree via a
   "uses" statement that does not appear inside a "grouping" statement,
   at which point they are bound to the namespace of the current module.

<tp>
Martin,

Thank you for the comprehensive reply;  it is what I would have remembered until I looked in RFC7950 to check my understanding and read that first sentence on p.104 'identifiers are .. resolved' which I took as identifiers being given a prefix and so given global uniqueness; the subsequent  sentence on prefixes, types etc. I am fine with , it is the 'resolved' that made me doubt my memory.  What is 'resolved' in this context?

Tom Petch

/martin



> And while I find teas-te almost impossible to understand because of
> how it is structured, yet
> the elements in
> type leafref {
>         path "/te/globals/named-path-constraints/"
>            + "named-path-constraint/name";
>       }
>
> would all appear to be in teas-te and not imported from
> e.g. te-typesin, in which case prefix are not needed.
>
> So while I am sure you are right, and this is a significant problem,
> yet I cannot see the chapter and verse to back it up.
>
> In passing, this construct, in teas-yang-path-computation has a
> history of YANG difficulties.
>
> Tom petch
>
>
>
>
>
>
>
>
> /martin
>
>
>
> _______________________________________________
> Teas mailing list
> Teas@ietf.org
> https://www.ietf.org/mailman/listinfo/teas
>
> _______________________________________________
> netmod mailing list
> netmod@ietf.org
> https://www.ietf.org/mailman/listinfo/netmod
>
> _______________________________________________
> netmod mailing list
> netmod@ietf.org
> https://www.ietf.org/mailman/listinfo/netmod