Re: [Last-Call] Genart last call review of draft-ietf-idr-bgp-ls-flex-algo-09

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Vijay, thank you for your review. I have entered a No Objection ballot for this document.

Lars


> On 2022-8-12, at 21:50, Vijay Gurbani via Datatracker <noreply@xxxxxxxx> wrote:
> 
> Reviewer: Vijay Gurbani
> Review result: Ready with Nits
> 
> I am the assigned Gen-ART reviewer for this draft. The General Area
> Review Team (Gen-ART) reviews all IETF documents being processed
> by the IESG for the IETF Chair.  Please treat these comments just
> like any other last call comments.
> 
> For more information, please see the FAQ at
> 
> <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.
> 
> Document: draft-ietf-idr-bgp-ls-flex-algo-??
> Reviewer: Vijay K. Gurbani
> Review Date: 2022-08-12
> IETF LC End Date: 2022-08-17
> IESG Telechat date: Not scheduled for a telechat
> 
> Summary: Draft is ready with nits for a Proposed Standard.
> 
> Major issues: 0
> 
> Minor issues: 1 (please see below)
> 
> Nits/editorial comments: 4 (please see below)
> 
> Minor:
> - Sec. 3.6: Note that Type is "TBD" here.  Should this be 1046, as shown in
> Table 1?  (Or is the use of 1046 still under discussion?)
> 
> Nits:
> - I note that certain acronyms --- IGP, NLRI, ASLA --- are not defined.  I
> suspect that these are well-known in the community, hence need no definition.
> Just in case they are not, you may consider expanding the rare ones on first
> use. - Sec. 1: s/Flexible algorithm is called so as/Flexible algorithm is so
> called because/ - Sec. 2: s/Definition(s) (FAD) advertised by a node
> is/Definition(s) (FAD) advertised by a node is (are)/
>  Reason: symmetry in the sentence construction
> - Sec. 3.6: Is Figure 7 missing the trailing "//" for sub-TLV tpes?
> 
> Thanks.
> 
> 
> 
> --
> last-call mailing list
> last-call@xxxxxxxx
> https://www.ietf.org/mailman/listinfo/last-call

Attachment: signature.asc
Description: Message signed with OpenPGP

-- 
last-call mailing list
last-call@xxxxxxxx
https://www.ietf.org/mailman/listinfo/last-call

[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Mhonarc]     [Fedora Users]

  Powered by Linux