Re: [Idr] Rtgdir early review of draft-ietf-idr-bgp-car-05

["Dhananjaya Rao (dhrao)" <[e-mail]>]

Hello Mike,

Thank you very much for your review. Your suggestions look good; will update in the next rev this week.

Regards,
-Dhananjaya

From: Mike McBride via Datatracker <[e-mail]>
Date: Thursday, January 4, 2024 at 5:41 PM
To: [e-mail] <[e-mail]>
Cc: [e-mail] <[e-mail]>, [e-mail] <[e-mail]>
Subject: Rtgdir early review of draft-ietf-idr-bgp-car-05
Reviewer: Mike McBride
Review result: Has Nits

Hello,

Looking good to me. The following are a few of my rtgdir early review comments,
for your consideration, using OLD/NEW:

1. Introduction:

OLD:
BGP CAR distributes distinct routes to a destination network endpoint such as a
PE router, for different intents or colors.

NEW:
BGP CAR distributes distinct routes to a destination network endpoint, such as
a PE router, for different intents or colors.

OLD:
BGP CAR adheres to the flat routing model of BGP-IP/LU...

NEW:
BGP CAR adheres to the flat routing model of BGP-IP/LU(Labeled Unicast)...
*Unless you think LU is as understood as IP and having no need for definition.

1.1 Terminology (Resolution vs Steering):

OLD:
In this document and consistently with the terminology of
the SR Policy document [RFC9256], steering is used to
describe the mapping of a service route onto a BGP CAR
path while the term resolution is preserved for the
mapping of an inter-domain BGP CAR route on an intra-
domain color-aware path.

NEW:
In this document, and consistently with the terminology of
the SR Policy document [RFC9256], steering is used to
describe the mapping of a service route onto a BGP CAR
path. The term resolution is preserved for the
mapping of an inter-domain BGP CAR route on an intra-
domain color-aware path.

1.2 Illustration

OLD:
*Control-Plane: a node should not install a (E, C) path if it does not need it.

NEW:
*Control-Plane: a node should not install a (E, C) path if it's not
participating in that color-aware path.

2.11.  Error Handling

OLD:
When the error determined allows for the router to skip the
malformed NLRI(s) and continue processing of the rest of the update
message, then it MUST handle such malformed NLRIs as 'Treat-as-
withdraw'.

NEW:
When the error determined allows for the router to skip the
malformed NLRI(s), and continue processing of the rest of the update
message, then it MUST handle such malformed NLRIs as 'Treat-as-
withdraw'.

7.1.1.  Routed Service SID

OLD:
The intent-aware transport path to the locator of the egress PE is
provided by underlay IP routing, for instance, IGP Flex-Algo
[RFC9350] within a domain, and BGP-CAR across multiple IGP domains
or BGP ASNs.

NEW:
The intent-aware transport path to the locator of the egress PE is
provided by underlay IP routing such as IGP Flex-Algo
[RFC9350] (within a domain) and BGP-CAR (across multiple IGP domains
or BGP ASNs).

*this makes more sense to me anyway.