Re: [Gen-art] Gen-ART Review of draft-ietf-l2vpn-vpls-pe-etree-10

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

 



Many thanks for your detailed review, Russ!

Yuanlong, will there be a new draft version or other edits before the draft
is approved?

Jari

On 17 Nov 2015, at 00:53, Jiangyuanlong <jiangyuanlong@xxxxxxxxxx> wrote:

> Russ,
> 
> Thanks a lot for your review and good observations.
> Please see my comments in line.
> 
> Best regards,
> Yuanlong
> 
>> -----Original Message-----
>> From: Russ Housley [mailto:housley@xxxxxxxxxxxx]
>> Sent: Saturday, November 14, 2015 1:28 AM
>> To: draft-ietf-l2vpn-vpls-pe-etree.all@xxxxxxxx
>> Cc: IETF Gen-ART; IETF
>> Subject: Gen-ART Review of draft-ietf-l2vpn-vpls-pe-etree-10
>> 
>> 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 wait for direction from your document shepherd or AD
>> before posting a new version of the draft.
>> 
>> For more information, please see the FAQ at
>> <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
>> 
>> Document: draft-ietf-l2vpn-vpls-pe-etree-10
>> Reviewer: Russ Housley
>> Review Date: 2015-11-13
>> IETF LC End Date: 2015-11-24
>> IESG Telechat date: unknown
>> 
>> Summary:  Almost Ready
>> 
>> 
>> Major Concerns: None
>> 
>> 
>> Minor Concerns:
>> 
>> Sections 4.1 and 5.3.1 inclues a reference to [802.1Q-2011].  Should this be
>> the 2014 version of the document?  If not, please add the informative
>> reference for [802.1Q-2011].
>> 
> [YJ] Yes, all these should refer to [802.1Q-2014].
> 
>> 
>> Other Editorial Comments:
>> 
>> The Abstract should appear on the title page.
>> 
> [YJ] OK.
> 
>> Section 3 needs a reference for MEF 6.1:
>> s/Specification MEF 6.1/Specification MEF 6.1 [MEF6.1]/ Also, [MEF6.1]
>> needs to be added as a normative reference.
>> 
> [YJ] All these will be updated to MEF 6.2.
> 
>> Section 3 needs a reference for IEEE 802.1Q-2003:
>> s/B.1.3 of IEEE 802.1Q-2003/B.1.3 of IEEE 802.1Q-2003 [802.1Q-2003]/ Also,
>> please add an informative reference for [802.1Q-2003].
>> 
> [YJ] Yes, we will add an informative reference for [802.1Q-2003].
> 
>> Third level section headings do not have space between the section number
>> and the section title.  For example:
>> s/5.3.1.PW Processing/5.3.1. PW Processing/
>> 
> [YJ] OK.
> 
>> In Fig 4, there is room to shift the figure to the right, this will allow the "AC"
>> labels to fit better on the left:
>> 
>>              +----------------------------------------+
>>              |           VPLS-capable PE model        |
>>              |   +---------------+          +------+  |
>>              |   |               |==========|TVSI-1|------------
>>   +---+  AC  |   |               ------------      |------------ PWs
>>   |CE |----------|     Bridge    ------------      |------------
>>   +---+      |   |               | Root &   +------+  |
>>              |   |     Module    | Leaf VLAN   o      |
>>              |   |               |             o      |
>>              |   |               |             o      |
>>              |   |               |             o      |
>>              |   |               |             o      |
>>   +---+  AC  |   |               |   VLAN-n +------+  |
>>   |CE |----------|               ------------VSI-n |-------------
>>   +---+      |   |               |==========|      |------------- PWs
>>              |   |               |     ^    |      |-------------
>>              |   +---------------+     |    +------+  |
>>              |                         |              |
>>              +-------------------------|--------------+
>>                            LAN emulation Interface
>> 
>>          Figure 4  A VPLS PE Model for E-Tree with a Single T-VSI
> 
> 
> [YJ] OK.
> 
> _______________________________________________
> Gen-art mailing list
> Gen-art@xxxxxxxx
> https://www.ietf.org/mailman/listinfo/gen-art

Attachment: signature.asc
Description: Message signed with OpenPGP using GPGMail


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