[Sidrops] Re: draft-ietf-sidrops-avoid-rpki-state-in-bgp-11 ietf last call Artart review

[Job Snijders <[e-mail]>]

Dear Henry,

Thank for this review. I've passed on the editorial nits to the RFC Editor:
https://mailarchive.ietf.org/arch/msg/auth48archive/9a61hMpfzjthDBCCRRITZMohRIM/

Kind regards,

Job

On Sun, Jun 28, 2026 at 11:13:43AM -0700, Henry Thompson via Datatracker wrote:
> Document: draft-ietf-sidrops-avoid-rpki-state-in-bgp
> Title: Guidance to Avoid Carrying RPKI Validation States in BGP Path Attributes
> Reviewer: Henry Thompson
> Review result: Ready with Nits
> 
> Document: draft-ietf-sidrops-avoid-rpki-state-in-bgp-11
> Title: Guidance to Avoid Carrying RPKI Validation States in BGP Path Attributes
> Reviewer: Henry S. Thompson
> Review Date: 2025-06-17
> 
> *Summary*
> 
> As far as I can tell as a non-specialist, the presentation of this
> proposed BCP is in good shape.  One minor suggestion and some
> editorial nits follow below.
> 
> *Minor point*
> 
> 1.
> 
>  ... "states leak through to EGBP speakers."  Would it be possible to
>  _briefly_ mention here at least one known way in which such leakage
>  can occur in a conformant router?  That way operators of 'good'
>  routers may realise this document is relevant to them.
> 
> *Nits*
> 
> 1.
> 
>  In this phrase
> 
>  "Annotating ... Attribute carried across EBGP session (based on validation
>   states) means that BGP UPDATE messages have to be sent to every
>   neighbor when the aforementioned validation states"
> 
>  there's a minor grammatical problem.  I would suggest the following
>  change
> 
>  "... Attribute carried across an EBGP session (based on validation
>   states) means that BGP UPDATE messages have to be sent to every
>   neighbor when the aforementioned validation states"
> 
> 4.
> 
>  "BGP communities" -> "BGP Communities" [here and in 4 other places below]
> 
>  "hold similar" -> "hold in a similar way"
> 
> 4.1
> 
>  "a large amounts" -> "large amounts"
> 
> 4.1.2
> 
>  "caused by operators to signal validation states" ->
>  "caused by operators signalling validation states"
> 
> 
>