Skip to main content

Last Call Review of draft-ietf-isis-auto-conf-04

Request Review of draft-ietf-isis-auto-conf
Requested revision No specific revision (document currently at 05)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2017-04-10
Requested 2017-03-22
Authors Bing Liu , Les Ginsberg , Bruno Decraene , Ian Farrer , Mikael Abrahamsson
I-D last updated 2017-04-11
Completed reviews Secdir Last Call review of -04 by Radia Perlman (diff)
Opsdir Last Call review of -04 by Will (Shucheng) LIU (diff)
Genart Last Call review of -04 by Robert Sparks (diff)
Assignment Reviewer Will (Shucheng) LIU
State Completed
Request Last Call review on draft-ietf-isis-auto-conf by Ops Directorate Assigned
Reviewed revision 04 (document currently at 05)
Result Ready
Completed 2017-04-11
I have reviewed review of draft-ietf-isis-auto-conf-04 as part of the
Operational directorate's ongoing effort to review all IETF documents being
processed by the IESG.  These comments were written with the intent of
improving the operational aspects of the IETF drafts. Comments that are not
addressed in last call may be included in AD reviews during the IESG review. 
Document editors and WG chairs should treat these comments just like any other
last call comments.

   "This document specifies IS-IS auto-configuration mechanisms.  The key
   components are IS-IS System ID self-generation, duplication detection
   and duplication resolution.  These mechanisms provide limited IS-IS
   functions, and so are suitable for networks where plug-and-play
   configuration is expected."

My overall view of the document is Ready for publication.

I read this draft long ago in its earlier version (probably
draft-liu-isis-auto-conf-06) and happy to see that issues (e.g. the confusing
sentence such as “octets of all 0”) are already fixed.

Some small ones below.

page 3:
> It SHOULD not be changed due to device status change (such as interface >
enable/disable, interface plug in/off, device reboot, firmware update etc.)

The term “due to” is confusing. It might be change to “It SHOULD not be changed
until the device status change”  or “It SHOULD not be changed as the device
status change” according to the meaning.

Page 8
> As specified in this document, there are two distinguisher need to be >
self-generated, which is System ID and Router-Fingerprint.

s/which is/which are

Page 9
>  In a network device, normally there are resources which provide an >
extremely high probability of uniqueness thus could be used as seeds to >
derive distinguisher (e.g. hashing or generating pseudo-random numbers), > such

Suggest to split the sentence to make it more readable.