Re: [Idr] AD Review of draft-ietf-idr-bgp-flowspec-oid-12

[Alvaro Retana <[e-mail]>]

On April 30, 2021 at 2:33:19 PM, Juan Alcaide wrote:


Juan:


Hi!

Just one point:

...
> --> I originally added AS_CONFED_SET because the
> [I-D.ietf-idr-deprecate-as-set-confed-set] is not an RFC yet. But I'll gladly
> remove AS_CONFED_SET reference for clarity if that's ok.
> And just add a note in case it's not deprecated yet. Only concern is that it
> was (could have been) mentioned in both sections 4.1 and 5. So I thought
> perhaps the best place to add the note would be at the end of the
> introduction. My proposal:
>
>
> Throughout this document, some references are made to
> AS_CONFED_SEQUENCE AS_PATH segments. If AS_CONFED_SET ASPATH
> segments are supported, same considerations MUST apply to them.
> Note, however, that AS_CONFED_SET segments are expected to be
> deprecated per [I-D.ietf-idr-deprecate-as-set-confed-set], and thus
> support for them will soon not be required.

Suggestion:

   Throughout this document, some references are made to AS_CONFED_SEQUENCE
   segments; see Sections 4.1 and 5.  If AS_CONFED_SET segments are also
   present in the AS_PATH, the same considerations apply to them.  Note,
   however, that the use of AS_CONFED_SET segments is not recommended
   [RFC6472].  Refer to [I-D.ietf-idr-deprecate-as-set-confed-set] as well.

Both references should be Informative.


Thanks!

Alvaro.