Re: [core] 🔔 WG Last Call of draft-ietf-core-senml-etch-03.txt

Klaus Hartke <hartke@projectcool.de> Mon, 24 June 2019 11:22 UTC

Return-Path: <hartke@projectcool.de>
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 24CC012008B for <core@ietfa.amsl.com>; Mon, 24 Jun 2019 04:22:15 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 Vv_qzE14dlag for <core@ietfa.amsl.com>; Mon, 24 Jun 2019 04:22:12 -0700 (PDT)
Received: from wp382.webpack.hosteurope.de (wp382.webpack.hosteurope.de [IPv6:2a01:488:42:1000:50ed:8597::]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id AAE87120048 for <core@ietf.org>; Mon, 24 Jun 2019 04:22:12 -0700 (PDT)
Received: from mail-qt1-f171.google.com ([209.85.160.171]); authenticated by wp382.webpack.hosteurope.de running ExIM with esmtpsa (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) id 1hfN32-0006FG-Rh; Mon, 24 Jun 2019 13:22:08 +0200
Received: by mail-qt1-f171.google.com with SMTP id a15so13997461qtn.7 for <core@ietf.org>; Mon, 24 Jun 2019 04:22:08 -0700 (PDT)
X-Gm-Message-State: APjAAAUTqgPBeQhEHXQL3+bU61RlsHD194kexKwec1PnhRU/yLmR381p NrUVEc6ARUJy8YIhrUd0m6LpVPDGLNb576AR7sI=
X-Google-Smtp-Source: APXvYqzgbOBFdAnhvZdn81QIfe/Xg0rJXnw0LjVGgwd6gGI1gorWBweIqqXORyAQjVzujPlQ4NLNKtYtnNAlLmdE/94=
X-Received: by 2002:aed:39e7:: with SMTP id m94mr51068537qte.0.1561375327791; Mon, 24 Jun 2019 04:22:07 -0700 (PDT)
MIME-Version: 1.0
References: <9AD3C4BB-7965-4776-84C4-6B5BFDCAA262@tzi.org> <e3a61d2c-1183-5ece-74d8-b1bad26ddfe6@ericsson.com> <3E80442D-9EBF-4973-89E1-7B4A69F42754@tzi.org> <E8AD8AB7-DF40-4049-901E-18C0655A3DAD@tzi.org>
In-Reply-To: <E8AD8AB7-DF40-4049-901E-18C0655A3DAD@tzi.org>
From: Klaus Hartke <hartke@projectcool.de>
Date: Mon, 24 Jun 2019 13:21:31 +0200
X-Gmail-Original-Message-ID: <CAAzbHvbT6Dyjh+33Mnb74wk1oh8iSWo-hUPq1hpBwgWyPe15Bg@mail.gmail.com>
Message-ID: <CAAzbHvbT6Dyjh+33Mnb74wk1oh8iSWo-hUPq1hpBwgWyPe15Bg@mail.gmail.com>
To: Carsten Bormann <cabo@tzi.org>
Cc: core <core@ietf.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
X-bounce-key: webpack.hosteurope.de; hartke@projectcool.de; 1561375332; 9ab4c01a;
X-HE-SMSGID: 1hfN32-0006FG-Rh
Archived-At: <https://mailarchive.ietf.org/arch/msg/core/iCF7TQKhsQGJp7h2Kgr6qtIza60>
Subject: Re: [core] =?utf-8?q?=F0=9F=94=94_WG_Last_Call_of_draft-ietf-core-se?= =?utf-8?q?nml-etch-03=2Etxt?=
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, 24 Jun 2019 11:22:15 -0000

I have performed a quick review of draft-ietf-core-senml-etch-03:

3.2. "For example, the following document could be given as (i)PATCH
payload to change/set values of two SenML Records for the example in
Section 1:" -- It would be nice if the example from section 1 was
duplicated here and the removal of a record was shown.

5.1 "All IDs are assigned from the "IETF Review or IESG Approval"
range." -- Remove this sentence.

5.1 "Table 1: CoAP Content-Format IDs" -- This table is missing the
"Content-Coding" (f.k.a. "Encoding") column.

General questions:

* It seems the new data formats are supersets of the existing data
formats. Also, it seems the same data format is used for fetching and
patching data. Is this what we, as CoRE WG, recommend (1 media type
for data, 1 media type for fetching and patching)? CoRAL is doing it
differently at the moment [1].

Klaus

[1] https://tools.ietf.org/html/draft-hartke-t2trg-coral-08#section-6.7





On Wed, 12 Jun 2019 at 15:10, Carsten Bormann <cabo@tzi.org> wrote:
>
> It seems we have received no (zero, nada) responses to this.
> We do need a few voices from people who have read it.
> Please speak up if you did!
>
> It’s a short document, so you can read it right now as well:
> https://tools.ietf.org/html/draft-ietf-core-senml-etch-03
>
> Grüße, Carsten
>
>
>
>
> > On Mar 11, 2019, at 23:40, Carsten Bormann <cabo@tzi.org> wrote:
> >
> > On Mar 6, 2019, at 00:40, Ari Keränen <ari.keranen@ericsson.com> wrote:
> >>
> >> If there are no further comments on these, I could merge the PR and
> >> submit a new version.
> >
> > … which the authors did (see below).
> >
> > This starts a working group last call for draft-ietf-core-senml-etch-03.txt,
> > ending on
> >
> >       24:00 CET on Monday, March 25, 2019.
> >
> > Please start a new email thread for each major issue that will need
> > discussion and make sure the subject line includes the draft name and
> > some sort of name for the issue.  (Minor issues such as typos can also
> > be sent to the authors.)
> >
> > If you read the draft and think it looks fine, please send a one line
> > email to the list or to the chairs letting us know that so we can get
> > a feel of how broad the review has been.
> >
> > (To reviewers and authors:)  If you are aware of any patent claims that
> > might apply to systems that implement these drafts, please review BCP 78
> > and BCP 79 and make any appropriate IPR declaration before the last-call
> > ends. If you are not sure whether you need to make a declaration or not,
> > please talk to the chairs and we will help.
> >
> > Grüße, Carsten
> >
> >>
> >> Cheers,
> >> Ari
> >>
> >> On 20/02/2019 17.00, Carsten Bormann wrote:
> >>> In preparation for the impending working-group last call, I have
> >>> reviewed draft-ietf-core-senml-etch-02.txt.  Substantive comments are
> >>> below; a set of small editorial comments is going to the authors in
> >>> parallel.
> >>>
> >>> # Minor
> >>>
> >>> 1. The text of the draft does not contain an argument that the
> >>>   application of Patch Packs is idempotent.  It probably should.
> >>>
> >>> 2. The text sometimes says "iPATCH" where it probably should be saying
> >>>   "PATCH and iPATCH".  This should be checked throughout.
> >>>
> >>> 3. The term "resource" is sometimes used in lieu of "subset of SenML
> >>>   records that, when resolved, match a given name".
> >>>
> >>> Grüße, Carsten
> >>>
> >>>
> >>>
> >>> _______________________________________________
> >>> core mailing list
> >>> core@ietf.org
> >>> https://www.ietf.org/mailman/listinfo/core
> >>>
> >>
> >>
> >>
> >
> >
> >
>
> _______________________________________________
> core mailing list
> core@ietf.org
> https://www.ietf.org/mailman/listinfo/core