Gen-Art LC review: draft-ietf-netmod-yang-metadata-04
Robert Sparks <rjsparks@nostrum.com> Tue, 01 March 2016 21:51 UTC
Return-Path: <rjsparks@nostrum.com>
X-Original-To: ietf@ietfa.amsl.com
Delivered-To: ietf@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C21C61B422A; Tue, 1 Mar 2016 13:51:55 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.906
X-Spam-Level:
X-Spam-Status: No, score=-1.906 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RP_MATCHES_RCVD=-0.006] autolearn=ham
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 IAte_Lwg-AeG; Tue, 1 Mar 2016 13:51:53 -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 8A3BB1B3C3B; Tue, 1 Mar 2016 13:51:50 -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 u21LpngQ011723 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=OK); Tue, 1 Mar 2016 15:51:50 -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
From: Robert Sparks <rjsparks@nostrum.com>
Subject: Gen-Art LC review: draft-ietf-netmod-yang-metadata-04
To: 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
Message-ID: <56D60EF5.7020001@nostrum.com>
Date: Tue, 01 Mar 2016 15:51:49 -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
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 7bit
Archived-At: <http://mailarchive.ietf.org/arch/msg/ietf/V5GT-4oaV_5j1i7Rbue_d_GGd-E>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.15
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: Tue, 01 Mar 2016 21:51:56 -0000
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". 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.
- 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