Draft: draft-ietf-isis-admin-tags-03.txt Reviewer: Suresh Krishnan [suresh.krishnan@ericsson.com] Review Date: 3/28/2007 IETF LC Date: 3/12/2007 Summary: This draft is on the right track but has open issues, described in the review. These need to be fixed before the document moves forward. Comments: Substantial =========== * The IANA considerations is incredibly vague. It does not specify what registry the draft is seeking allocation out of. Also the draft can only specify suggested values. It cannot mandate IANA to assign specific values. "The authors have chosen "1" as the type code of the 32-bits Administrative Tag Sub-TLV and "2" as the type code of the 64-bits Administrative Tag Sub-TLV. These values must be allocated by IANA." * The example IP addresses used in the document do not come from the RFC3330 Test-net block (i.e. 192.0.2.0/24) Minor ===== * The IPR boilerplate is a mix of boilerplate from RFC2026 and RFC3668. This needs to be switched to a consistent boilerplate (RFC3978 perhaps) * Unused references [1],[4]. * Downrefs to Informational RFCs [3],[5] * Newer version of draft-ietf-isis-ipv6 available Editorial ========= * Lots of long lines and pages * Whole lot of nits in the document. Please run a nits checker