I am the assigned Rtg reviewer for this draft. Please treat these comments just like any other last call comments.

Document: draft-ietf-spring-sr-replication-segment-10
Reviewer: Ines Robles
Review Date: 2022-11-14

Summary:

This document (with intended status Standards Track) describes the SR (Segment Routing) Replication segment for Multi-point service delivery. A SR Replication segment allows a packet to be replicated from a Replication Node to Downstream nodes.

The document mention two implementations. An example is described in Appendix A.


Major issues: None

Minor issues:

1- Requirements Language section should add RFC 8174, i.e. "...."NOT RECOMMENDED", "MAY", and "OPTIONAL" in this document are to be interpreted as described in BCP 14 [RFC2119] [RFC8174] when, and only when, they appear in all capitals, as shown here."

2- There is no terminology section. It would be nice to have a terminology section where inform to the reader which document to read to understand the terminology not defined in the document e.g. "..the operations NEXT, PUSH are defined in RFC8402 and the POP operation in RFC...; H.Encaps.Red is defined in ....."

3- Page 3: "Replication-ID can be a 32-bit number, but it can be extended or modified as required... " -> to which limit can be extended?

4- Question: since the replication state of the nodes can change over time. Is it possible that in some particular circumstances this change could trigger a loop in the replication segment? or this is not possible?

5- The security considerations states: "There are no additional security risks introduced by this design". Additional to what features? This is not clear to me. Perhaps it would be nice to rephrase it to something like: "The security considerations outlined in RFC 8402, RCF... also applies to this document"?

Nits/editorial comments:

6- Page 7: all the Must and SHOULD... => all the MUST and SHOULD?

7- Page 11- Figure 1: It would be nice if the caption of the figure could be descriptive 

8- Page 13 Paragraph 8 and 9: End.Replcate => End.Replicate?

Thanks for this document,

Ines.