[core] Lars Eggert's No Objection on draft-ietf-core-senml-versions-03: (with COMMENT)

Lars Eggert via Datatracker <noreply@ietf.org> Wed, 26 May 2021 12:33 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: core@ietf.org
Delivered-To: core@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 722C23A2D19; Wed, 26 May 2021 05:33:43 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Lars Eggert via Datatracker <noreply@ietf.org>
To: "The IESG" <iesg@ietf.org>
Cc: draft-ietf-core-senml-versions@ietf.org, core-chairs@ietf.org, core@ietf.org, jaime@iki.fi, jaime@iki.fi
X-Test-IDTracker: no
X-IETF-IDTracker: 7.30.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Lars Eggert <lars@eggert.org>
Message-ID: <162203242344.29582.10148358869813812866@ietfa.amsl.com>
Date: Wed, 26 May 2021 05:33:43 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/UzRYfv2F3_S2r4sikfWIA2KMsqQ>
Subject: [core] Lars Eggert's No Objection on draft-ietf-core-senml-versions-03: (with COMMENT)
X-BeenThere: core@ietf.org
X-Mailman-Version: 2.1.29
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, 26 May 2021 12:33:44 -0000

Lars Eggert has entered the following ballot position for
draft-ietf-core-senml-versions-03: No Objection

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-core-senml-versions/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Section 7.1, paragraph 4, comment:
>    [IANA.senml]
>               IANA, "Sensor Measurement Lists (SenML)",
>               <http://www.iana.org/assignments/senml>.

Not sure if a normative reference to an IANA registry page is technically OK; it
should probably cite the RFC that created that registry instead, and maybe add
an informative reference to the registry page?

-------------------------------------------------------------------------------
All comments below are about very minor potential issues that you may choose to
address in some way - or ignore - as you see fit. Some were flagged by
automated tools (via https://github.com/larseggert/ietf-reviewtool), so there
will likely be some false positives. There is no need to let me know what you
did with these suggestions.

Section 2.1, paragraph 4, nit:
-    decimal numbers, e.g. 26 (0b11010, 0x1a) for a version that adds the
+    decimal numbers, e.g., 26 (0b11010, 0x1a) for a version that adds the
+                         +

The text version of this document contains these HTML entities, which might
indicate issues with its XML source: &nbsp;

Section 1, paragraph 2, nit:
> es additional features over time. However in the case of SenML it is expecte
>                                   ^^^^^^^
Did you forget a comma after a conjunctive/linking adverb?

Section 3, paragraph 2, nit:
>  secondary unit names [RFC8798] MAY be be used in the "u" field of SenML Reco
>                                     ^^^^^
Possible typo: you repeated a word

These URLs in the document can probably be converted to HTTPS:
 * http://www.iana.org/assignments/senml