[art] Artart last call review of draft-ietf-dnsop-glue-is-not-optional-08

Gonzalo Salgueiro via Datatracker <noreply@ietf.org> Mon, 08 May 2023 19:21 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: art@ietf.org
Delivered-To: art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 70856C16951F; Mon, 8 May 2023 12:21:33 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Gonzalo Salgueiro via Datatracker <noreply@ietf.org>
To: art@ietf.org
Cc: dnsop@ietf.org, draft-ietf-dnsop-glue-is-not-optional.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 10.2.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <168357369344.28686.7293020736025361619@ietfa.amsl.com>
Reply-To: Gonzalo Salgueiro <gsalguei@cisco.com>
Date: Mon, 08 May 2023 12:21:33 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/art/0TZzS-tRlDBNB8klRAmGNupECNE>
Subject: [art] Artart last call review of draft-ietf-dnsop-glue-is-not-optional-08
X-BeenThere: art@ietf.org
X-Mailman-Version: 2.1.39
List-Id: Applications and Real-Time Area Discussion <art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/art>, <mailto:art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/art/>
List-Post: <mailto:art@ietf.org>
List-Help: <mailto:art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/art>, <mailto:art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 08 May 2023 19:21:33 -0000

Reviewer: Gonzalo Salgueiro
Review result: Ready

I am the assigned ART directorate reviewer for this document. These
comments were written primarily for the benefit of the ART area
directors. Document editors and WG chairs should treat these comments
just like any other last call comments.

This is a Standards Track document that establishes that Glue records
are expected to be returned as part of a referral and if they can't fit
in a UDP response TC=1 MUST be set to signal to the client to retry over
TCP. This is well written, clear, and concise. This document is ready
for publication.

Two very minor nits that you might optionally consider:

— Abstract —

I'm not sure it is grammatically incorrect but the following sounds
better to me:

OLD 
	Glue records are expected to be returned as part of a referral and
	if they cannot be fitted into the UDP response, TC=1 MUST be set to
	inform the client that the response is incomplete and that TCP SHOULD be
	used to retrieve the full response.

NEW 
	Glue records are expected to be returned as part of a referral and
	if they cannot be fit into the UDP response, TC=1 MUST be set to inform
	the client that the response is incomplete and that TCP SHOULD be used
	to retrieve the full response. 
END


I found myself looking up a few terms, so it might be useful to include
a reference to RFC 8499 (or draft-ietf-dnsop-rfc8499bis) for the
uninitiated.


Thanks, 
Gonzalo