ietf-wg-idr / draft-ietf-idr-bgp-ct

1 stars 2 forks source link

2nd WG LC draft-ietf-idr-bgp-ct-srv6 - Shepherd Report - G01-xx items - Must fix items in main text. #66

Closed suehares closed 3 months ago

suehares commented 4 months ago

Q1 - Issues in draft-ietf-idr-bgp-ct-srv6 - Editorial issues must be fixed prior to IESG

Q1-01 section 3, paragraph 1, section in BGP-CT.

Old text:/ The procedures for encoding a BGP Classful Transport (BGP CT) family route are specified in sections 6, section 4 in [BGP-CT]. These are followed, with the addition of SRv6 encapsulation information. /

Do you mean sections 4-7?

Q1-02, Section 4, paragraph 3, news to be reworded.

Old text:/ The BGP Classful Transport route update for SRv6 MUST include an attribute containing SRv6 SID information. This may be either the BGP Prefix-SID attribute as specified in [RFC9252] . / Possible reworking: Do you want to simply specify BGP Prefix-SID or any attribute that has the same information?

New text: The BGP Classful Transport route update for SRv6 MUST include an attribute containing SRv6 SID information. The BGP Prefix-SID attribute, as specified in [RFC9252] provides the correct information /

Q1-03, Section 5.1, paragraph ~8

Old text:/ The illustrations that follow, show how the BGP CT route for gold transport plane is originated, import processing done and propagated through this network. Similar processing is followed for the bronze transport plane route as well./

Possible New text:/ The illustrations that follow below, show the following things:

Why: Paragraph is difficult to follow.

Q1-04, Section 5.1, paragraph 11

Old text:/ The BGP CT routes for RD1:PE2-LPBK is further advertised towards ASBR1 via EBGP CT as follows. During this readvertisement, the next hop is set to self, and SID is rewritten to ASBR2-SRv6-gold- Replace. /

Comment: Rework these two sentences. The phrase "as follows" does not clearly reference something. If the second sentence is trying to describe how the route is revised as it is readvertised, it needs to mesh with the 1st of these two sentences clearly.

Q1-05: Section 9.2 - fix draft-wnag-idr-cpr-01 to draft-ietf-idr-cpr.

Old text:/.
[Colorful-Prefix-Routing-SRv6] Wang, Ed., "BGP Colorful Prefix Routing for SRv6 based Services", 26 March 2023, https://www.ietf.org/archive/id/draft-wang-idr-cpr- 01.html. /

suehares commented 4 months ago

If you want me to create a respository for draft-ietf-idr-bgp-ct-srv6 - let me know.

suehares commented 3 months ago

closed after reviewed difference between -02 and -03.