[core] RobW comments on draft-ietf-core-senml-more-units-04

"Rob Wilton (rwilton)" <rwilton@cisco.com> Mon, 17 February 2020 15:48 UTC

Return-Path: <rwilton@cisco.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 5C840120018; Mon, 17 Feb 2020 07:48:17 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.5
X-Spam-Level:
X-Spam-Status: No, score=-14.5 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=O7gjq992; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=ObW5ubdW
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 ILr4FTTsYfsn; Mon, 17 Feb 2020 07:48:14 -0800 (PST)
Received: from alln-iport-1.cisco.com (alln-iport-1.cisco.com [173.37.142.88]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1E0AB120072; Mon, 17 Feb 2020 07:48:14 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=12431; q=dns/txt; s=iport; t=1581954494; x=1583164094; h=from:to:subject:date:message-id:mime-version; bh=Moc1rvc34KnOxAObMOqbXTiDMklkT2QaANTx9iF64sU=; b=O7gjq992Y+saoXF6AdypVUh+5jwD6bNJHpGsw/USqjQuCMvWrZmQrVo5 DHD0VwZfspg2Ezq7AXoKH16GXYk5cKCmnDz52pek2dUFT9c7iS18fLMGz hOVcIDBqW+voyODpOYcRZ0V+r3CBvNXVbo7RDsDH0amZ4KMfj2objWcg2 g=;
IronPort-PHdr: =?us-ascii?q?9a23=3Az21aSRGGQ9wrzdxUlORKqZ1GYnJ96bzpIg4Y7I?= =?us-ascii?q?YmgLtSc6Oluo7vJ1Hb+e4z1A3SRYuO7fVChqKWqK3mVWEaqbe5+HEZON0pNV?= =?us-ascii?q?cejNkO2QkpAcqLE0r+eeT1bigmG8JqX15+9Hb9Ok9QS47z?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0DBGACGtUpe/5FdJa1mHgELHIMgL1A?= =?us-ascii?q?FbFggBAsqCodQA4p6TpVBhGGCUgNUCQEBAQwBAS0CBAEBhEACggQkOBMCAw0?= =?us-ascii?q?BAQUBAQECAQUEbYU3DIV5BhsTAQE4EQGBACYBBAEaGoMFgX1NAy4BAqEWAoE?= =?us-ascii?q?5iGKCJ4J/AQEFhQ8YggwJgTiMJBqBQT+BWIdXg0CCLJZGmTwKgjqWb5sajmm?= =?us-ascii?q?bMQIEAgQFAg4BAQWBaSKBWHAVGoMNUBgNjh2Dc4pTdIEpjGcBgQ8BAQ?=
X-IronPort-AV: E=Sophos;i="5.70,453,1574121600"; d="scan'208,217";a="430836072"
Received: from rcdn-core-9.cisco.com ([173.37.93.145]) by alln-iport-1.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 17 Feb 2020 15:48:11 +0000
Received: from XCH-ALN-004.cisco.com (xch-aln-004.cisco.com [173.36.7.14]) by rcdn-core-9.cisco.com (8.15.2/8.15.2) with ESMTPS id 01HFmBMM031819 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Mon, 17 Feb 2020 15:48:11 GMT
Received: from xhs-rtp-003.cisco.com (64.101.210.230) by XCH-ALN-004.cisco.com (173.36.7.14) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Mon, 17 Feb 2020 09:48:10 -0600
Received: from xhs-rtp-003.cisco.com (64.101.210.230) by xhs-rtp-003.cisco.com (64.101.210.230) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Mon, 17 Feb 2020 10:48:09 -0500
Received: from NAM02-CY1-obe.outbound.protection.outlook.com (64.101.32.56) by xhs-rtp-003.cisco.com (64.101.210.230) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Mon, 17 Feb 2020 10:48:09 -0500
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=ScY51JBuHB5EXU3YlVsDa4ty+PO/03IZLPmB+Cz+arUt4fXlIjmc3gMkiUheeUxyGUjMnNqLaIsxMRhJCSjqM5mh/nkJ3CGR17SDXpoyxYiyfzuuz2q/51NP438Uy23ETEa5K8BoWA3I3vPHncb0/Jc/dedHwW3n06uRbZN6FU8wovCZ2NWo3PiixNpA/BXutZYkoMSl0+BRMVRjQNwJ7LNd4znqLRRwKCqNSZ1N8PD8CqlLeJFG+iLA8aFytexVf8VuotCcoT2ih8C8E9hBGVBkHQ9n2u0UDKyheVwZ8YLMZhDMmt6T2nvK6DoqdTVL1gcW531tW1OCrLPabdSNpw==
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=vB4yVMl00b9RhnvZu3OT52qeOT369NyzrWkcqRKLoXk=; b=U0RAbJIKvI3YkUMzea89BlOZv/aSpUEs8MxojiVkXL2tX+4qxHRaNgMDommqsS58zgwd21d5No/nPKsIP8BmEfXDYqqizb5/6yYHWitW0s44K4ZFx4v58UE6CtUly7ZLUS9gerHcKZr5LYjQh13MxUR1xr4V6I/v8OAW480Fu11HsiXB00ESlTdpt6Wp5DMw+1ddRZx1+YX/S1E63ytRh7Y4JhHlq3CYlf2Hx5cZHVZE29TgeLoOnVAICdItZD/XHws16EWHdRTGZE5wM8zmThAdG6frMGUnmw4J+KPCVIKtJYOl9+K2yuVSOER1zbQRPujgLpbesCL2Xxz9bSxUEg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vB4yVMl00b9RhnvZu3OT52qeOT369NyzrWkcqRKLoXk=; b=ObW5ubdWb5TPXsYUyZLuhb3xV/WnNYGXFgp3ySlzMQxPn8dFPDI/gpWXCvX6EBdR9i3d0oDiSf5DWu8Qw2L+aasjL4ainESzag5FJYe/E8ej5ITAB/+kKEo8yvNQD55TGCJYCNHoN59fbiqEM0Lsli3RCOEVcGaQcGYCUZF2vMA=
Received: from MN2PR11MB4366.namprd11.prod.outlook.com (52.135.38.209) by MN2PR11MB4046.namprd11.prod.outlook.com (20.179.151.29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2729.25; Mon, 17 Feb 2020 15:48:08 +0000
Received: from MN2PR11MB4366.namprd11.prod.outlook.com ([fe80::b9ce:1058:5fa6:44a1]) by MN2PR11MB4366.namprd11.prod.outlook.com ([fe80::b9ce:1058:5fa6:44a1%7]) with mapi id 15.20.2729.031; Mon, 17 Feb 2020 15:48:08 +0000
From: "Rob Wilton (rwilton)" <rwilton@cisco.com>
To: "core@ietf.org" <core@ietf.org>, "core-chairs@ietf.org" <core-chairs@ietf.org>, The IESG <iesg@ietf.org>, "draft-ietf-core-senml-more-units@ietf.org" <draft-ietf-core-senml-more-units@ietf.org>
Thread-Topic: RobW comments on draft-ietf-core-senml-more-units-04
Thread-Index: AdXlqZeHMIQG61iWSRmLAl4AWyIkfw==
Date: Mon, 17 Feb 2020 15:48:08 +0000
Message-ID: <MN2PR11MB4366F25989E1D4062BBFBA1AB5160@MN2PR11MB4366.namprd11.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=rwilton@cisco.com;
x-originating-ip: [173.38.220.42]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: e3ce6066-d8b1-4c6e-5a15-08d7b3c0c90b
x-ms-traffictypediagnostic: MN2PR11MB4046:
x-microsoft-antispam-prvs: <MN2PR11MB40466AA73648C2C3F2EB8EFEB5160@MN2PR11MB4046.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:7691;
x-forefront-prvs: 0316567485
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(346002)(39860400002)(366004)(376002)(136003)(396003)(189003)(199004)(186003)(9686003)(316002)(55016002)(8676002)(2906002)(86362001)(71200400001)(8936002)(110136005)(6506007)(26005)(7696005)(33656002)(9326002)(450100002)(52536014)(478600001)(76116006)(66946007)(66476007)(66556008)(81166006)(81156014)(5660300002)(64756008)(66446008); DIR:OUT; SFP:1101; SCL:1; SRVR:MN2PR11MB4046; H:MN2PR11MB4366.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: dDmm1rGG6T3d7bo4CP3wx44tVwaAUybm4+cYcUjTWCzCk014L7SOiUyeiRPTxDqIRxQTQQXAv0a/PxO2xGplS3tNO82oxgE1GzgiUellN6DjriR5gPmVqYeigkpIpGxMJrESoZQ6PDdpw1PshEol8qYlYlu1EZ89dld1g+btJw6KXp7WnimH9SDAjl2lZ4im82kXqR0V1TdGV/PvYqLdxJTJZ7fO3Mm+WTeGjrSSZfSXdTV8LhOQ7O8KP4x4ro2GgzdPznzN8iL7pdtirBOZyqEqExC+4n5HQW9r9yOIyw3sjZ6xemRvQPlBC+zB9jW/HmcojrVAOQSDmeSJde6piy1NtfPwIIZYLEsmHUs+NWhJF+lfYvE/v8p+uDMM2PHEK4YDjL0ff/fUtEeBUECEtPaf/ED+mj8wqnZx8j6UpN2BMgXq3R5K+5SQeOuSHWCu
x-ms-exchange-antispam-messagedata: M/JoJhNMGvXIUnDaozYiff/BYt+WN/EqWCpRxKRreo+4ifiAscT57OnGgrJ0HJTxah7oiY+Ki3GYNQ16AuZN7P4bk0mMIqgKmMlp5/YMQX2C+wBFTkfhGh00sb0zaK0nYaA/TvSYHKu48TOfhFG/BA==
x-ms-exchange-transport-forked: True
Content-Type: multipart/alternative; boundary="_000_MN2PR11MB4366F25989E1D4062BBFBA1AB5160MN2PR11MB4366namp_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: e3ce6066-d8b1-4c6e-5a15-08d7b3c0c90b
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Feb 2020 15:48:08.2332 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: hk0faFk9SaKla821g8CZN7Na2SCqsAIqmCeuKnjk7lGH5txVAe/eHG4xhejAD2wvjbiXY2kMrJy6K7jChkYPUg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4046
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.36.7.14, xch-aln-004.cisco.com
X-Outbound-Node: rcdn-core-9.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/R35TkOgm326IezfIdEbkG2bkW00>
Subject: [core] RobW comments on draft-ietf-core-senml-more-units-04
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: Mon, 17 Feb 2020 15:48:18 -0000

Hi Carsten,

I'm providing some comments as an incoming Ops and Mgmt AD.

I found this document well written but think that it may be helpful to highlight the protocol usage of secondary units earlier in the document, i.e. in the introduction.

My understanding is that this document introduces both extra primary unit names, and also a new concept of secondary unit names, which I presume came about from deployment considerations.

Comments:

  1.  As per above, I think that it would be useful to add a comment in the introduction about how secondary units could be used.  E.g. may be extend



"The document also defines a registry for secondary Unit names that

   cannot be in SenML's main registry as they are derived by linear

   transformation from units already in that registry."



with the sentence "Although SenML version 10 [RFC8428] does not allow for the direct use of secondary units, they are planned to be supported via the use of SenML protocol extensions, such as [I-D.bormann-core-senml-versions]."


  1.  The introduction introduces the term "primary Unit names", but then doesn't refer to them using this term in the rest of the document.  It might be helpful if the title of section 2 was changed from "New Units" to "New primary Units", and the description could be changed from "... assign new units ..." to "... assign new primary unit names ...".



  1.  My reading of the "3. Rationale" section is that it relates to the new primary units, rather than secondary units.  The structure of the document might be clearer if this was made a subsection of the section 2 rather than its own top level section.



  1.  I would suggest renaming the section 4 title from "New Registry" to "New Registry for Secondary Units".





  1.  Regarding section 4 on secondary units:
     *   I observe that these are not using a generalized mechanism of using SI prefixes for scaling, but propose a separate custom mapping instead, which in some cases are just making use of SI prefixes.  I presume that there is a good reason for not wanting to generically use SI prefixes, perhaps for a bit more homogeneity in the scaling factors, but it might be helpful for the document to explain the rationale behind this.
     *   I note that you have similar units but with different scaling factors (e.g. KiB, GB, Mbit/s, MB/s).  Do we anticipate other scaling factors of these units?  E.g. if MB was requested as a secondary unit, then would that be added?  Would it make sense to define any more of these common scaling factors now?
     *   I also note that the names of some of the secondary units would overlap with SI prefixes (e.g. "min") for minutes.  This is okay, but would likely be an annoyance if it was ever desirable to use SI prefixes in a generic way.
     *   I wasn't sure whether "h" is really a great shorthand for hour, I would have gone for "hr" instead, but perhaps this is already widely used elsewhere?

Regards,
Rob