[netconf] mbj review of draft-ietf-netconf-notification-capabilities-01
Martin Bjorklund <mbj@tail-f.com> Mon, 25 March 2019 20:56 UTC
Return-Path: <mbj@tail-f.com>
X-Original-To: netconf@ietfa.amsl.com
Delivered-To: netconf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1])
by ietfa.amsl.com (Postfix) with ESMTP id A2A2212085C
for <netconf@ietfa.amsl.com>; Mon, 25 Mar 2019 13:56:42 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5
tests=[BAYES_00=-1.9, SPF_PASS=-0.001]
autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44])
by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024)
with ESMTP id l9q7X6MgyZ9Y for <netconf@ietfa.amsl.com>;
Mon, 25 Mar 2019 13:56:40 -0700 (PDT)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45])
by ietfa.amsl.com (Postfix) with ESMTP id 5AB99120863
for <netconf@ietf.org>; Mon, 25 Mar 2019 13:56:39 -0700 (PDT)
Received: from localhost (dhcp-9545.meeting.ietf.org [31.133.149.69])
by mail.tail-f.com (Postfix) with ESMTPSA id 725441AE00A0
for <netconf@ietf.org>; Mon, 25 Mar 2019 21:56:37 +0100 (CET)
Date: Mon, 25 Mar 2019 21:56:37 +0100 (CET)
Message-Id: <20190325.215637.1342845104682793774.mbj@tail-f.com>
To: netconf@ietf.org
From: Martin Bjorklund <mbj@tail-f.com>
X-Mailer: Mew version 6.7 on Emacs 24.5 / Mule 6.0 (HANACHIRUSATO)
Mime-Version: 1.0
Content-Type: Text/Plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/Br1wtmcRy12M6nuF2f8s_Phzkao>
Subject: [netconf] mbj review of
draft-ietf-netconf-notification-capabilities-01
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
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: Mon, 25 Mar 2019 20:56:48 -0000
Hi, Here are my (mostly minor) comments on draft-ietf-netconf-notification-capabilities-01 o Terminology 1. The term that is used in th YP draft is "YANG-Push". Please import that and use it consistently. (I see at least "YangPush" and "Yang-Push"). 2. This document uses the terms "YANG server" and "Yang server". I suggest you change this to "server", and import that from RFC 8342. o Abstract The abstract mentions YANG Instance Data, but it doesn't mention run-time. I think it should. o Section 1 s/Rutime/Runtime/ o Section 1 s/protocol/management protocol/ Should you really list HTTPS here? + add references to NETCONF and RESTCONF. o Section 3 Editorial: remove the text within the parentheses. o General We need to be able to specify on-change support per datastore. With support per datastore, it is not quite clear if notification-sent-for-config-default and notification-sent-for-state-default are needed, and/or how they should be interpreted. o General In YP, support for on-change is optional. There is a feature called "on-change". I think this document should state explicitly that this module is intended for servers that implement that feature. o Section 3.2 Editorial: We no longer list the WG Chairs in the description. Since you use 2119 language in the model, please include the boilerplate in the module description. The module description doesn't contain the proper copyright text, The module should be formatted similar to the rest of the IETF-published modules. I suggest you run pyang -f yang --keep-comments --yang-line-length 69 <FILE> and then ensure that any additions are consistent with that. o Section 3.2 The top-level container is currently called "yangpush-notification-capabilities". This doesn't really match the naming in the YP document. I suggest it is called "datastore-subscription-capabilities", so that we use a word that can be related to the the nodes YP defines, such as in "establish-subscription" ... "datastore" "datastore-subtree-filter" o Section 3.2 You have: units msec; units centiseconds; I think you probably should do s/msec/milliseconds/ o Section 3.2 minimum-dampening-period is optional. It should be stated what it means if this leaf is not present. (or should the default be 0?) o Section 3.2 The choice update-period is not mandatory, and doesn't have a default. What does it mean if this is not reported? o Section 3.2 The leaf max-objects-per-update is not mandatory. What does it mean if this is not reported? It can have the value 0. What does that mean? Perhaps make a union of uint32 0..max and the enum "infinite" and make that the default? (of course "infinite" is not really correct...) o General I missed some examples, perhaps in an Appendix. /martin
- [netconf] mbj review of draft-ietf-netconf-notifi… Martin Bjorklund
- Re: [netconf] mbj review of draft-ietf-netconf-no… Balázs Lengyel
- Re: [netconf] mbj review of draft-ietf-netconf-no… Martin Bjorklund
- Re: [netconf] mbj review of draft-ietf-netconf-no… Eric Voit (evoit)
- Re: [netconf] mbj review of draft-ietf-netconf-no… Martin Bjorklund