[Idr] Robert Wilton's No Objection on draft-ietf-idr-flow-spec-v6-19: (with COMMENT)

Robert Wilton via Datatracker <noreply@ietf.org> Wed, 04 November 2020 10:52 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 243793A0EED; Wed, 4 Nov 2020 02:52:12 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Robert Wilton via Datatracker <noreply@ietf.org>
To: "The IESG" <iesg@ietf.org>
Cc: draft-ietf-idr-flow-spec-v6@ietf.org, idr-chairs@ietf.org, idr@ietf.org, Jie Dong <jie.dong@huawei.com>, aretana.ietf@gmail.com, jie.dong@huawei.com
X-Test-IDTracker: no
X-IETF-IDTracker: 7.21.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Robert Wilton <rwilton@cisco.com>
Message-ID: <160448713212.21180.11299486836315882515@ietfa.amsl.com>
Date: Wed, 04 Nov 2020 02:52:12 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/idr/a_FvbHNxOaP4yi-vRgjmgy-D1rM>
Subject: [Idr] Robert Wilton's No Objection on draft-ietf-idr-flow-spec-v6-19: (with COMMENT)
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: Wed, 04 Nov 2020 10:52:12 -0000

Robert Wilton has entered the following ballot position for
draft-ietf-idr-flow-spec-v6-19: 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/iesg/statement/discuss-criteria.html
for more information about IESG DISCUSS and COMMENT positions.


The document, along with other ballot positions, can be found here:
https://datatracker.ietf.org/doc/draft-ietf-idr-flow-spec-v6/



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

Hi,

Thank you for this document.  Even without the specific domain knowledge I
found this document easy to read and understand.

A couple of minor comments that you may wish to consider:

    3.  IPv6 Flow Specification components

       Types 4, 5, 6, 9, 10 and 11, as defined in [I-D.ietf-idr-rfc5575bis],
       also apply to IPv6.

Also giving the descriptive names for these types might aid the reader here.

    3.3.  Type 3 - Upper-Layer Protocol

       This component uses the Numeric Operator (numeric_op) described in
       [I-D.ietf-idr-rfc5575bis] Section 4.2.1.1.  Type 3 component values
       SHOULD be encoded as single octet (numeric_op len=00).

The "(numeric_op len=00)" threw me off at first, until I referenced back to
section 4.2.1.1.  Possibly, this might be more clear as "(i.e., numeric_op len
field=00)".  Obviously, if you decide to change this, then there are other
places that need to be updated as well.

    3.4.  Type 7 - ICMPv6 Type

The text wasn't super clear to me whether the a Type 3 componet could/should be
specified to match protocol-value 58 as well as a Type 7 field. I would presume
that either is allowed, but I was unsure whether it might be helpful to clarify
this further.

Regards,
Rob