[dnsdir] Dnsdir telechat review of draft-ietf-alto-oam-yang-15

Ted Lemon via Datatracker <noreply@ietf.org> Mon, 23 October 2023 18:43 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: dnsdir@ietf.org
Delivered-To: dnsdir@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 74A5DC151710; Mon, 23 Oct 2023 11:43:35 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Ted Lemon via Datatracker <noreply@ietf.org>
To: dnsdir@ietf.org
Cc: alto@ietf.org, draft-ietf-alto-oam-yang.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 11.13.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <169808661546.45653.348035276258481740@ietfa.amsl.com>
Reply-To: Ted Lemon <mellon@fugue.com>
Date: Mon, 23 Oct 2023 11:43:35 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/dnsdir/lPhCoi0_CWnVEOaM90WZpWgNQls>
Subject: [dnsdir] Dnsdir telechat review of draft-ietf-alto-oam-yang-15
X-BeenThere: dnsdir@ietf.org
X-Mailman-Version: 2.1.39
List-Id: DNS Directorate <dnsdir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/dnsdir>, <mailto:dnsdir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/dnsdir/>
List-Post: <mailto:dnsdir@ietf.org>
List-Help: <mailto:dnsdir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/dnsdir>, <mailto:dnsdir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 23 Oct 2023 18:43:35 -0000

Reviewer: Ted Lemon
Review result: Ready with Nits

This is the third dnsdir review of this document. Previous reviews, done by a
different reviewer, marked it as ready or ready with nits, on the basis that
this document doesn't make any changes to how NAPTR is used, and that's the
only DNS-related content in the document.

I agree with this assessment. My one issue is that when I tried to actually
understand, by reading this document and the two RFCs to which it referred, how
the YANG model represents the NAPTR record, I failed. This may be because I'm
not smart enough, or lack experience in ALTO and/or YANG (both of which are
true).

However, if the authors intend that I be able to understand from this document
what part of the NAPTR record is represented by the data model, it might be
worth revisiting whether the model in fact accomplishes this. In particular,
NAPTR records contain quite a few fields, e.g. order and preference, and these
fields are not mentioned in the YANG data model. No fields at all are, which
makes me think that the data model is only representing one field, or perhaps
represents the owner name of the NAPTR record and doesn't represent the NAPTR
record's content at all.

If the authors intended that this be understood from what is written there, I
would encourage them to clarify the text. I'm calling this a nit rather than
raising it as an issue because I'm assuming that the problem is on my end—I
certainly didn't read the aforementioned RFCs closely. So perhaps someone who
understands those RFCs better than I do would not be confused by the text in
the data model document.