Re: [Netconf] [Technical Errata Reported] RFC8072 (5131)

Robert Wilton <rwilton@cisco.com> Wed, 11 October 2017 16:09 UTC

Return-Path: <rwilton@cisco.com>
X-Original-To: netconf@ietfa.amsl.com
Delivered-To: netconf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 2CFD51342D5 for <netconf@ietfa.amsl.com>; Wed, 11 Oct 2017 09:09:28 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.5
X-Spam-Level:
X-Spam-Status: No, score=-14.5 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.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 JAv7_2sZoRhH for <netconf@ietfa.amsl.com>; Wed, 11 Oct 2017 09:09:26 -0700 (PDT)
Received: from aer-iport-2.cisco.com (aer-iport-2.cisco.com [173.38.203.52]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 99B7B1342D3 for <netconf@ietf.org>; Wed, 11 Oct 2017 09:09:25 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=8927; q=dns/txt; s=iport; t=1507738165; x=1508947765; h=subject:to:cc:references:from:message-id:date: mime-version:in-reply-to; bh=2ZKw9vxwOd5awVDTsikTVHKHN+YkIj4LbjRJd5nWIqA=; b=Z/GiYAhDS4vVay9NghgmC622LwS/RVYVDkk3caRLnyZn59HXNf4oGuUJ vfgEROYJfFKLrSqC/XWkslu7TvxUcLL7U5FiPE/itGkjJUNwT2FYSMTSW uh2gaZ+BDowzxgRvBn8P7dxhs2XcMwATD2+Nsz2KRVPdn+58YPMwH2HXL 8=;
X-IronPort-AV: E=Sophos;i="5.43,362,1503360000"; d="scan'208,217";a="655392893"
Received: from aer-iport-nat.cisco.com (HELO aer-core-2.cisco.com) ([173.38.203.22]) by aer-iport-2.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 11 Oct 2017 16:09:24 +0000
Received: from [10.63.23.63] (dhcp-ensft1-uk-vla370-10-63-23-63.cisco.com [10.63.23.63]) by aer-core-2.cisco.com (8.14.5/8.14.5) with ESMTP id v9BG9Nbt002477; Wed, 11 Oct 2017 16:09:23 GMT
To: Kent Watsen <kwatsen@juniper.net>, Benoit Claise <bclaise@cisco.com>, "andy@yumaworks.com" <andy@yumaworks.com>, "mbj@tail-f.com" <mbj@tail-f.com>, "warren@kumari.net" <warren@kumari.net>, "mjethanandani@gmail.com" <mjethanandani@gmail.com>
Cc: "netconf@ietf.org" <netconf@ietf.org>
References: <20170928105016.B2A88B81896@rfc-editor.org> <b6c0cf18-eba4-f4e0-4802-bfe524095b57@cisco.com> <F149BE7A-07C8-42CE-92AF-4355E6B409E0@juniper.net>
From: Robert Wilton <rwilton@cisco.com>
Message-ID: <c5cd4554-34f5-31a3-e16c-8c858b5e1616@cisco.com>
Date: Wed, 11 Oct 2017 17:09:23 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0
MIME-Version: 1.0
In-Reply-To: <F149BE7A-07C8-42CE-92AF-4355E6B409E0@juniper.net>
Content-Type: multipart/alternative; boundary="------------7E630B0320D01A86FED040FD"
Content-Language: en-US
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/Ii6DT0iXbjmGoXjtKlUguMYMO3Q>
Subject: Re: [Netconf] [Technical Errata Reported] RFC8072 (5131)
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Network Configuration WG mailing list <netconf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netconf>, <mailto:netconf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netconf/>
List-Post: <mailto:netconf@ietf.org>
List-Help: <mailto:netconf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netconf>, <mailto:netconf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 11 Oct 2017 16:09:28 -0000

Hi Kent,

But the previous paragraph in the draft states:

    The "merge", "replace", "create", "delete", and "remove" edit
    operations have exactly the same meanings as those defined for the
    "operation" attribute described inSection 7.2 of [RFC6241] <https://tools.ietf.org/html/rfc6241#section-7.2>.


That NETCONF section allows "merge" and "replace" on nodes that don't 
exist (they are just created).  Also a "remove" on a data node that 
doesn't exist doesn't return any error.  It is just a silent no-op.

So, it seems that the YANG Patch semantics are not the same as for 
NETCONF, which seems to conflict with the previous paragraph quoted above.

Thanks,
Rob


On 11/10/2017 16:36, Kent Watsen wrote:
> I think the existing text is correct.  If the resource instance doesn't exist, it can only be created.  If it doesn't exist, then it cannot be merged, replaced, deleted, or removed.
>
> Kent (co-author)
>
> --
>
> Dear YANG Patch Media Type authors,
>
> What do you think of this proposed errata?
>
> Regards, B.
>> The following errata report has been submitted for RFC8072,
>> "YANG Patch Media Type".
>>
>> --------------------------------------
>> You may review the report below and at:
>> https://urldefense.proofpoint.com/v2/url?u=http-3A__www.rfc-2Deditor.org_errata_eid5131&d=DwICaQ&c=HAkYuh63rsuhr6Scbfh0UjBXeMK-ndb3voDTXcWzoCI&r=9zkP0xnJUvZGJ9EPoOH7Yhqn2gsBYaGTvjISlaJdcZo&m=KLQjPIYskU70WRcZ0nCQYZ4njobUB-tkMpQKKEBb8eE&s=5HhVTU7yZdrq0Ztpkqoqb2_yUmZJ9pk7jGCpEDJSLr0&e=
>>
>> --------------------------------------
>> Type: Technical
>> Reported by: Robert Wilton <rwilton@cisco.com>
>>
>> Section: 2.2
>>
>> Original Text
>> -------------
>> Regarding section 2.2 of RFC 8072, the third paragraph states:
>>
>>
>>                                          ... If the edit does not identify
>>       any existing resource instance and the operation for the edit is not
>>       "create", then the request MUST NOT be processed and a "404 Not
>>       Found" error response MUST be sent by the server.
>>
>> Corrected Text
>> --------------
>>                                         ... If the edit does not identify
>>      any existing resource instance and the operation for the edit is
>>      "delete" or "move" then the request MUST NOT be processed and a
>>      "404 Not Found" error response MUST be sent by the server.
>>
>> Notes
>> -----
>> As per the second paragraph of section 2.2 of RFC 8072, the operations are expected to mirror the semantics of the "operation" attribute described in Section 7.2 of [RFC6241].
>>
>> The spec also doesn't specify what happens if it is a "create" operation and the resource already exists.  It should probably also state that "400 Bad Request" is returned.
>>
>> Instructions:
>> -------------
>> This erratum is currently posted as "Reported". If necessary, please
>> use "Reply All" to discuss whether it should be verified or
>> rejected. When a decision is reached, the verifying party
>> can log in to change the status and edit the report, if necessary.
>>
>> --------------------------------------
>> RFC8072 (draft-ietf-netconf-yang-patch-14)
>> --------------------------------------
>> Title               : YANG Patch Media Type
>> Publication Date    : February 2017
>> Author(s)           : A. Bierman, M. Bjorklund, K. Watsen
>> Category            : PROPOSED STANDARD
>> Source              : Network Configuration
>> Area                : Operations and Management
>> Stream              : IETF
>> Verifying Party     : IESG
>> .
>>
>
>