Last Call Review of draft-ietf-rtgwg-remote-lfa-08

Request Review of draft-ietf-rtgwg-remote-lfa
Requested rev. no specific revision (document currently at 11)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2014-12-25
Requested 2014-12-15
Authors Stewart Bryant, Clarence Filsfils, Stefano Previdi, Mike Shand, Ning So
Draft last updated 2014-12-25
Completed reviews Genart Last Call review of -08 by Suresh Krishnan (diff)
Genart Last Call review of -09 by Suresh Krishnan (diff)
Secdir Last Call review of -09 by Simon Josefsson (diff)
Assignment Reviewer Suresh Krishnan 
State Completed
Review review-ietf-rtgwg-remote-lfa-08-genart-lc-krishnan-2014-12-25
Reviewed rev. 08 (document currently at 11)
Review result Ready with Issues
Review completed: 2014-12-25


I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at

Please wait for direction from your document shepherd or AD before 
posting a new version of the draft.

Document: draft-ietf-rtgwg-remote-lfa-10.txt
Reviewer: Suresh Krishnan
Review Date: 2015/01/06
IESG Telechat date: 2015/01/08

Summary: This draft is ready for publication as a Proposed Standard but 
I have some minor issues that the authors may wish to address.

This sentence is missing a verb (reach?)

The exclusion of routers reachable via an ECMP that includes S-E 
prevents the forwarding subsystem from attempting to <MISSING VERB> a 
repair endpoint via the failed link S-E.

Section 4.3:

The Compute_Neighbor_SPFs() function seems to be missing a check for 
verifying if the interface is not the failed interface. I think the 
following check should be added.

if (intf != fail_intf)