[RTG-DIR] 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: rtg-dir@ietf.org
Delivered-To: rtg-dir@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/rtg-dir/aGrBukurxTbj4MFx_f37zOSf7ao>
Subject: [RTG-DIR] Rtgdir early review of draft-ietf-babel-source-specific-01
X-BeenThere: rtg-dir@ietf.org
X-Mailman-Version: 2.1.22
List-Id: Routing Area Directorate <rtg-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-dir>, <mailto:rtg-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-dir/>
List-Post: <mailto:rtg-dir@ietf.org>
List-Help: <mailto:rtg-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-dir>, <mailto:rtg-dir-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)
- [RTG-DIR] Rtgdir early review of draft-ietf-babel… Joel Halpern
- Re: [RTG-DIR] Rtgdir early review of draft-ietf-b… Matthieu Boutier
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… Toke Høiland-Jørgensen
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… David Schinazi
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… Matthieu Boutier
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… Joel M. Halpern
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… Matthieu Boutier
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… Juliusz Chroboczek
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… Joel Halpern Direct
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… Toke Høiland-Jørgensen
- Re: [RTG-DIR] [babel] Rtgdir early review of draf… Juliusz Chroboczek