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

["Reshad Rahman (rrahman)" <[e-mail]>]

Hi,

I believe the document is ready to progress. I agree with the comments from Balazs on Sections 3 and 5, except that I think keeping the past paragraph in section 3 is worthwhile since it's a clarification.
All my comments are nits:
- Section 4 last sentence, s/in that draft/in that document/
- Figure 8 s/with Hint/with hint/
- A.3 Closing ) not needed?
- A.3.1 s/might be sent if over NETCONF if/might be sent over NETCONF if/

Regards,
Reshad.

On 2018-10-05, 10:13 AM, "Netconf on behalf of Eric Voit (evoit)" <[e-mail] on behalf of [e-mail]> wrote:

    Thanks Balazs.   Requested tweaks in Ch3 & Ch5 made.
    
    Eric
    
    > From Balázs Lengyel, October 5, 2018 8:17 AM
    > 
    > Hello,
    > 
    > I have reviewed the draft and I think it is ready to progress forward.
    > 
    > Some comments minor:
    > 
    > Generally I find the draft a bit too long and too detailed. Much of this can be
    > derived from the yang language, subscribed-notifications and yang-push.
    > Still I  can accept it; it is OK.
    > 
    > Ch 3)
    > 
    > s/is been received/has been received/
    > Last paragraph) true, but to me this seems trivial. If you don't support
    > something, don't advertise it. Still this does not harm anyone.
    > 
    > CH 5)
    > 
    > s/if the NETCONF session involved with the "establish-subscription"
    > terminates,/ if the NETCONF session that carried the "establish-subscription"
    > terminates/ s/RPC MUST be sent using same the NETCONF session/RPC MUST
    > be sent using the same NETCONF session/
    > 
    > regards Balazs
    > 
    > On 10/5/2018 12:10 AM, Kent Watsen wrote:
    > > This message starts a Last Call on draft-ietf-netconf-netconf-event-
    > notifications-13 (a.k.a. "netconf-notif"):
    > >
    > >
    > > https://tools.ietf.org/html/draft-ietf-netconf-netconf-event-notificat
    > > ions-13
    > >
    > > This, along with the restconf-notif Last Call, started in a separate
    > > message, marks the last of the last calls on the yang push suite of
    > > drafts. As explained before, given the size and number of documents,
    > > the chairs decided to break the reviews up into pieces so as to get
    > > focus on each in turn.  We chose to go top-down, starting with
    > > yang-push and ending with the "notif" drafts.  We plan to submit the
    > > drafts for publication when they are ready as a collective.
    > > The goal is to do all this prior to IETF 103.
    > >
    > > While it's always nice to receive messages of support, at this time,
    > > the question isn't so much if the working group supports the work than
    > > if the document is ready to progress.  The chairs need to see reviews
    > > that indicate thorough end-to-end reading of the text.  Of course, if
    > > there are any objections, these should be brought forward now as well.
    > >
    > > The current version (-13) of this draft was published on Oct 4th
    > > (today).  The datatracker page for the draft is here:
    > >
    > >    https://datatracker.ietf.org/doc/draft-ietf-netconf-netconf-event-
    > notifications.
    > >
    > >
    > > Thanks,
    > > Kent (and 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]
    > 
    
    _______________________________________________
    Netconf mailing list
    [e-mail]
    https://www.ietf.org/mailman/listinfo/netconf