[dhcwg] Robert Wilton's No Objection on draft-ietf-dhc-dhcpv6-yang-24: (with COMMENT)

Robert Wilton via Datatracker <noreply@ietf.org> Thu, 16 December 2021 14:48 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: dhcwg@ietf.org
Delivered-To: dhcwg@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 9EEBD3A0FDF; Thu, 16 Dec 2021 06:48:30 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Robert Wilton via Datatracker <noreply@ietf.org>
To: "The IESG" <iesg@ietf.org>
Cc: draft-ietf-dhc-dhcpv6-yang@ietf.org, dhc-chairs@ietf.org, dhcwg@ietf.org, Timothy Winters <tim@qacafe.com>, tim@qacafe.com
X-Test-IDTracker: no
X-IETF-IDTracker: 7.41.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Robert Wilton <rwilton@cisco.com>
Message-ID: <163966610986.326.18125841975372071697@ietfa.amsl.com>
Date: Thu, 16 Dec 2021 06:48:30 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/dhcwg/NOMFCQzFAq_hCmJ-zhozDjBVbGE>
Subject: [dhcwg] Robert Wilton's No Objection on draft-ietf-dhc-dhcpv6-yang-24: (with COMMENT)
X-BeenThere: dhcwg@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Dynamic Host Configuration <dhcwg.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dhcwg>, <mailto:dhcwg-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dhcwg/>
List-Post: <mailto:dhcwg@ietf.org>
List-Help: <mailto:dhcwg-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dhcwg>, <mailto:dhcwg-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 16 Dec 2021 14:48:41 -0000

Robert Wilton has entered the following ballot position for
draft-ietf-dhc-dhcpv6-yang-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/blog/handling-iesg-ballot-positions/
for more information about how to handle DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-dhc-dhcpv6-yang/



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

Thanks for publishing another YANG module.

I found this document well written, and in particular I thought that the
section on options extensibility and examples in the appendix to be thoughtful
and helpful.  I ​only have a few minor (non blocking) comments on the structure
of the YANG module that you may wish to consider:

1. You may want to consider putting the counters under a container to group
them together, and perhaps to make it easier for a targeted telemetry
subscription.

2. The paths on your RPCs and top level Notifications are using relative paths
- it would probably be clearer to use absolute paths here.

3. I would remove "container" from the Option description strings, to improve
help string readability.

4. Please make sure that you fix the 3 idnits YANG warnings (e.g., alignment of
the revision-date to the extracted file date).

Perhaps in a future draft the IETF will manage to standardize common
configuration for the class selection function since that will help interop but
I can appreciate that is tricky to get consensus on if the existing models/CLIs
are far apart, and you seem to have taken a pragmatic approach here.

Regards,
Rob