Re: [netconf] Comments on draft-tao-netconf-data-export-capabilities

Qin Wu <bill.wu@huawei.com> Wed, 04 August 2021 08:01 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 85C7F3A0C0C for <netconf@ietfa.amsl.com>; Wed, 4 Aug 2021 01:01:09 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.196
X-Spam-Level:
X-Spam-Status: No, score=-4.196 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.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=unavailable 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 fP-BRX84Ca5L for <netconf@ietfa.amsl.com>; Wed, 4 Aug 2021 01:01:05 -0700 (PDT)
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 50F013A0BF4 for <netconf@ietf.org>; Wed, 4 Aug 2021 01:01:05 -0700 (PDT)
Received: from fraeml708-chm.china.huawei.com (unknown [172.18.147.226]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4Gfkgs71pJz6GFb3 for <netconf@ietf.org>; Wed, 4 Aug 2021 16:00:41 +0800 (CST)
Received: from dggeml701-chm.china.huawei.com (10.3.17.134) by fraeml708-chm.china.huawei.com (10.206.15.36) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256) id 15.1.2176.2; Wed, 4 Aug 2021 10:00:55 +0200
Received: from dggeml753-chm.china.huawei.com (10.1.199.152) by dggeml701-chm.china.huawei.com (10.3.17.134) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.2176.2; Wed, 4 Aug 2021 16:00:53 +0800
Received: from dggeml753-chm.china.huawei.com ([10.1.199.152]) by dggeml753-chm.china.huawei.com ([10.1.199.152]) with mapi id 15.01.2176.012; Wed, 4 Aug 2021 16:00:53 +0800
From: Qin Wu <bill.wu@huawei.com>
To: Balázs Lengyel <balazs.lengyel=40ericsson.com@dmarc.ietf.org>, "netconf@ietf.org" <netconf@ietf.org>
Thread-Topic: Comments on draft-tao-netconf-data-export-capabilities
Thread-Index: AdeJBGefKPCEiczkTwuY/jEU2p/YgQ==
Date: Wed, 04 Aug 2021 08:00:53 +0000
Message-ID: <0cb25885b42c4f7aa18b874e0bba4bef@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.123.117]
Content-Type: multipart/alternative; boundary="_000_0cb25885b42c4f7aa18b874e0bba4befhuaweicom_"
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/wm5bUSxlqObsSLEEBiAR7pGHTHY>
Subject: Re: [netconf] Comments on draft-tao-netconf-data-export-capabilities
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: Wed, 04 Aug 2021 08:01:10 -0000

Hi, Balazs:
发件人: netconf [mailto:netconf-bounces@ietf.org] 代表 Balázs Lengyel
发送时间: 2021年7月31日 18:05
收件人: netconf@ietf.org
主题: [netconf] Comments on draft-tao-netconf-data-export-capabilities-0

Hello,
“Capability values on a smaller, more specific part of the
server's data always override more generic values.”

This not following the latest version of draft-ietf-netconf-notification-capabilities. Please check the description of module ietf-system-capabilities.



[Qin Wu] Thanks for correction, you are right, this statement is not applicable here any more and should be taken out. We will make yang push related server capability described in this draft align with the latest version of I-D.netconf-notification-capabilities.



2.1 tree diagram:

https://datatracker.ietf.org/doc/html/draft-ietf-netconf-notification-capabilities#section-4 states:

Every set of such capabilities SHOULD be

   wrapped in a container under the augment statement to cleanly

   separate different groups of capabilities.

This is not followed in your augment of /sysc:system-capabilities

[Qin Wu] Agree to follow the rule defined in I-D.netconf-notification-capabilities, to do so, we have two choice

add data-export-export-capabilities container on top of data-export-capabilities list;

change data-export-capabilities list into data-export-capabilities container and in addition, change all the leaf under data-export-capabilities list into leaf list to support e.g., multiple transport, encoding support.

My preference is the option b.



https://datatracker.ietf.org/doc/html/draft-ietf-netconf-notification-capabilities#section-4.2 states:

1) search for a datastore-capabilities list entry for

     the specific datastore. When stating a specific capability, the

     relative path for any specific capability must be the same

     under the system-capabilities container and under the

     per-node-capabilities list: the same grouping for defining

     the capabilities MUST be used.

Please follow that. Please specify every capability both on the under system-capabilities and under per-node-capabilities.

A specific capability might always be defined on system or datastore or per-node level on your system, other systems might have a more granular capability set. Even if a capability is available on e.g. per-node level, your system is free not to use it.

[Qin Wu] Agree to add per-node level capabilities.

Why is this model valid only for telemetry? It should be usable for other notifications too. E.g. transport-protocol or security is just as relevant for any other notification.

[Qin Wu] Yes, telemetry is the key use case. But I agree it is applicable other notification, will make this clear in the next version.

What is event based subscription mode?

[Qin Wu] Timer-event-support is similar to what is described in period subscription, i.e., push update is triggered periodically, the timer will be set to the update interval defined in RFC8641. I feel this parameter is a little bit redundant, I think we can take it out.

What is coutner based trigger mode ?

[Qin Wu] Counter threshold support is applicable to a set of data nodes which are integer built-in type.  The data node value are restricted not to exceed the specific threshold value. But not all the data nodes need to have such counter threshold support since not all of data nodes are integer built-in type. Even data nodes are integer built-in type, it is not mandatory to support threshold constraints, be default, this feature can be disabled.

Does this parameter make sense to you?



suppress-redundant, this mechanism is not described. What is suppressed?

[Qin Wu] Suppress-redundant indicate that duplicated data objects should be suppressed to be sent during each update interval.

The mechanism is to detect the occurrence of monitored data object in each update interval. If the occurrence is equal to or more than 2, the value of monitored data object will not be reported.


Regards Balazs

--
Balazs Lengyel                    Senior Specialist                       Ericsson Hungary Ltd.
Mobile: +36-70-330-7909              email: Balazs.Lengyel@ericsson.com<mailto:Balazs.Lengyel@ericsson.com>