Re: [nfsv4] [Errata Rejected] RFC5661 (5212)

Magnus Westerlund <magnus.westerlund@ericsson.com> Thu, 03 September 2020 12:57 UTC

Return-Path: <magnus.westerlund@ericsson.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 071D03A0B33 for <nfsv4@ietfa.amsl.com>; Thu, 3 Sep 2020 05:57:03 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.102
X-Spam-Level:
X-Spam-Status: No, score=-2.102 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, DKIM_VALID_EF=-0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.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 DlTm4sZLBhCx for <nfsv4@ietfa.amsl.com>; Thu, 3 Sep 2020 05:57:01 -0700 (PDT)
Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2071.outbound.protection.outlook.com [40.107.22.71]) (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 A5A6C3A0ADD for <nfsv4@ietf.org>; Thu, 3 Sep 2020 05:57:00 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=A+UO/P1egTHP7dEcD0F0hpNtginA57j6u7Nl0E0VDo09dX49AZL725sFLRC+C0g/SR4IZ82AHC/I2qoqzXlqqM7O/CogWs1SjMdGSU3Dae0YQki1uSG0Y8XnN9yGgMA/rHfsvgycmEmCMwEVedBoF9CJibVuvxW0ozGigGRsvx0XP5GlWSXDTyfUW1atQALCiUOujB2Ip0w37jPzRdvuuvJF8HKhlt0N8cCA2zNkCnp0uJT4yNywj21jD5x++czSWQIxrEcKD/9rGumubM/HStNgFMBsKBSmAMtwy6dINV4dA4THBg2dLGQZJyZipV7Dm4Eafd4kwjAhE0KVQSJBUA==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=KfxZ1sgyF/v4FFrVjVDTN6xRCfYOa3Dn6tRNJQmdvCU=; b=Fdi0H85F2mTg+GLffEEpeXNLvhGdxb3961toRjJyEsw3HzLkzCdntrebfmMocroYqv3bXJc2YS3ZZp3DUTx/6WwYmsOuLQ5LVU7taRCu/prjKHgpqJrAHwGadmNXbLh/2zKFl+soNYnng0Qn8uTHQAANMHXbe5QCo6oXjmYR1T+yS0j8QPR+NyglDtb2feH68HSwG3WfdoWwybA5MuqI/Ft8FeHRqWB57vOcayLYq9sj8zoJLxmuBv6lyqk5pmeRZEyHwvKJyXMGe/fFZRUDNbbjnk4Cdd/2hMKnzFa7Z3OTOQDGCV5pzDobm1u++EbLRbliuVqkEN6UldpzIg55dQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=KfxZ1sgyF/v4FFrVjVDTN6xRCfYOa3Dn6tRNJQmdvCU=; b=ImtBEjyMT18KAC+swX9ACUKqtT1aLixiFUWUf4oZNuUcm+DQypfps95eFAzaQB2EBeYUDDa93smipLMJv3fWyCGq+3I5HtHdNZAK93ZKiN8Tru1z6EfIKOwAzGly08JsFBKUa63+KQ5SU7DMcBV5Qc3AAuU8AhaibVgHFmtM/oY=
Received: from VI1PR0702MB3775.eurprd07.prod.outlook.com (52.134.8.30) by VI1PR07MB4064.eurprd07.prod.outlook.com (52.134.25.149) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3348.5; Thu, 3 Sep 2020 12:56:57 +0000
Received: from VI1PR0702MB3775.eurprd07.prod.outlook.com ([fe80::e8db:6218:4bd:1ce7]) by VI1PR0702MB3775.eurprd07.prod.outlook.com ([fe80::e8db:6218:4bd:1ce7%7]) with mapi id 15.20.3348.015; Thu, 3 Sep 2020 12:56:57 +0000
From: Magnus Westerlund <magnus.westerlund@ericsson.com>
To: "David.Noveck@netapp.com" <David.Noveck@netapp.com>, "loghyr@primarydata.com" <loghyr@primarydata.com>, "shepler@storspeed.com" <shepler@storspeed.com>, "mike@eisler.com" <mike@eisler.com>
CC: "nfsv4@ietf.org" <nfsv4@ietf.org>
Thread-Topic: [Errata Rejected] RFC5661 (5212)
Thread-Index: AQHWgeTbXWcqXORLpUeZSI6j0Th6c6lW2Q8AgAADfqA=
Date: Thu, 3 Sep 2020 12:56:57 +0000
Message-ID: <VI1PR0702MB3775494778825974558D64FD952C0@VI1PR0702MB3775.eurprd07.prod.outlook.com>
References: <20200903112436.4E681F40780@rfc-editor.org> <MN2PR06MB559720AE5B2EB67CFD9AFFC4E12C0@MN2PR06MB5597.namprd06.prod.outlook.com>
In-Reply-To: <MN2PR06MB559720AE5B2EB67CFD9AFFC4E12C0@MN2PR06MB5597.namprd06.prod.outlook.com>
Accept-Language: sv-SE, en-US
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator:
authentication-results: netapp.com; dkim=none (message not signed) header.d=none;netapp.com; dmarc=none action=none header.from=ericsson.com;
x-originating-ip: [158.174.116.90]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 20b6d0de-4fb2-4a94-7f36-08d85008d78f
x-ms-traffictypediagnostic: VI1PR07MB4064:
x-microsoft-antispam-prvs: <VI1PR07MB40644E270CC168A3E7FF1285952C0@VI1PR07MB4064.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:800;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: NwS2av9El6ZDZ9iYwaBrNoG/fs9g2JotV19aS2E4G2bugT0bWlWFBnWINEMrDPDqj3Mu4Qk+WJx6ey5QV4dp8UcGzuoDkZ9j3utotj8y583wGCQizUwKphMpC0Se5L6wjh1OpJEgCdtS1GwyvM2C799ctenVnQGjjALOgXhwpZPCLR1L2CPT2oZVkj5LckRxiXLIPejHMowpyeJJmYUu24UxVB1R+/IPoXVMS8yLnlWohRy0D2Mue3BTk/7xXdanPcs0/8Do5PL8u2izQZMSvkAo/RmUzqT+php1t18zXvgiQa4pxTmNn1L+orIkTsuseizSSRpFYYBPEduj8hj6wPaG5Y9QKARuZOYyD/ZXAw01rjK0iVYLIavcOISHj15H4Vu9HC2cvR0hwf7QU4FOnA==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:VI1PR0702MB3775.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(39860400002)(136003)(396003)(376002)(346002)(110136005)(4001150100001)(316002)(66616009)(64756008)(52536014)(8936002)(55016002)(66476007)(66446008)(66556008)(76116006)(4326008)(9686003)(66946007)(71200400001)(8676002)(53546011)(33656002)(478600001)(966005)(186003)(2906002)(5660300002)(44832011)(86362001)(83380400001)(26005)(99936003)(7696005)(6506007); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: KVMjcj4h24ndjVqDxeuHG4TW98xyT/209yjaJbzUxO6ELFZ7GwEYB0fgksRHcQQVHJaWJxGc3ZPa6K196vedPBqx77i0HncmdKyl0XXTPsLUv4RSbbqgYFDjuAFEFv+hmBLGJaJTa6A4q+e2YoGMln9pCf6HJ2TtUIAvD4F3nCX3KcMGVV6wSpjvNC7jYWjhYOSCY7UUR4/UB+9Qwn2Bbb7DMSoMFyJ9q9OZjvDAfCKpJAr+qcJyzqH6M6wVNWDLFF/gAUoIsI8mqWAH9tBIipxPiU89eZlbK1TMxv7qQjgzLEw+0csAUlERfG+/9C7dNBzBz0X/J372F4ARkO3qWcBUpCNYyX9uxgmFC1pPFFPhBo+c4/wmosR3cVOcLPPGJmfiQsadu/x3UIwA+DgwTUUtlSSxem5DISkxCCxfUpas48tMRoGtZXtNKDRQzlXtB798kjjXMyYfdLZhBpWz2JepqZ6S0UG4bHuv0X5u8RMPznANroOliSDDPxc0iDgkJev5Y/WzzNuL7HulBK1BXoLqyItcn74ThVtv9mtb0VapCMHyrKmDjop6P2byQz5RbsukpMJWU6VImJabQjMPYrXSQnuXFFEIU7mLHxEGM0XxyqipWHjzUelrM4sVsHBV9JELWpTE9hxZBvhxKtV2Cw==
x-ms-exchange-transport-forked: True
Content-Type: multipart/signed; micalg=SHA1; protocol="application/x-pkcs7-signature"; boundary="----=_NextPart_000_0009_01D68202.779865B0"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR0702MB3775.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 20b6d0de-4fb2-4a94-7f36-08d85008d78f
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Sep 2020 12:56:57.8831 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: BQOawHXeiRCwTZZ2t3JFMlCgOay/oTn8f+m7aocS/79rFHYXeeHk2vlfgF/iElifeVsFsr95y+CO7lxARJyKo5KrAUmiYNmOjm2CRqNcHV8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR07MB4064
Archived-At: <https://mailarchive.ietf.org/arch/msg/nfsv4/fc4MYgbs4lDutl1tygiEyebiNhQ>
Subject: Re: [nfsv4] [Errata Rejected] RFC5661 (5212)
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, 03 Sep 2020 12:57:03 -0000

Hi David,

I might have misjudged this one. However, from my perspective it falls under 
the following part from the instructions:

" or proposes a change to the RFC that should be done by publishing a new RFC 
that replaces the current RFC."
See Rejected:
https://www.rfc-editor.org/errata-definitions/

It could likely have been put in Held for Document update. However, I prefer 
those issues to be clearer in what is unclear or wrong and should be 
considered in the future.

You editors of the 8881bis will have to go though all Erratas against RFC 5661 
to determine which are already addressed and which are non-relevant and which 
needs considerations and discussion. Even before this one we had one Errata 
(https://www.rfc-editor.org/errata/eid2751) which was a significant change of 
the protocol that got rejected  due to it going beyond established consensus 
at time of publication.

If you feel strongly we can change it to Held for Document update.

Cheers

Magnus


> -----Original Message-----
> From: Noveck, David <David.Noveck@netapp.com>
> Sent: den 3 september 2020 14:33
> To: RFC Errata System <rfc-editor@rfc-editor.org>rg>; loghyr@primarydata.com;
> shepler@storspeed.com; mike@eisler.com
> Cc: Magnus Westerlund <magnus.westerlund@ericsson.com>om>;
> iesg@ietf.org; nfsv4@ietf.org
> Subject: RE: [Errata Rejected] RFC5661 (5212)
>
> The explanatory text, with I agree with, indicates why this Should be held
> over for update, presumably in rf5661bis.  However the subject line says
> "rejected" and I don't understand why.
>
> -----Original Message-----
> From: RFC Errata System <rfc-editor@rfc-editor.org>
> Sent: Thursday, September 3, 2020 7:25 AM
> To: loghyr@primarydata.com; shepler@storspeed.com; mike@eisler.com;
> Noveck, David <David.Noveck@netapp.com>
> Cc: magnus.westerlund@ericsson.com; iesg@ietf.org; nfsv4@ietf.org; rfc-
> editor@rfc-editor.org
> Subject: [Errata Rejected] RFC5661 (5212)
>
> NetApp Security WARNING: This is an external email. Do not click links or
> open attachments unless you recognize the sender and know the content is
> safe.
>
>
>
>
> The following errata report has been rejected for RFC5661, "Network File
> System (NFS) Version 4 Minor Version 1 Protocol".
>
> --------------------------------------
> You may review the report below and at:
> https://protect2.fireeye.com/v1/url?k=9b44cc6b-c5f451f3-9b448cf0-
> 861fcb972bfc-cf4c513c9d5ddf90&q=1&e=7b26cc7e-e55f-4723-adbc-
> 3949013eb251&u=https%3A%2F%2Fwww.rfc-
> editor.org%2Ferrata%2Feid5212
>
> --------------------------------------
> Status: Rejected
> Type: Technical
>
> Reported by: NFS4ERR_ROFS is not a valid error code for LAYOUTGET
> <loghyr@primarydata.com> Date Reported: 2017-12-19 Rejected by: Magnus
> Westerlund (IESG)
>
> Section: 15.2
>
> Original Text
> -------------
>    | LAYOUTGET            | NFS4ERR_ACCESS, NFS4ERR_ADMIN_REVOKED,     |
>    |                      | NFS4ERR_BADIOMODE, NFS4ERR_BADLAYOUT,      |
>    |                      | NFS4ERR_BADXDR, NFS4ERR_BAD_STATEID,       |
>    |                      | NFS4ERR_DEADSESSION, NFS4ERR_DELAY,        |
>    |                      | NFS4ERR_DELEG_REVOKED, NFS4ERR_DQUOT,      |
>    |                      | NFS4ERR_FHEXPIRED, NFS4ERR_GRACE,          |
>    |                      | NFS4ERR_INVAL, NFS4ERR_IO,                 |
>    |                      | NFS4ERR_LAYOUTTRYLATER,                    |
>    |                      | NFS4ERR_LAYOUTUNAVAILABLE, NFS4ERR_LOCKED, |
>    |                      | NFS4ERR_MOVED, NFS4ERR_NOFILEHANDLE,       |
>    |                      | NFS4ERR_NOSPC, NFS4ERR_NOTSUPP,            |
>    |                      | NFS4ERR_OLD_STATEID, NFS4ERR_OPENMODE,     |
>    |                      | NFS4ERR_OP_NOT_IN_SESSION,                 |
>    |                      | NFS4ERR_RECALLCONFLICT,                    |
>    |                      | NFS4ERR_REP_TOO_BIG,                       |
>    |                      | NFS4ERR_REP_TOO_BIG_TO_CACHE,              |
>    |                      | NFS4ERR_REQ_TOO_BIG,                       |
>    |                      | NFS4ERR_RETRY_UNCACHED_REP,                |
>    |                      | NFS4ERR_SERVERFAULT, NFS4ERR_STALE,        |
>    |                      | NFS4ERR_TOOSMALL, NFS4ERR_TOO_MANY_OPS,    |
>    |                      | NFS4ERR_UNKNOWN_LAYOUTTYPE,                |
>    |                      | NFS4ERR_WRONG_TYPE                         |
>
>
> Corrected Text
> --------------
>    | LAYOUTGET            | NFS4ERR_ACCESS, NFS4ERR_ADMIN_REVOKED,     |
>    |                      | NFS4ERR_BADIOMODE, NFS4ERR_BADLAYOUT,      |
>    |                      | NFS4ERR_BADXDR, NFS4ERR_BAD_STATEID,       |
>    |                      | NFS4ERR_DEADSESSION, NFS4ERR_DELAY,        |
>    |                      | NFS4ERR_DELEG_REVOKED, NFS4ERR_DQUOT,      |
>    |                      | NFS4ERR_FHEXPIRED, NFS4ERR_GRACE,          |
>    |                      | NFS4ERR_INVAL, NFS4ERR_IO,                 |
>    |                      | NFS4ERR_LAYOUTTRYLATER,                    |
>    |                      | NFS4ERR_LAYOUTUNAVAILABLE, NFS4ERR_LOCKED, |
>    |                      | NFS4ERR_MOVED, NFS4ERR_NOFILEHANDLE,       |
>    |                      | NFS4ERR_NOSPC, NFS4ERR_NOTSUPP,            |
>    |                      | NFS4ERR_OLD_STATEID, NFS4ERR_OPENMODE,     |
>    |                      | NFS4ERR_OP_NOT_IN_SESSION,                 |
>    |                      | NFS4ERR_RECALLCONFLICT,                    |
>    |                      | NFS4ERR_REP_TOO_BIG,                       |
>    |                      | NFS4ERR_REP_TOO_BIG_TO_CACHE,              |
>    |                      | NFS4ERR_REQ_TOO_BIG,                       |
>    |                      | NFS4ERR_RETRY_UNCACHED_REP, NFS4ERR_ROFS,  |
>    |                      | NFS4ERR_SERVERFAULT, NFS4ERR_STALE,        |
>    |                      | NFS4ERR_TOOSMALL, NFS4ERR_TOO_MANY_OPS,    |
>    |                      | NFS4ERR_UNKNOWN_LAYOUTTYPE,                |
>    |                      | NFS4ERR_WRONG_TYPE                         |
>
>
> Notes
> -----
> It could be argued that the OPEN takes care of a NFS4ERR_ROFS for a
> LAYOUTGET of a LAYOUTIOMODE4_RW, but that does not explain why
> WRITE is allowed to return a NFS4ERR_ROFS.
>
> With the Flex File Layout Type, the storage device depends on the metadata
> server enforcing the read-only filesystem semantics. An NFSv3 WRITE to the
> storage device might be accepted even though the filesystem might be RO.
> Further, if a snapshot is taken, the storage device might not be aware of 
> the
> fact that a data file is in a snapshot.
>
> Currently, if the underlying filesystem determines that the LAYOUTGET for a
> LAYOUTIOMODE4_RW is going to return NFS4ERR_ROFS, to be spec
> compliant, it MUST convert the error code to NFS4ERR_SERVERFAULT.  The
> client may then decide to perform IO through the metadata server with
> NFSv4 WRITE calls, which will in turn get a NFS4ERR_ROFS error. This change
> pushes the responsibility to be on the LAYOUTGET and allows the client to
> inform the application of an error earlier.
>
> AD Comments:
> This topic requires WG discussion and establishment of consensus. Thus for
> future document update.
>
>  --VERIFIER NOTES--
>    This topic requires WG discussion and establishment of consensus. Thus 
> for
> future document update.
>
>
> --------------------------------------
> RFC5661 (draft-ietf-nfsv4-minorversion1-29)
> --------------------------------------
> Title               : Network File System (NFS) Version 4 Minor Version 1 
> Protocol
> Publication Date    : January 2010
> Author(s)           : S. Shepler, Ed., M. Eisler, Ed., D. Noveck, Ed.
> Category            : PROPOSED STANDARD
> Source              : Network File System Version 4
> Area                : Transport
> Stream              : IETF
> Verifying Party     : IESG