Skip to main content

Early Review of draft-ietf-isis-node-admin-tag-08
review-ietf-isis-node-admin-tag-08-rtgdir-early-malis-2016-04-25-00

Request Review of draft-ietf-isis-node-admin-tag
Requested revision No specific revision (document currently at 11)
Type Early Review
Team Routing Area Directorate (rtgdir)
Deadline 2016-04-25
Requested 2016-04-14
Authors Pushpasis Sarkar , Hannes Gredler , Shraddha Hegde , Stephane Litkowski , Bruno Decraene
I-D last updated 2016-04-25
Completed reviews Genart Last Call review of -08 by Peter E. Yee (diff)
Genart Last Call review of -08 by Peter E. Yee (diff)
Secdir Last Call review of -08 by Catherine Meadows (diff)
Opsdir Last Call review of -08 by Jürgen Schönwälder (diff)
Rtgdir Early review of -08 by Andrew G. Malis (diff)
Assignment Reviewer Andrew G. Malis
State Completed
Request Early review on draft-ietf-isis-node-admin-tag by Routing Area Directorate Assigned
Reviewed revision 08 (document currently at 11)
Result Has issues
Completed 2016-04-25
review-ietf-isis-node-admin-tag-08-rtgdir-early-malis-2016-04-25-00
Hello,

I have been selected as the Routing Directorate reviewer for this draft. The
Routing Directorate seeks to review all routing or routing-related drafts as
they pass through IETF last call and IESG review, and sometimes on special
request. The purpose of the review is to provide assistance to the Routing ADs.
For more information about the Routing Directorate, please see ​

http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Although these comments are primarily for the use of the Routing ADs, it would
be helpful if you could consider them along with any other IETF Last Call
comments that you receive, and strive to resolve them through discussion or by
updating the draft.

Document: draft-ietf-isis-node-admin-tag-08.txt

Reviewer: Andy Malis

Review Date: April 20, 2016

IETF LC End Date: April 29, 2016

Intended Status: Standards Track

Summary:

I have some minor concerns about this document that I think should be resolved
before publication, if the AD agrees (see below for details).

Comments and minor concerns:

I have no technical concerns with this draft.

I have noted the two comments in the AD review of this draft, and agree with
them.

Given the similarity in functionality to RFC 7777 and the overlap in
authorship, I expected the draft to be more or less identical to the RFC,
except for the technical differences between OSPF and ISIS. However, there are
parts of the RFC that are editorially better (easier to read or understand)
than the equivalent text in the draft, starting with the title, Abstract, and
Introduction. In particular, the Introduction in the RFC looks like the result
of cleanup by the RFC Editor, but which still needs to be done in the draft.
Why not take advantage of the work already done by the RFC Editor? Also, the
Introduction in the draft doesn't include the usual reference to RFC 2119
terms, which is in the RFC. The Abstract in the RFC also includes more useful
detail than the Abstract in the draft.

As another example, these differences are also true in Section 4.1 of the
draft, when compared to the mostly equivalent Section 2.2.1 of the RFC. For
example, from an editorial standpoint there is a missing "The" in the first
line of the section, and there are other improvements as well. I also see
editorial corrections in Section 3 of the RFC when compared to Section 5 in the
draft.

I would recommend an editorial pass where the text is compared with the RFC,
and when obvious, editorially improved to take advantage of work already done.
This will make the RFC Editor's job easier. Alternatively, the AD could choose
to include a note to the RFC Editor, noting the similarity and asking the RFC
Editor to take advantage of the work that they already did for the RFC.
However, having this done by the document editor would take advantage of the
editor's knowledge of when differences between the two are deliberate.

Thanks,

Andy