Re: [babel] Shepherd review of draft-ietf-babel-v4viav6-03

Donald Eastlake <d3e3e3@gmail.com> Sun, 13 June 2021 02:21 UTC

Return-Path: <d3e3e3@gmail.com>
X-Original-To: babel@ietfa.amsl.com
Delivered-To: babel@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 39BB83A2B3E; Sat, 12 Jun 2021 19:21:59 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.847
X-Spam-Level:
X-Spam-Status: No, score=-1.847 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_ENVFROM_END_DIGIT=0.25, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, SPF_HELO_NONE=0.001, 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 H2N5V6L5nw8s; Sat, 12 Jun 2021 19:21:54 -0700 (PDT)
Received: from mail-io1-xd32.google.com (mail-io1-xd32.google.com [IPv6:2607:f8b0:4864:20::d32]) (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 6829E3A2B3D; Sat, 12 Jun 2021 19:21:54 -0700 (PDT)
Received: by mail-io1-xd32.google.com with SMTP id q3so12592789iop.11; Sat, 12 Jun 2021 19:21:54 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=vnOgL7gRPCKA7bcYRlfR6wHLn2bFNPxhiXOBhTqObxI=; b=NquhPjLb8arGScLG0FHsVT7BsDMjbIgcQqlXw2wZR72LMbET1Iv8n8jYxJjcfAVPF/ nRkqrXLTJ3jTbmFwp5tsUgKbKWt26ik5WcxwJJVJjufvgSCf/r9o5S6wRiooRBzAZd96 KutL6Hc22YdVI+p720Kayq4zlXPm+jdZMxwaj5U81lPa34vCCdN9L+yNkPos9ys7K2rI BkRWl3xNSh9vw3SDr2Dk3bINps9vq7yOREX9W77itnAJ87loaY6NeWj83GEGf5dkUUAo MxlN/5dQldtulx178pZo2q/sQDJMQrnYtLxZi2VW27D+2imjl6OPM0g/0k7l2TAPhv/P scAw==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=vnOgL7gRPCKA7bcYRlfR6wHLn2bFNPxhiXOBhTqObxI=; b=SlHvjHbXvhbTUqlR82qmenG8prKqchhUesg5FQkxflAhQAyUjiZgazeNbPqRCja9GH DVEL19vRJEKuGTniDSPD2YTy9ejqHBobXyaHtnGXzUDlxP1zq/Gxo3ZgJ0py7aHdgkQg r3N9ZSWdEFt8WUeRAwoWxsfCokQIwqpNhKSiRfX+zmCjCajnBzr9Td9Z1Kfvv8nIFjiZ oWfUUm1hQEQa58qYqr7BjNwcq9AEiGjSxNYr9LTQM+sjvvm3gIgu82BF49Kox7J4PIQ4 dyNis3M9lRAlvD3/SCDU1ta/pfEyvzCgDpPWocxrjTAEpW3OPJGKMPmfRS1/dFj2ggWO NYhA==
X-Gm-Message-State: AOAM530sldyFBNiOu5mwzoCwtH61R/Gnz/euz7TIKphS9diOUfK6SOJx RRV4C3zINy7Ta0s+DoUqeT5aagU2cudf7YXhHR0=
X-Google-Smtp-Source: ABdhPJwXOQQHpTzWUcWHY7SGwNjGBWaHpuXEYaV5nG9Sxjxx7UN3KLm2bODXpFuZHWF8wveN0sKzzPiNpPlwMnArNiI=
X-Received: by 2002:a5d:8190:: with SMTP id u16mr9357486ion.158.1623550911972; Sat, 12 Jun 2021 19:21:51 -0700 (PDT)
MIME-Version: 1.0
References: <CAF4+nEGXt+x6BsQNuXwL7MMgChEB574=dW77_y71XMoL_J1rBw@mail.gmail.com> <87v96nyzsv.wl-jch@irif.fr> <CAF4+nEHjmxKggg0S21QK=7rYVaQvb4xiVG+jaJEKV0PE928rzQ@mail.gmail.com>
In-Reply-To: <CAF4+nEHjmxKggg0S21QK=7rYVaQvb4xiVG+jaJEKV0PE928rzQ@mail.gmail.com>
From: Donald Eastlake <d3e3e3@gmail.com>
Date: Sat, 12 Jun 2021 22:21:41 -0400
Message-ID: <CAF4+nEHF72Z+GJPsSuTRSNeWCcJ=ePyS2bwCDb_nM6nhbsF7-A@mail.gmail.com>
To: Juliusz Chroboczek <jch@irif.fr>, Babel at IETF <babel@ietf.org>
Cc: babel-chairs <babel-chairs@ietf.org>
Content-Type: multipart/alternative; boundary="000000000000620a6405c49c6799"
Archived-At: <https://mailarchive.ietf.org/arch/msg/babel/qhTvxV2vCvE04GkyGyMLnCO9s90>
Subject: Re: [babel] Shepherd review of draft-ietf-babel-v4viav6-03
X-BeenThere: babel@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "A list for discussion of the Babel Routing Protocol." <babel.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/babel>, <mailto:babel-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/babel/>
List-Post: <mailto:babel@ietf.org>
List-Help: <mailto:babel-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/babel>, <mailto:babel-request@ietf.org?subject=subscribe>
X-List-Received-Date: Sun, 13 Jun 2021 02:21:59 -0000

Hi,

One more thing I had overlooked: If a draft obsoletes or updates an RFC,
this needs to be mentioned in the Abstract and Introduction. The header
information on the title page says this document updates RFC 8966. But the
Abstract and the body of the draft pretty consistently say that it
"extends" the Babel protocol. Generally, the IESG has been getting stricter
about the use of "updates" although it does not have unanimous agreement on
whether or not "updates" can be used when a draft extends a protocol.

I think the simplest thing would be to remove the "Updates: 8966" header
from the first page.

Thanks,
Donald
===============================
 Donald E. Eastlake 3rd   +1-508-333-2270 (cell)
 2386 Panoramic Circle, Apopka, FL 32703 USA
 d3e3e3@gmail.com


On Wed, Jun 9, 2021 at 10:09 AM Donald Eastlake <d3e3e3@gmail.com> wrote:

> Hi Juliusz,
>
> Latest version looks good. See below:
>
> On Wed, Jun 9, 2021 at 8:42 AM Juliusz Chroboczek <jch@irif.fr> wrote:
> >
> > > I've done a review of this draft as Shepherd. See below:
> >
> > Sorry, Donald, I'd missed your review.  I've just submitted a -05 that
> > takes it into account.
> >
> > > The introduction is generally quite good but, in the first paragraph,
> > > some wording feels just the slightest bit off.  For example, it appears
> > > to assume that ND or ARP would always be used when the next-hop to
> link-
> > > layer address mapping might be statically configured
> >
> > The introduction has been rewritten.  While this section is
> non-normative,
> > and therefore doesn't constrain implementations from doing what you
> > suggest, I've added an instance of the word "typically" as a concession
> to
> > implementations that might do what you suggest.
> >
> > > Globally TDB -> 4
> >
> > Done.
> >
> > > REFERENCES
> > >
> > > Update reference to RFC 5549 to RFC 8950.
> >
> > Most respectfully disagree.  The reference is used in the following
> > sentence:
> >
> >     The extension described in this document is inspired by a previously
> >     defined extension to the BGP protocol <xref target="RFC5549"/>.
> >
> > It is important to cite the older reference in order to give proper
> credit
> > and be clear who got there first.
>
> That makes sense. But I suspect other reviewers may trip over the same
> thing.
>
> > > Section 2, 1st line: It is more common in IETF documents to say
> > > "dual-stack" rather than "double-stack".
> >
> > Done.
> >
> > > anoncing -> announcing
> >
> > Done.
> >
> > > annoucements -> announcements
> >
> > Done.
> >
> > > administatoris -> administrators
> >
> > Done.
> >
> > > next hop -> next-hop   (7 times)
> >
> > Thanks.  I've checked that I consistently use "next hop" when it is a
> noun
> > and "next-hop" when it is used as an adjective.  This is consistent with
> > RFC 8966.
>
> OK
>
> > > eg. -> e.g.   (twice)
> >
> > Done.
> >
> > > It would be preferable, within the Normative and the Informative
> > > References, that RFCs be listed in numeric order.
> >
> > In the XML?  The plain-text version is being sorted by xml2rfc, so if
> > there's any issue, it's a bug in xml2rfc.
>
> Ahhh, I was too quick and didn't look closely enough! The informative
> references are in numeric order, including RFC 0826. But the normative
> references have an RFC 792 :-) which got sorted after RFC 2119.
> Obviously too minor to bother with now but I suggest that in some
> future revision you change that to RFC 0792.
>
> I'll go ahead and do the PROTO write-up and request RFC publication.
>
> Thanks,
> Donald
> ===============================
>  Donald E. Eastlake 3rd   +1-508-333-2270 (cell)
>  2386 Panoramic Circle, Apopka, FL 32703 USA
>  d3e3e3@gmail.com
>
> > > I notice this draft generally uses British spelling. I think that's OK
> > > as long as it is consistent.
> >
> > This is consistent with RFC 8966.
> >
> > Thanks,
> >
> > -- Juliusz
>