[ANCP] comments on draft-ietf-ancp-mib-an-10

["Wojciech Dec (wdec)" <[e-mail]>]

Hi All,

Here's my review/feedback regarding the draft-ietf-ancp-mib-an-10 draft:

Section 4.
While it may be obvious, it likely would be useful to add that the ANCP
Server MIB document is not scoped by this draft. Suggested text could be
something like:

"This document specifies a MIB module an ANCP client, such as an Access
Node (AN). An ANCP server, such as a NAS, would be typically expected to
implement an ANCP server MIB that is not covered by this document."

Section 5.4 (but applicable to whole document as it pops up throughout)
It would be preferable to recast sentences such as
 "This table is used to configure ANCP sessions at the AN towards a
      specific NAS"

To:
This table is used to configure ANCP sessions at the ANCP client towards a
      specific ANCP server.


Section 5.4
Text says: "Each session configured in ancpAnSessionConfigTable has a
corresponding row in ancpAnCurrentSessionTable."

As currently defined there seems to be little difference between the
ancpAnSessionConfigTable and the ancpAnCurrentSessionTable.
Shouldn't the intent be that only active sessions, makred as such in the
ancpAnSessionConfigTable, be represented in the ancpAnCurrentSessionTable?

Section 7.

AncpSessionCapabilities

The capabilities list does not reflect the capabilities we have in the
defined in other ANCP specs.
Missing are:
- Multicast 
(http://tools.ietf.org/html/draft-ietf-ancp-mc-extensions-08#section-6)
- PON (TBD)

The names of the capabilities shown should be aligned to what we have in:
http://tools.ietf.org/html/rfc6320#section-10.7

ancpAnSessionConfigNasIpAddressType and ancpAnCurrentSessionAnIpAddressType

Shouldn't the syntax also detail the allowed values for say
InetAddressType? Eg
SYNTAX     InetAddressType { ipv4(1), ipv6(2) }


ancpAnSessionConfigNasIpAddress and ancpAnCurrentSessionAnIpAddress

Shouldn't the syntax also indicate the size? Eg
SYNTAX InetAddress ( SIZE (4|16) )


Why isn't there an ancpANSessionConfigIPAddress* ? If the assumption is
that this address becomes derived only from the IF-MIB, and the assignment
of interfaces to the ANCP session, then it would be helpful to explain
that to the reader.

Regards,
Woj..