[Bier] Intdir telechat review of draft-ietf-bier-bar-ipa-12

Suresh Krishnan via Datatracker <noreply@ietf.org> Tue, 19 April 2022 03:54 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 51DB43A1044; Mon, 18 Apr 2022 20:54:08 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Suresh Krishnan via Datatracker <noreply@ietf.org>
To: int-dir@ietf.org
Cc: bier@ietf.org, draft-ietf-bier-bar-ipa.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.46.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <165034044824.2741.13284922661455065282@ietfa.amsl.com>
Reply-To: Suresh Krishnan <suresh.krishnan@gmail.com>
Date: Mon, 18 Apr 2022 20:54:08 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/bier/mDCNZxisfDt6Gkb9h8eJ8vzUIdg>
Subject: [Bier] Intdir telechat review of draft-ietf-bier-bar-ipa-12
X-BeenThere: bier@ietf.org
X-Mailman-Version: 2.1.29
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: Tue, 19 Apr 2022 03:54:09 -0000

Reviewer: Suresh Krishnan
Review result: Ready with Issues

This document is short and clear and easy to understand for someone who has a
basic understanding of BIER. I do have one minor issue that probably needs to
get addressed/clarified

* Section 3:

Step 3: "If BA is NULL" -> What exactly does this mean? Does this mean BAR==0?
If so, it might be good to state this explicitly and I suggest pulling forward
the text from Section 3.1. to clarify BAR==0 means that BA is NULL and BC is
NULL. If not, can you please clarify? i.e. do you expect there to be a NULL BA
with a BAR<>0?