Skip to main content

Last Call Review of draft-ietf-nfsv4-rfc3530bis-dot-x-22
review-ietf-nfsv4-rfc3530bis-dot-x-22-genart-lc-davies-2014-12-03-00

Request Review of draft-ietf-nfsv4-rfc3530bis-dot-x
Requested revision No specific revision (document currently at 24)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2014-10-06
Requested 2014-09-25
Authors Thomas Haynes , David Noveck
Draft last updated 2014-12-03
Completed reviews Genart Last Call review of -16 by Kathleen Moriarty (diff)
Genart Last Call review of -22 by Elwyn B. Davies (diff)
Genart Last Call review of -22 by Martin Thomson (diff)
Genart Telechat review of -22 by Elwyn B. Davies (diff)
Secdir Last Call review of -16 by Magnus Nystrom (diff)
Assignment Reviewer Elwyn B. Davies
State Completed
Review review-ietf-nfsv4-rfc3530bis-dot-x-22-genart-lc-davies-2014-12-03
Reviewed revision 22 (document currently at 24)
Result Ready with Nits
Completed 2014-12-03
review-ietf-nfsv4-rfc3530bis-dot-x-22-genart-lc-davies-2014-12-03-00
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-rfc3530bis-dot-x-22.txt
Reviewer: Elwyn Davies
Review Date: 2014-11-12
IETF LC End Date: 2014-10-06
IESG Telechat date: 2014-12-04

Summary:
The main aims of this review were to


1. Check that the extracted .x file actually compiled (it didn't) - fix 


below.



2. To check the text corresponded with draft-ietf-nfsv4-rfc3530bis-33/34

I found a few minor glitches but otherwise this draft is almost ready.

Major Issues:
(well not really major)
The extracted .x file does not currently compile with rpcgen.


The cause is the definition of linktext4 (at line 154 of the extracted 


file) which was just fixed in draft-ietf-nfsv4-rfc3530bis-34 but not in 


this one.



s/linktext4/linktext4<>/


Minor Issues:

The type used for the lease_time attribute in -dot-x-22 is uint32_t, i.e.,
typedef uint32_t fattr4_lease_time;


In 3530bis the type shown in Table 2 is nfs_lease4 but there is no 


mapping from this type to a base integer type in either document.  This 


needs cleaning up.




========================

Differences between rfc3530-33/34 and rfc35230bus-dot-x-22:
-----------------------------------------------------------



The items below need fixing up in 3530bis (apart from possibly the 


lease_time type) to make them consistent with -dot-x-22.






- Error codes NFS4ERR_CB_PATH_DOWN and NFS4ERR_SHARE_DENIED are missing 


from table 5 in rfc3530-33 and there are no corresponding sections in 


Section 13.1.1 of rfc3530-33.  However, they are mentioned in the text 


and in Tables 6 and 8.  BTW All these tables ought to have titles 


(assuming they remain after Barry's comments).  Also the ordering in 


Table 5 is slightly out of dictionary sort order if anyone cares.




- Required attributes (Table 2 in rfc3530bis):
  + change attribute type s/b changeid4
  + lease_time attribute the type used in -dot-x is uint32_t -
    nfs_lease4 used in 3530bis-33 is not defined in dot-x or copied
    into 3530bis.
  + fs_locations attribute type s/b fs_locations4 (not fs_locations)
  + owner attribute type s/b utf8str_mixed (not utf8<>)
  + owner_group attribute type s/b utf8str_mixed (not utf8<>)