[netconf] Éric Vyncke's No Objection on draft-ietf-netconf-subscribed-notifications-24: (with COMMENT)

Éric Vyncke via Datatracker <noreply@ietf.org> Tue, 30 April 2019 19:00 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: netconf@ietf.org
Delivered-To: netconf@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id A0772120341; Tue, 30 Apr 2019 12:00:11 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: =?utf-8?q?=C3=89ric_Vyncke_via_Datatracker?= <noreply@ietf.org>
To: "The IESG" <iesg@ietf.org>
Cc: draft-ietf-netconf-subscribed-notifications@ietf.org, Kent Watsen <kent+ietf@watsen.net>, netconf-chairs@ietf.org, kent+ietf@watsen.net, netconf@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.95.1
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: =?utf-8?q?=C3=89ric_Vyncke?= <evyncke@cisco.com>
Message-ID: <155665081164.7668.3304106941009307050.idtracker@ietfa.amsl.com>
Date: Tue, 30 Apr 2019 12:00:11 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/GYewpFsAMNZhwRyL_5Nqm9yjkCw>
Subject: [netconf] =?utf-8?q?=C3=89ric_Vyncke=27s_No_Objection_on_draft-i?= =?utf-8?q?etf-netconf-subscribed-notifications-24=3A_=28with_COMMENT=29?=
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.29
List-Id: NETCONF WG list <netconf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netconf>, <mailto:netconf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netconf/>
List-Post: <mailto:netconf@ietf.org>
List-Help: <mailto:netconf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netconf>, <mailto:netconf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 30 Apr 2019 19:00:17 -0000

Éric Vyncke has entered the following ballot position for
draft-ietf-netconf-subscribed-notifications-24: No Objection

When responding, please keep the subject line intact and reply to all
email addresses included in the To and CC lines. (Feel free to cut this
introductory paragraph, however.)


Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-netconf-subscribed-notifications/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

I appreciate the goal of this document to specify another telemetry streaming
than gRPC. As the I-D has been reviewed by YANG-doctors, I did not look in the
YANG module.

Comments
--------

C1) section 1.3, should the published check authorization before accepting an
subscription? There is some text in section 3.1 is about authorization but I
would prefer to have this authorization stated as early as possible

C2) end of section 1.3, "transport drafts" shouldn't rather be "transport
specifications" ?

C3) end of section 1.3, upon termination decided by the publisher, is there a
need for any explanation message sent to the subscriber?

C4) is there any reason why the YANG module validation but the datatracker
fails? Outdated/invalid PYANG ?

C5) section 2.2 "all event records on an event stream are to be sent", should
there be a mention of publisher being out of ressource ?

C6) section 2.4.1 "insufficient CPU or bandwidth available" but there may be
other reasons (e.g. memory), what about using "insufficient CPU, bandwidth
unavailable or other lack of ressource"

C7) for my curiosity, in section 2.4.2.1, how deep could realistically be a
replay buffer? Minutes?

C8) the term 'transport' is used quite often in the document but it seems to
refer to NETCONF and not so much to my understanding of 'transport' in an IETF
document (which is TCP, UDP, SCTP, ...). Is it obvious to the readers? If so,
then I do not mind. Else, it may be useful to clarify in section 1.2

Nits
----

N1) is there any reason why not all Cisco authors are not grouped together?
(even if another one has changed affiliation)

N2) abstract s/information/data/ also applicable in other sections IMHO

N3) section 1.3, expand RPC even if obvious

N4) section 2.3, expand QoS even if obvious