[Gen-art] Genart last call review of draft-ietf-babel-rfc6126bis-10

Russ Housley via Datatracker <noreply@ietf.org> Sat, 22 June 2019 17:23 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id DF79E12011A; Sat, 22 Jun 2019 10:23:26 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Russ Housley via Datatracker <noreply@ietf.org>
To: <gen-art@ietf.org>
Cc: draft-ietf-babel-rfc6126bis.all@ietf.org, ietf@ietf.org, babel@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.98.1
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Russ Housley <housley@vigilsec.com>
Message-ID: <156122420684.16418.15464371573444699739@ietfa.amsl.com>
Date: Sat, 22 Jun 2019 10:23:26 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/pJiQc-ZJP4n8ny-sLqf_FMoJElY>
Subject: [Gen-art] Genart last call review of draft-ietf-babel-rfc6126bis-10
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 22 Jun 2019 17:23:27 -0000

Reviewer: Russ Housley
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
<http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.

Document: draft-ietf-babel-rfc6126bis-10
Reviewer: Russ Housley
Review Date: 2019-06-22
IETF LC End Date: 2019-07-04
IESG Telechat date: Unknown

Summary: Ready with Nits

Major Concerns:

None


Minor Concerns:

None


Nits:

Section 3.7.2 says:

   ...  A node SHOULD NOT send triggered updates
   for other reasons, such as when there is a minor fluctuation in a
   route's metric, when the selected next hop changes, or to propagate a
   new sequence number (except to satisfy a request, as specified in
   Section 3.8).

This seem backwards to me.  Perhaps:

   ...  The node MUST send triggered updates to satisfy a request, as
   specified in Section 3.8; however, a node SHOULD NOT send
   triggered updates for other reasons, including a minor fluctuation
   in a metric for a route, the selected next hop changes, or to
   propagate a new sequence number.


Section 4 says:

   A Babel packet is sent as the body of a UDP datagram, with network-
   layer hop count set to 1, destined to a well-known multicast address
   or to a unicast address, over IPv4 or IPv6; in the case of IPv6,
   these addresses are link-local.
   
It seems to me that this should be reworded as MUST statements.

   A Babel packet MUST be sent as the body of a UDP datagram, with
   network-layer hop count set to 1, destined to a well-known multicast
   address or to a unicast address, over either IPv4 or IPv6.  When
   IPv6 addresses are used, the addresses MUST be link-local.


Section 4.1.2 says:

   A router-id is an arbitrary 8-octet value.  A router-id MUST NOT
   consist of either all zeroes or all ones.
   
I do not think you are referring to octets with a value of one.  I
think you mean that the router-id cannot be 0x0000000000000000 or
0xffffffffffffffff.  Please reword.