[netmod] Gen-ART LC review of draft-ietf-netmod-ip-cfg-09

Peter Yee <peter@akayla.com> Sat, 04 May 2013 08:30 UTC

Resent-From: Juergen Schoenwaelder <j.schoenwaelder@jacobs-university.de>
Resent-Date: Sat, 04 May 2013 10:30:47 +0200
Resent-Message-ID: <20130504083047.GB44790@elstar.local>
Resent-To: netmod@ietf.org
From: Peter Yee <peter@akayla.com>
To: draft-ietf-netmod-ip-cfg.all@tools.ietf.org
Date: Fri, 03 May 2013 23:26:07 -0700
Message-ID: <05fd01ce4890$45f5cbf0$d1e163d0$@akayla.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Thread-Index: Ac5Ijf7f2lgl5gtAS46HdVoU7WmpFw==
Content-Language: en-us
Resent-To: bclaise@cisco.com, david.kessens@nsn.com, j.schoenwaelder@jacobs-university.de, mbj@tail-f.com
X-MS-Exchange-Organization-AVStamp-Mailbox: MSFTFF;1;0;0 0 0
MIME-Version: 1.0
Cc: gen-art@ietf.org, ietf@ietf.org
Subject: [netmod] Gen-ART LC review of draft-ietf-netmod-ip-cfg-09
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.12
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: <http://www.ietf.org/mail-archive/web/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: Sat, 04 May 2013 08:30:55 -0000

I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>

Please resolve these comments along with any other Last Call comments you
may receive.

Document: draft-ietf-netmod-ip-cfg-09
Reviewer: Peter Yee
Review Date: May-03-2013
IETF LC End Date: May-03-2013
IESG Telechat date: May-16-2013

Summary: This draft is basically ready for publication as a Standards Track
RFC, but has nits that should be fixed before publication. [Ready with nits]

The abstract says it pretty well: This document defines a YANG data model
for management of IP implementations.

Major issues:

Minor issues:

Nits:

Page 7, bottom: The code copyright date says 2012.  Update it to 2013.

Page 10, in leaf phys-address, the description has an incorrect spelling of
"neighbor".

General: where a description of phys-address is given, in both cases it says
"The physical level address...".  It might be more correct to state "The
link layer address..." for most but not all cases.  

That's it.  Everything appears consistent within the limits of my
understanding of YANG.

		-Peter Yee

[netmod] Gen-ART LC review of draft-ietf-netmod-i… Peter Yee