Last Call Review of draft-ietf-nfsv4-flex-files-15
review-ietf-nfsv4-flex-files-15-genart-lc-holmberg-2017-12-21-00

Request Review of draft-ietf-nfsv4-flex-files
Requested rev. no specific revision (document currently at 17)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2018-01-16
Requested 2017-12-19
Other Reviews Secdir Last Call review of -15 by Brian Weis (diff)
Opsdir Last Call review of -15 by Linda Dunbar (diff)
Review State Completed
Reviewer Christer Holmberg
Review review-ietf-nfsv4-flex-files-15-genart-lc-holmberg-2017-12-21
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/TITh-eKj004DVwXaH9nugzRqNKI
Reviewed rev. 15 (document currently at 17)
Review result Almost Ready
Draft last updated 2017-12-21
Review completed: 2017-12-21

Review
review-ietf-nfsv4-flex-files-15-genart-lc-holmberg-2017-12-21

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair.  Please treat these comments just
like any other last call comments.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-nfsv4-flex-files-15
Reviewer: Christer Holmberg
Review Date: 2017-12-21
IETF LC End Date: 2018-01-16
IESG Telechat date: 2018-01-25

Summary: The document is well written, and almost ready for publication. However, I think some additional text is needed, as described below.

Major issues: None.

Minor issues: None:

Nits/editorial comments:

Q1:

The Abstract says:

   "The flexible file layout type is defined in this
   document as an extension to pNFS which allows the use of storage
   devices in a fashion such that they require only a quite limited
   degree of interaction with the metadata server, using already
   existing protocols."

…and the Introduction says:

   "There are different layout types for different storage
   systems and methods of arranging data on storage devices.  This
   document defines the flexible file layout type used with file-based
   data servers that are accessed using the Network File System (NFS)
   protocols: NFSv3 [RFC1813], NFSv4.0 [RFC7530], NFSv4.1 [RFC5661], and
   NFSv4.2 [RFC7862].”

But, there is no text about the existing file layout type, and how this file layout types will improve things. The Abstract mentions impacts on the interactions with the metadata server, but there is no text about that in the Introduction. I think a few sentences in the Introduction, describing the high-level advantages and changes that the flexible file layout type brings, would be good. 

Also, the Abstract mentions client mirroring, but there is nothing about that in the Introduction.
 
Q2:

Is section 2 specific to the flexible file layout type, or is it about pNFS in general? I think it would good to have a sentence describing the scope of the section.