Re: Gen-ART LC review of draft-ietf-nfsv4-lfs-registry-02

Tom Haynes <thomas.haynes@primarydata.com> Wed, 11 February 2015 21:14 UTC

Return-Path: <thomas.haynes@primarydata.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 013AF1A19FE for <ietf@ietfa.amsl.com>; Wed, 11 Feb 2015 13:14:11 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.6
X-Spam-Level:
X-Spam-Status: No, score=-2.6 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7] autolearn=unavailable
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 PGH6y6CmcYBp for <ietf@ietfa.amsl.com>; Wed, 11 Feb 2015 13:14:08 -0800 (PST)
Received: from mail-pa0-f45.google.com (mail-pa0-f45.google.com [209.85.220.45]) (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 5E0FD1A06E9 for <ietf@ietf.org>; Wed, 11 Feb 2015 13:14:08 -0800 (PST)
Received: by mail-pa0-f45.google.com with SMTP id et14so6587471pad.4 for <ietf@ietf.org>; Wed, 11 Feb 2015 13:14:07 -0800 (PST)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:content-type:mime-version:subject:from :in-reply-to:date:cc:content-transfer-encoding:message-id:references :to; bh=R1aGGCt1fNDG15vLR0UGLsPXJPZt6AMohBh9XMGFXpA=; b=JIfWrzV4KPiPixvKBZRXHAkCykJ9uno2SdS0ivTG0MNJJTxsaJ5V6heOX8Fe6yS0OY OkFxt0K8HdD4tbAVswTdd67n2FEFsj298lt+rCVCYS/jHH2IJVbE9nFed37oxHo008wX kQQco/YtroxmcLVTDOSTBKHO3cfLLSsX8oZjvOVAbZ2B1KmlhUWjQQOUAiYYNCSpxLZX iolN6z/l75XaMYhs7IfP9M+pevfhCbuaBGHZAYD8pgncCWFWtRnj4Muo8JEoYWHnOWrw jhK4bWlBcIRJXmCcFh9AO1qkvs5dineZog3SXGVQNCxzs9gPCoqBynhiuadmMWy9CtHg evbA==
X-Gm-Message-State: ALoCoQnOQIycvQJlMoM5zRPjd07k2s3x7x9Lk10v2RRKXsTu9lqkAlS/2tP9r0+OwMq485fyLfdX
X-Received: by 10.70.21.132 with SMTP id v4mr919064pde.114.1423689247846; Wed, 11 Feb 2015 13:14:07 -0800 (PST)
Received: from [10.30.8.5] ([50.242.95.105]) by mx.google.com with ESMTPSA id j9sm1708602pbq.66.2015.02.11.13.14.06 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Wed, 11 Feb 2015 13:14:07 -0800 (PST)
Content-Type: text/plain; charset="windows-1252"
Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2070.6\))
Subject: Re: Gen-ART LC review of draft-ietf-nfsv4-lfs-registry-02
From: Tom Haynes <thomas.haynes@primarydata.com>
In-Reply-To: <54DB4258.9050105@isode.com>
Date: Wed, 11 Feb 2015 13:14:05 -0800
Content-Transfer-Encoding: quoted-printable
Message-Id: <D85D2CD2-6C8B-4740-B934-986CD2AE6032@primarydata.com>
References: <54DB4258.9050105@isode.com>
To: Alexey Melnikov <alexey.melnikov@isode.com>
X-Mailer: Apple Mail (2.2070.6)
Archived-At: <http://mailarchive.ietf.org/arch/msg/ietf/LZrWrx4FynMF69jLoLGVwQnRkMk>
Cc: gen-art@ietf.org, ietf@ietf.org, draft-ietf-nfsv4-lfs-registry.all@tools.ietf.org
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 11 Feb 2015 21:14:11 -0000

Hi Alex,

Thanks for the review.

Comments inline.

Tom

> On Feb 11, 2015, at 3:51 AM, Alexey Melnikov <alexey.melnikov@isode.com> 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-nfsv4-lfs-registry-02
> Reviewer: Alexey Melnikov
> Review Date: 2015-02-11
> IETF LC End Date: 2015-02-16
> IESG Telechat date: N/A.
> 
> Summary: This draft is nearly ready for publication as a standard track RFC (with nits).
> 
> Major issues:
> Minor issues:
> 
> In Section 4:
> 
> "LSF" is used for the first time without being expanded. I suggest you introduce the abbreviation in the terminology section.

I think I prefer to expand it as there are two possible expansions and only one use of it:


4.  Security Considerations

   This document defines a mechanism to associate LFS identifier with a

->

4.  Security Considerations

   This document defines a mechanism to associate the Label Format Specifier to a


> 
> In Section 5:
> 
> Label Description: - what is the allowed character set for this field? Is it ASCII? Is it UTF-8 with some restrictions?


   Label Description:  A human readable ASCII text string that describes


> 
> >Status:  A short ASCII text string indicating the status of an entry
> >       in the registry.  The status field for most entries should have
> >       the value "active".  In the case that a label format selection
> >       entry is obsolete, the status field of the obsoleted entry should
> >       be "obsoleted by entry NNN".
> 
> What is entry NNN? Is it a document reference (e.g. An RFC)?


It is another entry in the registry. That new entry will provide the mapping to a document reference.



> Is it possible to obsolete without such entry?

No, Section 5.3 is clear on that.




> 
> In Section 5.3 - is it possible to update a label description document without requesting a new label? For example if changes are editorial and don't significantly affect label syntax and model.
> 


Two considerations:

1) Edit of “Description” - I don’t see a problem with allowing this to occur.

2) Edit of “Reference” - Which is what I think you are asking about here. 

If we consider IETF created RFCs, we know that a -bis is a legitimate need for an update as it obsoletes the earlier RFC.

And if we consider non-IETF created documents, I think we need to fall back Designated Expert reviewer to answer whether the new document requires a new label or we can allow an edit.

This is rough, but I’d envision a new Section 5.4:

5.4.  Modifying an Existing Entry in the Registry

  A request to modify  either the Description or the published
  label format specification will also require the Specification
  Required IANA policy to be applied. The Designated Expert reviewer
  will need to determine if the published label format specification
  either

  obsoletes the Label Format Specifier - follow the process in Section 5.2.

  updates the label syntax and/or model - approve the change.



> 
> Nits/editorial comments:
>