Re: [Netconf] IETF Last Call Gen-ART review of draft-ietf-netconf-restconf-15

worley@ariadne.com (Dale R. Worley) Mon, 26 September 2016 20:37 UTC

Return-Path: <worley@alum.mit.edu>
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 A026F12B34D for <netconf@ietfa.amsl.com>; Mon, 26 Sep 2016 13:37:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.934
X-Spam-Level:
X-Spam-Status: No, score=-1.934 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HEADER_FROM_DIFFERENT_DOMAINS=0.001, RCVD_IN_DNSWL_LOW=-0.7, SPF_SOFTFAIL=0.665] autolearn=no 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 vFRoY-MEfdl7 for <netconf@ietfa.amsl.com>; Mon, 26 Sep 2016 13:37:41 -0700 (PDT)
Received: from resqmta-ch2-01v.sys.comcast.net (resqmta-ch2-01v.sys.comcast.net [IPv6:2001:558:fe21:29:69:252:207:33]) (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 9DC3612B2F3 for <netconf@ietf.org>; Mon, 26 Sep 2016 13:37:23 -0700 (PDT)
Received: from resomta-ch2-19v.sys.comcast.net ([69.252.207.115]) by resqmta-ch2-01v.sys.comcast.net with SMTP id oceJbFWmNTaLwoceRbbUjd; Mon, 26 Sep 2016 20:37:23 +0000
Received: from hobgoblin.ariadne.com ([173.48.63.26]) by resomta-ch2-19v.sys.comcast.net with SMTP id occ5blOoXlRdooccBbkrEx; Mon, 26 Sep 2016 20:35:18 +0000
Received: from hobgoblin.ariadne.com (hobgoblin.ariadne.com [127.0.0.1]) by hobgoblin.ariadne.com (8.14.7/8.14.7) with ESMTP id u8QKYvmp003875; Mon, 26 Sep 2016 16:34:57 -0400
Received: (from worley@localhost) by hobgoblin.ariadne.com (8.14.7/8.14.7/Submit) id u8QKYuXS003872; Mon, 26 Sep 2016 16:34:56 -0400
X-Authentication-Warning: hobgoblin.ariadne.com: worley set sender to worley@alum.mit.edu using -f
From: worley@ariadne.com
To: Andy Bierman <andy@yumaworks.com>
In-Reply-To: <CABCOCHTmxNNrGopELX=w+XvG2Q=NoxrxEzJ0OzJ3o2yyxTDCLA@mail.gmail.com> (andy@yumaworks.com)
Sender: worley@ariadne.com
Date: Mon, 26 Sep 2016 16:34:56 -0400
Message-ID: <87ponq5rin.fsf@hobgoblin.ariadne.com>
X-CMAE-Envelope: MS4wfDI6ITnfZIayoM/HuhBl7e7PU1zVQHdr4mJlf1amPDxLzfw1HNZn/GeYxPt+/TZIdsvYXp39Hg0OvTD04C41ZbGu5K6VZMPq4nuzZ/LNdJ/sSS2fh6gt zZup3JZT642Hyee+xIU2jCHTWNTmGTuAAwpc3mYZyKe5/3D913MB0MwsHOSrh+agzKdWgIZrfBpyZVmTi2eAkzr3yA3R6XR9uM0=
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/1GTtIXAZbIkLQQHU3LQwsXB-oR8>
Cc: netconf@ietf.org
Subject: Re: [Netconf] IETF Last Call Gen-ART review of draft-ietf-netconf-restconf-15
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.17
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: Mon, 26 Sep 2016 20:37:42 -0000

Andy Bierman <andy@yumaworks.com> writes:
>>     8.  RESTCONF module
>>
>>              The following data-def-stmt sub-statements have special
>>              meaning when used within a yang-data-resource extension
>>              statement.
>>              - The list-stmt is not required to have a key-stmt defined.
>>              - The if-feature-stmt is ignored if present.
>>              - The config-stmt is ignored if present.
>>              - The available identity values for any 'identityref'
>>                leaf or leaf-list nodes is limited to the module
>>                containing this extension statement, and the modules
>>                imported into that module.
>>
>>     It seems like poor practice to have the extension be described as
>>     changing the semantics of Yang.  Better would be to turn these into
>>     constraints, so that the valid contents of yang-data are a subset of
>>     Yang, but that subset has the same semantics as Yang prescribes:
>>
>>              - The if-feature-stmt must not be present.
>>              - If the config-stmt is present, its value must be 'false'.
>>              - The available identity values for any 'identityref'
>>                leaf or leaf-list nodes is limited to the module
>>                containing this extension statement, and the modules
>>                imported into that module. [unchanged!]
>>
>>     The item "The list-stmt is not required to have a key-stmt defined."
>>     is redundant, since everything inside yang-data is not configuration
>>     data, and non-configuration lists need not have keys.
>
> The suggested edits are not accepted.
> They prevent groupings from being used within a YANG data template.

That bit about groupings is a subtle point I hadn't thought of.  If the
WG is OK with that, I'm OK with that.

Though I think the item "The list-stmt is not required to have a
key-stmt defined." is still redundant -- everything inside yang-data is
not configuration data, and non-configuration lists need not have keys.
Even if the list is inside a grouping, that doesn't change anything, in
that the list has a key-stmt or not depending on how the grouping is
defined, and either case is valid (in regard to the data template)
without a special rule.

Dale