Re: [netmod] Reminder: WGLC - draft-ietf-netmod-yang-tree-diagrams
Martin Bjorklund <mbj@tail-f.com> Mon, 15 January 2018 14:39 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 419E512D7EC; Mon, 15 Jan 2018 06:39:38 -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 0N-aTFOZSqu6; Mon, 15 Jan 2018 06:39:36 -0800 (PST)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id 37CBF12D7E2; Mon, 15 Jan 2018 06:39:36 -0800 (PST)
Received: from localhost (unknown [173.38.220.56]) by mail.tail-f.com (Postfix) with ESMTPSA id 23CDC1AE03DD; Mon, 15 Jan 2018 15:39:35 +0100 (CET)
Date: Mon, 15 Jan 2018 15:39:33 +0100
Message-Id: <20180115.153933.1215610340924130656.mbj@tail-f.com>
To: rwilton@cisco.com
Cc: joelja@bogus.com, netmod@ietf.org, draft-ietf-netmod-yang-tree-diagrams@ietf.org
From: Martin Bjorklund <mbj@tail-f.com>
In-Reply-To: <66857af8-80b4-8c7c-f952-f04c3f2adaa7@cisco.com>
References: <2cde8b64-0455-a513-4719-feb61c87a952@bogus.com> <d66db346-9ca8-c08d-ea25-4c239d265ad4@bogus.com> <66857af8-80b4-8c7c-f952-f04c3f2adaa7@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/kg2yCTayVBoJQ_TNolaCmRJO_mg>
Subject: Re: [netmod] Reminder: 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: Mon, 15 Jan 2018 14:39:38 -0000
Hi, Thanks for the review! Comments inline. Robert Wilton <rwilton@cisco.com> wrote: > Hi Lou, Martin > > I've done a quick review of draft -04. > > It looks well written to me. > > I have a spotted a few minor nits/questions. > > Section 1: > > (i) "Such diagrams are commonly used to provided " => "Such diagrams > are used to provide"? Ok. > (ii) "This document provides the syntax used in YANG Tree Diagrams." > => "This document describes the syntax used in YANG Tree Diagrams", or > if not "describes", perhaps "specifies"? I changed to "describes". > (iii) "common practice is include" => "common practice is to include" Ok. > Section 2: > (iv) Are the top level data nodes really offset by 4 spaces, or should > this be 2 spaces? The example in section 2, and section 4 seem to > differ here. The submodule example in Sec 2.1 looks like it is using > 2 spaces. It should be 4 spaces. I fixed the example in 2.1. > (v) "is prefaces with" => "is prefaced with" Ok. > (vi) Section 2.2, should there be an example of an unexpanded uses > statement? I was wondering if this section was under specified? I have added: For example, the following diagram shows the "tls-transport" grouping from [RFC7407] unexpanded: +--rw tls +---u tls-transport If the grouping is expanded, it could be printed as: +--rw tls +--rw port? inet:port-number +--rw client-fingerprint? x509c2n:tls-fingerprint +--rw server-fingerprint? x509c2n:tls-fingerprint +--rw server-identity? snmp:admin-string > Section 2.6: > (vii) "If the node is augmented into the tree from another module, its > name is printed as <prefix>:<name>." Does there need to be a > clarification that the prefix name used is the one used by the > module's import statement? Or does it use the prefix statement from > the imported modules instead (I know that these should normally be the > same, but this is not guaranteed). Since this is used when a node is augmented *into* the main tree, it can't be the prefix in import, since the augmenting module is not imported from the augmented module. I did: OLD: If the node is augmented into the tree from another module, its name is printed as <prefix>:<name>. NEW: If the node is augmented into the tree from another module, its name is printed as <prefix>:<name>, where <prefix> is the prefix defined in the module where the node is defined. > Section 3.2. > (viii) It would be slightly easier to read if there wasn't a linebreak > between "--" and "tree-print-groupings", not sure if that is feasible > to force this. I don't think I can enforce this, but I'll look into it. If nothing else, the RFC editor will fix this. /martin > > Thanks, > Rob > > On 10/01/2018 16:16, joel jaeggli wrote: > > Just a reminder given the date that this was posted. This last call is > > expected to complete Monday 1/15/18. > > > > Thanks > > > > joel > > > > > > On 1/1/18 2:01 PM, joel jaeggli wrote: > >> Greetings, > >> > >> We hope the new year is a time to make great progess on outstanding > >> documents before preparation for the London IETF begins in earnest. > >> > >> This starts a two-week working group last call on: > >> > >> YANG Tree Diagrams > >> draft-ietf-netmod-yang-tree-diagrams > >> > >> https://datatracker.ietf.org/doc/draft-ietf-netmod-yang-tree-diagrams/ > >> > >> Please send email to the list indicating your support or concerns. > >> > >> We are particularly interested in statements of the form: > >> > >> * I have reviewed this draft and found no issues. > >> * I have reviewed this draft and found the following issues: ... > >> > >> > >> Thank you, > >> NETMOD WG Chairs > >> > >> > >> > >> > > > > _______________________________________________ > > netmod mailing list > > netmod@ietf.org > > https://www.ietf.org/mailman/listinfo/netmod > > . > > >
- [netmod] WGLC - draft-ietf-netmod-yang-tree-diagr… joel jaeggli
- [netmod] Reminder: WGLC - draft-ietf-netmod-yang-… joel jaeggli
- [netmod] 答复: Reminder: WGLC - draft-ietf-netmod-y… Qin Wu
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… t.petch
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Benoit Claise
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Robert Wilton
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Robert Wilton
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Martin Bjorklund
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Robert Wilton
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Acee Lindem (acee)
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Lou Berger
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Martin Bjorklund
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Robert Wilton
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Vladimir Vassilev
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Martin Bjorklund
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Vladimir Vassilev
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… Benoit Claise
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Martin Bjorklund
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Vladimir Vassilev
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Robert Wilton
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Martin Bjorklund
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Lou Berger
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Robert Wilton
- Re: [netmod] Reminder: WGLC - draft-ietf-netmod-y… joel jaeggli
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… joel jaeggli
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Vladimir Vassilev
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Juergen Schoenwaelder
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Alexander Clemm
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Martin Bjorklund
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Juergen Schoenwaelder
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Vladimir Vassilev
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Juergen Schoenwaelder
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Robert Wilton
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… t.petch
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Martin Bjorklund
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Robert Wilton
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Juergen Schoenwaelder
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Benoit Claise
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… Jeff Tantsura
- Re: [netmod] WGLC - draft-ietf-netmod-yang-tree-d… t.petch
- [netmod] Closure - : WGLC - draft-ietf-netmod-yan… joel jaeggli
- Re: [netmod] Closure - : WGLC - draft-ietf-netmod… Martin Bjorklund