Re: [secdir] review of draft-ietf-netconf-nmda-restconf-04

Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de> Mon, 16 July 2018 10:42 UTC

Return-Path: <j.schoenwaelder@jacobs-university.de>
X-Original-To: secdir@ietfa.amsl.com
Delivered-To: secdir@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 447B3130FEC; Mon, 16 Jul 2018 03:42:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, URIBL_BLOCKED=0.001] autolearn=ham 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 WzxWeTXba4Ta; Mon, 16 Jul 2018 03:42:29 -0700 (PDT)
Received: from anna.localdomain (anna.eecs.jacobs-university.de [IPv6:2001:638:709:5::7]) by ietfa.amsl.com (Postfix) with ESMTP id B259B130FE9; Mon, 16 Jul 2018 03:42:29 -0700 (PDT)
Received: by anna.localdomain (Postfix, from userid 501) id 41650233FEEE; Mon, 16 Jul 2018 12:42:25 +0200 (CEST)
Date: Mon, 16 Jul 2018 12:42:25 +0200
From: Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de>
To: Mahesh Jethanandani <mjethanandani@gmail.com>
Cc: Daniel Harkins <dharkins@lounge.org>, Kent Watsen <kwatsen@juniper.net>, The IESG <iesg@ietf.org>, "secdir@ietf.org" <secdir@ietf.org>, "draft-ietf-netconf-nmda-restconf.all@ietf.org" <draft-ietf-netconf-nmda-restconf.all@ietf.org>
Message-ID: <20180716104225.hlg3r3x62q4ipq2x@anna.jacobs.jacobs-university.de>
Reply-To: Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de>
Mail-Followup-To: Mahesh Jethanandani <mjethanandani@gmail.com>, Daniel Harkins <dharkins@lounge.org>, Kent Watsen <kwatsen@juniper.net>, The IESG <iesg@ietf.org>, "secdir@ietf.org" <secdir@ietf.org>, "draft-ietf-netconf-nmda-restconf.all@ietf.org" <draft-ietf-netconf-nmda-restconf.all@ietf.org>
References: <f919a44f-d93b-f399-cc5d-1353c1c5b57d@lounge.org> <20180704124128.qpr7tunjw5quiex6@anna.jacobs.jacobs-university.de> <9b2f8091-9ead-e188-ee34-1acfead2dcd2@lounge.org> <20180704183436.zjzwz4vowqi5phz7@anna.jacobs.jacobs-university.de> <14e7ae2d-90ae-32c5-c814-a2d31e9f1a4e@lounge.org> <DB8ED828-D18F-4AEF-A7AB-884D085ECDA7@juniper.net> <8dbe0263-de63-0ddd-86db-04fa7744155c@lounge.org> <99EB8497-BCC0-45A3-A412-6FC5036243F6@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <99EB8497-BCC0-45A3-A412-6FC5036243F6@gmail.com>
User-Agent: NeoMutt/20180622
Archived-At: <https://mailarchive.ietf.org/arch/msg/secdir/WQm-H7p-6w-jufw8QQTURt22HQ8>
Subject: Re: [secdir] review of draft-ietf-netconf-nmda-restconf-04
X-BeenThere: secdir@ietf.org
X-Mailman-Version: 2.1.27
Precedence: list
List-Id: Security Area Directorate <secdir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/secdir>, <mailto:secdir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/secdir/>
List-Post: <mailto:secdir@ietf.org>
List-Help: <mailto:secdir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/secdir>, <mailto:secdir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 16 Jul 2018 10:42:33 -0000

On Mon, Jul 09, 2018 at 05:48:53PM -0700, Mahesh Jethanandani wrote:
> There have been a few comments that have been made to the version of the draft that has been submitted to IESG. While we wait for other comments to be provided, and direction of when an update should be provided, who amongst the authors is updating GitHub with these changes? 
> 
> I am concerned we might lose track of the changes that need to be made.
> 
> Here are the changes that I am aware of for nmda-restconf. I can compile a similar list for nmda-netconf if needed.

Having a list of outstanding edits for nmda-netconf would be nice as well
so we can make the edits on github.
 
> Kent’s comment on nmda-restconf operations (thread <https://www.ietf.org/mail-archive/web/netconf/current/msg14998.html> from today).

Given that lock and unlock require session semantics, I think the
resolution should be to add.

   A RESTCONF server supporting NMDA datastores MAY implement the
   "ietf-netconf-nmda" [I-D. ietf-netconf-nmda-netconf] module to
   enable the NETCONF operations defined in this draft to appear
   under the {+restconf}/operations resource.

But then, what is the value of this? You do not need get-data and
edit-data with RESTCONF since you can send GET/POST/PATCH straight to
the datastore resources. I guess I am still unclear which problem we
are trying to solve by adding this (or a similar) statement.

> Dan’s comment on this thread (also from today, but is not on netconf mailing list)

Not sure which one you mean.

> Russ’s Genart review (thread <https://www.ietf.org/mail-archive/web/netconf/current/msg14834.html> from June 28)

Not sure what the resolution is, removing the example text or trying
to better explain that this is example text.

> Rohit’s change-2 comments provided for nmda-netconf, that are applicable for nmda-restconf (thread <https://www.ietf.org/mail-archive/web/netconf/current/msg14607.html> from June 1)

The inclusion of an example showing the usage of
negated-origin-filter? This was an nmda netconf command. We do not
seem to have an origin filter in nmda restconf, this is an nmda
netconf only feature?

> Anything else that I missing?

Not that I recall.

/js

-- 
Juergen Schoenwaelder           Jacobs University Bremen gGmbH
Phone: +49 421 200 3587         Campus Ring 1 | 28759 Bremen | Germany
Fax:   +49 421 200 3103         <https://www.jacobs-university.de/>