Last Call Review of draft-ietf-rtgwg-remote-lfa-09
review-ietf-rtgwg-remote-lfa-09-genart-lc-krishnan-2015-03-02-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 2015-01-06
Requested 2014-12-29
Authors Stewart Bryant , Clarence Filsfils , Stefano Previdi , Mike Shand , Ning So
I-D last updated 2015-03-02
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
Request Last Call review on draft-ietf-rtgwg-remote-lfa by General Area Review Team (Gen-ART) Assigned
Reviewed revision 09 (document currently at 11)
Result Ready w/issues
Completed 2015-03-02
review-ietf-rtgwg-remote-lfa-09-genart-lc-krishnan-2015-03-02-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