Re: Gen-ART review of draft-ietf-pwe3-mpls-eth-oam-iwk-06

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Dave,
Sorry for a late reply addressing your comments. Please, see inline.

Thanks,
Nabil

From: "Black, David" <david.black@xxxxxxx>
Date: Monday, August 20, 2012 8:57 PM
To: "dinmohan@xxxxxxxxxxx" <dinmohan@xxxxxxxxxxx>, "Bitar, Nabil N" <nabil.n.bitar@xxxxxxxxxxxxxxx>, Sagessi <sajassi@xxxxxxxxx>, "gen-art@xxxxxxxx" <gen-art@xxxxxxxx>
Cc: "Black, David" <david.black@xxxxxxx>, "pwe3@xxxxxxxx" <pwe3@xxxxxxxx>, "ietf@xxxxxxxx" <ietf@xxxxxxxx>
Subject: Gen-ART review of draft-ietf-pwe3-mpls-eth-oam-iwk-06

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-eth-oam-iwk-06
Reviewer: David L. Black
Review Date: August 20, 2012
IETF LC End Date: August 20, 2012

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

<NB> Thanks.

 

This draft covers defect behavior for Ethernet pseudowires,

including defect state mapping and PE defect reporting behavior.

The draft is generally in good shape.  I found a few minor nits.

 

1) The draft uses a lot of acronyms - while each acronym appears to

be expanded on first use, an additional section near the start of the

draft listing all of them would be helpful.


<NB> Done.

 

2) There's a typo in the first paragraph of section 2:

 

     covers the following Ethernet OAM (Opertaions, Administration and

 

Opertaions -> Operations.


<NB> Thanks. Done.

 

3) The following normative reference is incomplete - please add additional

information that will enable a reader to locate the referenced document:

 

     [MEF16] "Ethernet Local Management Interface", MEF16, January 2006.


[MEF16] "Ethernet Local Management Interface", Metro Ethernet Forum Technical Specification MEF16, January 2006.

<NB> changed it to : 

 

4) idnits 2.12.13 did not like the pagination:

 

  == The page length should not exceed 58 lines per page, but there was 22
     longer pages, the longest (page 1) being 61 lines

<NB> That will be fixed.
 
Thanks,
--David
----------------------------------------------------
David L. Black, Distinguished Engineer
EMC Corporation, 176 South St., Hopkinton, MA  01748
+1 (508) 293-7953             FAX: +1 (508) 293-7786
david.black@xxxxxxx        Mobile: +1 (978) 394-7754
----------------------------------------------------

[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Fedora Users]