[Idr] Shepherd Report for draft-ietf-idr-ct-srv6-02

[Susan Hares <[e-mail]>]

Nats, Kaliraj, and Reshma:

I've attached the shepherd report for the document in a text file.   I have also included the text below.

The errors are categorized as:

Q1-xx - Must fix editorial errors - 5
Q2-00- IANA errors   - none
Q3-00 - Main text - Editorial errors for English


I will upload this shepherd report to IDR WG github (probably by 10am PT 3/3).

Sue

===============

Shepherd's report for draft-ietf-idr-bgp-ct-srv6-02

Q1 - Issues that need to be taken care of

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 section 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 anything 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:
     - how the BGP CT route for gold transport plane is originated, and
                 - how import route processing done,  and
                 - how the route is propagated through this network.

   Similar processing is followed for the bronze transport plane route as well.
/

Why: Sparagraph 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.
/

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 need to clearly
mesh with the 1st of these two sentences.


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>.
/



=========
Q2-None


=====
Editorial only
Q3-01 - Section 3, paragraph 4 - use And

old text:/
   Aspects regarding Interoperability between nodes supporting different
   forwarding technologies is discussed in Section 6.3, Section 11.3.2
   of [BGP-CT].
/
New text:/
   Aspects regarding Interoperability between nodes supporting different
   forwarding technologies is discussed in Section 6.3 and Section 11.3.2
   of [BGP-CT].

Q3-02 - section 5.1, paragraph begining "Futhermore".

I recommend removing "Furthermore" and starting the paragraph "Any service"

Starting two paragraphs "Furthermore" and then "Similarly"
seems a bit stilted.  Like

Old text:/
   Furthermore, any service routes received with next hop as PE2-LPBK
   and Mapping Community as Color:0:100 indicating Gold SLA will use the
   Resolution Scheme associated with its Mapping Community to resolve
   over the PE2-LPBK CT route installed in the gold TRDB, and push the
   SRv6-gold SID stack to reach PE2.
/

Q3-04, Section 5.1, 2 line wraps need to be fixed and spelling error (repace)
old text:/
PE1: Service routes FIB

      [BGP INET] SVC_PFX1, color:0:100
        NH: EncapSID "PE2-SRv6-S1-DT4, ASBR1-SRv6-gold-Repace, Gold-SRv6-Tunnel-to-ASBR1(outer)"

      [BGP INET] SVC_PFX2, color:0:200
        NH: EncapSID "PE2-SRv6-S1-DT4, ASBR1-SRv6-bronze-Replace, Bronze-SRv6-Tunnel-to-ASBR1(outer)"
/
New text:/
PE1: Service routes FIB

      [BGP INET] SVC_PFX1, color:0:100
        NH: EncapSID "PE2-SRv6-S1-DT4, ASBR1-SRv6-gold-Replace,
                                Gold-SRv6-Tunnel-to-ASBR1(outer)"

      [BGP INET] SVC_PFX2, color:0:200
        NH: EncapSID "PE2-SRv6-S1-DT4, ASBR1-SRv6-bronze-Replace,
                                Bronze-SRv6-Tunnel-to-ASBR1(outer)"
/