Re: [netconf] more comments on netconf-adaptive-subscription revision -08

Qin Wu <bill.wu@huawei.com> Mon, 07 February 2022 02:27 UTC

Return-Path: <bill.wu@huawei.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 AACD33A11C2 for <netconf@ietfa.amsl.com>; Sun, 6 Feb 2022 18:27:36 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.896
X-Spam-Level:
X-Spam-Status: No, score=-1.896 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_BLOCKED=0.001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, 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 fw0bZZhfwwKc for <netconf@ietfa.amsl.com>; Sun, 6 Feb 2022 18:27:32 -0800 (PST)
Received: from frasgout.his.huawei.com (frasgout.his.huawei.com [185.176.79.56]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2CB123A11BC for <netconf@ietf.org>; Sun, 6 Feb 2022 18:27:32 -0800 (PST)
Received: from fraeml734-chm.china.huawei.com (unknown [172.18.147.201]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4JsVQH6R6Cz67hkH; Mon, 7 Feb 2022 10:26:47 +0800 (CST)
Received: from canpemm100006.china.huawei.com (7.192.104.17) by fraeml734-chm.china.huawei.com (10.206.15.215) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21; Mon, 7 Feb 2022 03:27:27 +0100
Received: from canpemm500005.china.huawei.com (7.192.104.229) by canpemm100006.china.huawei.com (7.192.104.17) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.21; Mon, 7 Feb 2022 10:27:25 +0800
Received: from canpemm500005.china.huawei.com ([7.192.104.229]) by canpemm500005.china.huawei.com ([7.192.104.229]) with mapi id 15.01.2308.021; Mon, 7 Feb 2022 10:27:25 +0800
From: Qin Wu <bill.wu@huawei.com>
To: Michael Richardson <mcr@sandelman.ca>, "netconf@ietf.org" <netconf@ietf.org>
Thread-Topic: [netconf] more comments on netconf-adaptive-subscription revision -08
Thread-Index: AdgbyhIM2a9Uy6mqSamvt9s8SP/4Yw==
Date: Mon, 07 Feb 2022 02:27:25 +0000
Message-ID: <a307cc378fd44dacb19269676fad4ed5@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.100.16]
Content-Type: text/plain; charset="gb2312"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/whMFjU6i8idoF3Q3AOvshCQkYog>
Subject: Re: [netconf] more comments on netconf-adaptive-subscription revision -08
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: NETCONF WG 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, 07 Feb 2022 02:27:37 -0000

Thanks Michael for valuable review, see reply inline below.
-----邮件原件-----
发件人: netconf [mailto:netconf-bounces@ietf.org] 代表 Michael Richardson
发送时间: 2022年1月31日 22:19
收件人: netconf@ietf.org
主题: [netconf] more comments on netconf-adaptive-subscription revision -08

On 2021-06-24 15:14, Michael Richardson made some comments on:
> 
> I read:
>          Title           : Adaptive Subscription to YANG Notification
> 	Filename        : draft-wang-netconf-adaptive-subscription-05.txt
> 

I have reviewed a diff from -05 to -08, which you can see at:

https://www.ietf.org/rfcdiff?url1=draft-wang-netconf-adaptive-subscription-05&url2=draft-wang-netconf-adaptive-subscription-08

(I know that we have a new rfcdiff service being tested, but I can't recall where it is now)

I found the introduction of section 3, on XPath Evaluation is an interesting and welcome new addition.  I can't judge on the content of it, but it seems like it is useful things to explain.

[Qin] Yes, this piece was introduced based on comment raised by Rob in IETF 111 and IETF 112. The question is whether we select Arbitrary XPath and how complex such XPATH evaluation is. As we clarified in the section 3, we use similar selection filter defined in RFC8641 to express a standard XPATH Evaluation Criterion against targeted data object which has already defined in section 3.4 of RFC7950. In addition, we follow design rules defined in XPATH1.0, We are not using arbitrary XPATH evaluation, e.g, Support XPath Evaluation criteria against every data object, instead, we only support XPath Evaluation criteria against constant number. Therefore such xpath evaluation complexity is controllable.

leaf watermark {	
  	               type uint32;	
  	               description	
  	                 "The watermark for targeted data object. The higher	
  	                  or lower watermark can be specified for the	
  	                  targeted data object.";	
  	             }

probably this description should explain where the kind of watermark (high or low) is actually specified.
[Qin] Take thermostat module as example:
module thermostat {
…
leaf actual-temp {
type int32;
config false;
units “degrees Celsius”;
description “The measured temperature”;
}
}
We need to monitor the temperature change on specific function component in the device, We may need to control temperature change to make sure the highest temperature is not greater than high watermark and the lowest temperature is not less than Low watermark.
In addition, the watermark value is part of XPATH expression "xpath-external-eval", e.g., actual-temp<high-watermark&&atual-templ>low-watermark.
In this expression, high-watermark and low-watermark are both constant integer values. 
Since the watermark value is part of XPATH expression, therefore watermark parameter is an optional parameter, in some cases, it doesn't need.
Hope this clarifies.


I guess that I would never specify both, as that would be just two rules.
[Qin]: See our model design, we define a list of adaptive-periods, for each entry of adaptive periods, we will specify a watermark and corresponding xpath-external-eval.
Therefore we can specify both high watermark and low watermark using two entries. In other words, we can specify multiple rules in one subscprtion.

Are centiseconds already a typical unit for NETCONF?
[Qin]Yes, see similar example in yang push subscription model defined in RFC8641.

I see that you have rewritten the example YANG module for wifi based upon the CSA/MATTER data model.  It's good to have a realistic model.
I still wonder why an existing YANG model couldn't have been used.
[Qin]: CSA/MATTER only provide an information model, therefore we create a data model based on this information model defined in CSA/MATTER specification.
If CSA/MATTER data model has already been available somewhere, please let me know.

The text in section B.2 hints that the access point will change not just how often it reports, but how often it actually collects the data.
I'm not sure if that is the intention?
[Qin]: Yes, you are right, the goal is to focus on how often it reports, we can make this clear in the next version.
I think when it says "server", I think you mean access point?
[Qin]Yes, the access point will collect the data and report back to Wireless Lan access controller (Known as AC)[Qin Wu] .

_______________________________________________
netconf mailing list
netconf@ietf.org
https://www.ietf.org/mailman/listinfo/netconf