draft-ietf-magma-mgmd-mib-08 comments

Greetings.

While implementing this MIB draft, we found a few bugs.

This draft is past the WG stage and with the IESG, but it looks like it needs a revision before it can be published.

So I hope these comments help when you come to work on that revision.

Regards,
David McWalter
[email protected]


1) mgmdRouterInterfaceQuerier - should have size constraint (SIZE(4|16)) as for mgmdRouterCacheAddress.  We assume no ipv4z/ipv6z/dns support is intended.
gmdRouterCacheLastReporter - ditto

2) mgmdRouterInterfaceJoins - has MAX-ACCESS read-only, so the default is not needed.
mgmdRouterInterfaceGroups - ditto
mgmdRouterInterfaceLastMembQueryCount - ditto
mgmdRouterInterfaceStartupQueryInterval - ditto
mgmdRouterInterfaceStartupQueryCount - ditto

3) mgmdInverseRouterCacheTable - name is not consistent with other tables in the draft, would be better called
mgmdRouterInverseCacheTable, and ditto for all fields in this table.

4) mgmdInverseRouterCacheEntry - this extends the mgmdRouterInterfaceTable, so it should use the index objects from that (mgmdRouterInterfaceIfIndex, mgmdRouterInterfaceQuerierType) not define new new ones (mgmdInverseRouterCacheIfIndex, mgmdInverseRouterCacheAddressType).
mgmdRouterSrcListEntry - this extends the mgmdRouterCacheTable, so again should use index fields from that where possible.  i.e.  mgmdRouterCacheAddressType, mgmdRouterCacheAddress, and mgmdRouterCacheIfIndex instead of mgmdRouterSrcListAddressType, mgmdRouterSrcListAddress, mgmdRouterSrcListIfIndex.

5) mgmdRouterSrcListHostAddress - this object is an index, and there is another columnar object in the table, so it should have MAX-ACCESS not-accessible, rather than read-only.