Re: [Lsr] John Scudder's No Objection on draft-ietf-lsr-isis-srv6-extensions-14: (with COMMENT)

Christian Hopps <chopps@chopps.org> Wed, 19 May 2021 09:53 UTC

Return-Path: <chopps@chopps.org>
X-Original-To: lsr@ietfa.amsl.com
Delivered-To: lsr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 85E283A24CF; Wed, 19 May 2021 02:53:57 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.896
X-Spam-Level:
X-Spam-Status: No, score=-1.896 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_BLOCKED=0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001, URIBL_BLOCKED=0.001] autolearn=unavailable autolearn_force=no
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 hUd8LluzXEtk; Wed, 19 May 2021 02:53:52 -0700 (PDT)
Received: from smtp.chopps.org (smtp.chopps.org [54.88.81.56]) by ietfa.amsl.com (Postfix) with ESMTP id 0240B3A24C9; Wed, 19 May 2021 02:53:51 -0700 (PDT)
Received: from ja.int.chopps.org.chopps.org (047-026-251-217.res.spectrum.com [47.26.251.217]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (2048 bits) server-digest SHA256) (Client did not present a certificate) by smtp.chopps.org (Postfix) with ESMTPSA id DF35C80E6C; Wed, 19 May 2021 09:53:50 +0000 (UTC)
References: <162092059520.16031.2606295470570253120@ietfa.amsl.com> <CAMMESsw7V4YrsgUf9RR7GOqmTrc9T0gxVs7omF4E34zg0R2RgQ@mail.gmail.com> <8702605F-CF59-4F1E-B4CE-02951B894D1F@juniper.net> <BY5PR11MB4337994AA5C89060CAEE3E2FC1519@BY5PR11MB4337.namprd11.prod.outlook.com> <ddcfb6db-f5e4-1aa7-e45f-c9b5905de147@cisco.com> <CAMMESsyk0=ro8V3bVQddU=kn+Z7howEiCZ6mGP7bmFnWvMqXOA@mail.gmail.com> <CF010668-3238-4C01-BAC3-EB8A349EA169@cisco.com> <66662452-4BC0-4A8D-9392-12D1A3DAFF7E@juniper.net> <DB87B31E-22AD-468E-9EA9-4702BB270BE0@cisco.com> <a5991c52-1536-8d1d-6289-66d8fe2ae713@cisco.com> <BF31A944-CE8D-4598-A0CE-0971D93C4DFF@cisco.com> <BY5PR11MB4337B6F0E98672F7A3250A11C12B9@BY5PR11MB4337.namprd11.prod.outlook.com>
User-agent: mu4e 1.5.13; emacs 27.2
From: Christian Hopps <chopps@chopps.org>
To: "Les Ginsberg (ginsberg)" <ginsberg@cisco.com>
Cc: "Acee Lindem (acee)" <acee@cisco.com>, "Peter Psenak (ppsenak)" <ppsenak@cisco.com>, John Scudder <jgs=40juniper.net@dmarc.ietf.org>, Alvaro Retana <aretana.ietf@gmail.com>, "draft-ietf-lsr-isis-srv6-extensions@ietf.org" <draft-ietf-lsr-isis-srv6-extensions@ietf.org>, "lsr@ietf.org" <lsr@ietf.org>, "lsr-chairs@ietf.org" <lsr-chairs@ietf.org>, The IESG <iesg@ietf.org>, Christian Hopps <chopps@chopps.org>
Date: Wed, 19 May 2021 05:52:15 -0400
In-reply-to: <BY5PR11MB4337B6F0E98672F7A3250A11C12B9@BY5PR11MB4337.namprd11.prod.outlook.com>
Message-ID: <m2o8d7aw2a.fsf@ja.int.chopps.org>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature"
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsr/N-69fLZQPqqSxVku2rQWAPfa5AU>
Subject: Re: [Lsr] John Scudder's No Objection on draft-ietf-lsr-isis-srv6-extensions-14: (with COMMENT)
X-BeenThere: lsr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Link State Routing Working Group <lsr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lsr>, <mailto:lsr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lsr/>
List-Post: <mailto:lsr@ietf.org>
List-Help: <mailto:lsr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lsr>, <mailto:lsr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 19 May 2021 09:53:58 -0000

"Les Ginsberg (ginsberg)" <ginsberg@cisco.com> writes:

> Perhaps the somewhat unfortunate choice of the name of the registry - which inserts the codepoint for all the TLVs supported by the registry into the registry name - feeds into this confusion.
> If so, I would suggest renaming the registry to:  "Sub-TLVs for TLVs advertising NLRI".
> The registry name would then never have to be changed and we could more easily avoid being drawn into an inappropriate use of "update".

The existing names are mouthful that's for sure. I think updating the name might make sense.

Thanks,
Chris.

>
>    Les
>
>
>
>> -----Original Message-----
>> From: Acee Lindem (acee) <acee@cisco.com>
>> Sent: Monday, May 17, 2021 7:03 AM
>> To: Peter Psenak (ppsenak) <ppsenak@cisco.com>om>; John Scudder
>> <jgs=40juniper.net@dmarc.ietf.org>
>> Cc: Alvaro Retana <aretana.ietf@gmail.com>om>; Les Ginsberg (ginsberg)
>> <ginsberg@cisco.com>om>; John Scudder via Datatracker <noreply@ietf.org>rg>;
>> draft-ietf-lsr-isis-srv6-extensions@ietf.org; lsr@ietf.org; lsr-chairs@ietf.org;
>> The IESG <iesg@ietf.org>rg>; Christian Hopps <chopps@chopps.org>
>> Subject: Re: John Scudder's No Objection on draft-ietf-lsr-isis-srv6-
>> extensions-14: (with COMMENT)
>>
>> Hi Peter,
>>
>> On 5/17/21, 9:07 AM, "Peter Psenak" <ppsenak@cisco.com> wrote:
>>
>>     Hi Acee,
>>
>>     On 17/05/2021 14:56, Acee Lindem (acee) wrote:
>>     > Hi John,
>>     >
>>     > Yes – I think “updates” should be removed. Registries are created
>>     > explicitly for the purpose of tracking extensions and every document
>>     > that adds to a registry should not update the document creating that
>>     > registry. Now if the definition or application of the registry were
>>     > changed, which I don’t believe is the case here, then we could consider
>>     > “updates”.
>>
>>     RFC 7370 created the registry by merging multiple existing registries.
>>     It did not really defined any new functionality.
>>
>>     We are changing the name of that merged registry. Given that RFC 7370
>>     did not define anything new, just defined the merged registry, one can
>>     consider the name change as an update to RFC 7370.
>>
>> Ok - I missed that. I agree with that it updates RFC 7370 since the registry
>> name changed as opposed to simply adding code points.
>>
>> Thanks,
>> Acee
>>
>>     thanks,
>>     Peter
>>     >
>>     > Thanks,
>>     >
>>     > Acee
>>     >
>>     > *From: *John Scudder <jgs=40juniper.net@dmarc.ietf.org>
>>     > *Date: *Monday, May 17, 2021 at 8:48 AM
>>     > *To: *Acee Lindem <acee@cisco.com>
>>     > *Cc: *Alvaro Retana <aretana.ietf@gmail.com>om>, "Les Ginsberg
>> (ginsberg)"
>>     > <ginsberg@cisco.com>om>, "Peter Psenak (ppsenak)"
>> <ppsenak@cisco.com>om>, John
>>     > Scudder via Datatracker <noreply@ietf.org>rg>,
>>     > "draft-ietf-lsr-isis-srv6-extensions@ietf.org"
>>     > <draft-ietf-lsr-isis-srv6-extensions@ietf.org>rg>, "lsr@ietf.org"
>>     > <lsr@ietf.org>rg>, "lsr-chairs@ietf.org" <lsr-chairs@ietf.org>rg>, The IESG
>>     > <iesg@ietf.org>rg>, Christian Hopps <chopps@chopps.org>
>>     > *Subject: *Re: John Scudder's No Objection on
>>     > draft-ietf-lsr-isis-srv6-extensions-14: (with COMMENT)
>>     >
>>     > Acee,
>>     >
>>     > I think you are saying you prefer to remove the “updates”. Is that
>>     > right? It was a little confusing given the reply chain.
>>     >
>>     > (I’ve already given my opinion but said I’m not going to go to the mat
>>     > over it.)
>>     >
>>     > —John
>>     >
>>     >
>>     >
>>     >     On May 17, 2021, at 8:21 AM, Acee Lindem (acee)
>>     >     <acee=40cisco.com@dmarc.ietf.org> wrote:
>>     >
>>     >     That we be my preference as well. We’ve had several discussions on
>>     >     what constitutes “update” and I believe that the consensus was that
>>     >     a document isn’t “updated” unless the current behavior is changed.
>>     >     If we’ve done our jobs, protocols are designed to be extended and
>>     >     these extensions shouldn’t constitute updates.
>>     >
>>     >     Thanks,
>>     >
>>     >     Acee
>>     >
>>     >     *From: *Alvaro Retana <aretana.ietf@gmail.com>
>>     >     *Date: *Monday, May 17, 2021 at 6:55 AM
>>     >     *To: *"Les Ginsberg (ginsberg)" <ginsberg@cisco.com>om>, "Peter Psenak
>>     >     (ppsenak)" <ppsenak@cisco.com>om>, John Scudder
>>     >     <jgs=40juniper.net@dmarc.ietf.org>
>>     >     *Cc: *John Scudder via Datatracker <noreply@ietf.org>rg>,
>>     >     "draft-ietf-lsr-isis-srv6-extensions@ietf.org"
>>     >     <draft-ietf-lsr-isis-srv6-extensions@ietf.org>rg>, "lsr@ietf.org"
>>     >     <lsr@ietf.org>rg>, "lsr-chairs@ietf.org" <lsr-chairs@ietf.org>rg>, The
>>     >     IESG <iesg@ietf.org>rg>, Christian Hopps <chopps@chopps.org>
>>     >     *Subject: *Re: John Scudder's No Objection on
>>     >     draft-ietf-lsr-isis-srv6-extensions-14: (with COMMENT)
>>     >     *Resent-From: *<alias-bounces@ietf.org>
>>     >     *Resent-To: *Yingzhen Qu <yingzhen.ietf@gmail.com>om>, Christian
>> Hopps
>>     >     <chopps@chopps.org>rg>, Acee Lindem <acee@cisco.com>
>>     >     *Resent-Date: *Monday, May 17, 2021 at 6:54 AM
>>     >
>>     >     Peter:
>>     >
>>     >     Hi!
>>     >
>>     >     As John mentioned, "Since for better or worse we don’t have a firm
>>     >     definition of when we do, and don’t, use “updates”, it comes down to
>>     >     a matter of personal taste in the end.”
>>     >
>>     >     I rather you leave it in.
>>     >
>>     >     Thanks!
>>     >
>>     >     Alvaro.
>>     >
>>     >     On May 17, 2021 at 6:42:48 AM, Peter Psenak (ppsenak@cisco.com
>>     >     <mailto:ppsenak@cisco.com>) wrote:
>>     >
>>     >         John, Alvaro,
>>     >
>>     >         do we have a consensus whether we need the update to RFC 7370 or
>>     >         not?
>>     >
>>     >
>>     >         thanks,
>>     >         Peter
>>     >
>>     >
>>     >
>>     >         On 13/05/2021 21:12, Les Ginsberg (ginsberg) wrote:
>>     >          > Alvaro –
>>     >          >
>>     >          > FWIW, I agree w John here.
>>     >          >
>>     >          > There are many examples – to cite a few:
>>     >          >
>>     >          > Sub-TLVs for TLVs 22, 23, 25, 141, 222, and 223 (Extended IS
>>     >          > reachability, IS Neighbor Attribute, L2 Bundle Member
>>     >         Attributes,
>>     >          > inter-AS reachability information, MT-ISN, and MT IS Neighbor
>>     >         Attribute
>>     >          > TLVs)
>>     >          >
>>     >          > …
>>     >          >
>>     >          > Reference
>>     >          >
>>     >          >     [RFC5305][RFC5316][RFC7370][RFC8668]
>>     >          >
>>     >          > RFC 8868 is not marked as updating RFC 7370.
>>     >          >
>>     >          > RFC 7370 is not marked as updating RFC 5316/RFC 5305.
>>     >          >
>>     >          > Sub-TLVs for TLVs 135, 235, 236, and 237 (Extended IP
>>     >         reachability, MT
>>     >          > IP. Reach, IPv6 IP. Reach, and MT IPv6 IP. Reach TLVs)
>>     >          >
>>     >          > …
>>     >          >
>>     >          > Reference
>>     >          >
>>     >          >     [RFC5305][RFC7370]
>>     >          >
>>     >          > Again, RFC7370 is not marked as updating RFC 5305.
>>     >          >
>>     >          > I think it is sufficient to request that IANA add the new RFC
>>     >         to the
>>     >          > list of References for the modified registry.
>>     >          >
>>     >          >    Les
>>     >          >
>>     >          > *From:* Lsr <lsr-bounces@ietf.org
>>     >         <mailto:lsr-bounces@ietf.org>> *On Behalf Of *John Scudder
>>     >          > *Sent:* Thursday, May 13, 2021 11:00 AM
>>     >          > *To:* Alvaro Retana <aretana.ietf@gmail.com
>>     >         <mailto:aretana.ietf@gmail.com>>
>>     >          > *Cc:* John Scudder via Datatracker <noreply@ietf.org
>>     >         <mailto:noreply@ietf.org>>; Christian Hopps
>>     >          > <chopps@chopps.org <mailto:chopps@chopps.org>>;
>>     >         lsr-chairs@ietf.org <mailto:lsr-chairs@ietf.org>; The IESG
>>     >         <iesg@ietf.org <mailto:iesg@ietf.org>>;
>>     >          > draft-ietf-lsr-isis-srv6-extensions@ietf.org
>>     >         <mailto:draft-ietf-lsr-isis-srv6-extensions@ietf.org>;
>>     >         lsr@ietf.org <mailto:lsr@ietf.org>
>>     >          > *Subject:* Re: [Lsr] John Scudder's No Objection on
>>     >          > draft-ietf-lsr-isis-srv6-extensions-14: (with COMMENT)
>>     >          >
>>     >          > On May 13, 2021, at 1:20 PM, Alvaro Retana
>>     >         <aretana.ietf@gmail.com <mailto:aretana.ietf@gmail.com>
>>     >          > <mailto:aretana.ietf@gmail.com
>>     >         <mailto:aretana.ietf@gmail.com>>> wrote:
>>     >          >
>>     >          >   This documents updates RFC 7370 by modifying an existing
>>     >          > registry.
>>     >          >
>>     >          > Also, this doesn’t seem to me like an update to RFC 7370. It’s
>>     >          > normal for an
>>     >          > RFC to update an IANA registry, without saying it updates a
>>     >          > previous RFC that
>>     >          > established that registry. I think the “updates” just confuses
>>     >          > matters and
>>     >          > clutters things up, and should be removed.
>>     >          >
>>     >          >
>>     >          > In this case the document is not only registering a value.
>>     >         It is
>>     >          > changing the name of the registry, adding an extra column, and
>>     >          > updating all the other entries (§11.1.*).  The Updates tag is
>>     >         used
>>     >          > because it significantly changes the registry.
>>     >          >
>>     >          > Still seems unnecessary to me, registries are moving targets,
>>     >         citation
>>     >          > of all the relevant RFCs in their references should be
>>     >         sufficient. So,
>>     >          > the registry would be updated so that it cited both this spec
>>     >         and 7370,
>>     >          > and someone wanting to know “how did the registry get this
>>     >         way?” would
>>     >          > be able to work it out.
>>     >          >
>>     >          > I’m not going to fight about it; the “updates” is not very
>>     >         harmful. I
>>     >          > say “not very” because the diligent reader might be led to
>>     >         think they
>>     >          > need to go read RFC 7370 in order to properly understand this
>>     >         spec, and
>>     >          > waste some time realizing that isn’t true. Since for better
>>     >         or worse we
>>     >          > don’t have a firm definition of when we do, and don’t, use
>>     >         “updates”, it
>>     >          > comes down to a matter of personal taste in the end.
>>     >          >
>>     >          > $0.02,
>>     >          >
>>     >          > —John
>>     >          >
>>     >
>>