Re: Gen-Art LC review: draft-ietf-netmod-yang-metadata-04
Robert Sparks <rjsparks@nostrum.com> Thu, 10 March 2016 23:57 UTC
Return-Path: <rjsparks@nostrum.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7901712DEEB; Thu, 10 Mar 2016 15:57:26 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-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 vvkVxpLPAj4D; Thu, 10 Mar 2016 15:57:24 -0800 (PST)
Received: from nostrum.com (raven-v6.nostrum.com [IPv6:2001:470:d:1130::1]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id BA6E712DEE6; Thu, 10 Mar 2016 15:57:24 -0800 (PST)
Received: from unnumerable.local (pool-173-57-158-165.dllstx.fios.verizon.net [173.57.158.165]) (authenticated bits=0) by nostrum.com (8.15.2/8.14.9) with ESMTPSA id u2ANvNLL059725 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=OK); Thu, 10 Mar 2016 17:57:24 -0600 (CST) (envelope-from rjsparks@nostrum.com)
X-Authentication-Warning: raven.nostrum.com: Host pool-173-57-158-165.dllstx.fios.verizon.net [173.57.158.165] claimed to be unnumerable.local
Subject: Re: Gen-Art LC review: draft-ietf-netmod-yang-metadata-04
To: Ladislav Lhotka <lhotka@nic.cz>, General Area Review Team <gen-art@ietf.org>, "ietf@ietf.org" <ietf@ietf.org>, netmod@ietf.org, draft-ietf-netmod-yang-metadata.all@ietf.org
References: <56D60EF5.7020001@nostrum.com> <m27fhbvb07.fsf@birdie.labs.nic.cz>
From: Robert Sparks <rjsparks@nostrum.com>
Message-ID: <56E209E3.4030805@nostrum.com>
Date: Thu, 10 Mar 2016 17:57:23 -0600
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.10; rv:38.0) Gecko/20100101 Thunderbird/38.6.0
MIME-Version: 1.0
In-Reply-To: <m27fhbvb07.fsf@birdie.labs.nic.cz>
Content-Type: text/plain; charset="windows-1252"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/ietf/9CH-x680LkiLFnnbtiPOrWlLlLE>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.17
Precedence: list
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 10 Mar 2016 23:57:26 -0000
On 3/9/16 11:04 AM, Ladislav Lhotka wrote: > Hi Robert, > > thanks for the review, I apologize for replying late, please see my responses inline: > > Robert Sparks <rjsparks@nostrum.com> writes: > >> I am the assigned Gen-ART reviewer for this draft. The General Area >> Review Team (Gen-ART) reviews all IETF documents being processed >> by the IESG for the IETF Chair. Please treat these comments just >> like any other last call comments. >> >> For more information, please see the FAQ at >> >> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. >> >> Document: draft-ietf-netmod-yang-metadata-04 >> Reviewer: Robert Sparks >> Review Date: 1Mar2016 >> IETF LC End Date: 9Mar2016 >> IESG Telechat date: not yet scheduled >> >> Summary: Ready with nits >> >> 1) I might be missing something obvious, but the introduction has two >> statements that don't seem aligned: >> >> " Values of annotations are not limited to strings; any YANG built-in or >> derived type may be used for them" >> and >> "annotations are scalar values and cannot be further structured". > These two statements are not in conflict: YANG data types (built-in or > derived) apply to scalar values. I don't know what it means for a data type to apply to a value. Can you say that more simply? I think this is just a language thing, and being precise in the text will get past it. Are you saying all YANG data types (built-in or derived) have only scalar values? If so, you could change the second point in the document to say "Since annotations have only built-in or derived types, they can only have scalar values." But then, placing the point under "This document deliberately adopts some restrictions" doesn't make sense, and I suggest restructuring the discussion to only call out the restrictions (which appears to be to not place annotations on lists or leaf-lists) below that sentence. > >> If I'm not missing something, that may be more of an open issue than a nit. >> >> 2) The shepherd writeup calls out the tension in figuring out whether to >> make this an extension or a new built-in statement. Please consider >> capturing the reasoning for the path you chose in the draft itself. > I will try, the main reason was that most people felt that introducing a > new built-in statement is too big a change for the upcoming maintenance > version of YANG (1.1) and YANG 2.0 is nowhere in sight. > > Thanks, Lada > >> >> >> >> >> >> >> >>
- Gen-Art LC review: draft-ietf-netmod-yang-metadat… Robert Sparks
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… Ladislav Lhotka
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… Robert Sparks
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… Ladislav Lhotka
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… tom p.
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… Ladislav Lhotka
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… Juergen Schoenwaelder
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… Ladislav Lhotka
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… Robert Sparks
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… tom p.
- Re: Gen-Art LC review: draft-ietf-netmod-yang-met… Juergen Schoenwaelder
- Re: [Gen-art] Gen-Art LC review: draft-ietf-netmo… Jari Arkko