[Bier] Genart last call review of draft-ietf-bier-tether-04
Joel Halpern via Datatracker <noreply@ietf.org> Thu, 15 February 2024 23:09 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: bier@ietf.org
Delivered-To: bier@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 369D5C1CAF58; Thu, 15 Feb 2024 15:09:45 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Joel Halpern via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: bier@ietf.org, draft-ietf-bier-tether.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 12.5.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <170803858520.33924.988542142430727148@ietfa.amsl.com>
Reply-To: Joel Halpern <jmh@joelhalpern.com>
Date: Thu, 15 Feb 2024 15:09:45 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/bier/kaRf3wzShE72v8SlSaAcB3JUOI8>
Subject: [Bier] Genart last call review of draft-ietf-bier-tether-04
X-BeenThere: bier@ietf.org
X-Mailman-Version: 2.1.39
List-Id: "\"Bit Indexed Explicit Replication discussion list\"" <bier.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/bier>, <mailto:bier-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/bier/>
List-Post: <mailto:bier@ietf.org>
List-Help: <mailto:bier-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/bier>, <mailto:bier-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 15 Feb 2024 23:09:45 -0000
Reviewer: Joel Halpern Review result: Ready with Issues 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://wiki.ietf.org/en/group/gen/GenArtFAQ>. Document: draft-ietf-bier-tether-04 Reviewer: Joel Halpern Review Date: 2024-02-15 IETF LC End Date: 2024-02-29 IESG Telechat date: Not scheduled for a telechat Summary: This document is almost ready for publication as a proposed standard Major issues: Section 3.1 on IGP Signaling states "The helper node (BFRx) MUST advertise one or more BIER Helped Node sub-sub-TLVs". However, I only find a vague outline of this sub-sub TLV. The code point for it is requested in the IANA considerations section, but the description is a single sentence easily misread and lacking the conventional diagrams and precision that is used to define routing TLVs (and sub or sub-sub TLVs.) Minor issues: In the paragraph about multiple helpers helping a single non-supporting router, I think I missed how one case works properly. (Section 2, additional considerations, paragraph 6). The text says that the sending BFR (BFR1 can choose to use multiple helpers if they are available. Assuming that BFR1 chooses to use BFR2 and BFR 3 to reach BFRs 4 .. BFR N, the text is clear that this results in BFR2 and BFR 3 both sending copies of the packet to Router X. That is fine. It is load, but it is a tradeoff. However, it appears that both BFR2 and BFR 3 would send packets to BFR4, and to all the other BFR children of X. This results in duplicate packets in the rest of the tree. Is there some assumption I missed that prevents this? Nits/editorial comments:
- [Bier] Genart last call review of draft-ietf-bier… Joel Halpern via Datatracker
- Re: [Bier] Genart last call review of draft-ietf-… Jeffrey (Zhaohui) Zhang
- Re: [Bier] Genart last call review of draft-ietf-… Joel Halpern
- Re: [Bier] Genart last call review of draft-ietf-… Jeffrey (Zhaohui) Zhang
- Re: [Bier] Genart last call review of draft-ietf-… Joel Halpern