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 treat these comments just like any other last call comments. For more information, please see the FAQ at <http://wiki.tools.ietf.org/area/gen/trac/wiki/GenArtfaq>.
Document: draft-ietf-idr-deprecate-30-31-129-00 Reviewer: Meral Shirazipour Review Date: 2016-12-15 IETF LC End Date: 2016-12-16 IESG Telechat date: NA Summary: This draft is ready to be published as Standards Track RFC but I have some comments. Major issues: Minor issues: -[Page 2]: "This has led to deployment problems for new technologies such as Large BGP Communities [I-D.ietf-idr-large-community]." Not sure if draft-ietf-idr-large-community explains the issues caused by the use of these attribute values ? If yes it would be good to point to that section. If not it would be good to add short summary here. Nits/editorial comments: -[Page 2], Section 4 please refer to latest version of [I-D.ietf-idr-large-community]. -[Page 2]: " The squatting of values 30, 31, 129, 241, 242 and 243 has been confirmed by the involved vendors or through source code review." A bit confusing, somehow it seems 30 is already deprecated as per: https://www.ietf.org/assignments/bgp-parameters/bgp-parameters.xml with reference to [draft-ietf-idr-large-community]. The switch between used of 30 to use of 32 happened between v04 and v05 of [draft-ietf-idr-large-community]? Perhaps https://www.ietf.org/assignments/bgp-parameters/bgp-parameters.xml should be updated to point to this draft instead? Best Regards, Meral --- Meral Shirazipour Ericsson Research www.ericsson.com
_______________________________________________ Gen-art mailing list [email protected] https://www.ietf.org/mailman/listinfo/gen-art
