Last Call Review of draft-ietf-nfsv4-layout-types-03
review-ietf-nfsv4-layout-types-03-opsdir-lc-dodge-2015-05-15-00

Request Review of draft-ietf-nfsv4-layout-types
Requested rev. no specific revision (document currently at 13)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2015-05-26
Requested 2015-03-28
Other Reviews Genart Last Call review of -03 by Meral Shirazipour (diff)
Genart Telechat review of -03 by Meral Shirazipour (diff)
Secdir Last Call review of -03 by Joseph Salowey (diff)
Genart Last Call review of -09 by Meral Shirazipour (diff)
Review State Completed
Reviewer Menachem Dodge
Review review-ietf-nfsv4-layout-types-03-opsdir-lc-dodge-2015-05-15
Posted at http://www.ietf.org/mail-archive/web/ops-dir/current/msg01070.html
Reviewed rev. 03 (document currently at 13)
Review result Has Nits
Draft last updated 2015-05-15
Review completed: 2015-05-15

Review
review-ietf-nfsv4-layout-types-03-opsdir-lc-dodge-2015-05-15

Hello,

I have reviewed this document as part of the Operational directorate's ongoing effort to review all IETF documents being processed by the IESG.  These comments were written with the intent of improving the operational aspects of the IETF drafts. Comments that are not addressed in last call may be included in AD reviews during the IESG review.  Document editors and WG chairs should treat these comments just like any other last call comments.

This document is for the Standards Track and if approved Updates RFC 5661. This document is a clarification of RFC 5661 - "

Network File System (NFS) Version 4 Minor Version 1 Protocol"

NITS

====

The tool has found the following: 

 == Outdated reference: A later version (-05) exists of
     draft-ietf-nfsv4-flex-files-02

Additional NITS
===============

Abstract First Sentence: Repetition of the word "those".

This document provides help in distinguishing between the requirements for Network File System (NFS) version 4.1's Parallel NFS (pNFS) and 

those those

 specifically directed to the pNFS File Layout.

Section 2: Definitions - Not sure whether the word :striped" was intended here.

Data Server (DS): is one of the pNFS servers which provide the
 contents of a file system object which is a regular file.
 Depending on the layout, there might be one or more data servers
 over which the data is 

striped

. 

Section 2: Definitions - Suggest "lay out" rather than "lays out".

Layout Type: describes both the storage protocol used to access the
 data and the aggregation scheme used to 

lays 

out the file data on
 the underlying storage devices. 

Section 3.3: 

Editorial Requirements - Suggest "separately" rather than "separably"

"While these could be envisioned as one section in that the fencing
 issue might be the only security issue, it is recommended to deal
 with them 

separably

."

Other than the above nits I have no further issues with this document.

Thank you kindly.

Best Regards,

Menachem Dodge