[RTG-DIR]Re: Rtgdir early review of draft-ietf-idr-cpr-02

["Dongjie (Jimmy)" <[e-mail]>]

Hi Yingzhen, 

Thanks for the review and comments, we will prepare a new revision to address them. 

And please see some replies inline:

> -----Original Message-----
> From: Yingzhen Qu via Datatracker <[e-mail]>
> Sent: Friday, May 31, 2024 9:52 PM
> To: [e-mail]
> Cc: [e-mail]; [e-mail]
> Subject: Rtgdir early review of draft-ietf-idr-cpr-02
> 
> Reviewer: Yingzhen Qu
> Review result: Has Nits
> 
> This is an Early Review, requested by the WG chair.
> 
> Document: draft-ietf-idr-cpr-02
> Reviewer: Yingzhen Qu
> Review Date: May 31, 2024
> Intended Status: Informational
> 
> Summary:
> This draft proposes inter-domain intent-aware routing based on existing
> mechanisms to advertise IPv6 Colored Prefixes: BGP IPv6 Unicast Address Family
> and color extended community.
> 
> Comments and nits:
> 
> The Abstract is a bit long, the authors may consider moving some content from
> the abstract to the introduction.

OK, will reduce the text in the abstract. 

> 
> 
> 233    While in some special cases, one intent may be represented as
> 234    different color value in different domains, then the Color Extended
> 235    Community in the CPR routes may be updated at the border nodes of
> the
> 236    domains based on the color-mapping policy.
> 
> [Q]: why "may be updated" instead of "SHOULD be updated"?
> [comments]: Please consider adding an example of one intent with different
> colors in different domains.

Whether the color needs to be updated or not depends on the color-mapping policy between the adjacent domains. We will add some text to clarify this. 

> 
> 323    forwarding process using H.Insert.Red behavior is shown as below:
> 
> [comments]: Please add a reference for H.Insert.Red .

OK.

> 
> 334    The packet encapsulation and forwarding process using H.Encaps.Red
> 335    behavior is shown as below:
> 
> [comments]: Please add a reference for H.Encaps.Red.

OK. 

> 
> 420    There may be multiple inter-domain links between network domains,. A
> 
> nits: extra ","

Thanks for catching the nit, will fix it in next revision. 

Best regards,
Jie