[netconf] Comments on draft-ietf-netconf-netconf-event-notifications-16

[Rohit R Ranade <[e-mail]>]

Hi All,

Editorial:

1.       "resynch-subscription" is used in 4 places with an additional 'h' after sync. It is "resync-subscription" in draft-ietf-netconf-yang-push-22

2.       Section 7  bullet 3, "this MAY but does not have to be included", can be replaced with just "optional" ?

3.       Section 7, bullet 4,  "section 2.4.6" link looks to be broken.

4.       Section 5 refers to many RPCs, but there is no reference to the documents which define these RPCs.

5.       Since there are many RPC names used, there are too many hyperlinks pointing to their sources in the document. Whether we can have a separate section which provides the list of RPCs used in this draft and their source document. The increase in hyperlinks leads to distraction while reading.


Technical:

1.       Section 3 "reply with the [RFC6241] error "operation-not-supported"". I think this should be an "<rpc-error> element is returned with an <error-tag> value of "operation-not-supported". Similar occurrence in another place in same section.

2.       Is the intention that RFC5277 be implemented and then this draft be supported on top of it ? I see that there are 7 references to RFC5277 , including definitions of <notification> and its encoding mechanism. Whether we can add a statement regarding this point ?

With Regards,
Rohit R