[Gen-art] genart LC review of draft-ietf-nfsv4-rpc-netid-03.txt (and -04)

Robert Sparks <rjsparks@nostrum.com> Thu, 04 December 2008 19:52 UTC

Return-Path: <gen-art-bounces@ietf.org>
X-Original-To: gen-art-archive@optimus.ietf.org
Delivered-To: ietfarch-gen-art-archive@core3.amsl.com
Received: from [127.0.0.1] (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id 0027B3A67AB; Thu, 4 Dec 2008 11:52:13 -0800 (PST)
X-Original-To: gen-art@core3.amsl.com
Delivered-To: gen-art@core3.amsl.com
Received: from localhost (localhost [127.0.0.1]) by core3.amsl.com (Postfix) with ESMTP id A28843A6A64 for <gen-art@core3.amsl.com>; Thu, 4 Dec 2008 11:52:11 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.599
X-Spam-Level:
X-Spam-Status: No, score=-2.599 tagged_above=-999 required=5 tests=[BAYES_00=-2.599, HTML_MESSAGE=0.001, SPF_PASS=-0.001]
Received: from mail.ietf.org ([64.170.98.32]) by localhost (core3.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 5VsLOJbuiIen for <gen-art@core3.amsl.com>; Thu, 4 Dec 2008 11:52:10 -0800 (PST)
Received: from nostrum.com (nostrum-pt.tunnel.tserv2.fmt.ipv6.he.net [IPv6:2001:470:1f03:267::2]) by core3.amsl.com (Postfix) with ESMTP id 15C7C3A67AB for <gen-art@ietf.org>; Thu, 4 Dec 2008 11:52:09 -0800 (PST)
Received: from [192.168.2.6] (pool-71-170-80-166.dllstx.fios.verizon.net [71.170.80.166]) (authenticated bits=0) by nostrum.com (8.14.3/8.14.3) with ESMTP id mB4Jq2qP062688 (version=TLSv1/SSLv3 cipher=AES128-SHA bits=128 verify=NO); Thu, 4 Dec 2008 13:52:02 -0600 (CST) (envelope-from rjsparks@nostrum.com)
Message-Id: <32E142FF-7258-48DA-A55F-61D12DA8A52D@nostrum.com>
From: Robert Sparks <rjsparks@nostrum.com>
To: gen-art@ietf.org
Mime-Version: 1.0 (Apple Message framework v929.2)
Date: Thu, 04 Dec 2008 13:52:01 -0600
X-Mailer: Apple Mail (2.929.2)
Received-SPF: pass (nostrum.com: 71.170.80.166 is authenticated by a trusted mechanism)
X-Virus-Scanned: ClamAV 0.94.2/8721/Thu Dec 4 07:26:10 2008 on shaman.nostrum.com
X-Virus-Status: Clean
Cc: nsfv4-chairs@tools.ietf.org, mark@eisler.com, Lars Eggert <lars.eggert@nokia.com>
Subject: [Gen-art] genart LC review of draft-ietf-nfsv4-rpc-netid-03.txt (and -04)
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.9
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/pipermail/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>
Content-Type: multipart/mixed; boundary="===============0980946600=="
Sender: gen-art-bounces@ietf.org
Errors-To: gen-art-bounces@ietf.org

I have been selected as the General Area Review Team (Gen-ART)
reviewer for this draft (for background on Gen-ART, please see
http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html).

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-nfsv4-rpc-netid-03.txt (modified to take -04 into  
account)
Reviewer: Robert Sparks
Review Date: 4 Dec 2008
IETF LC End Date: 5 Dec 2008
IESG Telechat date: not known

Summary: This draft is basically ready for publication, but has nits  
that should be fixed before publication.

Comments:

I reviewed -03, but see -04 was submitted yesterday. I've modified my  
comments below based on -04, but any references to section numbers  
still point into the values the section had in -03.

1) There is an inconsistency between the text in 3.1 claiming "Netids  
can be up to 2^32 -1 octets in length" and the instructions in the  
rest of the document which restrict them to 128 octets.

2) The constraints placed on the "constant name" put a constraint  
around case-sensitivity on netid that you are probably assuming, but  
have not explicitly stated. Do you want to allow someone to register a  
netid of "foo" and someone else to register "Foo"? (btw - in the  
protocols that use these, is it clear whether they are case-sensitive?).

3) In a couple of places, the document says things like "For  
assignments made on a Standards Action basis, the point of contact is  
always the IESG". I suggest this should be something like "the point  
of contact is determined by the IESG" to make it easier to allow the  
IESG to delegate when appropriate.

4) Section 3.2 (IANA considerations for Uaddr Formats) is conflicted  
on what review is required for non-Standards Action registrations. It  
says First Come First Served at the top of the section, and Expert  
Review at the end. This same conflict appears in the first sentence of  
3.2.2.

5) Why are you putting length restrictions on what can go into the  
descriptive parts of the registry? (The description and point of  
contact)? Where did the limit values you selected come from?

6) If the explanation in 3.2.3.5 (Uaddr format for icmp) is complete  
the point is just to prevent someone else registering "icmp" or  
"icmp6". Why not just reserve those instead of registering them in  
this document? That would avoid leading someone into thinking  
something might send those values to them in a protocol message. Or is  
there more to why they're registered than what's explained in this  
section?



_______________________________________________
Gen-art mailing list
Gen-art@ietf.org
https://www.ietf.org/mailman/listinfo/gen-art