[IPv6]draft-ietf-6man-rfc6724-update-23 telechat Intdir review

Timothy Winters via Datatracker <noreply@ietf.org> Wed, 23 July 2025 08:48 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: ipv6@ietf.org
Delivered-To: ipv6@mail2.ietf.org
Received: from [10.244.4.63] (unknown [104.131.183.230]) by mail2.ietf.org (Postfix) with ESMTP id 1F37C491FE94; Wed, 23 Jul 2025 01:48:25 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Timothy Winters via Datatracker <noreply@ietf.org>
To: int-dir@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 12.43.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <175326050497.2199754.12871003611259735374@dt-datatracker-7ff4b97469-cdtd8>
Date: Wed, 23 Jul 2025 01:48:24 -0700
Message-ID-Hash: 55A5MJRXJCQX56QWTOOKYJUPCFA6JW6T
X-Message-ID-Hash: 55A5MJRXJCQX56QWTOOKYJUPCFA6JW6T
X-MailFrom: noreply@ietf.org
X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-ipv6.ietf.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header
CC: draft-ietf-6man-rfc6724-update.all@ietf.org, ipv6@ietf.org, last-call@ietf.org
X-Mailman-Version: 3.3.9rc6
Reply-To: Timothy Winters <tim@qacafe.com>
Subject: [IPv6]draft-ietf-6man-rfc6724-update-23 telechat Intdir review
List-Id: "IPv6 Maintenance Working Group (6man)" <ipv6.ietf.org>
Archived-At: <https://mailarchive.ietf.org/arch/msg/ipv6/y_zJh-f2M-Nj3tGZJb279x3VSwc>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ipv6>
List-Help: <mailto:ipv6-request@ietf.org?subject=help>
List-Owner: <mailto:ipv6-owner@ietf.org>
List-Post: <mailto:ipv6@ietf.org>
List-Subscribe: <mailto:ipv6-join@ietf.org>
List-Unsubscribe: <mailto:ipv6-leave@ietf.org>

Document: draft-ietf-6man-rfc6724-update
Title: Prioritizing known-local IPv6 ULAs through address selection policy
Reviewer: Timothy Winters
Review result: Ready with Issues

I am the assigned int-dir reviewer for this draft. These comments were written
with the intent of improving the Internet area aspects of the IETF drafts.
Please wait for direction from your document shepherd or AD before posting a
new version of the draft. For background on int-dir, please see the
[FAQ](https://wiki.ietf.org/en/group/intdir).

I think this document is ready, there are three minor technical issues that
need to be addressed before publication.

Technical:

Section 3.1

$known_local/48 - Later in the document, in Section 3.3 the range is documented
as /40 to /48.  I would recommend putting the range in this table /48 to /40?
---
Section 3.3
These known-local ULA prefixes are inferred from ULA addresses assigned to
interfaces or learned from Prefix Information Options (PIOs) in Router
Advertisements (RAs) [RFC4861] received on any interface regardless of how the
PIO flags are set.

This reads to me that an implementation will need to merge all the ULA prefixes
they receive on any interface.  I don't think that is the intention, I would
suggest clarifying.
---
Section 3.3
7. Entries MUST be removed from the known-local ULA list and the Policy Table
when the announced RIOs or PIOs are deprecated, or an interface address is
removed, and there is no covering RIO or PIO.

RIO can't be deprecated, they only have a valid lifetime this should be
invalidated?
---

Nits:
---
OLD:
It further clarifies the unconditional requirement for implementing Rule 5.5 of
RFC 6724

NEW:
It introduces a requirement to implement Rule 5.5 of RFC 6724
---
OLD:
This document therefore introduces two changes to RFC6724 to support a node
implementing

NEW:
This document therefore introduces two requirements to RFC6724 to support a
node implementing
---
OLD:
Known-local ULA: A ULA prefix that an individual organization/site has
determined to be local to a given node/network/administrative domain

NEW:
Known-local ULA: A ULA prefix that an node has determined to be local to a
given network domain.
---
Section 3.3
Tools that display a node's current policy table MUST show all currently
inserted known-local ULA prefixes.

What defines a tool?
---