Meral, Thanks for the review!
Greg, It looks like most, but not all, of these comments were already caught in -04. This is really up to Stewart, but IMHO, the remaining comments can go in instructions to the RFC Editor, or wait for a revision triggered by further IESG or IETF-wide review. Cheers, Andy On Thu, Jan 2, 2014 at 4:55 PM, Meral Shirazipour <[email protected]> wrote: > 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 resolve these comments along with any other Last Call comments you > may receive. > > > > Document: draft-ietf-pwe3-mpls-tp-cv-adv-03 > > Reviewer: Meral Shirazipour > > Review Date: 2014-01-02 > > IETF LC End Date: 2014-01-16 > > IESG Telechat date: NA > > > > > > Summary: > > This draft is ready to be published as Standards Track RFC but I have some > editorial comments (mostly nits). > > > > > > Nits/editorial comments: > > Nits: > > -[Page 3] Introduction, line 4, "PW s"--extra space-->"PWs". > > > > -[Page 3] Section 2, "types of CV been defined"--missing have-->"types of CV > have been defined" > > > > -[Page 4] Section 2.1 "It it isrecommended"--extra it and no space--->"It is > recommended" > > > > -[Page 4] Section 2.1, this statement is repeated twice, another time on the > sentence above it: > > "The Reserved field must be set to zeroes on transmit and ignored on > receive. > > " > > -[Page 4], Table 1 "Signalling"--for consistency with the rest of the > occurrences-->"Signaling" > > > > -[Page 5], Section 2.4 "advertized"--more common-->"advertised" > > > > -[Page 5], Section 2.4 missing "a" and "are": > > > > old: > > "and common set of proactive CV types advertized" > > > > new: > > "and a common set of proactive CV types are advertised" > > > > -[Page 5], Section 2.4 missing "are" and "the" > > > > old: > > " > > If multiple MPLS-TP CV types advertised by both PEs, then following list > sorted in > > descending priority order is used: > > " > > new: > > " > > If multiple MPLS-TP CV types are advertised by both PEs, then the following > list sorted in > > descending priority order is used: > > " > > > > General suggestion: > > - > > In the abstract something is missing to make it more suitable for standard > track. Right now it only says "this document specifies how use of ABC > affects XYZ". > > Examples (to inspire from), Section 1 and 2 mention: > > > > " > > This document specifies how signaling > > and selection processes are modified to ensure backward compatibility > > and allow use of proactive CV-CC-RDI over MPLS-TP PWs. > > " > > > > and > > > > " > > This document introduces four > > new CV types and, to accommodate them, a new VCCV Extended CV > > parameter for PW Interface Parameters Sub-TLV is defined. > > " > > > > > > Best Regards, > > Meral > > --- > > Meral Shirazipour > > Ericsson Research > > www.ericsson.com _______________________________________________ Gen-art mailing list [email protected] https://www.ietf.org/mailman/listinfo/gen-art
