[netconf] Comments on draft-claise-netconf-metadata-for-collection-02
Balázs Lengyel <balazs.lengyel@ericsson.com> Sun, 01 August 2021 14:21 UTC
Return-Path: <balazs.lengyel@ericsson.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 7A2D73A3E1C for <netconf@ietfa.amsl.com>; Sun, 1 Aug 2021 07:21:32 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.552
X-Spam-Level:
X-Spam-Status: No, score=-2.552 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.452, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=ericsson.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 gQJldIUfduZ0 for <netconf@ietfa.amsl.com>; Sun, 1 Aug 2021 07:21:27 -0700 (PDT)
Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05on2046.outbound.protection.outlook.com [40.107.20.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E2EB33A27A4 for <netconf@ietf.org>; Sun, 1 Aug 2021 07:21:26 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MCKoO2SO1IKs6Z2+MTy+90whVdwDIKbDJusE1cWsFWz4DJlTgV5vOlTfsp7YFTmSe9HSOVZnHi0+VMjPaJXkAeYD23dXMg9yYbM4ozMVuxnBpfQ6uWZaBLd42G7P0TDPpxJxoqHzN/mGAMUhdMlzH07foZ/5Yv9sG/vrlArVNUuOOyT0mMG2pkxdvLAPMdkL2cf49aqRew8am0FInvlIecfdP+IVytmjCK9dzZLVlgMzLeJCsTn5kYnHNOJyTcADgj5cJe8nDAlRNccRIH5Tel/FPpftb4Hy+i/RMFpjZZljz4KTUmGI+cdMce/7GMXBvf6mgKVPu3yvdHBfr8Dslw==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vheEvbh4JRoAR6FiEWTwB9vV9rtYEFox+2UzfLQ5iCY=; b=Zjhq7pujPWSWXBKIPggq+rXpg3BHq17pMzsWqF3JxgZ54glSuWFV2bBMtrEHMoPGSZeXvVt895/L31BBifV45Knmjij60Fu4YTmxii8RLOFDj3tEqTi2kVJhWIWV9URT18pnzMAhIkg4fOTda+AE2IcpiYtvVospbBRqfff7q/iM0dKy8NxNzB0IggMUw1cXtaXn1zPPS6OynHCYXBg2ZAzGhNJ+FzgUHJQwrEM0XBUyn0UVdp9dwNmRMihf/u+MjBBGuNbhuPYuwvd8hfbtsuuMG73GPRpOggxHn/1bYTFz7eda/0bKNr8/CwtHycyOHuTMNuuKUuWDt90nQtv9sw==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vheEvbh4JRoAR6FiEWTwB9vV9rtYEFox+2UzfLQ5iCY=; b=kGKgt7XPSJFdsPY/zg+06YoFm6lgudA5rd71UMWuVNn5ZkRiBNiukLNO5PFHryXTfkcRp8IJo7fm8j64fKiOt4xo8TK5KRpt3gXlH4uLmV1uuRlumXxZcuwnWVOmSMUh03oED5rs0pnLzoFvmzPPmb5PXDi6wtfUDzNdWvu8fhc=
Received: from AM8PR07MB8230.eurprd07.prod.outlook.com (2603:10a6:20b:325::15) by AM8PR07MB8190.eurprd07.prod.outlook.com (2603:10a6:20b:321::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4352.21; Sun, 1 Aug 2021 14:21:24 +0000
Received: from AM8PR07MB8230.eurprd07.prod.outlook.com ([fe80::dd5e:8a67:d8c3:4625]) by AM8PR07MB8230.eurprd07.prod.outlook.com ([fe80::dd5e:8a67:d8c3:4625%9]) with mapi id 15.20.4373.018; Sun, 1 Aug 2021 14:21:24 +0000
From: Balázs Lengyel <balazs.lengyel@ericsson.com>
To: "netconf@ietf.org" <netconf@ietf.org>
Thread-Topic: Comments on draft-claise-netconf-metadata-for-collection-02
Thread-Index: AdeGWTGMCp7qLP3ZQhytMbWLqTYUWQ==
Date: Sun, 01 Aug 2021 14:21:23 +0000
Message-ID: <AM8PR07MB8230835C53319F0065C14D15F0EE9@AM8PR07MB8230.eurprd07.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator:
authentication-results: ietf.org; dkim=none (message not signed) header.d=none;ietf.org; dmarc=none action=none header.from=ericsson.com;
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 0e1e1937-6dcc-4af1-4af8-08d954f7a458
x-ms-traffictypediagnostic: AM8PR07MB8190:
x-microsoft-antispam-prvs: <AM8PR07MB81906726CFE44B073B61192CF0EE9@AM8PR07MB8190.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: hAawQZxSPojXQlcwV8ere0xx6CB482XWe7Oz5n2iGKXMsZIzdXaJbBgfOP4W1lAlPmIwwaQ1eKIwb167Zv7BEmTwGlpdOhYsFceo50+l/oFSHzEBjnlpXLxq9vYDkFE6AMY+pdlIIYN5S0rpkUEvQ0/KW3UyxJFLSGt6wXlVprKukyAbdH4tcHDEPs8aVbAUHJacQeTmoqx9XasuPJjUxKvWEchwSqG3Stueqyu5C712DKni3of9xk+ryWnJ3s3cFqE2ybGrJB7CLj7ffYAtl1YetlPF0hiuW3we7GY6FBR8wmsa260eiJNjLu1x/6oFeCgFQPDgZy7fSgrmRR4mWvqqY5mIw0UaRlVFLSCwWVoFy1Y4yw5HcoSDjN2FPicyMfp/QwFAZhFBlgkJGGBzrt6g3Uz9zmsArpRJBpZdl9HzGHv1LfOxS/fRJ3uMPJt818cUppNMk9+Y8NW67ZX9dQzyHaavy48P0whKJEqa5DfmT22s+nSNzoW69Ld2Gks+/rg3gFHu9mJh9p0Ykl5h+yfyQ55+MnOSOjeEGRQ/oCpMGL3whP7FmbUPw61dStXSZoBuNsDJJK3qvu/ydI/7vTIWxT2vb13NISQvV7ve3jd50eYdDImf67N0ExwHfwnCfbFJI2BxR0Eu/I3OwZaCg91PergOzKkpTSrX9chdWhATDKgvwoeK5RH3lZuYvjCuRnf5ABI7O8RKuyNTolocXIBzEtk96eZ87E+n7Q1gYSyho6nM8Ie7Y2qQbDhH7KiwAnUlnKFDBG/gjyZFee1fcHjJmWBgRNny1kUUjvEkH+uBaOdr82TMeOGf5EKOla+vYjpBubpHo7S2XblkYyZYSw==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM8PR07MB8230.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(376002)(396003)(366004)(39860400002)(346002)(136003)(186003)(52536014)(166002)(66946007)(99936003)(478600001)(5660300002)(6506007)(83380400001)(76116006)(8936002)(86362001)(55016002)(8676002)(66556008)(38100700002)(33656002)(316002)(26005)(4326008)(38070700005)(64756008)(9326002)(7696005)(66446008)(66616009)(71200400001)(6916009)(66476007)(966005)(122000001)(9686003)(2906002); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: vcAM3lyclNLj5aw0Nam1e6qiBgXiOnnOEab2rnlHRRAGdzwXa6FIZmZN6sMwEMByxIuAA+JydA8lshyMW1mVoarUdw8ZG/4r/UTd2vh2H22Y063ZoqrNd4I7AhYjXAw3rhKQXS/SwLeaYxpUCqRuIYmu9/e7L85tbrUmpEZ12sUvJCEz3hTX0Q33BMdiaCyEtoNuI8TIslflDxlcBpu6DVY6na6N/d/P8luOADfJO7wuA2PhoGmKYW7adTIX/VZTorDopKnIkD9mxUC96yNbyMMrplYYKFyRzBgGokrfjt51vWNKa2A3k1zuVLzyVsZHRNErjs5x5FHghp+dQHtYfzTAs2uRed9U8rPqNwmIEXn3P+TfKUEFGepmoaLxDqQ3ibrHH8MeiFMn1TPQ2t6vAgB4tr1msjifbZFkf6Ff0gIpQy5kaPy1dwhOMmqqNzI0SiWRLWm/SGjp8xCiL2xBae6A55/LTq/VM29e8pca/PEYIUqwQS2wy8G3U9waVXtMdNXJhG7Np0vNTIlJUkwAi8ooIS0jKSgD0qGZPQ0qCJmK1mgktMaWgP64xDw3qKlfnCpycuXB3P0r+wEB9+qLfzWRBnTjJ3rPuXbUN/0TE3QLK2Uz+atQh+vKFBDK4ViaEzvaRBjcUtcshMcQ9LiPqRlmNzATGTX7re0ee0ZHoZrxKr+b+cCBuKXcBQKBEwFF+VlpxI8XcqZhsZDfOnUUUm2YL1Wxc2TltB8Z2bn8x+upWJMHFoc8SYw0BDsBtCp5GXouf4gitlfDQsbsDkckQ173FfhJsCKYfUOPy2KfatkBhOTUT+pVfaBLW3YvKsEl8MwaHFKFCNwNfhYry4qDeLcExwnK7ktqgkSseRtjOzaUBSx5kBstI55kSDr4XQoP5789Y3g67Sp6RccLBGHAb+IzjogMRo387yOzJP0mBY2bLZbi2D3l+PIbp8ZKEulgNGvarB/QlCindlDJWes4Kl3cukH1x4va1sBF/UPCXp725rIOohx1Ix9yNiZCBqdbJ4Pi/K6aH48wJEVTAdmv9ew2Dx9hDcYhDFYt4JTjJy0pzw1JxQgH7xNZF38IrXPeg9ZFlXFrB9OWPs5o62M17x/fOgc2tYTXj+Bur33XGQyVGQSYEv8LJ1ASia4Membhn1UHo2GENOsuQnkdOoGGiR0RsbRRTAnmeliYkM1IiykaJIUucdL5Jf2F2B+zbD6hGvMrO5xnRYqbwrWJocKk9zDbe5X3QAr2sOfrz9pN9Inu5wHvJ9X/aqfz4f7jXFXzC80N7B9QU95nScHqrQeIaRCJDe3Z+AXwuJvjRNuDGeM=
x-ms-exchange-transport-forked: True
Content-Type: multipart/signed; protocol="application/x-pkcs7-signature"; micalg="SHA1"; boundary="----=_NextPart_000_00DF_01D786F1.3CC62D60"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM8PR07MB8230.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0e1e1937-6dcc-4af1-4af8-08d954f7a458
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 Aug 2021 14:21:23.9326 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: d+p8K/t7gDlp3aPjzskzB8VdFaBJtqKXd2y4dzTr+2BWvugRbCnj9vlzVb019xX7PQzOytQF+7B5YZCc21y/OCIwykXVBaChyiEhnS6/KBM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM8PR07MB8190
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/sPFXSUQhUX5ZjPCQiS9mFHK8B7U>
Subject: [netconf] Comments on draft-claise-netconf-metadata-for-collection-02
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: Sun, 01 Aug 2021 14:21:33 -0000
Hello, The draft contains good ideas, but it needs more work. Comments below. General: The base drafts use the term capabilities not metadata. Unless you have a reason for it, please stick to capabilities. Ch.3 suggested-observable-period: why is this available only at implementation time? IMHO it should be available in runtime too. I dont understand the description. Those 5 seconds ... What does this refer too? optimized-measurement-point: How should this effect a YANG-Push subscription ? Some examples would be good. corresponding-mib-oid: Wouldnt it be easier to define a YANG extension statement that defines this SNMP OIDs in the YANG models itself? These are static design time values, that cannot change in run-time. related-node: A relationship can be of many-many types. IMHO for this to be usable in an interoperable manner you need to narrow down what you mean by relationship. E.g. config-state-node-relationship: The capability is used for YANG models that use separate branches to specify the configuration and the state data for management objects, For a config=true container or list this capability specifies the list/container that carries the state data for the same managed object. A generic RPC... remove the word generic. It does not mean anything specific. Per-node dynamic metadata includes, part of the get-measurement-metadata RPC I think you mean: optimized-measurement-point: up the data tree chain change to up the containment hierarchy. The sentence could use some rewording. Can the node selector be any selector, or maybe it should point at a specific schema node? Otherwise, it would be very hard to specify the answer. computed-observable-period: This is the same as suggested-observable-period after it is dynamically adjusted. It serves the same purpose. IMHO it is a general principle of ALL capabilities, that * Capabilities are potentially available both in implementation time and in run-time. However, a network node may decide that it has no useful information in implementation time, so it does not publish a value * During runtime the system MAY or MAY NOT be change/update a capability value. Just because the value may be updated dynamically it is still the same capability. active-measurements: Needs rewording. It is unclear what this returns, why do you need to calculate measurement-periods to return the subscription IDs Ch.4.1 According to https://datatracker.ietf.org/doc/html/draft-ietf-netconf-notification-capabi lities#section-4 Every set of such capabilities SHOULD be wrapped in a container under the augment statement to cleanly separate different groups of capabilities. These "wrapper containers" SHALL be augmented in at /sysc:system-capabilities and /sysc:system-capabilities/sysc:datastore-capabilities/sysc:per-node- capabilities. Please follow it. grouping system-node-metadata-info: this is a very generic name, that says nothing about what capabilities are contained in the grouping. Find a better name. Generally the base draft uses the term capability not metadata. minimum-observable-period: I would call this either measurement-update-period or granularity (widely used by other SDOs) This really states the update period of the counter. The fact that there is no real sense in observing it more frequently is just a consequence. suggested-observable-period: only available at implementation time Why? Do you hide the value that was available in implementation-time? If yes why? If this is part of the YANG model it should be available in run-time. I propose to remove this half-sentence. related-node: description should be improved. In case of an NMDA server with a non-NMDA model (e.g. ietf-system) this could still be useful. get-measurement-metadata: The input node selector MAY point to multiple nodes if some key values are missing. This is not handled. RPC that returns the optimized what does optimized mean in this case? Avoid the word, describe instead what you mean. This RPC is added to allow clients to learn dynamically changing metadata You could learn dynamic metadata(? Use capabilities) with a simple <get> an RPC is not needed. State the real purpose. I would rather call the rpc something like: get-per-node-metadata Why is this rpc only valid for measurement metadata? It could be useful for any capability. What if there are multiple optimized-measurement-points above the input node? Specify. Why are minimum-observable-period, corresponding-mib-oid and related-node not returned. Illogical. list active-measurements: What is a measurement in this context? Is it the YangPush subscription for the counter? If yes return a subscriptionId. Or did you mean the smallest reporting period of all YangPush subscriptions? What is returned if there are on-change Yang-Push subscriptions for the counter (or whatever data node)? get-system-node-capabilities The difference between the 2 RPCs is not clear. The input node selector MAY point to multiple nodes if some key values are missing. Is this intentional? Why is this rpc only valid for measurement metadata? It could be useful for any capability. If the input node is not leaf/leaf-list then metadata of its subtree nodes is returned. The metadata/capabilities can be very different for different schema nodes in the subtree. Will you return a separate set of capabilities for every child node? That could be long! Output: Node should be a key node-selector-capability: the name is wrong. This is a list of capabilities not a new node-selector-capability. Regards Balazs -- Balazs Lengyel Senior Specialist Ericsson Hungary Ltd. Mobile: +36-70-330-7909 email: Balazs.Lengyel@ericsson.com
- [netconf] Comments on draft-claise-netconf-metada… Balázs Lengyel