Telechat Review of draft-ietf-l2vpn-vpls-mib-14
review-ietf-l2vpn-vpls-mib-14-genart-telechat-shirazipour-2014-02-19-00

Request Review of draft-ietf-l2vpn-vpls-mib
Requested rev. no specific revision (document currently at 15)
Type Telechat Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2014-02-14
Requested 2014-02-13
Authors Thomas Nadeau, Kiran Koushik, Rohit Mediratta
Draft last updated 2014-02-19
Completed reviews Genart Last Call review of -14 by Meral Shirazipour (diff)
Genart Telechat review of -14 by Meral Shirazipour (diff)
Secdir Last Call review of -14 by Alexey Melnikov (diff)
Opsdir Last Call review of -14 by Sarah Banks (diff)
Assignment Reviewer Meral Shirazipour
State Completed
Review review-ietf-l2vpn-vpls-mib-14-genart-telechat-shirazipour-2014-02-19
Reviewed rev. 14 (document currently at 15)
Review result Ready with Nits
Review completed: 2014-02-19

Review
review-ietf-l2vpn-vpls-mib-14-genart-telechat-shirazipour-2014-02-19






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 wait for direction from your document shepherd or AD before posting a new version of the draft.




 




Document: draft-ietf-l2vpn-vpls-mib-14




Reviewer: Meral Shirazipour




Review Date: 2013-02-14




IETF LC End Date:  2014-02-14




IESG Telechat date: 2014-02-20




 




Summary:




This draft is ready to be published as Standards Track RFC but I have some editorial comments .




 




 




Nits/editorial comments:




It was noticed that Unsigned32 is used with 16 bit encoding. It would good to mention it in the descriptions like other MIB documents do.




Look for "Unsigned32 (1..65535) in the draft :




-Offline discussion with one of the authors, a suggestion could be "the number of VPLS Edge devices(VEid) is limited to 65535 in any implementation"




 




 




 




Best Regards,




Meral




---




Meral Shirazipour




Ericsson




Research




www.ericsson.com