[yang-doctors] Yangdoctors early review of draft-ietf-netconf-yang-push-06
Bert Wijnen <bwietf@bwijnen.net> Tue, 16 May 2017 12:37 UTC
Return-Path: <bwietf@bwijnen.net>
X-Original-To: yang-doctors@ietf.org
Delivered-To: yang-doctors@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 7818412EB41; Tue, 16 May 2017 05:37:51 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Bert Wijnen <bwietf@bwijnen.net>
To: yang-doctors@ietf.org
Cc: draft-ietf-netconf-yang-push.all@ietf.org, ietf@ietf.org, netconf@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.50.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <149493827145.11944.9758454721784881844@ietfa.amsl.com>
Date: Tue, 16 May 2017 05:37:51 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/yang-doctors/PKnkH2R9gMcwy06x61pHjj9Unww>
Subject: [yang-doctors] Yangdoctors early review of draft-ietf-netconf-yang-push-06
X-BeenThere: yang-doctors@ietf.org
X-Mailman-Version: 2.1.22
List-Id: email list of the yang-doctors directorate <yang-doctors.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/yang-doctors/>
List-Post: <mailto:yang-doctors@ietf.org>
List-Help: <mailto:yang-doctors-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/yang-doctors>, <mailto:yang-doctors-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 16 May 2017 12:37:52 -0000
Reviewer: Bert Wijnen Review result: On the Right Track - last para of sect 3.5. This seems to me to make it difficult to create interoperable implementations. Or is there a way for a client to figure out what is or is not support, other than tryal and error? - page 41: /* YANG Parser Pyang crashing on the following syntax below So does the definition get skipped? Or what needs to happen here? Consistency - last bullet on page 7 talks about "YANG subtrees". I do not see that term in netconf or yang documents. Those just talk about "subtrees". Maybe I am not looking good enough? - top of page 8 I see the words "xpath", "Xpath" and "XPath" is there a difference? Nits - you may want to check the reference/citation occurrences of [subscribe] at several places it points to draft-ietf-netconf-yang-push-06#ref-subscribe whereas I think it intends to point to the [subscribe] in the normative references section - first bullet on page 5: Enhancements to filters. Specifically the filter MUST at identify at least one targeted yang s/at// -- the first "at" seems superfluous plus, you are using capitalized MUST with out reference/citation of RFC2119 - page 36: leaf dependency { type sn:subscription-id; description "Provides the Subscription ID of a parent subscription which has absolute priority should that parent have push updates ready to egress the publisher. In other words, there should be no streaming of objects from the current subscription if of the parent has something ready to push."; reference "RFC-7540, section 5.3.1"; } s/if of/if/ ??
- [yang-doctors] Yangdoctors early review of draft-… Bert Wijnen
- Re: [yang-doctors] Yangdoctors early review of dr… Mehmet Ersue
- Re: [yang-doctors] Yangdoctors early review of dr… Eric Voit (evoit)
- Re: [yang-doctors] Yangdoctors early review of dr… Andy Bierman
- Re: [yang-doctors] Yangdoctors early review of dr… Eric Voit (evoit)