[Idr] Opsdir last call review of draft-ietf-idr-flow-spec-v6-17

Qin Wu via Datatracker <noreply@ietf.org> Tue, 20 October 2020 14:08 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: idr@ietf.org
Delivered-To: idr@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 5DEAF3A099F; Tue, 20 Oct 2020 07:08:31 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Qin Wu via Datatracker <noreply@ietf.org>
To: <ops-dir@ietf.org>
Cc: draft-ietf-idr-flow-spec-v6.all@ietf.org, last-call@ietf.org, idr@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.20.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <160320291135.11937.7363678904982610426@ietfa.amsl.com>
Reply-To: Qin Wu <bill.wu@huawei.com>
Date: Tue, 20 Oct 2020 07:08:31 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/RdjcfvgFUm3YvbIXV2m7Kf5Nj0Q>
Subject: [Idr] Opsdir last call review of draft-ietf-idr-flow-spec-v6-17
X-BeenThere: idr@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Inter-Domain Routing <idr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/idr>, <mailto:idr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/idr/>
List-Post: <mailto:idr@ietf.org>
List-Help: <mailto:idr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/idr>, <mailto:idr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 20 Oct 2020 14:08:32 -0000

Reviewer: Qin Wu
Review result: Ready

I have reviewed this document on behalf of the Operations and Management
Directorate. I believe this document is well written. One clarification
question I want to ask here is why not consolidate this document into
I-D.ietf-idr-rfc5575bis since rc5575bis has just begun. Regarding Type 13 -
Flow Label , I am wondering why Type 13 component values can not be be encoded
as 8-byte quantities? why len=11 is not supported for IPv6 case? Regarding "the
Sub-Type always TBD" in section 6.1, I want to suggest to add reference to IANA
section, i.e., section 8.2.