[ippm] Murray Kucherawy's No Objection on draft-ietf-ippm-route-09: (with COMMENT)

Murray Kucherawy via Datatracker <noreply@ietf.org> Wed, 12 August 2020 05:45 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: ippm@ietf.org
Delivered-To: ippm@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 9B30D3A1041; Tue, 11 Aug 2020 22:45:14 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Murray Kucherawy via Datatracker <noreply@ietf.org>
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-ippm-route@ietf.org, ippm-chairs@ietf.org, ippm@ietf.org, Brian Trammell <ietf@trammell.ch>
X-Test-IDTracker: no
X-IETF-IDTracker: 7.13.0
Auto-Submitted: auto-generated
Precedence: bulk
Reply-To: Murray Kucherawy <superuser@gmail.com>
Message-ID: <159721111418.1137.15547353822405197957@ietfa.amsl.com>
Date: Tue, 11 Aug 2020 22:45:14 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/ippm/1F_-z8cyJZOn9Azj2UOiNk8-Tzc>
Subject: [ippm] Murray Kucherawy's No Objection on draft-ietf-ippm-route-09: (with COMMENT)
X-BeenThere: ippm@ietf.org
X-Mailman-Version: 2.1.29
List-Id: IETF IP Performance Metrics Working Group <ippm.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ippm>, <mailto:ippm-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ippm/>
List-Post: <mailto:ippm@ietf.org>
List-Help: <mailto:ippm-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ippm>, <mailto:ippm-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 12 Aug 2020 05:45:15 -0000

Murray Kucherawy has entered the following ballot position for
draft-ietf-ippm-route-09: 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-ippm-route/



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

Section 3's title made me expect a glossary, but it also lays out actual
normative requirements.  This seems odd.  Personal preference, but I suggest
separating them out.   I note that Warren also tripped over this.

Also in Section 3, I tripped over "class C" as Warren did.  But I think what
you're doing is trying to introduce a label "C" to define the class of packet
types in this context, so maybe:

OLD:

    A route that treats equally a class C of different ...

NEW:

    A route that treats equally a class, referred to here as "C", of different
    ...

Then you can just refer to "C" later and drop the confusing references to
"class C".

Section 3.1 starts with something that isn't a sentence.  I suggest making it
one.

Section 4.1 uses SHOULD in a number of places that leave me wondering, "Or else
what?"  You're presenting the implementer with a choice here, but it doesn't
feel like these choices are fully developed.  The same occurs in 4.3.  Are
nodes not conforming to these SHOULDs malfunctioning, misconfigured, or simply
opting out?