[netmod] comments on draft-ietf-netmod-yang-instance-file-format-04
Martin Bjorklund <mbj@tail-f.com> Thu, 10 October 2019 12:05 UTC
Return-Path: <mbj@tail-f.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 98388120C34 for <netmod@ietfa.amsl.com>; Thu, 10 Oct 2019 05:05:55 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, 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 OI1EwhaD4gQb for <netmod@ietfa.amsl.com>; Thu, 10 Oct 2019 05:05:53 -0700 (PDT)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id 8E4D8120C1D for <netmod@ietf.org>; Thu, 10 Oct 2019 05:05:53 -0700 (PDT)
Received: from localhost (unknown [173.38.220.41]) by mail.tail-f.com (Postfix) with ESMTPSA id 42A181B0671F for <netmod@ietf.org>; Thu, 10 Oct 2019 14:05:52 +0200 (CEST)
Date: Thu, 10 Oct 2019 14:05:25 +0200
Message-Id: <20191010.140525.904627955349075516.mbj@tail-f.com>
To: netmod@ietf.org
From: Martin Bjorklund <mbj@tail-f.com>
X-Mailer: Mew version 6.8 on Emacs 25.2
Mime-Version: 1.0
Content-Type: Text/Plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/i78yIlINgUqzkgMff5yoK_GMFvQ>
Subject: [netmod] comments on draft-ietf-netmod-yang-instance-file-format-04
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.29
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: Thu, 10 Oct 2019 12:05:55 -0000
Hi, I have some mostly cosmetic comments on this draft. o "YANG" should be spelled "YANG". Not Yang etc. o "NETCONF" should be spelled "NETCONF". o leaf-list module The type of this leaf-list is a string with: pattern '.+@\d{4}-\d{2}-\d{2}\.yang'; I think the revision needs to be optional, and the suffix ".yang" dropped, since it doesn't add any value: pattern '.+(@\d{4}-\d{2}-\d{2})?'; (same for inline-spec). o schema-uri The description says: A reference to another YANG instance data file. This instance data file will use the same set of target YANG modules, revisions, supported features and deviations as the referenced YANG instance data file. I don't understand what this means. Does it mean that the schema for this document is the same as the schema defined in the schema-uri file, or that the schema-uri file defines the schema in its content-data? I *think* it is the former. In either case, the name of the leaf can perhaps be changed to reflect the semantics, rather than the syntax (i.e., don't call it xxx-uri just b/c its type is an uri). Perhaps 'same-schema-as-file'. o Data node naming. The current structure of the model is: +--rw (content-schema-spec)? | +--:(simplified-inline) | +--rw module* string | +--:(inline) | | +--rw inline-spec* string | | +--rw inline-content-schema <anydata> | +--:(uri) | +--rw schema-uri? inet:uri ... +--rw content-data? <anydata> To make the instance document more understandable, I suggest the following structure, which adds a wrapping container for the schema, and renames the inline and uri nodes: +--rw content-schema +--rw (content-schema-spec)? | +--:(simplified-inline) | +--rw module* string | +--:(inline) | | +--rw inline-module* string | | +--rw inline-schema <anydata> | +--:(uri) | +--rw same-schema-as-file? inet:uri ... +--rw content-data? <anydata> o Format the YANG module I suggest you run the YANG module through: pyang -f yang --keep-comments --yang-line-length 69 o 3.2 The element "<netconf-state>" needs a namespace declaration. /martin /martin
- [netmod] comments on draft-ietf-netmod-yang-insta… Martin Bjorklund
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Martin Bjorklund
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Martin Bjorklund
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Martin Bjorklund
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Martin Bjorklund
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Martin Bjorklund
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Rob Wilton (rwilton)
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Andy Bierman
- Re: [netmod] comments on draft-ietf-netmod-yang-i… Balázs Lengyel