Re: [lisp] WGLC for draft-ietf-lisp-mib-05

[Gregg Schudel <[e-mail]>]

Thank you for the comments Dan

On 9/11/12 5:59 AM, Romascanu, Dan (Dan) wrote:
> Hi,
>
> There are a few issues that need to be corrected and clarified before
> submitting this document to the IESG. There may be more, but I did not
> have the time to perform a full MIB Doctor review, this is what I found
> at a first pass reading of the document.
>
> - Runing smilint indicates a number of problems - most derive from the
> fact that indices of many table are of the SYNTAX of Integer32 with no
> range restriction

thanks - fixed. (added ranges)

> - LispAddressType TC is defined as a four-tuple with several variants -
> how can it have 0 as its lower size?

thanks - fixed.

> - Having a REFERENCE clause that says "[LISP]" is not useful. People may
> use the MIB module but not the RFC, and even if they know what the
> reference is just one high level reference is useless. To be useful to
> implementers REFERENCE clauses of specific objects must point to the
> exact paragraphs that define the origin of the MIB definition in the
> LIPS RFC

thanks - fixed. added pertinent references w/ sections.

> - Some of the Integer objects have obviously a range, but this is not
> specified - for example lispMapCacheLocatorRlocPriority or
> lispMapCacheLocatorRlocWeight

thanks - fixed. added ranges.

> - The indexation of lispIidToVrfTable seems broken - if the value of
> VPNIdOrZero is zero because a VPN ID could not be determined, how are to
> rows in such situation distinguished?

thanks - fixed. added index.

> - There is no discontinuity indicator for the (many) counter objects in
> the MIB module. See section 4.6.1.2 in RFC 4181 for some advice on this
> issue

thanks - fixing...

> - IANA Considerations - IANA must allocate a branch under mib-2 for this
> MIB module (marked xxx in the MIB module)

thanks - yes, we had this on the "to do" list. will get this going.

>
> Regards,
>
> Dan

cheers
gregg

-- 
--------------------------------------------------------------------
  .:|:.:|:.  | gregg schudel (ccie#9591) LISP technical mrkting engr
    cisco    | mobile: [phone]   email: [e-mail]
--------------------------------------------------------------------
cisco corporate legal statement:
http://www.cisco.com/web/about/doing_business/legal/cri/index.html
--------------------------------------------------------------------