RE: [MIB-DOCTORS] MIB-DOCTOR review of draft-ietf-magma-mgmd-mib-10.txt

A few comments (clipped away all the stuff that I agree with or
have no comments on).

> 
> Many objects:
> >   SYNTAX     InetAddressType (SIZE(4|16))
> 
> This is incorrect.  This is an integer, not an octet string, 
> so the SIZE restriction is wrong.  It should instead be:
> >   SYNTAX     InetAddressType { ipv4(1), ipv6(2) }
> if you want to restrict what is legal to IPv4 and IPv6.
> 
> mgmdRouterInterfaceRobustness has a syntax restriction of 
> (1..255) which is fine, but 
> mgmdHostInterfaceVersion3Robustness has none and just has the 
> statement "The variable must be a non-zero value" in the DESCRIPTION.
> It should instead have a range restriction.
> 
> mgmdHostCacheLastReporter's DESCRIPTION says:
> > ... If no membership report has been received, this object 
> has a value of 0.
> 
> Since this is an InetAddress object, shouldn't this say 
> 0.0.0.0 for IPv4 or :: for IPv6?  (And same for 
> mgmdRouterCacheLastReporter)
> 

Or one could allow for 'unknown' as AddressType and zero length string
for Address for the "no memeber report received".
Just mentioning it as an option. What Dave suggests is fine too I think.

Bert