Telechat Review of draft-ietf-l2vpn-ipls-15
review-ietf-l2vpn-ipls-15-genart-telechat-krishnan-2014-10-29-00

Request Review of draft-ietf-l2vpn-ipls
Requested rev. no specific revision (document currently at 16)
Type Telechat Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2014-10-28
Requested 2014-10-16
Authors Himanshu Shah, Eric Rosen, Fran├žois Le Faucheur, Giles Heron
Draft last updated 2014-10-29
Completed reviews Genart Last Call review of -14 by Suresh Krishnan (diff)
Genart Telechat review of -15 by Suresh Krishnan (diff)
Secdir Last Call review of -14 by Dacheng Zhang (diff)
Opsdir Last Call review of -14 by Warren Kumari (diff)
Assignment Reviewer Suresh Krishnan
State Completed
Review review-ietf-l2vpn-ipls-15-genart-telechat-krishnan-2014-10-29
Reviewed rev. 15 (document currently at 16)
Review result Ready
Review completed: 2014-10-29

Review
review-ietf-l2vpn-ipls-15-genart-telechat-krishnan-2014-10-29

I have been selected as the General Area Review Team (Gen-ART)
reviewer for this draft (for background on Gen-ART, please see


http://www.alvestrand.no/ietf/gen/art/gen-art-FAQ.html

).

Please wait for direction from your document shepherd
or AD before posting a new version of the draft.

Document: draft-ietf-l2vpn-ipls-15.txt
Reviewer: Suresh Krishnan
Review Date: 2013/10/28
IESG Telechat date: 2013/10/30


Summary: This document is ready for publication as a Historic RFC, but I 
have some comments you may wish to address.

* Section 7.1

The Length field of the IP address TLV in case the IP address is IPv6 is 
not defined. It should be defined to be 18 (2 bytes for address family 
and 16 bytes for the IPv6 address)


* Address family reference

The IANA registry at 


http://www.iana.org/assignments/address-family-numbers/address-family-numbers.xhtml

 
is a much better reference for address families than the currently 
referred RFC3232 that does not contain any of the relevant information.

Thanks
Suresh