Eric, Please find my responses below.
Thanks, -Rishabh On Wed, Jun 28, 2023 at 2:17 AM Éric Vyncke via Datatracker < nore...@ietf.org> wrote: > Éric Vyncke has entered the following ballot position for > draft-ietf-spring-sr-replication-segment-15: No Objection > > > ---------------------------------------------------------------------- > COMMENT: > ---------------------------------------------------------------------- > > > # Éric Vyncke, INT AD, comments for > draft-ietf-spring-sr-replication-segment-15 > > Thank you for the work put into this document. It is quite dense and not > too > easy to read though, perhaps adding some graphics? > [RP] What type of graphics do you have in mind? Do you think the illustrations in appendix help in clarifying the concepts in the document? > > # COMMENTS > > ## Section 1 > > The reader would probably welcome use case of this protocol: is it for > multicast ? or more like a span port for monitoring/troubleshooting ? > > Waiting until section 3 is not really reader friendly. > [RP] I will add some text about uses cases in the Introduction in the next revision. > ## Section 2 > > `When the PCE signals a Replication segment to its node` what is 'its > node' ? > [RP] It should have been "When a PCE signals a Replication Segment to a node ...", but after reading again, I don't think this sentence is necessary in the paragraph. I will remove it. > ## Section 2.2 > > In the 2nd paragraph, is the segment left field also decremented ? > [RP] No, the SL in SRH is not decremented because the Downstream Replication SID that is put in the IPv6 DA field for a replication comes from the Replication State, not SRH. > > ## Section 2.2.3 > > Thanks for this section (no need to reply). > > # NITS > > ## Section 4.2 > > s/has all the Must and SHOULD clause/has all the MUST and SHOULD clauses/ ? > [RP] I will fix this. > > > _______________________________________________ > spring mailing list > spring@ietf.org > https://www.ietf.org/mailman/listinfo/spring >
_______________________________________________ spring mailing list spring@ietf.org https://www.ietf.org/mailman/listinfo/spring
