Skip to main content

Last Call Review of draft-ietf-dhc-dhcpv4-active-leasequery-05
review-ietf-dhc-dhcpv4-active-leasequery-05-genart-lc-holmberg-2015-09-06-00

Request Review of draft-ietf-dhc-dhcpv4-active-leasequery
Requested revision No specific revision (document currently at 07)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2015-09-08
Requested 2015-08-27
Authors Kim Kinnear , Mark Stapp , Bernie Volz , Neil Russell
I-D last updated 2015-09-06
Completed reviews Genart Last Call review of -05 by Christer Holmberg (diff)
Genart Telechat review of -06 by Christer Holmberg (diff)
Secdir Last Call review of -05 by Taylor Yu (diff)
Assignment Reviewer Christer Holmberg
State Completed
Request Last Call review on draft-ietf-dhc-dhcpv4-active-leasequery by General Area Review Team (Gen-ART) Assigned
Reviewed revision 05 (document currently at 07)
Result Ready w/nits
Completed 2015-09-06
review-ietf-dhc-dhcpv4-active-leasequery-05-genart-lc-holmberg-2015-09-06-00

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>

Document:

draft-ietf-dhc-dhcpv4-active-leasequery-05

Reviewer:                                     Christer Holmberg

Review Date:                               6 September 2015

IETF LC End Date:                       8 September 2015

IETF Telechat Date:                   N/A

Summary:         The document is well written, but I have a few comments and
issues I’d like the authors to address.

Major Issues: None

Minor Issues: None

Editorial Issues:



General:

-----------

QGEN_1:

The text says that the document updates

RFC 6926, but it is a little unclear to figure out exactly what is updated.

I think it would be good to have an explicit “Update to RFC 6926” section which
explains exactly which parts are updated.



QGEN_2:

The draft talks about “secure mode” and “insecure mode” in a few places, and
defined different procedures based on which mode is used.

However, there is no generic definition for “secure mode” and “insecure mode”.
I wonder whether it would be useful to add some text somewhere, e.g. to section
 2?



Abstract:

------------



QA_1:



The text says:



“The Dynamic Host Configuration Protocol for IPv4 (DHCPv4) has been

              extended with a Leasequery capability that allows a requestor to

              request information about DHCPv4 bindings.”



Please indicate in which specification (RFC?) this extension has been done.





Section 1 (Introduction):

---------------------------------



Q1_1:



The text says:



              “Requirements exist for external entities to keep up to date on
              the

              correspondence between DHCPv4 clients and their bindings.”



Are these documented requirements, or generic requirements coming from the
industry? Please clarify.





Q1_2:



The text says:



              “This document updates DHCPv4 Bulk Leasequery [RFC6926] in that it

              specifies the DHCPv4 server should close the TCP connection if…”



Is “should” the correct wording? Section 8.4 contains both MAY, SHOULD and MUST
procedures, and I am not quite sure which procedure(s) the text above refers.