Re: [nfsv4] Review of draft-ietf-nfsv4-rfc5661sesqui-msns-01
David Noveck <davenoveck@gmail.com> Fri, 20 September 2019 16:14 UTC
Return-Path: <davenoveck@gmail.com>
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 65D2612026E; Fri, 20 Sep 2019 09:14:40 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.997
X-Spam-Level:
X-Spam-Status: No, score=-1.997 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 70wClqbirK2N; Fri, 20 Sep 2019 09:14:37 -0700 (PDT)
Received: from mail-ot1-x335.google.com (mail-ot1-x335.google.com [IPv6:2607:f8b0:4864:20::335]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0F597120113; Fri, 20 Sep 2019 09:14:37 -0700 (PDT)
Received: by mail-ot1-x335.google.com with SMTP id b2so6625791otq.10; Fri, 20 Sep 2019 09:14:37 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=wb7kMSPRv0nY1Ogy+zftGCjFR+lW9rVPyb7js3OwxkA=; b=LaUNWGDtFyGEr3xGVCZT0xPlLLKAVMt3zxhVM/kqmqohzAnD0qLbh+zE5a3Hi02CUB f3BXkgItLVkaADsdjNScuafkXqBlXd/ZhkCprgFAtE8buLsG2YzVnIIgC6igPS5F9fWq D3qJWOeepY2yoZ0GQM+pE26j2DJ/l5ec8D3yN7eaQCIAekezfWa3KRSRNWhOMRGfQY17 mgUi6iw/D5g/DsxiZGKgDTjo4wa4kCerbCTNkARoIyEOgCwG4bUzN8DIlKu0ccWAk64I 4FIoBryPoFSxEz4L2faUh1E5wqooKLubPKGZy9r9RgRpyny8X+T6DkSn6g389z1mC92R +PAQ==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=wb7kMSPRv0nY1Ogy+zftGCjFR+lW9rVPyb7js3OwxkA=; b=ihyaqIx5XgS3cr2TWYoZ8vJWdPafy+357k5w9qBxTUZHp9KKP9b69Jw8rm4ulOBbVe 3CVZwc2m2iP5HouwCPkizRYHoDekvxdFH3ghPCwsBehUvVS/v52QvJbiTauyQV8Bt5Dd 1rCIfDHMkV+/1OVxFR3OiVeN0ZTAEl/QcoHJxAiITsxJe9vT9p7rwAf0JL0aXHFYXitK nXMu8g3ve8qfVybYMNUpkx86NicfFgWR6EoH6Zh74o1TDL+1p5QuYifU82Yj290w5StK VKgBf0Fm9vCYz4+94vTchLAfTkjDiKlIotYJuYFiTt50N1k2jhYned9c/p8o8cw/oi5H Hzhw==
X-Gm-Message-State: APjAAAWSiy+Mq8YEAtEWbKXcLecJjgBjpdcAu8DzuxkIsqoVMZbQcIKz CNTQAkoR+XZNGX6FTzzxa3iRdHoFMZDVZBymJcc=
X-Google-Smtp-Source: APXvYqypQqHyaXKjuNL8Nk/3PpOoZGFrypBrLBoHEqVYhZDDLiFAskIjc7Vz1Un8TX8aHq5/nuXQr4mzLw/Z6OHJ5uI=
X-Received: by 2002:a9d:7a83:: with SMTP id l3mr11733482otn.359.1568996076241; Fri, 20 Sep 2019 09:14:36 -0700 (PDT)
MIME-Version: 1.0
References: <DB7PR07MB5736DCA699046998B565A5FA95880@DB7PR07MB5736.eurprd07.prod.outlook.com>
In-Reply-To: <DB7PR07MB5736DCA699046998B565A5FA95880@DB7PR07MB5736.eurprd07.prod.outlook.com>
From: David Noveck <davenoveck@gmail.com>
Date: Fri, 20 Sep 2019 12:14:25 -0400
Message-ID: <CADaq8jftcTAgsrAebe-UrLjDvbxyvPB+e9LWvkOzMJenuGJCaw@mail.gmail.com>
To: Magnus Westerlund <magnus.westerlund@ericsson.com>
Cc: "nfsv4@ietf.org" <nfsv4@ietf.org>, "draft-ietf-nfsv4-rfc5661sesqui-msns@ietf.org" <draft-ietf-nfsv4-rfc5661sesqui-msns@ietf.org>
Content-Type: multipart/alternative; boundary="000000000000c749330592fe5d2f"
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/-hwKNEmc8myHAJwRpgi20enTHqw>
Subject: Re: [nfsv4] Review of draft-ietf-nfsv4-rfc5661sesqui-msns-01
X-BeenThere: nfsv4@ietf.org
X-Mailman-Version: 2.1.29
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, 20 Sep 2019 16:14:41 -0000
On Fri, Sep 20, 2019 at 11:38 AM Magnus Westerlund < magnus.westerlund@ericsson.com> wrote: > Hi, > > > > I have now done my review of draft-ietf-nfsv4-rfc5661sesqui-msns-01. > Thanks. > The review focuses on introduction, appendix and other things related to > the format of the update. I have also gone through the Diff to glance > through the changes. > > > > > > 1. Section 1.1. > Work would have to be done with regard to RFC8178 [61] with which > RFC5661 [60] is curretly inconsistent, in order to arrive at a > situation in which there would be no need for RFC8178 to update > the NFSv4.1 specfication. > > I would recommend that you include some words, either title or some > summary of what RFC 8178 is in this text as it is the first occurrence of > the reference. > > I will do that. > > 1. The same comment as above also applies to the next paragraph in > respect to RFC 8434. > > OK. > > 1. Section 1.1: Regarding the second bullet: > > o Work would have to be done with regard to RFC8434 [64] which > > curently updates RFC5661 [60]. When that work is done and the > > resulting document approved, the new NFSv4.1 specfication will > > obsolete RFC8434 as well as RFC5661. > > 1. When it comes to updating this document, there are no purpose in a > single document. What I understand of the goal with creating complete RFCs > with updates is to avoid having to combine descriptions. However, if 8434 > is a self contained functionality it could actually remain as is, and > simply be normatively referenced by 5661bis as being part of NFS 4.1 > protocol. Now, I haven’t read through the documents to understand if this > is a possibility or that integrating it is the obvious way forward. > > I think it is a possibility, although I'm not sure yet whether this is the best choice. My feeling, based on what we've seen so far of the pNFS-related errata is that substantial work will be needed to clarify this area of the spec so we cannot simple leave Sections 12 and 13 as they are and leave 8434 as it is. I'd really like to hear Tom's opinion on the best way to proceed. I can rework the paragraph to leave it open, for now, about what approach to take. > 1. What I wrote in 3. Is why there clearly is a possibility to resolve > i18n in a manner that applies to all minor versions . > > I don't see any reason for these to be different in 4.0 and 4.1. Although the transition from mnor version zero hanged lots of things, it did not change this. One possible problem is that if do publish "I18N in V4" RFC, it is unclear how it will affect rfc7530. It wouldn't update rfc750 since it would say exactly what rfc7530 says now. Stiill there is a value in having this all in one place even if the existing treatment of v4.0 I18N remains as it is until an eventual rfc7530bis. Section 1.1. I think this text should drop the use of a single document in favor of talking about a set of authorative correct specifications. I think you could consider the language Adam Roach uses in https://datatracker.ietf.org/doc/draft-roach-bis-documents/ to motivate this approach. OK. > 1. Section 1.1: > There is a need for a revised treatment of security of in NFSv4.1. > The issues with the existing treatment are discussed in > Appendix C. > Note the two “of” in first sentence. > > Will fix. Section 1.1: I think the first paragraphs should be a bit more explicit about that this is a limited scope update. I would suggest the following: The revised description of the NFS version 4 minor version 1 (NFSv4.1) protocol presented in this update is necessary to enable full use of trunking in connection with multi-server namespace features and to enable the use of transparent state migration in connection with NFSv4.1. This document is in the form of a updated description of the NFS 4.1 protocol previously defined in [RFC5661]. RFC5661 is obsoleted by this document. However, the update has a limited and focused scope on enabling full use of trunking, the need for these changes are discussed in Appendix A. Appendix B described the specific changes made to arrive at the current text. > 1. > This limited scope update is applied on the main NFSv4.1 RFC is > intending to provide an authorative complete specification, the motivation > for this is discussed in [I.D-roach-bis-documents], addressing the issues > within the scope of the update. However, it will not address issues that > are known but outside of this limited scope as could expected by a full > update of the protocol. Below are some areas which are known to need > addressing in a future update of the protocol. > > Like this text. will incorporate. Section 22: IANA Consideration As this document is going to obsolete RFC5661 it will be necessary to instruct IANA to update all registry entries and registry rules references that points to RFC5661 to point to this document instead. OK. I intend to have these updates available some time next week. Are you expecting a -02 that includes the changes to address your review together with the work previously posted to the list to address the oe errata relevan to rfc5661sesqui, i.e. 2006?.
- [nfsv4] Review of draft-ietf-nfsv4-rfc5661sesqui-… Magnus Westerlund
- Re: [nfsv4] Review of draft-ietf-nfsv4-rfc5661ses… David Noveck
- Re: [nfsv4] Review of draft-ietf-nfsv4-rfc5661ses… David Noveck
- Re: [nfsv4] Review of draft-ietf-nfsv4-rfc5661ses… Magnus Westerlund