[babel] Rtgdir early review of draft-ietf-babel-source-specific-01

Joel Halpern <jmh@joelhalpern.com> Sat, 04 November 2017 21:19 UTC

Return-Path: <jmh@joelhalpern.com>
X-Original-To: babel@ietf.org
Delivered-To: babel@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 6395913FB6C; Sat, 4 Nov 2017 14:19:58 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Joel Halpern <jmh@joelhalpern.com>
To: rtg-dir@ietf.org
Cc: draft-ietf-babel-source-specific.all@ietf.org, babel@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.64.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <150983039829.26601.3109842491855567125@ietfa.amsl.com>
Date: Sat, 04 Nov 2017 14:19:58 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/babel/-V3XN6IHqAEVjIyCpztO9PsoCvA>
Subject: [babel] Rtgdir early review of draft-ietf-babel-source-specific-01
X-BeenThere: babel@ietf.org
X-Mailman-Version: 2.1.22
List-Id: "A list for discussion of the Babel Routing Protocol." <babel.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/babel>, <mailto:babel-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/babel/>
List-Post: <mailto:babel@ietf.org>
List-Help: <mailto:babel-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/babel>, <mailto:babel-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sat, 04 Nov 2017 21:19:58 -0000

Reviewer: Joel Halpern
Review result: Not Ready

This is a routing directorate early review.  It is intended to assist the
working group and the routing ADs in processing the advancing document.

This document is nearly ready for publication as a Proposed Standard RFC.

Isses:
Major: N/A

Moderate:
Please address the issues reported by id-nits.  Specifically, repair the
abstract and add an explicit reference to RFC 2119

Please add an informative reference to draft-ietf-rtgwg-dst-src-routing
indicating the the routing policy described here aligns with the ongoing work
in the routing area for how to handle source specific routes.  This could be
added in section 4.

The base Babel (bis) specification does not talk about the handling of
duplicate sub-TLVs.  Are multiple source-specific sub-TLVs allowed on a given
destination prefix advertisement?  Please indicate what the intended /
permitted handling is in the text.

Minor:
As far as I can tell, the behavior described in section 3.1 for 0/0 source
addresses and their relationship to routes without source addresses is correct
and matches draft-ietf-rtgwg-dst-src-routing.  However, the wording is
sufficiently different as to cause this reader to wonder about it.  If
practical consider additional wording to make the alignment clear.  This would
seem to apply to 3.2 as well.  The most obvious fix is to say that a Babel node
supporting this extensions treats all advertisements received without a source
specific prefix as if they had the 0/0 source prefix?  (The text in section 5.2
says this.  I would like to see it in section 3.1)