Re: [netconf] mbj review of draft-ietf-netconf-notification-capabilities-01
Martin Bjorklund <mbj@tail-f.com> Tue, 26 March 2019 13:01 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 0B94C120043 for <netconf@ietfa.amsl.com>; Tue, 26 Mar 2019 06:01:10 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.9
X-Spam-Level:
X-Spam-Status: No, score=-1.9 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_PASS=-0.001, URIBL_BLOCKED=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 QU9nJVFZ0U4Y for <netconf@ietfa.amsl.com>; Tue, 26 Mar 2019 06:01:07 -0700 (PDT)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id 81986120003 for <netconf@ietf.org>; Tue, 26 Mar 2019 06:01:07 -0700 (PDT)
Received: from localhost (dhcp-97ad.meeting.ietf.org [31.133.151.173]) by mail.tail-f.com (Postfix) with ESMTPSA id 505E01AE08F1; Tue, 26 Mar 2019 14:01:05 +0100 (CET)
Date: Tue, 26 Mar 2019 14:01:02 +0100
Message-Id: <20190326.140102.310852457337101560.mbj@tail-f.com>
To: evoit@cisco.com
Cc: balazs.lengyel@ericsson.com, netconf@ietf.org
From: Martin Bjorklund <mbj@tail-f.com>
In-Reply-To: <cfb2f23fb199437eb714d67ad86e44a1@XCH-RTP-013.cisco.com>
References: <e333e4e3-2c40-2f19-5035-d6d24a0adcab@ericsson.com> <20190326.121231.1546381642017000114.mbj@tail-f.com> <cfb2f23fb199437eb714d67ad86e44a1@XCH-RTP-013.cisco.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/qdKB8_BgICDvDyGUmuyimQPLBB0>
Subject: Re: [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: Tue, 26 Mar 2019 13:01:10 -0000
"Eric Voit (evoit)" <evoit@cisco.com> wrote: > > > 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. > > > > > > BALAZS: OK, but some other capacity related proerties are still valid. > > > > What other capacity related properties do you mean? > > The latest draft allows minimum periodic reporting intervals to be set. Ok. Then perhaps the nodes that are specific to on-change should have a if-feature "yp:on-change". > > > 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?) > > > > > > BALAZS: If not present it means that the system does not tell you what > > > it is. No special meaning. I do not see a reasonable minimum value > > > (eventhough 0 would look strange). IMHO it is the responsibility of > > > the server to provide a meaningful value. If it says zero, that's > > > obviously crazy, just ignore it. I don't thing we need to have rules, > > > like: do not provide stupid state data. > > > > Note that 0 is the default in YP, so I don't know why you think it is crazy. > > > > I suggest you add default "0" to this. > > I agree. > > > > 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? > > > > > > BALAZS: If not present it means that the system does not tell you what > > > it is. No special meaning. > > > > I think this should be clarified in the description. > > > > > 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? > > > > > > BALAZS: If not present it means that the system does not tell you what > > > it is. No special meaning. > > > > > > I do not see a reasonable minimum value (eventhough 0 would look > > > strange). IMHO it is the responsibility of the server to provide a > > > meaningful value. If it says zero, that's obviously crazy, just ignore > > > it. > > > > I suggest you add a range "1..max" to the type. > > Is max necessary? I would assume that when this is not populated, > it is by default max (per below). Currently the type is just a uint32. I think we should limit the range so that 0 is not a valid value. The way to do that in YANG is: type uint32 { range "1..max"; } /martin > > Eric > > > > Perhaps make a union of uint32 0..max and the enum "infinite" and > > > make that the default? (of course "infinite" is not really > > > correct...) > > > > > > BALAZS: IMHO infinite is unreasonable. If the server doesn't know the > > > real limit, it should just not provide a value. > > > > Ok, makes sense. I think this should be clarified in the description. > > > > > > /martin > > > > _______________________________________________ > > netconf mailing list > > netconf@ietf.org > > https://www.ietf.org/mailman/listinfo/netconf >
- [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