Last Call Review of draft-ietf-rtgwg-remote-lfa-08
review-ietf-rtgwg-remote-lfa-08-genart-lc-krishnan-2014-12-25-00

Request Review of draft-ietf-rtgwg-remote-lfa
Requested revision 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
I-D 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 Snapshot
Review review-ietf-rtgwg-remote-lfa-08-genart-lc-krishnan-2014-12-25
Reviewed revision 08 (document currently at 11)
Result Ready with Issues
Completed 2014-12-25
review-ietf-rtgwg-remote-lfa-08-genart-lc-krishnan-2014-12-25-00
I am the assigned Gen-ART reviewer for this draft. For background on
Gen-ART, please see the FAQ at
<

http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>

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.

Section 4.2.1.1:
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)

Thanks
Suresh