Last Call Review of draft-ietf-netmod-rfc6021-bis-01
review-ietf-netmod-rfc6021-bis-01-genart-lc-halpern-2013-04-20-00

Request Review of draft-ietf-netmod-rfc6021-bis
Requested rev. no specific revision (document currently at 03)
Type Last Call Review
Team General Area Review Team (Gen-ART) (genart)
Deadline 2013-05-01
Requested 2013-04-17
Authors Jürgen Schönwälder
Draft last updated 2013-04-20
Completed reviews Genart Last Call review of -01 by Joel Halpern (diff)
Assignment Reviewer Joel Halpern
State Completed
Review review-ietf-netmod-rfc6021-bis-01-genart-lc-halpern-2013-04-20
Reviewed rev. 01 (document currently at 03)
Review result Almost Ready
Review completed: 2013-04-20

Review
review-ietf-netmod-rfc6021-bis-01-genart-lc-halpern-2013-04-20

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>.

Please resolve these comments along with any other Last Call comments
you may receive.

Document: draft-ietf-netmod-rfc6021-bis-01
    Common YANG Data Types
Reviewer: Joel M. Halpern
Review Date: 19-April-2013
IETF LC End Date: 1-May-2013
IESG Telechat date: N/A



Summary: This document is nearly ready for publication as a Standards 


Track RFC




Major issues:
    (The following may well be a non-issue.)


    In the revision of the ietf-inet-types, the patterns for the new 


ip4-address-no-zone and ipv6-address-no-zone are drastically simplified 


from the ipv4-address and ipv6-address patterns.  The new 


ipv4-address-no-zone allows any sequence of decimal digits an periods, 


while the original was carefully defined as dotted quads of 0..255. 


Similarly, te ipv6-address-no-zone allows any arbitrary sequence of hex 


digits and colons.  The original patterns were very careful to match 


rules for validity.  Is there a reason for the change.




Minor issues:

Nits/editorial comments: