Hi Carlos, Thanks for the review. Some thoughts in-line... > From: Carlos Pignataro, April 5, 2019 10:58 PM > > Reviewer: Carlos Pignataro > Review result: Has Nits > > Hi, > > I have reviewed this document as part of the Operational directorate's ongoing > effort to review all IETF documents being processed by the IESG. These > comments were written with the intent of improving the operational aspects of > the IETF drafts. Comments that are not addressed in last call may be included in > AD reviews during the IESG review. Document editors and WG chairs should > treat these comments just like any other last call comments. > > Summary: Almost Ready > > This is a comprehensive very well written document. > > >From an operational point of view, as per the ops-dir review, I have no > concerns or comments. Might be nice to collect some of the operational points > in an Operations Consideration section, particularly given the document > structure of Section 5.x, "ZZZ Considerations". In "implementation considerations" there are a couple which are really operational considerations. For example the first and third bullets might qualify as operational. The hard part is that the operational considerations to expose will vary by the subset of feature which are selected. So I can imagine that much text could be written on best operational practices. I am hoping that such documents could follow this one based on specific deployment contexts. > I do have one important question for consideration, which is: what is *really* > the relationship of this (+ draft-ietf-netconf-netconf-event-notifications > potentially) with RFC 5277? A Normative reference, this doc has no metadata > regarding Updating, Obsoleting, etc. Yet lots of it is indeed a superset of it. We went around and around in the WG on this for a bit. The initial decision was that we were going to obsolete RFC-5277. However the problem was that RFC-5277 Section 4 has a <notification> element defined which we are not replacing yet. This <notification> is used in several important places. Examples include RFC-8040 section 6.4 and RFC-7950 Section 7.16.2. So we didn't want to recommend an obsolete without a replacement of that <notification>. The good news is that we are working on a replacement for this notification. See: draft-ietf-netconf-notification-messages But this still has a while to go before it is ready. As a result we are left with Section 1.4 of draft-ietf-netconf-subscribed-notifications which describes the relationship with RFC-5277. > I recommend the authors+ADs consider whether there is any more formal > relationship with RFC 5277 that would require meta-tagging the RFC. I think that this is fully appropriate as we finish draft-ietf-netconf-notification-messages Thanks! Eric > Thanks, > > Carlos Pignataro.
