[core] Review of draft-ietf-core-yang-cbor-15

Francesca Palombini <francesca.palombini@ericsson.com> Wed, 24 February 2021 12:41 UTC

Return-Path: <francesca.palombini@ericsson.com>
X-Original-To: core@ietfa.amsl.com
Delivered-To: core@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id BB68E3A1517; Wed, 24 Feb 2021 04:41:26 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.671
X-Spam-Level:
X-Spam-Status: No, score=-2.671 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.57, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-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=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 i6yBCmEw7JtF; Wed, 24 Feb 2021 04:41:25 -0800 (PST)
Received: from EUR03-AM5-obe.outbound.protection.outlook.com (mail-eopbgr30062.outbound.protection.outlook.com [40.107.3.62]) (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 F099A3A1516; Wed, 24 Feb 2021 04:41:24 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KrDDRWp+buN0ZcR9oPXihdg6EbrEd4l3C8RuTucGGJqiUtb8grbLSHLXV+aeCeyrH5500GeHuVJrgkn10HwHNbxQ1LswpnRKVccr43Bk2T4CfoF1qUEpSArbcy7sCDl/HSEVl8I1o6jQIHmoFoCAMuPqnejoIwtr9MoaCZnYHExvN1KbRqDzSK1kex9uVvmAgqFV8z9usK7aOxfE4iHC1Q2GbP/zdzNctA2lAOarwgCxl3iNDUluAkGBIDuAWrHmZtVSwnGSm06g/samTALMmg8SoIbMMyuHa0hv1AsxDIQbpOuzBXDEmO89RubkE7r/TdZB4vq0CXJjBJuEPySGHQ==
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=B/uxp9esruQOVVKyGPSUvF6cIFLA0X4rYUp+Eg27ocU=; b=S5f3RMeNG/ge3wFzq6I+r8olSTbfMtGLl2dZTR1K2xILfg6J4AVjVklhJoGePSJqy6I7rvT7UddsP4+moMdln0qvZcmyh9487UK4RulXzs9XVoXCvfDXc8NZyW4+FHE/HChOOHWpOT6Ijnyzi+N9l3fSUHgiC5NX517PGbvu+z8M3L6QQAo9HiQ6oL7NLUM13sbQWU8isJ5a+XVUgQBzAsfX2MOKOK5gJKHNkfk2WB9A4WBc1B1xIl38/YjeWr/LmhXBkgTFHVLo8pcp0hx5M84cCxMJGkdV1h2P49gwq1t0lZmnXaPhg2vt2udfcZdMnvgmddMHRx9WykuHznM6bg==
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=B/uxp9esruQOVVKyGPSUvF6cIFLA0X4rYUp+Eg27ocU=; b=kkcrR+ppdxIgSFpZXCO03OoTcxwAz707WLbVGxOIxGu1uRsqZfoWotUibRWaGi88CF/Sev6b69NAIY62z6+eYbZrzskLqIFDtMg1YOsWPGB6Yvgwlcg9zFYJ25Otm0VFzTXJNDjSrjaD8XUmzaG9PGtp0pAdWOTQXbaQ5rYjAoE=
Received: from VI1PR07MB4477.eurprd07.prod.outlook.com (2603:10a6:803:74::33) by VI1PR0701MB7022.eurprd07.prod.outlook.com (2603:10a6:800:17e::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3868.14; Wed, 24 Feb 2021 12:41:23 +0000
Received: from VI1PR07MB4477.eurprd07.prod.outlook.com ([fe80::c5e9:fb9a:e4a0:e7a4]) by VI1PR07MB4477.eurprd07.prod.outlook.com ([fe80::c5e9:fb9a:e4a0:e7a4%5]) with mapi id 15.20.3890.018; Wed, 24 Feb 2021 12:41:23 +0000
From: Francesca Palombini <francesca.palombini@ericsson.com>
To: "draft-ietf-core-yang-cbor@ietf.org" <draft-ietf-core-yang-cbor@ietf.org>
CC: "core@ietf.org" <core@ietf.org>, "core-chairs@ietf.org" <core-chairs@ietf.org>, "barryleiba@computer.org" <barryleiba@computer.org>
Thread-Topic: Review of draft-ietf-core-yang-cbor-15
Thread-Index: AQHXCqpbWqAoYRZiFki+vT40wc8LMw==
Date: Wed, 24 Feb 2021 12:41:22 +0000
Message-ID: <AC9738EF-04A8-42FA-BE88-884CDDDDF4D2@ericsson.com>
Accept-Language: en-GB, en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/16.46.21021202
authentication-results: ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=none action=none header.from=ericsson.com;
x-originating-ip: [2001:1ba8:147a:c100:2419:35d:8efa:cace]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 14f80e43-92ab-47ec-2c8e-08d8d8c17e36
x-ms-traffictypediagnostic: VI1PR0701MB7022:
x-ms-exchange-minimumurldomainage: rfc-editor.org#8321
x-microsoft-antispam-prvs: <VI1PR0701MB7022823DB962B171F0985E0C989F9@VI1PR0701MB7022.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: IdHXX1+WhLaXVZSYYMaX17D6HjJFHBoIlp7DYUVUsVvKviXObdurmQ2WT14Ksg6Cc9z9UdJb2mI5Xb8bikl2MYC5aCCuOQBZrYddE+vZr/kJQY0ssnIOsrV2DYR27I5XFQ5+ZJmwCx818VKRk11MD8mFD1Jc8cBQrNvtZ3GlglV22k5IUkBL7TTo+m02htuh3U6r7CanylB5bYwvaaT/FX8aJH7LIk0ikRR3i9Ff5i3QVkh+kceSRV0aUeq3jFC917BGgmC24yqxEMgBHuWmz/c1LZfQj74Q7/GX/hXyYIDw/pTJw3jS7rWK8V7N7OTi+mjj93X3jkJo5D07Y8WkIGwyPFieIQpih0X6QRrQ3PXicnBATrWMR0Ptyk12bxzyOuF9WrAn5dcFBayNjCGvStZQKEcPx7AiMaVbBdsz88sAPeejBa5M5mZRpxyJM4Zn4qcv81nl2yIckfUdqkrRyFMTjXjAdSp3lss7LR41fFE9Si0HwqeFjiQ0ZB195iTz/Be30IO2WcfHFFWoFS+DDe2wIIInc02tH2egG2XmP99WaNBH5geYsCvaSrq485HosWzDQnUxZMgZgN4lKWUDFGOXNd+HNuY0yahV/6cNi3Q/muBFH7RrsZJb8ChHiPFkYqeMFN8gsPl4fjsupZo8kWmY91zXJbVDfyj/R6DILp0=
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR07MB4477.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(346002)(366004)(396003)(376002)(136003)(39860400002)(8676002)(54906003)(64756008)(186003)(6916009)(316002)(6486002)(33656002)(66556008)(91956017)(966005)(66476007)(86362001)(5660300002)(4326008)(6512007)(66446008)(8936002)(2616005)(478600001)(76116006)(66946007)(83380400001)(36756003)(6506007)(44832011)(2906002)(71200400001)(45980500001); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: =?utf-8?B?c3NCSEN4ZnpmL0ZqaHJ3Y2V2UjNiZUZpRmVyTSs0VEoxQjhXUElneFR6SnJT?= =?utf-8?B?MFRJRDhYVUZpOU1jRkVYUm5qa2RIa0MxelJQMHUvd0MyVDZSQVhFckdraitM?= =?utf-8?B?VWNBenFjYWhlcWNnclBJaFdXY250bEV6bWZHMHoybWppZWpQeUtlSUxaelVV?= =?utf-8?B?T2YvcndxYmdaZDdyMVA2bXNFZ0NuUm1oWlhhWlROVDh1ZW54SUQ3MlBwM0gr?= =?utf-8?B?am94cENYd0FYc3RRVGo3d2JXbG8rYjQ0d3ZseWtNY3Q3cUF5WkJtSks1RVA3?= =?utf-8?B?LzJQNHlmSXp3aWRNY1MxdWdtY0tQZThMR0tBaVJmKzQ4NjQ5RldIVWYzMEdG?= =?utf-8?B?MjlRT0JQczV0RkQwM2dKL1JFM2VRKzVwb2QxalVXam5qanF2RUN0S28wZXdI?= =?utf-8?B?NXdDK3UvS25IMUhYTUR5TFN3L2lSeC9uQ3FjU0M0WlNrSGJ3c0dScEg4WHBH?= =?utf-8?B?WTFIOGNHdVpZSUlQRTR0S0N3MjZ3SzdJcVpVS0xWU29PemhXdUhCTnJDbmdM?= =?utf-8?B?M2QvTkRrTlFmVnJnMDFpcHV1Qzl5STNXZmhDWE1jMmp3U2JCZjdKNFJES3pP?= =?utf-8?B?bWRKVHVZRkpCNFFlNjI3dW1jL0U2dnVLN0JZMmwyNDloNjd6aXEyWEZWVDU2?= =?utf-8?B?M1c2ZHJGSHJzM1RzVGUzTWxqaitKT3RVYlBlQVVWU0RUaFdpYUdWZW00VDlo?= =?utf-8?B?ZnhWVlpQTTRQS0wrNXh6RnVFYWVuRUlHTThWN3R6UDROWUNKSDBlL0RleWRX?= =?utf-8?B?L2lOR2lpNzV3ZlRzaHJQK1hLeVp1bHNZbllVUHc5NUZYaUFpRndhYWU0Skxy?= =?utf-8?B?ZGN4N255REpFcmFDbEt4alNsQUFCcnZBcnZ6aGcxc3lPZW40SmdKa3hKdlFP?= =?utf-8?B?UUNpTHk3Qzl0dkVQb0c0elVZUnd6aCt0MzJvdE5rYURsd1d5ZVBHUWtxYitk?= =?utf-8?B?Tjc1Zk5nQjZ3Sm1qSmpqZXFFTWRLMm9PQ1p4OFpNR1ViUjFQNnMvTEw2SnIr?= =?utf-8?B?YThvT09QMW1panVxL2U3MjRBOVpRaVdzSGlZM1RqMVNsd0Frc0hkQUFFdjdD?= =?utf-8?B?ZXo1dW1WWmFOdWJpZWl6SzNkcFB0QzQ0ZklvYnl1K1pGS2tiaUw4RDVnamp5?= =?utf-8?B?dUZwc0JYTHNUS2E0am1WMmZKUjZYSGVxS0Z0QThvbERCaWt2VngyU05VSndq?= =?utf-8?B?ZEpERkRHL0hIQ0FuU0d2TGNGUmNkeS9jTzc3RWZmZElpSmY5ZGhCRURtQ2hi?= =?utf-8?B?UnpoeXNmb0Jmd0lybUNpYTNFS1c5cXFINkNXcE1OTWF6NUltNG9tZVNTdFFL?= =?utf-8?B?TWkrMEQxQUxyTjNZRVFyaGo4OHhzZHlmOGRicEtVSkx3NEVBMTExNHZNS01l?= =?utf-8?B?WFFNT3RFNkJoZ3dDMkNIdlg1S1ZqMHpqVjQrOFcrK1FjYVEvck1hR1JFSTRZ?= =?utf-8?B?dFFyWVBXYXZtYkM3Z0FIRVJHdnZIMlB1c0lMV0tYaFFsQlNQVGdaczdnSFV4?= =?utf-8?B?RWgxYTFGLzJ1SlNaZHo4c0NtYW9qZkxFL2I1YzUvb3QweDVIajFTbHc3ZmZF?= =?utf-8?B?aDIvL3BINHZSRUdiSnNudGJ0Q3J6cW9qOTJxQzEwdC9MRVpYT2tEcE93bXpB?= =?utf-8?B?Q1Y3cUJjMlNwZmdzekxBTDhqYUxrQnIwcnc1R2hNTDNVaWd3ajg2TGNWcmNF?= =?utf-8?B?YzNkc3d6dFFFUTkwdldaeHhsNDgrcTVCVEFLOUkxY0k1WEZKUVQ2SDB0Ynpt?= =?utf-8?B?a2dVSk91WkxhcXpUY0phaFVQb1lUTkNtYjZEY2x3a1VuUW5helFiUk5xYXlH?= =?utf-8?B?MTFvRzQycXZyVEEzSHdOcHovTDE1b0hXOElvTm01WDVWVjVYRzNLWHhqQ2g4?= =?utf-8?B?K3BGZkhLRmZ5SmcybURzclV4SGRkQVhVMCtYK3Z2REVmSkE9PQ==?=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-ID: <AD848DF3240F804D9014E1175A61F872@eurprd07.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR07MB4477.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 14f80e43-92ab-47ec-2c8e-08d8d8c17e36
X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Feb 2021 12:41:23.0052 (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: N42Wkt+NqJ+6ZhxFDX74tsQQiyOOpYzAmvR/l8RfWDTgNQ5tyxNmwIi8NqZoz84Ci2vPRGrqTmzc6tvtJaiHurkWOugB14z+BSYk9dVYs3xG97Kdu25QcB3ZAjHhiIvp
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR0701MB7022
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/3w1kO2b5KBylJlkWCAuNw2OtG7U>
Subject: [core] Review of draft-ietf-core-yang-cbor-15
X-BeenThere: core@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Constrained RESTful Environments \(CoRE\) Working Group list" <core.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/core>, <mailto:core-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/core/>
List-Post: <mailto:core@ietf.org>
List-Help: <mailto:core-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/core>, <mailto:core-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 24 Feb 2021 12:41:27 -0000

 Thanks for the work on this document. I have reviewed it and have some comments. Please handle these comments together with the Last Call comments.

Chairs: I believe it would make sense to request a review from YANG Doctors for this document. Let me know if you need help setting that up.

Thanks,
Francesca

==

4.4.  The 'list' and 'list' instance(s) - should this second 'list' be "subset of 'list'"

==

Section 4.6:

anyxml value MAY
   contain CBOR data items tagged with one of the tags listed in
   Section 9.3, these tags shall be supported.

Why not BCP 14 SHALL?

==

* General question: about the SID values used throughout the document: as far as I can tell these are not allocated right now (and they will take some time to get registered as defined in core-sid document) It might be good to add a sentence about where the SID values from the examples come from.

==

Section 6.6 (and other sections that use tags defined by this document):

There is a new CBOR tag appearing in this section's example, tag 44. This is quite surprising without any text. Although the registration follows the template from RFC8949, I would have expected to see some descriptive text about the tag (and not only appearing in the sentence "The encoding MUST be enclosed by the enumeration CBOR tag as specified in Section 9.3." - note that section 9.3 does not specify how to use the tags).
This is valid for the following sections specifying new CBOR tags as well.

==

Section 6.7:

I find it hard to understand the example, without any indication on which bits are set. 

==

Nits

- expand acronyms on first use ( e.g. NETCONF, RPC, see https://www.rfc-editor.org/materials/abbrev.expansion.txt )

- move YANG SID terminology description before delta description.

Section 4.5

   module event-log {
     ...
     anydata last-event;                # SID 60123

Please close the parenthesis of this example