Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-diagrams

Martin Bjorklund <mbj@tail-f.com> Wed, 17 January 2018 10:25 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 9A7AE126DD9 for <netmod@ietfa.amsl.com>; Wed, 17 Jan 2018 02:25:00 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.91
X-Spam-Level:
X-Spam-Status: No, score=-1.91 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_PASS=-0.001, T_RP_MATCHES_RCVD=-0.01, 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 EFI9nyxzvP1b for <netmod@ietfa.amsl.com>; Wed, 17 Jan 2018 02:24:59 -0800 (PST)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id F24B4126DFF for <netmod@ietf.org>; Wed, 17 Jan 2018 02:24:58 -0800 (PST)
Received: from localhost (h-85-209.A165.priv.bahnhof.se [94.254.85.209]) by mail.tail-f.com (Postfix) with ESMTPSA id 5F67B1AE0385; Wed, 17 Jan 2018 11:24:57 +0100 (CET)
Date: Wed, 17 Jan 2018 11:24:57 +0100
Message-Id: <20180117.112457.2094197769485990789.mbj@tail-f.com>
To: rwilton@cisco.com
Cc: alexander.clemm@huawei.com, vladimir@transpacket.com, netmod@ietf.org
From: Martin Bjorklund <mbj@tail-f.com>
In-Reply-To: <52dd8202-605b-0dcb-2a2e-6f38a44ef72c@cisco.com>
References: <644DA50AFA8C314EA9BDDAC83BD38A2E0EADB145@sjceml521-mbx.china.huawei.com> <20180117.084318.1633996450082512639.mbj@tail-f.com> <52dd8202-605b-0dcb-2a2e-6f38a44ef72c@cisco.com>
X-Mailer: Mew version 6.7 on Emacs 24.5 / Mule 6.0 (HANACHIRUSATO)
Mime-Version: 1.0
Content-Type: Text/Plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/JzzOvo3KzxN6PRWPWkEaLGufKC4>
Subject: Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-diagrams
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.22
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: Wed, 17 Jan 2018 10:25:00 -0000

Hi,

Robert Wilton <rwilton@cisco.com> wrote:
> On 17/01/2018 07:43, Martin Bjorklund wrote:
> > Alexander Clemm <alexander.clemm@huawei.com> wrote:
> >> +1 to (2) as preference, followed by (1).  I don't think (3) is needed
> >> here.  The purpose is to make this human-readable and provide readers
> >> a good sense of the overall structure.  The authoritative
> >> specification is still the .yang itself.  Providing some guidance for
> >> how to represent the tree is good but let's not over-engineer this; I
> >> believe retaining some flexibility is good.
> > The last sentence summarizes my personal view.  I prefer (1), followed
> > by (2).
> OK, so the "Providing some guidance for how to represent the tree is
> good" is along the lines of what I was thinking for the "algorithm"
> would be for 2.
> 
> E.g. something along the lines of:
> "Arbitrary whitespace is allowed between any of the whitespace
> separated fields (e.g. <opts> and <type>).  Additional whitespace may
> be used to column align fields (e.g. within a list or container) to
> improve readability".

This is fine with me.  It is something in between (1) and (2) which
people preferred.  I will add this text.

> But just saying implementations can use arbitrary whitespace is also
> fine with me.  I certainly err on the side of not being overly
> prescriptive on this ..
> 
> I'm not really a fan of the comparing tree output as a method of
> verifying a YANG parser idea.

This is getting off-topic, but if an implementor wants to do this in
order to verify his implementation, go for it!  It doesn't have to be
discussed on this ML.  Speaking from own experience of two different
implementations, I can tell you that it is a useful test to do ;-)


/martin