Adam Roach's No Objection on draft-ietf-rtgwg-yang-rip-08: (with COMMENT)
Adam Roach <adam@nostrum.com> Thu, 11 January 2018 06:15 UTC
Return-Path: <adam@nostrum.com>
X-Original-To: rtgwg@ietf.org
Delivered-To: rtgwg@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 75B11126CD6; Wed, 10 Jan 2018 22:15:17 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Adam Roach <adam@nostrum.com>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-rtgwg-yang-rip@ietf.org, Yingzhen Qu <yingzhen.qu@huawei.com>, rtgwg-chairs@ietf.org, yingzhen.qu@huawei.com, rtgwg@ietf.org
Subject: Adam Roach's No Objection on draft-ietf-rtgwg-yang-rip-08: (with COMMENT)
X-Test-IDTracker: no
X-IETF-IDTracker: 6.68.3
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <151565131747.31352.6717246747415005776.idtracker@ietfa.amsl.com>
Date: Wed, 10 Jan 2018 22:15:17 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtgwg/Cop43A_DZHUhS6KoEYUdfnmeu5s>
X-BeenThere: rtgwg@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Routing Area Working Group <rtgwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtgwg>, <mailto:rtgwg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtgwg/>
List-Post: <mailto:rtgwg@ietf.org>
List-Help: <mailto:rtgwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtgwg>, <mailto:rtgwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 11 Jan 2018 06:15:18 -0000
Adam Roach has entered the following ballot position for draft-ietf-rtgwg-yang-rip-08: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-rtgwg-yang-rip/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- I have a few minor comments. In the YANG module, I find this definition: leaf address { type inet:ip-prefix; description "IPv4 address, in the form A.B.C.D, and the prefix length, separated by the slash (/) character; or IPv6 address, in the form A:B:C:D:E:F:G:H, and the prefix length, separated by the slash (/) character."; } The description -- which clearly says that IPv6 addresses need to be represented as eight discreet components -- implies that the use of :: (https://tools.ietf.org/html/rfc5952#section-4.1) and embedded IPv4 addresses (https://tools.ietf.org/html/rfc5952#section-5) is forbidden. This seems to be a restriction above and beyond that defined for ipv6-prefix in RFC 6991. Since I don't find any explanation of this in the accompanying text, I suspect this is an error. I think you want to cite RFC 5952 rather than trying to to describe the various valid formats for IPv6 addresses here. ____ I believe you want to update the copyright date that appears at the top of the YANG module definition. ____ Please consider updating the example in appendix A to conform to the IAB guidance found at <https://www.iab.org/2016/11/07/iab-statement-on-ipv6/>. This would involve changing the example to use IPv6 instead of IPv4, or adding an IPv6 example in addition to the IPv4 example.