Last Call Review of draft-ietf-lsr-isis-fast-flooding-05
review-ietf-lsr-isis-fast-flooding-05-rtgdir-lc-andersson-2023-11-06-00

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-lsr-isis-fast-flooding (the current version is -05)
Reviewer: Loa Andersson
Review Date: 2023-10-08
IETF LC End Date:
Intended Status: Experimental

Summary:

This document is basically ready for publication; I only found one issue 
- the number of authors listed.

Document Overview:
Current Link State Protocol Data Unit (PDU) flooding rates are much 
slower than what modern networks can support.  The use of IS-IS at 
larger scale requires faster flooding rates to achieve desired 
convergence goals.  This document discusses the need for faster 
flooding, the issues around faster flooding, and some example approaches 
to achieve faster flooding.  It also defines protocol extensions 
relevant to faster flooding.

Comments:

The draft is well-written and easy to read. I gone over the IANA 
Considerations and allocations, and not found anything that need to be 
addressed.

Major Issues:

Number of authors: There are 7 authors, that is more the the "allowed" 5 
authors.

I have no background why there 7 authors listed, this has to be 
addressed in some way:

- reduce the number of authors to five

- keep the number of authors at seven, and the Shepherd will have to
    address this in the SWU,

I have put this as a "major issue" since I don't know where to put it.

My personal opinion is that anyone that has contributed text to the 
document, and participated in the authors discussions, should be listed 
as an author.

"No minor issues found."

Nits:

The nits-tool only finds a  Miscellaneous warning:

-- The document date (5 September 2023) is 32 days in the past.  Is this
      intentional?

This warning is a bit annoying since it is impossible to avoid.

I have not found any other nits.


/Loa


-- 
Loa Andersson                        email: loa@pi.nu
Senior MPLS Expert                          loa.pi.nu@gmail.com
Bronze Dragon Consulting             phone: +46 739 81 21 64