[RTG-DIR] Rtgdir last call review of draft-ietf-spring-sr-replication-segment-10

[Ines Robles via Datatracker <[e-mail]>]

Reviewer: Ines Robles
Review result: Has Issues

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.