Last Call Review of draft-ietf-rtgwg-rlfa-node-protection-09
review-ietf-rtgwg-rlfa-node-protection-09-genart-lc-shirazipour-2016-12-29-00
Request | Review of | draft-ietf-rtgwg-rlfa-node-protection |
---|---|---|
Requested revision | No specific revision (document currently at 13) | |
Type | Last Call Review | |
Team | General Area Review Team (Gen-ART) (genart) | |
Deadline | 2017-01-04 | |
Requested | 2016-12-21 | |
Authors | Pushpasis Sarkar , Shraddha Hegde , Chris Bowers , Hannes Gredler , Stephane Litkowski | |
I-D last updated | 2016-12-29 | |
Completed reviews |
Rtgdir Early review of -02
by Mike Shand
(diff)
Genart Last Call review of -09 by Meral Shirazipour (diff) Opsdir Last Call review of -13 by Éric Vyncke Genart Telechat review of -10 by Meral Shirazipour (diff) |
|
Assignment | Reviewer | Meral Shirazipour |
State | Completed | |
Request | Last Call review on draft-ietf-rtgwg-rlfa-node-protection by General Area Review Team (Gen-ART) Assigned | |
Reviewed revision | 09 (document currently at 13) | |
Result | Ready w/nits | |
Completed | 2016-12-29 |
review-ietf-rtgwg-rlfa-node-protection-09-genart-lc-shirazipour-2016-12-29-00
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 <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>. Document: draft-ietf-rtgwg-rlfa-node-protection-09 Reviewer: Meral Shirazipour Review Date: 2016-12-28 IETF LC End Date: 2017-01-04 (extension 2017-01-11) IESG Telechat date: 2017-01-05 (extended) Summary: This draft is ready to be published as Standards Track RFC but I have comments. Major issues: Minor issues: Nits/editorial comments: -Please spell out Loop-Free Alternate (LFA), Shortest Path First (SPF) at first use. -typo in draft header: "Internet-Draft R-LFA Node-Protection and Manageabilty December 2016" "Manageabilty"--should be-->"Manageability" -Same typo in Section 3 header: "Manageabilty of Remote-LFA Alternate Paths" "Manageabilty"--should be-->"Manageability" -[Page 6] Section 2.2.3. "on any of the shortest path", path -->"paths" -[Page 6] Section 2.2.3. "from the node Y to primary nexthop E": "one ECMP path from the node Y": -->the example in this draft did not use the letter Y for any nodes. Would it be clearer to say node Y is defined in Section 2.2.4? -Few occurences of "w.r.t to ", the "to" is redundant. -[Page 1], "can be utilised "--->"can be utilized" -[Page 9], Section 2.3 "Sections Section 2.3.1 and Section 2.3.2 shows "--->"Section 2.3.1 and Section 2.3.2 show" -[Page 11], "To determine wether"--->"To determine whether" -[Page 11], "primary nexthop node"-->"primary nexthop nodes" -[Page 13], "choose only the ones that does"--->"choose only the ones that do" -[Page 13], "not gaurantee"--->"not guarantee" -[Page 14], "Figure 7: Toplogy with multiple ECMP primary nexthops"--->"Figure 7: Topology with multiple ECMP primary nexthops" -[Page 14], "node-proecting"--->"node-protecting" -[Page 15], "paths tp PQ-node R2"--->"paths to PQ-node R2" -[Page 16], "gaurantees node-protection"--->"guarantees node-protection" -[Page 17], "above example above"--->"above example" -[Page 17], "also allow user"--->"also allow the user" -[Page 18], "the the computing"---->"the computing" -[Page 18], "in section Section 2.3.2."---->"in Section 2.3.2." (2 occurrences) -[Page 18], "in section Section 2.3 the"---->"in Section 2.3 the" -[Page 18], "i.e from "---->"i.e. from " -[Page 18], "two Remote-LFA alternate path"--->"two Remote-LFA alternate paths" -[Page 19], "the approach proposed"----->"the proposed approach " -[Page 19], "is needed keep "---->"is needed to keep" -[Page 19], "entire toplogy"---->"entire topology" -General: Is there any proof or extensive simulation that has proved that the mechanism proposed works for various network topologies and not only the one shown in the examples? Best Regards, Meral --- Meral Shirazipour Ericsson Research www.ericsson.com