From: Alvaro Retana <aretana.ietf@gmail.com>
MIME-Version: 1.0
Date: Tue, 11 Feb 2020 16:31:30 +0000
Message-ID: <CAMMESsz1AFH8mAVHCBJg6gSAydfFP0aWmox5AzgQkPYKrmaN4Q@mail.gmail.com>
To: draft-ietf-idr-capabilities-registry-change@ietf.org
Cc: Susan Hares <shares@ndzh.com>, idr-chairs@ietf.org,
 "idr@ietf. org" <idr@ietf.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/OFQOEM3lBjZbhUtGhh5QUH5DJC4>
Subject: [Idr] AD Review of draft-ietf-idr-capabilities-registry-change-06
Precedence: list

John:

Hi! =C2=A0Thanks for your work on this document!

I have a couple of important items that need to be addressed before
moving this document forward, specifically about change control and a
couple more values that should be deprecated. =C2=A0Please see in-line.

I will wait for an update before starting the IETF LC.

Thanks!!

Alvaro.



[Line numbers from idnits.]


...
11 =C2=A0 =C2=A0Abstract

13 =C2=A0 =C2=A0 =C2=A0 This document updates RFC 5492 by making a change t=
o the registration
14 =C2=A0 =C2=A0 =C2=A0 procedures for BGP Capability Codes. =C2=A0Specific=
ally, the range
15 =C2=A0 =C2=A0 =C2=A0 formerly designated "Reserved for Private Use" is d=
ivided into three
16 =C2=A0 =C2=A0 =C2=A0 new ranges, respectively designated as "First Come =
First Served",
17 =C2=A0 =C2=A0 =C2=A0 "Experimental" and "Reserved".

[nit] s/Experimental/Experimental Use


...
63 =C2=A0 =C2=A01. =C2=A0Introduction

65 =C2=A0 =C2=A0 =C2=A0 [RFC5492] designates the range of Capability Codes =
128-255 as
66 =C2=A0 =C2=A0 =C2=A0 "Reserved for Private Use". =C2=A0Subsequent experi=
ence has shown this to
67 =C2=A0 =C2=A0 =C2=A0 be not only useless, but actively confusing to impl=
ementors. =C2=A0BGP
68 =C2=A0 =C2=A0 =C2=A0 Capability Codes do not meet the criteria for "Priv=
ate Use" described
69 =C2=A0 =C2=A0 =C2=A0 in [RFC8126] section 4.1. =C2=A0An example of a leg=
itimate "private use"
70 =C2=A0 =C2=A0 =C2=A0 code point might be a BGP community [RFC1997] value=
 assigned for use
71 =C2=A0 =C2=A0 =C2=A0 within a given Autonomous System, but no analogous =
use of
72 =C2=A0 =C2=A0 =C2=A0 Capabilities exists.

[nit/potential rathole] I don=E2=80=99t see why Capabilities Advertisement
can=E2=80=99t meet the =E2=80=9Cfor private or local use only=E2=80=9D crit=
eria in rfc8126. =C2=A0I
guess we just don=E2=80=99t know of any local-only capability! =C2=A0;-)
Seriously, I=E2=80=99m not going to object the justification, or strongly a=
sk
you to change it, but others in the IESG might. =C2=A0Suggestion: Leave
just the first two sentences.


...
85 =C2=A0 =C2=A02. =C2=A0Discussion
...
100 =C2=A0 =C2=A0 =C2=A0 The reason for designating "IESG" as the change co=
ntroller for all
101 =C2=A0 =C2=A0 =C2=A0 registrations is that while it should be easy to o=
btain a Capability
102 =C2=A0 =C2=A0 =C2=A0 Code, once registered it's not a trivial matter to=
 safely and
103 =C2=A0 =C2=A0 =C2=A0 interoperably change the use of that code, and thu=
s working group
104 =C2=A0 =C2=A0 =C2=A0 consensus should be sought before changes are made=
 to existing
105 =C2=A0 =C2=A0 =C2=A0 registrations.

[minor] I don=E2=80=99t think this paragraph is needed because =E2=80=9CFor=
 registries
created by RFCs in the IETF stream, change control for the registry
lies by default with the IETF, via the IESG.=E2=80=9D [rfc8126]

[] I think it is confusing (at least to me) the text about =E2=80=9Cworking
group consensus=E2=80=9D if the change controller is the IESG, and we're
dealing with FCFS. =C2=A0Again, I don=E2=80=99t think this paragraph is nee=
ded.


107 =C2=A0 =C2=A0 =C2=A0 Finally, we invite implementors who have used valu=
es in the range
108 =C2=A0 =C2=A0 =C2=A0 128-255 to contribute to this draft, so that the v=
alues can be
109 =C2=A0 =C2=A0 =C2=A0 included in the registry. =C2=A0Values that have b=
een reported, are
110 =C2=A0 =C2=A0 =C2=A0 included.

[nit] The invitation is not needed, since the draft is now =E2=80=9Cdone=E2=
=80=9D and
the values are already included...


112 =C2=A0 =C2=A03. =C2=A0IANA Considerations
...
119 =C2=A0 =C2=A0 =C2=A0 Registry Owner/Change Controller: IESG

[] See above.

121 =C2=A0 =C2=A0 =C2=A0 Registration procedures:

123 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 +---------+-------------------------+
124 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 | =C2=A0Range =C2=A0| Registration Procedures |
125 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 +---------+-------------------------+
126 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 | =C2=A0 1-63 =C2=A0| IETF Review =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0=
 =C2=A0 |
127 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 | =C2=A064-238 | First Come First Served |
128 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 | 239-254 | Experimental =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|
129 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 +---------+-------------------------+

[minor] Please add Figure/Table numbers.

131 =C2=A0 =C2=A0 =C2=A0 Note: a separate "owner" column is not provided be=
cause the owner of
132 =C2=A0 =C2=A0 =C2=A0 all registrations, once made, is "IESG".

[major] The IANA review (which I now notice was only sent to Sue and
me), says: "Since part of the registry will be First Come First
Served, we'll be adding a change controller column, per RFC 8126."
[That is the full review...] =C2=A0"Change controller" and "owner" are the
same thing; I think it would be ok to rename the "Reference" column
(below) to "Reference / Change Controller".

134 =C2=A0 =C2=A0 =C2=A0 IANA is requested to perform the following new all=
ocations within the
135 =C2=A0 =C2=A0 =C2=A0 "Capability Codes" registry:

137 =C2=A0 =C2=A0 =C2=A0 +-------+-----------------------------------------=
------+-----------+
138 =C2=A0 =C2=A0 =C2=A0 | Value | Description =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 | Reference |
139 =C2=A0 =C2=A0 =C2=A0 +-------+-----------------------------------------=
------+-----------+
140 =C2=A0 =C2=A0 =C2=A0 | =C2=A0128 =C2=A0| Prestandard Route Refresh (dep=
recated) =C2=A0 =C2=A0 =C2=A0 =C2=A0| (this =C2=A0 =C2=A0 |
141 =C2=A0 =C2=A0 =C2=A0 | =C2=A0 =C2=A0 =C2=A0 | =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 | document) =
|
142 =C2=A0 =C2=A0 =C2=A0 +-------+-----------------------------------------=
------+-----------+
143 =C2=A0 =C2=A0 =C2=A0 | =C2=A0129 =C2=A0| Prestandard Outbound Route Fil=
tering =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| (this =C2=A0 =C2=A0 |
144 =C2=A0 =C2=A0 =C2=A0 | =C2=A0 =C2=A0 =C2=A0 | (deprecated), prestandard=
 draft-li-idr- =C2=A0 =C2=A0 =C2=A0 | document) |
145 =C2=A0 =C2=A0 =C2=A0 | =C2=A0 =C2=A0 =C2=A0 | flowspec-rpd-04 (deprecat=
ed) =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 |
146 =C2=A0 =C2=A0 =C2=A0 +-------+-----------------------------------------=
------+-----------+
147 =C2=A0 =C2=A0 =C2=A0 | =C2=A0130 =C2=A0| Prestandard Outbound Route Fil=
tering =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| (this =C2=A0 =C2=A0 |
148 =C2=A0 =C2=A0 =C2=A0 | =C2=A0 =C2=A0 =C2=A0 | (deprecated) =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| document) |
149 =C2=A0 =C2=A0 =C2=A0 +-------+-----------------------------------------=
------+-----------+
150 =C2=A0 =C2=A0 =C2=A0 | =C2=A0255 =C2=A0| Reserved =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0| (this =C2=A0 =C2=A0 |
151 =C2=A0 =C2=A0 =C2=A0 | =C2=A0 =C2=A0 =C2=A0 | =C2=A0 =C2=A0 =C2=A0 =C2=
=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =
=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 | document) =
|
152 =C2=A0 =C2=A0 =C2=A0 +-------+-----------------------------------------=
------+-----------+

[major] There is mention in the mail archive of other values that
should also be deprecated [1].

- "131 (CISCO multi-session)"
- "184, cumulus hostname draft"
- "185, operational draft"

[1] https://mailarchive.ietf.org/arch/msg/idr/eIoTAz3wS6dJMKBoIxwrAk98lxM