IETF Logo Mail Archive

Re: [netmod] Fw: New Version Notification for draft-bjorklund-netmod-rfc7277bis-00.txt

Robert Wilton <rwilton@cisco.com> Fri, 01 September 2017 16:34 UTC

Return-Path: <rwilton@cisco.com>
X-Original-To: netmod@ietfa.amsl.com
Delivered-To: netmod@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 9FF891330AE for <netmod@ietfa.amsl.com>; Fri, 1 Sep 2017 09:34:09 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -12.6
X-Spam-Level:
X-Spam-Status: No, score=-12.6 tagged_above=-999 required=5 tests=[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 N9YdkeIKKPfD for <netmod@ietfa.amsl.com>; Fri, 1 Sep 2017 09:34:07 -0700 (PDT)
Received: from aer-iport-4.cisco.com (aer-iport-4.cisco.com [173.38.203.54]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3C59E132113 for <netmod@ietf.org>; Fri, 1 Sep 2017 09:34:07 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=6135; q=dns/txt; s=iport; t=1504283647; x=1505493247; h=subject:to:references:from:message-id:date:mime-version: in-reply-to; bh=bj03oEFmeBXYu6fns7OULXTf/bm9sX8uUZcOGMwwV7A=; b=kRVN0n0Eit6/HuBXx17K/z4Fm6PGm+pzqax0ByzyZZWow2YTFALxfugF pZi7QkeQGEOx1DFl2KCFEpdB8czPyEgH1Biq3pwJvmcwXHToUAh5rACyv YmzPy1PZz8fVgDhBzyqgRN4CvRkRdzu/6DKMEL9CEoT9glXCV3eDTfLlx s=;
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0BuAQDwiqlZ/xbLJq1dGQEBAQEBAQEBAQEBBwEBAQEBgy2BEYEVjwuRGpBphT+CEiEBCoRMTwKEVRcBAgEBAQEBAQFrKIUZAQEBAwEBbAkSCw4KLicwBgEMBgIBAYotELEyJ4srAQEBAQEBAQEBAQEBAQEBAQEBAQEBGAWDKoNQgWMrgn2KaQWgc5RRghOJQIcdiXmDWoh0IAE2gQ0yIQgcFUmFGBwZgU8/Nop/AQEB
X-IronPort-AV: E=Sophos;i="5.41,459,1498521600"; d="scan'208,217";a="657175621"
Received: from aer-iport-nat.cisco.com (HELO aer-core-3.cisco.com) ([173.38.203.22]) by aer-iport-4.cisco.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 01 Sep 2017 16:34:02 +0000
Received: from [10.63.23.169] (dhcp-ensft1-uk-vla370-10-63-23-169.cisco.com [10.63.23.169]) by aer-core-3.cisco.com (8.14.5/8.14.5) with ESMTP id v81GY2Qt005797; Fri, 1 Sep 2017 16:34:02 GMT
To: Martin Bjorklund <mbj@tail-f.com>, netmod@ietf.org
References: <20170821.140610.1599460825983098893.mbj@tail-f.com>
From: Robert Wilton <rwilton@cisco.com>
Message-ID: <aa4a1c8f-19e2-ac9f-3aa7-a4ad5828cd20@cisco.com>
Date: Fri, 01 Sep 2017 17:34:02 +0100
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0
MIME-Version: 1.0
In-Reply-To: <20170821.140610.1599460825983098893.mbj@tail-f.com>
Content-Type: multipart/alternative; boundary="------------9CAB2239288BAE497748B03C"
Content-Language: en-US
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/ekAqoj-TTbj5BzcN4jPhZQq0S-4>
Subject: Re: [netmod] Fw: New Version Notification for draft-bjorklund-netmod-rfc7277bis-00.txt
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: NETMOD WG list <netmod.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netmod>, <mailto:netmod-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netmod/>
List-Post: <mailto:netmod@ietf.org>
List-Help: <mailto:netmod-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netmod>, <mailto:netmod-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 01 Sep 2017 16:34:10 -0000

Hi Martin,

I have reviewed this document (mainly as a diff against the 7277).

In summary, the transformation to NMDA looks good.   Given that it is a 
fairly formulaic conversion, I hope that the WG will be able to quickly 
move to WG adoption, and even WG last call!


I did have a few minor comments (and spotted one bug):

1. leaf "origin" under IPv4 neighbors should be marked as "config false".


2. Sometimes the YANG module description refers to "the intended 
configuration datastore" or "the operational state datastore", but I 
think that it would be better to just refer to "intended configuration" 
and "operational state".  I.e. I think that it is OK for the draft to 
reference the different datastores, but I think that it might be better 
if the YANG modules don't.  This is partly in the sense that the YANG 
modules are just schema for configuration and state data, and that I see 
the use of datastores is effectively an IETF decision on how that 
configuration/state is instantiated.

To take the analogy further, I think that the it is conceivable that the 
source OpenConfig YANG models could also be usefully structured in the 
same NMDA style.  This would allow the OpenConfig models to work better 
with NETCONF/RESTCONF implementations that support NMDA.  An alternative 
implementation choice would be to use a script to convert NMDA style 
OpenConfig YANG models to equivalent existing OpenConfig style with 
their split config/state containers.

A second justification (if you don't like the one above), is that the 
configurable leaves could also be set via a dynamic configuration 
datastore such as via I2RS, hence it wouldn't necessarily always be via 
the intended datastore.


3. I think that it might be useful to add a comment to the IPv6 
link-layer-address leaf description to indicate that it wouldn't expect 
to be populated if the associated "state" is set to "incomplete".


Thanks,
Rob


On 21/08/2017 13:06, Martin Bjorklund wrote:
> Hi,
>
> This document defines an NMDA-compliant update to the IP model
> (RFC 7277).
>
> I would like to ask the WG to adopt this individual draft as a working
> group document.
>
>
> /martin
>
>
>
> _______________________________________________
> netmod mailing list
> netmod@ietf.org
> https://www.ietf.org/mailman/listinfo/netmod

v2.23.0 | Report a Bug | By Email