[yang-doctors] Yangdoctors last call review of draft-ietf-netmod-rfc6991-bis-16
Martin Björklund via Datatracker <noreply@ietf.org> Tue, 03 September 2024 14:33 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: yang-doctors@ietf.org
Delivered-To: yang-doctors@ietfa.amsl.com
Received: from [10.244.2.118] (unknown [104.131.183.230]) by ietfa.amsl.com (Postfix) with ESMTP id 8023DC16942E; Tue, 3 Sep 2024 07:33:39 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Martin Björklund via Datatracker <noreply@ietf.org>
To: yang-doctors@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 12.23.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <172537401912.1444563.10355942395544427055@dt-datatracker-68b7b78cf9-q8rsp>
Date: Tue, 03 Sep 2024 07:33:39 -0700
Message-ID-Hash: GPUPPBGQNRFR4BUEP4PW2XW2YLROETAA
X-Message-ID-Hash: GPUPPBGQNRFR4BUEP4PW2XW2YLROETAA
X-MailFrom: noreply@ietf.org
X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-yang-doctors.ietf.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header
CC: draft-ietf-netmod-rfc6991-bis.all@ietf.org, last-call@ietf.org, netmod@ietf.org
X-Mailman-Version: 3.3.9rc4
Reply-To: Martin Björklund <mbj+ietf@4668.se>
Subject: [yang-doctors] Yangdoctors last call review of draft-ietf-netmod-rfc6991-bis-16
List-Id: Email list of the yang-doctors directorate <yang-doctors.ietf.org>
Archived-At: <https://mailarchive.ietf.org/arch/msg/yang-doctors/Z5aMiDX_n52gTbrU1EDTKd1JKRA>
List-Archive: <https://mailarchive.ietf.org/arch/browse/yang-doctors>
List-Help: <mailto:yang-doctors-request@ietf.org?subject=help>
List-Owner: <mailto:yang-doctors-owner@ietf.org>
List-Post: <mailto:yang-doctors@ietf.org>
List-Subscribe: <mailto:yang-doctors-join@ietf.org>
List-Unsubscribe: <mailto:yang-doctors-leave@ietf.org>
Reviewer: Martin Björklund Review result: Ready with Nits Here is my YANG doctor's review of draft-ietf-netmod-rfc6991-bis-16. o typedef email-address The domain part of "email-address" is different from the type "domain-name". This looks a bit odd. If special characters can occur in the domain part of an email address, one would assume that they can occur in a domain name as well. o typedef protocol-number "The protocol-number type represents an 8-bit Internet protocol number, carried in the 'protocol' field of the IPv4 header or in the 'next header' field of the IPv6 header. If IPv6 extension headers are present, then the protocol number type represents the upper layer protocol number, i.e., the number of the last next header' field ^^^ ' missing of the IPv6 extension headers. o typedef ipv6-address-and-prefix "The ipv6-address-and-prefix type represents an IPv6 address and an associated ipv4 prefix. s/ipv4 prefix/IPv6 prefix/ o typedef ipv4-address-and-prefix "The ipv4-address-and-prefix type represents an IPv4 address and an associated ipv4 prefix. s/ipv4 prefix/IPv4 prefix/ o "schema node instance" This term is used in a few places in ietf-yang-types, for example: A schema node instance of this type will be set to zero (0) on creation This isn't correct, since a schema node is a node in the schema tree, and doesn't have a value. With RFC 7950 terminology, it would be "a node in the data tree". It is unfortunate that there is no specific term for this in RFC 7950. Perhaps it would be easier to just write "An instance of this type...". (I know that this was not correct RFC 6991 either) /martin
- [yang-doctors] Yangdoctors last call review of dr… Martin Björklund via Datatracker
- [yang-doctors] Re: Yangdoctors last call review o… Jürgen Schönwälder