Re: [Bier] Warren Kumari's No Objection on draft-ietf-bier-architecture-07: (with COMMENT)

Tony Przygienda <tonysietf@gmail.com> Thu, 06 July 2017 21:08 UTC

Return-Path: <tonysietf@gmail.com>
X-Original-To: bier@ietfa.amsl.com
Delivered-To: bier@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B58CB12EA58; Thu, 6 Jul 2017 14:08:09 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.999
X-Spam-Level:
X-Spam-Status: No, score=-1.999 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id M0o23V-eLTgd; Thu, 6 Jul 2017 14:08:07 -0700 (PDT)
Received: from mail-wr0-x231.google.com (mail-wr0-x231.google.com [IPv6:2a00:1450:400c:c0c::231]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 2534813180A; Thu, 6 Jul 2017 14:08:07 -0700 (PDT)
Received: by mail-wr0-x231.google.com with SMTP id c11so19741931wrc.3; Thu, 06 Jul 2017 14:08:07 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=9sDU/7y7t4IpAgGBOf/AUOZ+1/DsPzR1MJ8XgEzhOxs=; b=mtwRHovXRCkAz/4iuhLMLwHb19kBvd0grd9xX4k8wvbxKgxXKEEsI2czi5IcT8i2DM hV3So0JigGp3bhw+KQtdizW0yRFIPKoRX1AGBXEoyzRM6O9kjP5ZXQW39gw994UxcjBG hjOsg7YUx4bVnYiEmtBSruaGftcs+QZqao5GnImrHYilFBVb6mDJnszJI7AjBIGAboN4 I0o1xWgaHx27Ik854ZeGZSm3ZFqZBo36HuxVe0D2CZp7ZSLdOdW7w7lGaySF7nFvWvtR S1Yuk/QVOzccjHobd2ZFOJ+cvl/KKinrgKPKQsJLUjS/gBgeUy/qoBFFxUjo1o+mTEDE 24cg==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=9sDU/7y7t4IpAgGBOf/AUOZ+1/DsPzR1MJ8XgEzhOxs=; b=ilkvsXt7hnBKFyjk4p2T6f5WXkuQ9HwcgOfsLmg3ZX/aIHW5jo7y9GtP7HRovQLT9i sCGKX78nOQmOPc2HPnhRJoJzuXoexQvcZDwara1dXdLlvfG4WSIJWp8MkBWUeQIN7fE+ 1V0+L1LhQh5x+vbRSeS2JkR2FN3nK8Ufc3e/WvTrEaQlEv/AxCLBtjVCnOX2bTp4W92v XIdw66nu9F/VqnyDsayKPbn3d3PGYC+yY6Qw2k/K2WfVAzSuYUidrm66zmgdEg6JdorT yJ8gvVyUvrHleXC71y468OUZ0sCk0Mj4p9IUewB6OWW087sdj/zgIHHUoMCSwB0WP07w hpDw==
X-Gm-Message-State: AIVw111uw/laA8XQ0ZbRIZjKzpWUtsD3hi4zfoUHmBzOSN2i6v52Zr4j QrTNcevVAbsfcnVN6SYRaBnJqt4JxN5H
X-Received: by 10.80.161.71 with SMTP id 65mr599739edj.183.1499375285747; Thu, 06 Jul 2017 14:08:05 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.80.183.131 with HTTP; Thu, 6 Jul 2017 14:07:25 -0700 (PDT)
In-Reply-To: <149928257588.22631.11073122843340523128.idtracker@ietfa.amsl.com>
References: <149928257588.22631.11073122843340523128.idtracker@ietfa.amsl.com>
From: Tony Przygienda <tonysietf@gmail.com>
Date: Thu, 6 Jul 2017 14:07:25 -0700
Message-ID: <CA+wi2hOi3p2jaZo86f6wXs+8aA-gXppXb35EmaPgqHE9t0_=fg@mail.gmail.com>
To: Warren Kumari <warren@kumari.net>
Cc: The IESG <iesg@ietf.org>, draft-ietf-bier-architecture@ietf.org, Greg Shepherd <gjshep@gmail.com>, "bier@ietf.org" <bier@ietf.org>, bier-chairs@ietf.org
Content-Type: multipart/alternative; boundary="94eb2c0d9a5c4ac7260553ac83a1"
Archived-At: <https://mailarchive.ietf.org/arch/msg/bier/wYW9liEuSDIgFKrd-9SmxgBuSyI>
Subject: Re: [Bier] Warren Kumari's No Objection on draft-ietf-bier-architecture-07: (with COMMENT)
X-BeenThere: bier@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
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, 06 Jul 2017 21:08:10 -0000

>
> Nits and notes:
> 1: " A router that supports BIER is known as a "Bit-Forwarding Router"
> (BFR)."
> -- this makes me sad; the BFR acronym was already in use, and had
> interesting
> history behind it (Big er... Fast Router) -
> http://photos.kumari.net/Technology/Networking/i-k4vCdwv/A


Some of the authors are known as mild punsters so for the old hands it may
look like some of the acronym choices have not been innocuous ;-)


>
>
> 2: "MVPN signaling also has several components that depend on the
>    type of "tunneling technology" used to carry multicast data though
>    the network." -- though the network what?! I suspect you meant
> "through" :-)
> 3: Section 6.1.  Overview
> "If a BFR receives a BIER-encapsulated packet whose sub-domain, SI and
> BitString identify that BFR itself, ..." -- identifies.
>

ack


>
> O: "When BIER on a BFER is to pass a packet to the multicast flow overlay,
> it
> of course decapsulates..." P: "When BIER on a BFER is to pass a packet to
> the
> multicast flow overlay, it, of course decapsulates" C: Missing a comma. I
> suspect: "When BIER on a BFER is to pass a packet to the multicast flow
> overlay, it decapsulates..." would be even better.
>
> Section 6.9.  When Some Nodes do not Support BIER
> "In this section, we assume that the routing underlay is an SPF-based IGP
> that
> computes a shortest path tree from each node to all other nodes in the
> domain."
> C: I *think* that this should actually be "the shortest path", but I'm not
> sure; everything talks about e.g Dijkstra's algorithm finding the shortest
> path, but what if there are multiple equal cost paths? But, this is a
> singular
> tree, but may not be unique, so any SP tree will... erk... Good this this
> is a
> nit and I can move on :-)
>

in fact, it is a DAG with SPF properties and BIER is free to use the DAG to
ECMP. The ECMP section should make that pretty obvious.



>
> Section 6.10.3.  Transitioning from One BitStringLength to Another
> Typo:  Dispostion ->  Disposition
>

ack