Skip to main content

Last Call Review of draft-ietf-homenet-babel-profile-06
review-ietf-homenet-babel-profile-06-opsdir-lc-chown-2018-02-26-00

Request Review of draft-ietf-homenet-babel-profile
Requested revision No specific revision (document currently at 07)
Type Last Call Review
Team Ops Directorate (opsdir)
Deadline 2018-02-26
Requested 2018-02-12
Authors Juliusz Chroboczek
I-D last updated 2018-02-26
Completed reviews Secdir Early review of -02 by Leif Johansson (diff)
Rtgdir Telechat review of -06 by Ron Bonica (diff)
Opsdir Last Call review of -06 by Tim Chown (diff)
Genart Last Call review of -05 by Stewart Bryant (diff)
Assignment Reviewer Tim Chown
State Completed
Request Last Call review on draft-ietf-homenet-babel-profile by Ops Directorate Assigned
Reviewed revision 06 (document currently at 07)
Result Has nits
Completed 2018-02-26
review-ietf-homenet-babel-profile-06-opsdir-lc-chown-2018-02-26-00
Hi,

I have reviewed this document 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 draft specifies the subset of the Babel protocol and its extensions that
is required by an implementation of the Homenet protocol suite, as well as the
interactions between Babel and HNCP.

This draft reads well, is clearly written, and includes good, concise
rationales for the stated requirements.

The draft is Ready for publication, with minor Nits.

I only have two minor comments:

Abstract:
------------

You say "This document defines the subset of the Babel routing protocol and its
extensions that a Homenet router must implement"

but the REQs are a mix of MUST and SHOULDs, not "musts", so perhaps use the
sentence in paragraph two of Section 1, i.e.:

"This document specifies the exact subset of the Babel protocol and its
extensions that is required by an implementation of the Homenet protocol suite."

Section 2.1
----------------

REQ1: IPv4 also has "link local" addresses, under 169.254.0.0/16, as per RFC
3927, so perhaps here make it clearer why IPv6 link locals make implementations
simpler and more reliable (or why IPv4 link locals do not).  I know what you
mean, but given the document defines IPv4 and IPv6 requirements, a little extra
clarity here might be useful.