[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.
- [Gen-art] Genart last call review of draft-ietf-b… Russ Housley via Datatracker
- Re: [Gen-art] [babel] Genart last call review of … Juliusz Chroboczek
- Re: [Gen-art] [babel] Genart last call review of … Russ Housley
- Re: [Gen-art] [babel] Genart last call review of … Juliusz Chroboczek
- Re: [Gen-art] [babel] Genart last call review of … Alissa Cooper