Genart last call review of draft-ietf-rtgwg-yang-rip-06

Roni Even <ron.even.tlv@gmail.com> Sun, 10 December 2017 10:49 UTC

Return-Path: <ron.even.tlv@gmail.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 ED978126CC7; Sun, 10 Dec 2017 02:49:59 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Roni Even <ron.even.tlv@gmail.com>
To: gen-art@ietf.org
Cc: rtgwg@ietf.org, ietf@ietf.org, draft-ietf-rtgwg-yang-rip.all@ietf.org
Subject: Genart last call review of draft-ietf-rtgwg-yang-rip-06
X-Test-IDTracker: no
X-IETF-IDTracker: 6.67.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <151290299991.21913.13558438364274541815@ietfa.amsl.com>
Date: Sun, 10 Dec 2017 02:49:59 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtgwg/ktp9ywA8RdWTn0KSybwFMIv5kA8>
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: Sun, 10 Dec 2017 10:50:00 -0000

Reviewer: Roni Even
Review result: Ready with Nits

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

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-rtgwg-yang-rip-??
Reviewer: Roni Even
Review Date: 2017-12-10
IETF LC End Date: 2017-12-12
IESG Telechat date: 2018-01-11

Summary:
The document is ready for publication with nits as a standard track RFC

Major issues:

Minor issues:

Nits/editorial comments:

1.  in section  4  " type enumeration {
               enum 1 {
                 description "IS-IS level 1 routes.";
               }
               enum 2 {
                 description "IS-IS level 1  routes.";
               }"

enum 2 is IS-IS level 2 routes

2. In Appendix A , the figure with eth0 and the data tree with eth1. Also there
is "ipv4-prefix": "198.51.100.0/24" which are not in the figure