Re: [netmod] WG Last Call: draft-ietf-netmod-rfc6087bis-14

Qin Wu <bill.wu@huawei.com> Tue, 17 October 2017 05:12 UTC

Return-Path: <bill.wu@huawei.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 6FA9C132949 for <netmod@ietfa.amsl.com>; Mon, 16 Oct 2017 22:12:50 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.22
X-Spam-Level:
X-Spam-Status: No, score=-4.22 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_MED=-2.3, RCVD_IN_MSPIKE_H3=-0.01, RCVD_IN_MSPIKE_WL=-0.01, SPF_PASS=-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 vMrWZOdkyxoT for <netmod@ietfa.amsl.com>; Mon, 16 Oct 2017 22:12:48 -0700 (PDT)
Received: from lhrrgout.huawei.com (lhrrgout.huawei.com [194.213.3.17]) (using TLSv1 with cipher RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 312B3132924 for <netmod@ietf.org>; Mon, 16 Oct 2017 22:12:47 -0700 (PDT)
Received: from 172.18.7.190 (EHLO LHREML713-CAH.china.huawei.com) ([172.18.7.190]) by lhrrg02-dlp.huawei.com (MOS 4.3.7-GA FastPath queued) with ESMTP id DQT93382; Tue, 17 Oct 2017 05:12:44 +0000 (GMT)
Received: from NKGEML412-HUB.china.huawei.com (10.98.56.73) by LHREML713-CAH.china.huawei.com (10.201.108.36) with Microsoft SMTP Server (TLS) id 14.3.361.1; Tue, 17 Oct 2017 06:12:43 +0100
Received: from NKGEML513-MBX.china.huawei.com ([169.254.1.105]) by nkgeml412-hub.china.huawei.com ([10.98.56.73]) with mapi id 14.03.0235.001; Tue, 17 Oct 2017 13:04:56 +0800
From: Qin Wu <bill.wu@huawei.com>
To: NetMod WG <netmod@ietf.org>
Thread-Topic: [netmod] WG Last Call: draft-ietf-netmod-rfc6087bis-14
Thread-Index: AQHTRl0k7vUwulvIVEqGWP7/9n94aaLnavqg
Date: Tue, 17 Oct 2017 05:04:56 +0000
Message-ID: <B8F9A780D330094D99AF023C5877DABA9ABF4F91@nkgeml513-mbx.china.huawei.com>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.136.79.163]
Content-Type: multipart/alternative; boundary="_000_B8F9A780D330094D99AF023C5877DABA9ABF4F91nkgeml513mbxchi_"
MIME-Version: 1.0
X-CFilter-Loop: Reflected
X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020201.59E5914D.0078, ss=1, re=0.000, recu=0.000, reip=0.000, cl=1, cld=1, fgs=0, ip=169.254.1.105, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32
X-Mirapoint-Loop-Id: 5fcc494bb5ef979f6997c9a15e26cdc5
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/cjugHN3UgPuaQjXXnl6hn7pa5G0>
Subject: Re: [netmod] WG Last Call: draft-ietf-netmod-rfc6087bis-14
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: Tue, 17 Oct 2017 05:12:50 -0000

I have reviewed this draft and have the following comments to this draft:

1.      I want to understand the relationship between draft-dsdt-nmda-guidelines-01 and draft-ietf-netmod-rfc6087bis-14, it looks some of pieces have been merged into draft-ietf-netmod-rfc6087bis-14 and some of pieces have been moved to draft-ietf-netmod-revised-datastores-04, Would you like to clarify this in details.

2.      Section 4.6.5, 1st paragraph, 1st sentence

Have difficulty to parse this sentence, what it is emphasize here is when combined with several modules, the XPATH expression being evaluated is different from one that has been evaluated within a single module, am I correct? If the answer is yes, I believe this sentence needs to be improved.

3.      Section 4.6.5

s/ when /foo/services/*/active/" when /foo/services/*/active"

4.      Section 4.11.4

What the second version is referred to? Correct one?

5.      Section 4.11.5

"When used within a list key, only one value can (and must)

exist for this key leaf.  The type "empty" SHOULD NOT be used for a

key leaf since it is pointless."

It looks these two sentences contradict with each other? The first sentence says 'empty' data type could be used as a list key(The confusion is what one value is is not clear),

The second sentence says the 'empty' data type should not be used as list key. Could you give an example on how to use 'empty' data type with a list key leaf.

6.      Section 4.12.2

s/-top-level/top-level

Can not parse the first paragraph.

7.      Section 4.23.2, 2nd paragraph

What the value set is referred to? Take admin-state leaf and oper-state leaf as an example, if both admin-state and oper-state are set to uint32, do you mean values range set for admin-state

e.g.,range "0..100" is different from value range set for oper-state, e.g.,"101..200", Do you mean "admin-state" value set is configured value set?

8. Section 4.23.2 3rd paragraph, 4th paragraph

    How the 4th  paragraph is related to 3nd paragraph?

In 3nd paragraph, it discuss two list have two different key

In 4th paragraph, it discusses two list have keys with same type

9.      Section 4.23.3.1

Why there is no example for Temporary non-NMDA Modules?

10.  Section 4.23.3.1, 2nd paragraph

s/ the deprecated <get>

operation/with the deprecated <get> operation

11.  Section 4.23.3.1 3rd paragraph

when we need to create temporary non-NMDA model from NMDA model? Since when we transition to NMDA model, why we should transition back to temporary Non-NMDA model?

How temporary non-NMDA Modules is different from temporary NMDA modules? This is something I feel very confused.

12.  Section 4.23.3.1 said:
"   o  Retain or create only the top-level nodes that have a "config"
      statement value "false".  These subtrees represent config=false
      data nodes that were combined into the configuration subtree, and
      therefore not available to non-NMDA aware clients.  Set the
      "status" statement to "deprecated" for each new node.

"

Where  these subtrees are defined? It is not clear to me by reading the first sentence in this bullet.

13.  Section 4.23.3.4

How "create a Temporary NMDA Module" is different from "Convert an old Non-NMDA Module" described in section 4.23.3.3

When do we need to create a temporary NMDA module?

14.  Section 4.26.4
s/access access control/access control

-Qin
-------- Forwarded Message --------
Subject:

[netmod] WG Last Call: draft-ietf-netmod-rfc6087bis-14

Date:

Tue, 12 Sep 2017 18:21:49 +0000

From:

Kent Watsen <kwatsen@juniper.net><mailto:kwatsen@juniper.net>

To:

netmod@ietf.org<mailto:netmod@ietf.org> <netmod@ietf.org><mailto:netmod@ietf.org>

CC:

netmod-chairs@ietf.org<mailto:netmod-chairs@ietf.org> <netmod-chairs@ietf.org><mailto:netmod-chairs@ietf.org>, draft-ietf-netmod-rfc6087bis@ietf.org<mailto:draft-ietf-netmod-rfc6087bis@ietf.org> <draft-ietf-netmod-rfc6087bis@ietf.org><mailto:draft-ietf-netmod-rfc6087bis@ietf.org>



This starts a two-week working group last call on:



    Guidelines for Authors and Reviewers of YANG Data Model Documents

    https://tools.ietf.org/html/draft-ietf-netmod-rfc6087bis-14



Please send email to the list indicating your support or concerns.



We are particularly interested in statements of the form:

  * I have reviewed this draft and found no issues.

  * I have reviewed this draft and found the following issues: ...





Thank you,

NETMOD WG Chairs







_______________________________________________

netmod mailing list

netmod@ietf.org<mailto:netmod@ietf.org>

https://www.ietf.org/mailman/listinfo/netmod

.



________________________________