Skip to main content

Last Call Review of draft-ietf-mpls-ri-rsvp-frr-07
review-ietf-mpls-ri-rsvp-frr-07-genart-lc-enghardt-2019-10-30-00

Request Review of draft-ietf-mpls-ri-rsvp-frr
Requested revision No specific revision (document currently at 16)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2019-11-05
Requested 2019-10-22
Authors Chandrasekar R , Tarek Saad , Ina Minei , Dante Pacella
I-D last updated 2019-10-30
Completed reviews Rtgdir Last Call review of -16 by Carlos Pignataro
Rtgdir Last Call review of -05 by Julien Meuric (diff)
Genart Last Call review of -07 by Reese Enghardt (diff)
Assignment Reviewer Reese Enghardt
State Completed
Request Last Call review on draft-ietf-mpls-ri-rsvp-frr by General Area Review Team (Gen-ART) Assigned
Posted at https://mailarchive.ietf.org/arch/msg/gen-art/w4OEk-qCV5xFprZA0I_is8RHSTg
Reviewed revision 07 (document currently at 16)
Result Ready w/nits
Completed 2019-10-30
review-ietf-mpls-ri-rsvp-frr-07-genart-lc-enghardt-2019-10-30-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

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-mpls-ri-rsvp-frr-07
Reviewer: Theresa Enghardt
Review Date: 2019-10-30
IETF LC End Date: 2019-11-05
IESG Telechat date: Not scheduled for a telechat

Summary: This draft is basically ready for publication, but has nits that
should be fixed before publication.

Major issues: None.

Minor issues: None.

Nits/editorial comments:

The abstract is hard to parse and could be improved by fixing the following
nits.

OLD: "Facility backup method allows"
NEW: "The facility backup method allows"

OLD: "The many-to-one nature of local repair technique"
NEW: "The many-to-one nature of this local repair technique"

OLD: "This document enumerates facility backup procedures in RFC 4090"
NEW: "This document enumerates the facility backup procedures in RFC 4090" [or
"several facility backup procedures", if not all procedures are enumerated here]

OLD: "and hence make facility backup method refresh-interval dependent."
NEW: "and hence make the facility backup method refresh-interval dependent."

OLD: "The RSVP-TE extensions defined in this document will enhance"
NEW: "This document defines RSVP-TE extensions to enhance"

Are "facility backup method", "facility backup procedure" and "facility backup
protection mechanism" all the same thing? Please consider using only a single
term to avoid confusion.

Please double-check the use of definite and indefinite articles in the rest of
the document as well.