Re: [Gen-art] [Bier] Genart last call review of draft-ietf-bier-architecture-07

Tony Przygienda <tonysietf@gmail.com> Thu, 29 June 2017 15:46 UTC

Return-Path: <tonysietf@gmail.com>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id DEE0112EC2C; Thu, 29 Jun 2017 08:46:35 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.698
X-Spam-Level:
X-Spam-Status: No, score=-2.698 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_LOW=-0.7, SPF_PASS=-0.001, URIBL_BLOCKED=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 u9dfyMR0LKCd; Thu, 29 Jun 2017 08:46:33 -0700 (PDT)
Received: from mail-wm0-x22d.google.com (mail-wm0-x22d.google.com [IPv6:2a00:1450:400c:c09::22d]) (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 A0E26129B09; Thu, 29 Jun 2017 08:46:32 -0700 (PDT)
Received: by mail-wm0-x22d.google.com with SMTP id b184so19208946wme.1; Thu, 29 Jun 2017 08:46:32 -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=EVNOJ64IHfTg6zGmzvF0dr2gspWHTSxQjBK51arzKeM=; b=Qa8NbQg8m3kcUSn8dxyvQzhN14igeO0SQggGu+UNy9Yjuqw/6N0q9IlsoX/Qr8mbvX aovar6Wija5+BvoJL84AdA4VSRKNuOgKfwGiiedySnjKnJJUSDy27+Xlh1Y7iwVWESQz 2Y17xr74tqrRp3hxt+swP89eB3Z+Ecmoomb6aOOavUSp8HB9iKZUHy6N4VcmMFRCXWGA 2U4AfblfTXJNE6+O60x5f1S2Y43vc9ijc0x2jNDVU5FggpnkEIQGx3tk8oMQ4O7A68Nk 7ymSUR7euKwNbg6u704NUjeG2pRWbBn8EEjjZQrG1NJpcUu094KbWAepbL9pV4IOrFi+ dfFQ==
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=EVNOJ64IHfTg6zGmzvF0dr2gspWHTSxQjBK51arzKeM=; b=KlxvadoUhdQgmn3wa7ONm1MGl9y/ONGbDT/1+wNjcqciGtityLbUDv8M8Hwjj9/+Tw LmmE/VzRiZydoH52cUra6D+S9Q1QXjbuUSOQYOw6P9Y19Li1yk6emJXVcP63lcZeYTMD aw01dm9AMc2C77Knu4/Q9ax8YCdl75N5vAD+1C5cAf4wMDN1A0rKN2p5mM8gCJdvdgH1 ncoZ1ZyTBMsGQLgjMoC6J6il782V+47EoS17jEKQswCLoCS0YdbhjZycuwNVpzRwxuln fzpaQ6lRIqmYyKKWSf3M4eZ6lRG3iZGk6qmkGUTO1tewwt8tPCFt1j53u/An/NghjNfl BDrw==
X-Gm-Message-State: AKS2vOwe5qoEPqP4UfE7ZWBT6p/AWWr8skaKyesPEBRXNPpoPC3xnyx/ 4hhanf97vafbCXqcFZ2XtU4XIKDgoQ==
X-Received: by 10.80.178.166 with SMTP id p35mr2241060edd.118.1498751191231; Thu, 29 Jun 2017 08:46:31 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.80.183.131 with HTTP; Thu, 29 Jun 2017 08:45:50 -0700 (PDT)
In-Reply-To: <7d872d02-4154-989d-e83d-4d7408cc6073@juniper.net>
References: <149838804788.3251.829139224134758886@ietfa.amsl.com> <7d872d02-4154-989d-e83d-4d7408cc6073@juniper.net>
From: Tony Przygienda <tonysietf@gmail.com>
Date: Thu, 29 Jun 2017 08:45:50 -0700
Message-ID: <CA+wi2hPVp_z3CNaD7M2Bw+ihFZzmyh+x3U+1O1X66jsuyhX6kQ@mail.gmail.com>
To: Eric C Rosen <erosen@juniper.net>
Cc: Dan Romascanu <dromasca@gmail.com>, gen-art@ietf.org, "bier@ietf.org" <bier@ietf.org>, ietf@ietf.org, draft-ietf-bier-architecture.all@ietf.org
Content-Type: multipart/alternative; boundary="f403045c8a025c36ef05531b34ad"
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/tZQCWvprOhLdmr4Q4f2ofZuKyYY>
Subject: Re: [Gen-art] [Bier] Genart last call review of draft-ietf-bier-architecture-07
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 29 Jun 2017 15:46:36 -0000

I picked up shepherd of this doc. Will reply by end of week to Dan's,
Eric's comments ...

thanks

-- tony

On Tue, Jun 27, 2017 at 7:42 AM, Eric C Rosen <erosen@juniper.net> wrote:

> On 6/25/2017 6:54 AM, Dan Romascanu wrote:
>
> Reviewer: Dan Romascanu
> 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://trac.ietf.org/trac/gen/wiki/GenArtfaq> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
>
> Document: draft-ietf-bier-architecture-?
> ?
> Reviewer: Dan Romascanu
> Review Date: 2017-06-25
> IETF LC End Date: 2017-06-29
> IESG Telechat date: 2017-07-06
>
> Summary:
>
> This document specifies a new architecture known as "Bit Index Explicit
> Replication" (BIER) for the forwarding of multicast data packets through a
> "multicast domain".  It does not require a protocol for explicitly building
> multicast distribution trees, nor does it require intermediate nodes to
> maintain any per-flow state. This architecture is .  While the Abstract and
> Introduction of the document mentions Architecture as the principal scope, this
> document goes well beyond the scope of a typical architectural document.
> including detailed definitions of the procedures, terminology and normative
> algorithms required for BIER.
>
> The document is clear and detailed. Because of its structure, I am missing some
> information that usually can be found in architecture documents. I included
> these in the 'minor issues' list. Although none of these may be a show-stopper,
> I believe that addressing these before document approval can improve the
> quality of the document and of the overall BIER work.
>
> Major issues:
>
> Minor issues:
>
> 1. As the document is targeting 'Experimental' it would be useful to mention
> what is the scope of the experiment.The charter actually says:
>
> ' The scope of the experiment will be
> documented in the output of the Working Group.'
>
> Would not the Architecture document be the right place for this? If not, is
> there another document that deals or is planned to define the scope of the
> experiment?
>
>
> I don't really know what is meant by "the experiment" or "scope of the
> experiment", but I'm pretty sure it is not relevant to the architecture (or
> to the forwarding rules).
>
> I don't know if there is another document discussing "scope of the
> experiment", or if such a document is actually needed.  That would be a
> question for the WG chairs.
>
> 2. While the Abstract and Introduction of the document mentions Architecture as
> the principal scope, this document is different from a typical architectural
> document. While it defines well the procedures, terminology and normative
> algorithms required for BIER Intra-domain forwarding, it goes well beyond the
> level of detail that other similar documents go. Specifications of the
> procedures and normative algorithm should be mentioned in Abstract and
> Introduction, they occupy the same or more space than architecture.
>
>
> I can add a few sentences to the abstract and introduction to make it
> clear that the procedures for fowarding BIER packets within a BIER domain
> specified in this document.
>
> 3. On the other hand I am missing the relationship with other work items in the
> BIER charter - there is no manageability section for example, there is no
> reference to the performance impact in networks. Maybe these are dealt with in
> a different document or documents or BIER, if so it would be good at least to
> mention and reference these here.
>
>
> There is no requirement to include a manageability section.
>
> I believe there is ongoing work having to do with Operations and
> Management of BIER, but as that does not help to understand the
> architecture (or forwarding procedures), I don't think it would be
> appropriate to reference that work.
>
> With regard to the performance impact, if the question is speed of
> forwarding, there is mention of the fact that the number of lookups needed
> to forward a BIER packet is on the order of the number of neighbors.   I
> don't know what else can really be said at this level of detail, as the
> actual forwarding performance will depend a great deal on the
> implementation.  I'm not worried too much about that, because if BIER
> implementations do not perform well, the technology will not catch on.
>
> 4. I also would have expected the architecture document to refer the use cases
> document and note which of the use cases are being addressed and how -
> draft-ietf-bier-use-cases is not even included in the references.
>
>
> I don't see any reason why the architecture document should reference the
> "use cases" document.  An explanation of how to apply the architecture to
> each use case is not within the scope of the architecture document.
>
> 5. Sections 3 to 6 mentioned repeatedly provisioning. As there is no Operations
> and Manageability section as in many other Routing Area documents, it is not
> clear how this is expected to happen.
>
>
> How OAM is "expected to happen" would be outside the scope of this
> document.
>
> For example draft-ietf-bier-bier-yang is
> not mentioned or referred. I suggest adding a note (and maybe references) for
> clarity.
>
>
> I don't see any reason to reference that document.
>
> 6. In section 8 I found:
>
> 'Every BFR must be provisioned to know which of its interfaces lead to
>    a BIER domain and which do not.  If two interfaces lead to different
>    BIER domains, the BFR must be provisioned to know that those two
>    interfaces lead to different BIER domains. '
>
> It seems that the two 'must' in these sentences would rather be capitalized.
>
>
>
>
> I will make that change.
>
>
> _______________________________________________
> BIER mailing list
> BIER@ietf.org
> https://www.ietf.org/mailman/listinfo/bier
>
>


-- 
*We’ve heard that a million monkeys at a million keyboards could produce
the complete works of Shakespeare; now, thanks to the Internet, we know
that is not true.*
—Robert Wilensky