Hi Suresh, Thanks for your Gen-Art review. The email you sent finally reached me. I will get back to you with my response soon.
Thanks, Young -----Original Message----- From: Suresh Krishnan [mailto:[email protected]] Sent: Monday, March 02, 2015 3:55 AM To: draft-ietf-ccamp-gmpls-general-constraints-ospf-te....@tools.ietf.org; General Area Review Team Subject: Gen-ART Telechat review of draft-ietf-ccamp-gmpls-general-constraints-ospf-te-09.txt 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-ccamp-gmpls-general-constraints-ospf-te-09.txt Reviewer: Suresh Krishnan Review Date: 2015/03/02 IESG Telechat date: 2015/03/05 Summary: This draft is ready for publication as a Proposed Standard, but I do have some comments you may wish to address. Minor: * Section 5 Isn't separating static and dynamic info of the same TE link another method of scaling? (e.g. encoding the Port Label Restrictions sub-TLV into a separate top level link TLV and advertising in a separate LSA) If so, suggest adding a subsection here to describe it. * Security Considerations: I have an uneasy (potentially unjustified) feeling about the lack of any specific security considerations for the Connectivity Matrix sub-TLV. It looks pretty compute intensive to me. Somebody should probably take a look at the DoS possibilities with a multiple complex decomposed Connectivity Matrices. * IANA Considerations: Looking at the IANA registry, the Sub-TLVs do not have the text sub-TLV tag attached to them. Suggest renaming to OLD: TBD1 | Connectivity Matrix sub-TLV | [This.I-D] NEW: TBD1 | Connectivity Matrix | [This.I-D] OLD: TBD2 | Port Label Restrictions sub-TLV | [This.I-D] NEW: TBD2 | Port Label Restrictions | [This.I-D] Thanks Suresh _______________________________________________ Gen-art mailing list [email protected] https://www.ietf.org/mailman/listinfo/gen-art
