[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

RE: mib-doctor review of draft-ietf-ipsp-spd-mib



One thing that immediately caught my eye when quickly browsing this 
response was:


>    4.  The following is a specific example of the notation used in the
>        following section's examples of this MIB's usage.  In order to
>        set the address status column to deprecated for a row in the
>        IP-MIB::ipAddressTable with the index values of ipAddressAddrType
>        = ipv4 and ipAddressAddr = = 10.0.0.1.  The example notation
>        would look like the following:
> 
>        ipAddressEntry(ipAddressAddrType = 1,           -- ipv4
>                       ipAddressAddr     = 0x0a000001 ) -- 10.0.0.1
>                    = (ipAddressStatus   = 2)           -- deprecated
> 
Pls, when you want to use example IPv4 addresses, stick to and use the ones 
assigned/reserved for that purpuse as per RFC3330:

   192.0.2.0/24 - This block is assigned as "TEST-NET" for use in
   documentation and example code.  It is often used in conjunction with
   domain names example.com or example.net in vendor and protocol
   documentation.  Addresses within this block should not appear on the
   public Internet.

And if you are going to do a revision (which is what it looks like), then
the following may be considered too:

IDNITs (as available at: http://ietf.levkowetz.com/tools/idnits/)
results looks reasonably OK:
--------
$ idnits draft-ietf-ipsp-spd-mib-01.txt
idnits 1.58

draft-ietf-ipsp-spd-mib-01.txt:

  Checking nits according to http://www.ietf.org/ID-Checklist.html :

  * The document seems to lack an IANA Considerations section.
    Checking conformance with RFC 3667/3668 boilerplate...
    the boilerplate looks good.

  Checking nits according to http://www.ietf.org/ietf/1id-guidelines.txt :

  - The page length should not exceed 58 lines per page, but there was 65 longer pages, the
    longest (page 63) being 70 lines
  - It seems as if not all pages are separated by form feeds - found 0 form feeds but 65 pages

  Miscellaneous warnings:

    None.

    Run idnits with the --verbose option for more detailed information.
----

reference checking:
---------
!! Missing citation for Normative reference:
  P062 L040:    [RFC3411]  Harrington, D., Presuhn, R. and B. Wijnen, "An

!! Missing citation for Normative reference:
  P062 L046:    [RFC3412]  Case, J., Harrington, D., Presuhn, R. and B. Wijnen,

!! Missing citation for Normative reference:
  P062 L052:    [RFC3413]  Levi, D., Meyer, P. and B. Stewart, "Simple Network

!! Missing citation for Normative reference:
  P062 L057:    [RFC3414]  Blumenthal, U. and B. Wijnen, "User-based Security Model

!! Missing citation for Normative reference:
  P062 L062:    [RFC3415]  Wijnen, B., Presuhn, R. and K. McCloghrie, "View-based
------------

Bert