[nfsv4] comments w.r.t. flex layout draft 12

Rick Macklem <rmacklem@uoguelph.ca> Fri, 21 July 2017 21:41 UTC

Return-Path: <rmacklem@uoguelph.ca>
X-Original-To: nfsv4@ietfa.amsl.com
Delivered-To: nfsv4@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2287C12EC11 for <nfsv4@ietfa.amsl.com>; Fri, 21 Jul 2017 14:41:44 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.902
X-Spam-Level:
X-Spam-Status: No, score=-1.902 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_PASS=-0.001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 uaVefun3Er6P for <nfsv4@ietfa.amsl.com>; Fri, 21 Jul 2017 14:41:42 -0700 (PDT)
Received: from CAN01-QB1-obe.outbound.protection.outlook.com (mail-eopbgr660062.outbound.protection.outlook.com [40.107.66.62]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2092C12EA95 for <nfsv4@ietf.org>; Fri, 21 Jul 2017 14:41:41 -0700 (PDT)
Received: from YTXPR01MB0189.CANPRD01.PROD.OUTLOOK.COM (10.165.218.133) by YTXPR01MB0190.CANPRD01.PROD.OUTLOOK.COM (10.165.218.134) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1261.13; Fri, 21 Jul 2017 21:41:40 +0000
Received: from YTXPR01MB0189.CANPRD01.PROD.OUTLOOK.COM ([10.165.218.133]) by YTXPR01MB0189.CANPRD01.PROD.OUTLOOK.COM ([10.165.218.133]) with mapi id 15.01.1261.024; Fri, 21 Jul 2017 21:41:40 +0000
From: Rick Macklem <rmacklem@uoguelph.ca>
To: "nfsv4@ietf.org" <nfsv4@ietf.org>, Thomas Haynes <loghyr@primarydata.com>
Thread-Topic: comments w.r.t. flex layout draft 12
Thread-Index: AQHTAmkEOtZ5rSlip0aWin7YxO1ZKQ==
Date: Fri, 21 Jul 2017 21:41:40 +0000
Message-ID: <YTXPR01MB0189332A64D2E7485CFC8F2CDDA40@YTXPR01MB0189.CANPRD01.PROD.OUTLOOK.COM>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=rmacklem@uoguelph.ca;
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; YTXPR01MB0190; 7:RaKAwPD7y0vh469Ou816I/3VB46Tgi939hYoLq/26NNhyey1hdTEM0z8lerMMQ6rJYDpQUDrh4WLboJhH+fDshGTW1A7+UKyxMG6Rd/TJdswr2ui08Avc4ztf18RGXWlAorap+WgZXlPIKI5/qeI7IHTHNmWZNjy/w3jNJYEbYA1lhOn4dI2XVkoD+jsOkb0NbjFSiXiB2rt5kag1yoTI0qx1CmRbT/PBvg0QuEvsOF9ibSoVzSLX7aKSwQiZorXhi2gzrq9DuOkdHnworKMX26sdLQCrvzjxusnEgW0xc5J8Httxffwf8+Xu0kZNOBolh7Mfa5jrpFyeteQJ4w0fRPIz3p+V1o0kqrnl+HgLksKiiQpyOMbGf/tDBgLcmSfS0IHLU2/nH2q3s+jlBx65PseH1NQolcq023oz0flMnEKKQvGBBpGNw8jkVtk4Ar8iV8uWL9ETDg1FmhjHKjRXkRxQIkJMnNxMTYI0kO2gh8Z0DeeV/GBxURsyK+jDs0/2LDAnYNX9/FfLxvX2wEXM6gwC/L4W02f4GWkw+78Y56wxuTNvYvjsJa4Qt/EL1TWTI+tblzxECciYPbXGarYGVSnoFtqivsnFmPnE5tvGZUWtyrgrHvh+E6h49DUWRIKeSLFMHLqLIIQ8LSGTFEx7sUgZw0BpkQvt/Co/PIWgVeRkw8fmH5WKZwBHN87dpDfDBe28639AGac87A8yxt6si+nIzWEPSqgSMx5mMUDIYoa7fTe4WGdAYrEY6vgRl2D/ZyRqdRhaQfl0Xp6it1H+Qo3ZimtwbhQVS9q44NjWDM=
x-ms-office365-filtering-correlation-id: 08270e7f-f773-4686-4e75-08d4d0814580
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(300000500095)(300135000095)(300000501095)(300135300095)(22001)(300000502095)(300135100095)(2017030254075)(300000503095)(300135400095)(2017052603031)(201703131423075)(201703031133081)(201702281549075)(300000504095)(300135200095)(300000505095)(300135600095)(300000506095)(300135500095); SRVR:YTXPR01MB0190;
x-ms-traffictypediagnostic: YTXPR01MB0190:
x-exchange-antispam-report-test: UriScan:(158342451672863)(192374486261705);
x-microsoft-antispam-prvs: <YTXPR01MB0190AAACFBD134A0DADD305DDDA40@YTXPR01MB0190.CANPRD01.PROD.OUTLOOK.COM>
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(100000700101)(100105000095)(100000701101)(100105300095)(100000702101)(100105100095)(6040450)(2401047)(8121501046)(5005006)(3002001)(10201501046)(93006095)(93001095)(100000703101)(100105400095)(6041248)(20161123555025)(20161123564025)(201703131423075)(201702281529075)(201702281528075)(201703061421075)(201703061406153)(20161123562025)(20161123560025)(20161123558100)(6072148)(100000704101)(100105200095)(100000705101)(100105500095); SRVR:YTXPR01MB0190; BCL:0; PCL:0; RULEID:(100000800101)(100110000095)(100000801101)(100110300095)(100000802101)(100110100095)(100000803101)(100110400095)(100000804101)(100110200095)(100000805101)(100110500095); SRVR:YTXPR01MB0190;
x-forefront-prvs: 0375972289
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(6009001)(39840400002)(39410400002)(39450400003)(39850400002)(39400400002)(199003)(189002)(77096006)(3280700002)(189998001)(106356001)(478600001)(97736004)(3660700001)(74316002)(68736007)(14454004)(74482002)(81166006)(8936002)(6436002)(81156014)(2900100001)(5660300001)(102836003)(54356999)(105586002)(53936002)(101416001)(305945005)(25786009)(8676002)(2906002)(38730400002)(2501003)(7696004)(33656002)(55016002)(6506006)(9686003)(86362001)(50986999); DIR:OUT; SFP:1101; SCL:1; SRVR:YTXPR01MB0190; H:YTXPR01MB0189.CANPRD01.PROD.OUTLOOK.COM; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en;
received-spf: None (protection.outlook.com: uoguelph.ca does not designate permitted sender hosts)
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: uoguelph.ca
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Jul 2017 21:41:40.0852 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: be62a12b-2cad-49a1-a5fa-85f4f3156a7d
X-MS-Exchange-Transport-CrossTenantHeadersStamped: YTXPR01MB0190
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/4Du9GaOifhq0aDW0IpSlLE0U9OE>
Subject: [nfsv4] comments w.r.t. flex layout draft 12
X-BeenThere: nfsv4@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: NFSv4 Working Group <nfsv4.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/nfsv4>, <mailto:nfsv4-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/nfsv4/>
List-Post: <mailto:nfsv4@ietf.org>
List-Help: <mailto:nfsv4-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/nfsv4>, <mailto:nfsv4-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 21 Jul 2017 21:41:44 -0000

Hi,

Overall, it looks good to me.

First off, there is one generic issue I thought I'd bring up...
- w.r.t. Kerberos...shouldn't it be talking about Rpcsec_gss instead of Kerberos,
  just in case someone uses another security mechanism someday?
- Also, I'll admit I didn't understand the last two sentences of Sec. 15.1.1.
 - Why is the metadata server generating TGTs. Is this referring to the case of
   the metadata doing I/O on the storage server? Or is this specific to some part of
   RPCSEC_GSS_V3 (which I'll admit I've never looked at;-). If the latter, just ignore
   my ignorance;-)

Now, here are the minor typo like things I spotted. (Feel free to ignore any you
disagree with):
344:   wrong credential for the deisred access will get a NFS4ERR_ACCESS
- spelling of "desired"

855:   ... Note that
856:   even if the data protocol is at least NFSv4.1, it may still be the
857:   case that there is loose coupling is in effect.  If
- one too many "is"? -->"there is loose coupling in effect."

957:   This document defines structure associated with the layouttype4 value
- "structures" (ie. plural)?

1105:   If an implementation needs a different statedid per file handle, then
- spelling of "stateid"

1166:   with a LAYOUTEGT which returns an error of either
- spelling of "LAYOUTGET"

1326:   ... While the client
1327:   may provide a hint to how many copies it wants (see Section 12), the
- "hint as to how many"
1328:   metadata server can ignore that hint and in any event, the client has
1329:   no means to dictate neither the storage device (which also means the
- "dictate either the"

1349:   (see Section 8.2.  If the client is writing to the layout segments
- closing ")"

1374:   ... The requirements for writing to a mirrored layout
1375:   segments are presented below.
- single or multiple layouts? "a mirrored layout segment" or "to mirrored layout segments"

1387:   client then MUST inform the metadata server (see Section 8.2.3.  The
- closing ")"

1472:   been done, and these data must have been committed to stable storage
- "and the data must"

2039:   modifying the syntethic uid or gid on the data file.
- spelling of "synthetic"