Re: [Netconf] LC on subscribed-notifications-10 review

[Balazs Lengyel <[e-mail]>]

Hello,

My last call review. Sorry if some of the comments are already mentioned.
I  think the document is in a good shape, ready to progress onwards (if 
my MAJOR comments are clarified.).

Abstract:  "Also defined are delivery mechanisms for instances of the 
resulting notification messages."
Aren't these in other drafts?

P5)
Dynamic subscriptions can only be modified via an RPC request made by 
the original subscriber.
Is it stated/documented that at least for netconf the same session must 
be used?

P7))
"If access control permissions are in use to secure publisher content,
    then for event records to be sent to a receiver, that receiver MUST
    be allowed access to all the event records on the stream."

So if in Yang-push I subscribe to 2 separate objects, but I only have
rights to one of them, I will not get notifications when the other changes?

P10) MAJOR
In the tree it is indicated that establish-subscription has an 
output(subscription-id). This is missing from the Yang Model

P13) It should be stated that delete-subscription MUST always succeed if 
the subscription exists.

P15) second list: transport parameter should be mentioned

P22)
The publisher may decide to terminate a subscription for any other 
reason too (license expiry, configuration switch, internal error, etc.)

P26) MAJOR
The tree indicates everything as readOnly while a lot of this can be 
configured.

P32) Shouldn't the transport identities be defined in the relevant 
transport drafts/RFCs?

P34) There really should be a filter example to show how it is applied 
against the notification.

What happens if multiple event records are carried in one notification 
message. Is that possible? It should be stated explicitly somewhere.
How is the filter applied in this case?

P37) Why is require-instance for leaf stream false?

P39) grouping receiver-info: "Defines where and how to get notification 
messages" IMHO use send instead of get in the first line of the description

P48) "A list of pre-positioned filters"   use pre-configured or 
pre-provisioned

regards Balazs


On 3/1/2018 12:09 AM, Kent Watsen wrote:
> WG,
>
> The authors of netconf-event-notifications have indicated privately that they believe this document is now ready for Last Call.
>
> This starts a 2.5-week Last Call on draft-ietf-netconf-subscribed-notifications-10 [1]
> The LC will end on March 17, or when active threads peter out.
>
> Please send your comments on this thread. Reviews of the document, and statement of support, are particularly helpful to the authors.  If you have concerns about the document, please state those too.
>
> Authors please indicate if you are aware of any IPR on the document.
>
>
> [1] https://tools.ietf.org/html/draft-ietf-netconf-subscribed-notifications-10
>
>
> Kent & Mahesh
>
>
>
>
> _______________________________________________
> Netconf mailing list
> [e-mail]
> https://www.ietf.org/mailman/listinfo/netconf
>

-- 
Balazs Lengyel                       Ericsson Hungary Ltd.
Senior Specialist
Mobile: [phone]              email: [e-mail]