Re: [Netconf] WGLC on netconf-event-notifications-13

[Martin Bjorklund <[e-mail]>]

Hi,

Thanks for fixing these issues.  Two comments, trimming:


"Eric Voit (evoit)" <[e-mail]> wrote:
> Thanks very much Martin.   Changes made as described in-line.
> 
> > From: Martin Bjorklund, October 10, 2018 5:15 AM

[...]

> > But this illustrates a problem with SN; the "transport" leaf is
> > mandatory if the feature "configured" is supported.  The problem
> > is that just b/c a server supports the "configured" feature, it
> > doesn't mean that there is an identity defined for all transports;
> > e.g., there is no identity defined for NETCONF.  So the leaf
> > "transport" should probably not be mandatory.
> 
> Made optional within the SN grouping "subscription-policy", and
> tweaked the definition as follows:
> 
>     leaf transport {
>       if-feature "configured";
>       type transport;
>       description
>         "For a configured subscription, this leaf specifies the 
>         transport used to deliver messages destined to all receivers 
>         of that subscription.";
>     }

Is the intention that this leaf is not present in the operational
state for dynamic subscriptions?  If so, I think this should be
explicitly stated.

[...]

> > o  Spelling of XPath
> > 
> >   s/xpath filter/XPath filter/  (a couple of places)
> 
> Found one.  Fixed.

In A.2.2:

   seen, the modifications being attempted are the application of a new
   xpath filter as well as the setting of a new periodic time interval.

In A.4:

   For more examples of xpath filters, see [XPATH].



/martin