Re: [netconf] Adoption poll for draft-tgraf-netconf-yang-notifications-versioning

Jean Quilbeuf <jean.quilbeuf@huawei.com> Fri, 02 June 2023 11:24 UTC

Return-Path: <jean.quilbeuf@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 5F573C15108E; Fri, 2 Jun 2023 04:24:29 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.894
X-Spam-Level:
X-Spam-Status: No, score=-1.894 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_MSPIKE_H5=0.001, RCVD_IN_MSPIKE_WL=0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=unavailable autolearn_force=no
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id ma6v7wCYsjNI; Fri, 2 Jun 2023 04:24:25 -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 17982C151094; Fri, 2 Jun 2023 04:24:25 -0700 (PDT)
Received: from frapeml500001.china.huawei.com (unknown [172.18.147.200]) by frasgout.his.huawei.com (SkyGuard) with ESMTP id 4QXgZz401nz67bMk; Fri, 2 Jun 2023 19:22:35 +0800 (CST)
Received: from frapeml500001.china.huawei.com (7.182.85.94) by frapeml500001.china.huawei.com (7.182.85.94) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.23; Fri, 2 Jun 2023 13:24:22 +0200
Received: from frapeml500001.china.huawei.com ([7.182.85.94]) by frapeml500001.china.huawei.com ([7.182.85.94]) with mapi id 15.01.2507.023; Fri, 2 Jun 2023 13:24:22 +0200
From: Jean Quilbeuf <jean.quilbeuf@huawei.com>
To: "Rob Wilton (rwilton)" <rwilton=40cisco.com@dmarc.ietf.org>
CC: netconf <netconf@ietf.org>, netconf-chairs <netconf-chairs@ietf.org>
Thread-Topic: [netconf] Adoption poll for draft-tgraf-netconf-yang-notifications-versioning
Thread-Index: AQHZh23oZ5CiVAtuM0GdJ/xr3h5msa9meWgAgAALlYCADBkigIAEz9tw
Date: Fri, 02 Jun 2023 11:24:22 +0000
Message-ID: <cbe33f18a1ad4e649b00eff66124c70c@huawei.com>
References: <E78B3C0B-B415-4E8C-9080-ADFB69D4464E@gmail.com> <AC6138B1-ADA3-4D27-959A-47053D195914@imdea.org> <ghneulz67ajhfmlvs4kkx2ixvtl54ycjtr63dp6diqkqth5hdp@dt3zayd4keas> <BY5PR11MB41965C8427EF343AD6B8EC72B54B9@BY5PR11MB4196.namprd11.prod.outlook.com>
In-Reply-To: <BY5PR11MB41965C8427EF343AD6B8EC72B54B9@BY5PR11MB4196.namprd11.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.206.138.136]
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/MmBP5pWXP42Wx1EGBy2LVkzSfjk>
Subject: Re: [netconf] Adoption poll for draft-tgraf-netconf-yang-notifications-versioning
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.39
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: Fri, 02 Jun 2023 11:24:29 -0000

Hello,

I think this draft is needed to simplify the metric collection and I support its adoption.

I have the same comment as Rob and Andy about a single filter (subtree or xpath) potentially touching several modules and thus a list of involved modules is necessary.

Best,
Jean

-----Original Message-----
From: netconf <netconf-bounces@ietf.org> On Behalf Of Rob Wilton (rwilton)
Sent: Tuesday, May 30, 2023 12:10 PM
Cc: netconf <netconf@ietf.org>; netconf-chairs <netconf-chairs@ietf.org>
Subject: Re: [netconf] Adoption poll for draft-tgraf-netconf-yang-notifications-versioning

Hi,

I think that it would be helpful to decouple the discussion about whether the YANG version number should be changed to the NETMOD WGLC review of the module versioning draft.  I don't think that part of the discussion is particularly relevant to the adoption call for this document.

My understanding of the goal here is to make it easier to export network device telemetry data (exposed via YANG push) in a way that can integrate well with message bus services like Kafka.  I think that this is a good aim for the industry, and thus I support efforts (like this draft) to try and achieve this.

I'm not intimately familiar with Kafka but I understand that it assumes that schema for the data being published can changed over time (e.g., perhaps due to a software upgrade on a publisher, or different instances of publishers running slightly different versions of software).  Hence Kafka maintains a schema registry and allows a reference to the schema to be associated with the published data records.  This allows entities in the data pipeline to check and confirm that the telemetry data conforms to the schema, and potentially to handle the data differently, if needed.

I appreciate that RFC 6020 and RFC 7950 both are defined to try and ensure that YANG definitions only evolve in a backwards compatible way, but I see many examples within the wider industry and some examples in standards bodies where definitions are changed in ways that may impact clients. 

Hence, I think that this problem space, and draft, is about how to easily/robustly connect YANG data sources into Kafka data buses:
- one choice is to do this entirely off the box, having the entities managing the subscriptions also tracking changes to the device schema (e.g., through subscribing to YANG library updates).  Perhaps this can be made to work, but probably adds additional timing complexity relating to when the subscriptions actually start sending data relative to receiving notifications that the content in YANG library has changed.  Another choice here could be to always tear down all subscriptions if the device schema changes at all.
- the alternative is to augment the subscription-started/modified notifications to give an indication that the data for that subscription has changed.  All data sent before this notification uses a previous schema, and all data sent after that notification uses the new schema.  This allows the schema to change without completely tearing down existing subscriptions.

I'm less sure about the ability to restrict subscription requests to particular versions.  At a minimum I would suggest putting this under a feature statement, but I'm not quite sure whether it should be needed at all.

I also agree with Andy's comment that we need to resolve how the revision-date/label applies to all the potential data under a subscription.  To reuse part of Andy's example:

   <A:top>
      <B:list1 />
         <C:container1 />
   <A:top>

If the subscription is to B/list1, but during upgrade the only change was to the contents of C:container1 then for the subscription the revision-date/label for B:list1 wouldn't have changed, but the data received in the subscription would have changed.  I.e., at the moment, YANG has no way to version everything under a subscription.  Hence, I think that the notification would need to list all the module versions that are being used by the subscription, but I'm not convinced that is easy to do in practice.  The alternative would be to just version the entire schema of the datastore (e.g., either a YANG library checksum, or a version label for a YANG package instance that represents the datastore schema), but this has the downside that all subscriptions would see a reported change in schema if any single module is changed on the device, even one that isn't being used (directly or indirectly) by the subscription.  But then an off-device tool can figure out if the actual schema for the subscription has changed.

In summary, I support NETCONF working in this space, and I believe that this draft is a reasonable starting point, but it still requires some refinement, which could be done as part of the WG work on this problem.

Rob

// As an individual contributor.


> -----Original Message-----
> From: netconf <netconf-bounces@ietf.org> On Behalf Of Jürgen 
> Schönwälder
> Sent: 22 May 2023 19:25
> To: Camilo Cardona <juancamilo.cardona@imdea.org>
> Cc: netconf <netconf@ietf.org>; netconf-chairs 
> <netconf-chairs@ietf.org>
> Subject: Re: [netconf] Adoption poll for 
> draft-tgraf-netconf-yang-notifications-
> versioning
> 
> On Mon, May 22, 2023 at 12:43:20PM -0500, Camilo Cardona wrote:
> 
> > - I like the insights shared by Jürgen and Andy regarding how hacky
> >   versioning is for YANG 1.1. It makes sense, but I guess nobody
> >   wants to start a 1.2 project..
> 
> If it is too hard to change the YANG version number, why should I 
> believe it will be easy to change all the future module version 
> numbers?
> 
> If people are serious about versioning, the first thing they should do 
> is proposing to bump the YANG version number when they make 
> incompatible changes.
> 
> /js
> 
> --
> Jürgen Schönwälder              Constructor University Bremen gGmbH
> Phone: +49 421 200 3587         Campus Ring 1 | 28759 Bremen | Germany
> Fax:   +49 421 200 3103         <https://constructor.university/>
> 
> _______________________________________________
> netconf mailing list
> netconf@ietf.org
> https://www.ietf.org/mailman/listinfo/netconf
_______________________________________________
netconf mailing list
netconf@ietf.org
https://www.ietf.org/mailman/listinfo/netconf