Re: [Gen-art] [core] Review: draft-ietf-core-link-format-11

Zach Shelby <zach@sensinode.com> Thu, 08 March 2012 12:55 UTC

Return-Path: <zach@sensinode.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DE57121F8611; Thu, 8 Mar 2012 04:55:27 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -3.654
X-Spam-Level:
X-Spam-Status: No, score=-3.654 tagged_above=-999 required=5 tests=[AWL=-0.055, BAYES_00=-2.599, RCVD_IN_DNSWL_LOW=-1]
Received: from mail.ietf.org ([12.22.58.30]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id nM0Zii84-K0z; Thu, 8 Mar 2012 04:55:27 -0800 (PST)
Received: from auth-smtp.nebula.fi (auth-smtp.nebula.fi [217.30.180.105]) by ietfa.amsl.com (Postfix) with ESMTP id E072121F8607; Thu, 8 Mar 2012 04:55:26 -0800 (PST)
Received: from [62.145.172.52] ([62.145.172.52]) (authenticated bits=0) by auth-smtp.nebula.fi (8.13.8/8.13.4) with ESMTP id q28CtNsZ011294; Thu, 8 Mar 2012 14:55:24 +0200
Mime-Version: 1.0 (Apple Message framework v1084)
Content-Type: text/plain; charset="us-ascii"
From: Zach Shelby <zach@sensinode.com>
In-Reply-To: <4F3FFB16.6030502@joelhalpern.com>
Date: Thu, 08 Mar 2012 14:55:23 +0200
Content-Transfer-Encoding: quoted-printable
Message-Id: <78A0D3BA-5EFE-4AD7-ACD6-56196CB588CF@sensinode.com>
References: <4F3D931C.1000101@nostrum.com> <4F3FFB16.6030502@joelhalpern.com>
To: "Joel M. Halpern" <jmh@joelhalpern.com>
X-Mailer: Apple Mail (2.1084)
X-Mailman-Approved-At: Thu, 08 Mar 2012 04:57:11 -0800
Cc: gen-art@ietf.org, Carsten Bormann <cabo@tzi.org>, core WG <core@ietf.org>
Subject: Re: [Gen-art] [core] Review: draft-ietf-core-link-format-11
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.12
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 08 Mar 2012 12:55:28 -0000

Joel,

Thanks for the review. See comments regarding your suggestions in-ine:

On Feb 18, 2012, at 9:25 PM, Joel M. Halpern wrote:

> I am the assigned Gen-ART reviewer for this draft. For background on Gen-ART, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
> 
> Please resolve these comments along with any other Last Call comments you may receive.
> 
> Document: draft-ietf-core-link-format-11
>    CoRE Link Format
> Reviewer: Joel M. Halpern
> Review Date: 18-Feb 2012
> IETF LC End Date: 28-Feb-2012
> IESG Telechat date: N/A
> 
> Summary: This document is almost ready for publication as a proposed standard.
> 
> Major issues:
>    What is the registration / collision avoidance strategy for resource type (rt) and interface description (if) values?  They are both defined as opaque strings which can happen to be URIs.  So there seems to be potential for collision.

There is definitely a possibility for collision for those two attributes, especially as there will be a mix of specifications that define specific values to be used, and developers using their own values. Currently while those are being defined in CoRE drafts we can avoid collisions, but when other WGs or SDOs start defining them...

We have deliberated on the idea of defining registries for rt= and if= values, but it has not been clear if that should be done in this document. Recently several CoRE drafts have been written that do specify well-known values for those fields, so it starts to become obvious that those registries would be useful. If I understand right, your recommendation would be that we define those registries in the IANA section of this document? I would be in favor of doing that.

> Minor issues:
>    Would it be sensible, in the introduction, when the well-known URI is first introduced, to refer to it as a "well-known relative URI"?

Sure, will fix.

>    Should this document register a resource type (rt) and interface description (if) for the server resource manager itself?  The text refers to using multicast to find resources, with filtering based on rt and if.  (This would, I think, be defined in section 4, and registered in the IANA considerations section?)  Presumably all devices would support the interface, which is why a resource type for the master server would seem useful.

I am not sure what you mean by "master server" here. Maybe you are referring to something like a resource directory defined in [http://tools.ietf.org/id/draft-shelby-core-resource-directory-02.txt]? Yes, that specification does specify a well-known rt= value "core-rd" with which any node can discovery the location of a resource directory.

Thanks,
Zach

> 
> Nits/editorial comments:
> _______________________________________________
> core mailing list
> core@ietf.org
> https://www.ietf.org/mailman/listinfo/core

-- 
Zach Shelby, Chief Nerd, Sensinode Ltd.
http://www.sensinode.com
http://zachshelby.org  - My blog "On the Internet of Things"
http://6lowpan.net - My book "6LoWPAN: The Wireless Embedded Internet"
Mobile: +358 40 7796297