[RTG-DIR]Re: Rtgdir last call review of draft-ietf-rtgwg-segment-routing-ti-lfa-13

[Ahmed Bashandy <[e-mail]>]

Sorry for the garbled subject. I am re-sending the reply with subject 
intact for easy tracking

Thanks a lot for the comments

Version 16 addresses the comments

See #Ahmed


On 2/28/24 7:36 AM, Ben Niven-Jenkins via Datatracker wrote:
> Reviewer: Ben Niven-Jenkins
> Review result: Has Nits
>
> Hello,
>
> I have been selected as the Routing Directorate reviewer for this draft. The
> Routing Directorate seeks to review all routing or routing-related drafts as
> they pass through IETF last call and IESG review, and sometimes on special
> request. The purpose of the review is to provide assistance to the Routing ADs.
> For more information about the Routing Directorate, please see
> https://wiki.ietf.org/en/group/rtg/RtgDir
>
> Although these comments are primarily for the use of the Routing ADs, it would
> be helpful if you could consider them along with any other IETF Last Call
> comments that you receive, and strive to resolve them through discussion or by
> updating the draft.
>
> Document: draft-ietf-rtgwg-segment-routing-ti-lfa-13
> Reviewer: Ben Niven-Jenkins
> Review Date: 28th February 2024
> IETF LC End Date: Not known
> Intended Status: Proposed Standard
>
> Summary:
> This document is basically ready for publication but has nits that should be
> considered prior to publication.
>
> Comments:
> The draft is well written and the scenarios and solutions described in the
> draft are understandable and easy to follow.
>
> Major Issues:
> No major issues found.
>
> Minor Issues:
> The paragraphs at the bottom of the introduction section provide good summary
> of each section but skips over (does not provide a summary for) sections 8, 9 &
> 10. Was this intentional?
#Ahmed Version 16 provides a summary for sections 8, 9, and 10
>
> Nits:
> Section 1 - Acronyms lists TLDP as "TLDP: Target Label Distribution Protocol."
> s/Target/Targeted/
##Ahmed: We no longer need to use this acronym and we use "Targeted 
Label Distribution Protocol" only once . So it is removed from the 
acronyms section.
>
> Thanks
> Ben
>
>
>