Re: [nfsv4] Progressing RFC errata for RFC 5661

Chuck Lever <chuck.lever@oracle.com> Thu, 24 October 2019 14:49 UTC

Return-Path: <chuck.lever@oracle.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 EAD2F1208F5 for <nfsv4@ietfa.amsl.com>; Thu, 24 Oct 2019 07:49:02 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.299
X-Spam-Level:
X-Spam-Status: No, score=-4.299 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001, UNPARSEABLE_RELAY=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=oracle.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 ddqdvnrN3o7C for <nfsv4@ietfa.amsl.com>; Thu, 24 Oct 2019 07:49:00 -0700 (PDT)
Received: from userp2130.oracle.com (userp2130.oracle.com [156.151.31.86]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0E2A71208A6 for <nfsv4@ietf.org>; Thu, 24 Oct 2019 07:48:59 -0700 (PDT)
Received: from pps.filterd (userp2130.oracle.com [127.0.0.1]) by userp2130.oracle.com (8.16.0.27/8.16.0.27) with SMTP id x9OEhsFK016092; Thu, 24 Oct 2019 14:48:55 GMT
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oracle.com; h=from : message-id : content-type : mime-version : subject : date : in-reply-to : cc : to : references; s=corp-2019-08-05; bh=zquk7mEJ5IqRRm1CKEF0shW9+E+YdrQyChiXenehixM=; b=UfX4GQJKS3wFR+VZTMYlb6+OPsChc0DWtJxoCX+foGAZ9nM3guEvbB+JbVya5Nrzl3Wn QpcTijTXMk6mNMaz4Yu4bwygTLj2aCBg2RbONIIAxyKyAeos24dGx3pw45qu658GAgED MLfM2RnrJ0DnYKHvuNlCY1BQWZzUaZgwHh+JCv6P6DvwDDhajMuR7Itz6qXIlFua15BO Ecvc0oZUYX2fbRUcTOM0oIgkwOp/X6/r1kHXJrPX3WLBxLHNFFZP23GG1x9Mm0s4a5ej IWRmM8jWPlU8a/LupSzI0tnQYw0qoxhKjPoqw4aewSpIrgT32guO+upEF2OAs0wAUvcr Ew==
Received: from userp3030.oracle.com (userp3030.oracle.com [156.151.31.80]) by userp2130.oracle.com with ESMTP id 2vqswtv9gx-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 24 Oct 2019 14:48:54 +0000
Received: from pps.filterd (userp3030.oracle.com [127.0.0.1]) by userp3030.oracle.com (8.16.0.27/8.16.0.27) with SMTP id x9OEmXEq164031; Thu, 24 Oct 2019 14:48:54 GMT
Received: from aserv0122.oracle.com (aserv0122.oracle.com [141.146.126.236]) by userp3030.oracle.com with ESMTP id 2vtjkjumpn-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Thu, 24 Oct 2019 14:48:54 +0000
Received: from abhmp0019.oracle.com (abhmp0019.oracle.com [141.146.116.25]) by aserv0122.oracle.com (8.14.4/8.14.4) with ESMTP id x9OEmrQc013400; Thu, 24 Oct 2019 14:48:53 GMT
Received: from anon-dhcp-153.1015granger.net (/68.61.232.219) by default (Oracle Beehive Gateway v4.0) with ESMTP ; Thu, 24 Oct 2019 07:48:52 -0700
From: Chuck Lever <chuck.lever@oracle.com>
Message-Id: <484BC9B0-BA1A-4D1F-9F2C-3EE953AA6226@oracle.com>
Content-Type: multipart/alternative; boundary="Apple-Mail=_04C1D46B-EB47-4DCE-A964-11C985A80687"
Mime-Version: 1.0 (Mac OS X Mail 12.4 \(3445.104.11\))
Date: Thu, 24 Oct 2019 10:48:51 -0400
In-Reply-To: <DB7PR07MB5736124B2F507DA20F317BC195B30@DB7PR07MB5736.eurprd07.prod.outlook.com>
Cc: "nfsv4@ietf.org" <nfsv4@ietf.org>
To: Magnus Westerlund <magnus.westerlund@ericsson.com>
References: <DB7PR07MB5736124B2F507DA20F317BC195B30@DB7PR07MB5736.eurprd07.prod.outlook.com>
X-Mailer: Apple Mail (2.3445.104.11)
X-Proofpoint-Virus-Version: vendor=nai engine=6000 definitions=9419 signatures=668684
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 suspectscore=0 malwarescore=0 phishscore=0 bulkscore=0 spamscore=0 mlxscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1908290000 definitions=main-1910240136
X-Proofpoint-Virus-Version: vendor=nai engine=6000 definitions=9419 signatures=668684
X-Proofpoint-Spam-Details: rule=notspam policy=default score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1015 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1908290000 definitions=main-1910240136
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/Ce1LaKWtZWG8vDtmkQmD6Uz0Fkw>
Subject: Re: [nfsv4] Progressing RFC errata for RFC 5661
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: Thu, 24 Oct 2019 14:49:03 -0000


> On Sep 13, 2019, at 4:27 AM, Magnus Westerlund <magnus.westerlund@ericsson.com> wrote:
> 
> Hi,
>  
> To make progress on two aspects, both ensure that the RFC errata are dealt with, and to progress draft-ietf-nfsv4-rfc5661sesqui-msns-01 I like to do the following:
>  
> From the discussion it appears that some think that the reported errata should mostly be accepted. Based on that I want at least one WG participant to look at each errata and public state their opinion on what Status that particular errata should be in. Before doing that please read https://www.rfc-editor.org/errata-definitions/ <https://www.rfc-editor.org/errata-definitions/> and the IESG statement on errata https://www.ietf.org/about/groups/iesg/statements/processing-rfc-errata/ <https://www.ietf.org/about/groups/iesg/statements/processing-rfc-errata/> . 
>  
> Below you have the list of errata in reported state. When you have looked at one and judged, you respond to this email and write below the specific errata your recommendation. I will then review that errata to see if I concur with the judgment and then make it so. 

4711
The text in RFC 5661 Section 15.1.5.5 is clearly incorrect.
Compare with text in RFC 7530 Section 13.1.5.5.
I recommend this errata is assigned status Verified.
4712
The text in RFC 5661 Section 6.2.1.3.1 matches the text in RFC 7530 Section 6.2.1.3.1. Should there be a similar errata filed against RFC 7530?
Should the Notes section of the errata be added to Section 6.2.1.3.2 ?
Are there already server implementations that reject RENAME operations in these cases, or do all server implementations permit RENAME?
I recommend this errata is assigned status Rejected, and the issues addressed in rfc5661bis after further discussion.
Additional Editorial correction: The Discussion subsections of ACE4_DELETE and ACE4_DELETE are missing the word "how". 
5040
The text in RFC 5661 Section 18.46.3 is inconsistent with the text in Section 18.50.3. Section 18.50.3 clearly intends to allow DESTROY_CLIENTID to be submitted as a single operation without a SEQUENCE. 
I recommend this errata is assigned status Verified.
5212
There was no follow-up discussion of this errata on the mailing list.
Does this errata propose a change in WG consensus? Will existing client implementations handle an NFS4ERR_ROFS result to LAYOUTGET?
I recommend this errata is assigned status Rejected, and the issues addressed in rfc5661bis after further discussion.

>  
> RFC Number
> (Errata ID)
> Section
> Type
> Source of RFC <https://www.rfc-editor.org/source/>	
> Submitted By
> Publication Format
> Date Submitted
> RFC5661 (2505 <https://www.rfc-editor.org/verify_errata_select.php?eid=2505>)
> 12.5.4.
> Technical
> nfsv4 (tsv)
> Michael Eisler
> TEXT
> 2010-08-31
> RFC5661 (2751 <https://www.rfc-editor.org/verify_errata_select.php?eid=2751>)
> GLOBAL
> Technical
> nfsv4 (tsv)
> Ricardo Labiaga
> TEXT
> 2011-03-21
> RFC5661 (3379 <https://www.rfc-editor.org/verify_errata_select.php?eid=3379>)
> 18.50.3
> Technical
> nfsv4 (tsv)
> Asmita Karandikar
> TEXT
> 2012-10-15
> RFC5661 (3714 <https://www.rfc-editor.org/verify_errata_select.php?eid=3714>)
> 13.4.4.
> Technical
> nfsv4 (tsv)
> Yuri Radchenko
> TEXT
> 2013-08-31
> RFC5661 (3901 <https://www.rfc-editor.org/verify_errata_select.php?eid=3901>)
> 18.43.3
> Technical
> nfsv4 (tsv)
> Trond Myklebust
> TEXT
> 2014-02-25
> RFC5661 (4119 <https://www.rfc-editor.org/verify_errata_select.php?eid=4119>)
> 12.2.10
> Technical
> nfsv4 (tsv)
> Christoph Hellwig
> TEXT
> 2014-09-17
> RFC5661 (4492 <https://www.rfc-editor.org/verify_errata_select.php?eid=4492>)
> 18.17.4
> Technical
> nfsv4 (tsv)
> David Noveck
> TEXT
> 2015-10-05
> RFC5661 (4711 <https://www.rfc-editor.org/verify_errata_select.php?eid=4711>)
> 15.1.5.5
> Technical
> nfsv4 (tsv)
> Tom Haynes
> TEXT
> 2016-06-16
> RFC5661 (4712 <https://www.rfc-editor.org/verify_errata_select.php?eid=4712>)
> 6.2.1.3.1.
> Technical
> nfsv4 (tsv)
> Trond Myklebust
> TEXT
> 2016-06-16
> RFC5661 (5040 <https://www.rfc-editor.org/verify_errata_select.php?eid=5040>)
> 18.46.3
> Technical
> nfsv4 (tsv)
> Jonathan Price
> TEXT
> 2017-06-12
> RFC5661 (5212 <https://www.rfc-editor.org/verify_errata_select.php?eid=5212>)
> 15.2
> Technical
> nfsv4 (tsv)
> NFS4ERR_ROFS is not a valid error code for LAYOUTGET
> TEXT
> 2017-12-19
> RFC5661 (4572 <https://www.rfc-editor.org/verify_errata_select.php?eid=4572>)
> 18.35.3
> Editorial
> nfsv4 (tsv)
> Sai Chakravarthy Tangudu
> TEXT
> 2015-12-30
> RFC5661 (4914 <https://www.rfc-editor.org/verify_errata_select.php?eid=4914>)
> 14.3.3
> Editorial
> nfsv4 (tsv)
> Dylan Simon
> TEXT
> 2017-01-22
> RFC5661 (5476 <https://www.rfc-editor.org/verify_errata_select.php?eid=5476>)
> 18.42.3
> Editorial
> nfsv4 (tsv)
> Tigran Mkrtchyan
> TEXT
> 2018-08-23
>  
> Cheers
>  
> Magnus Westerlund
> _______________________________________________
> nfsv4 mailing list
> nfsv4@ietf.org <mailto:nfsv4@ietf.org>
> https://www.ietf.org/mailman/listinfo/nfsv4 <https://www.ietf.org/mailman/listinfo/nfsv4>
--
Chuck Lever