[Tsv-art] Tsvart last call review of draft-ietf-rift-applicability-03

Tommy Pauly via Datatracker <noreply@ietf.org> Wed, 06 January 2021 16:48 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: tsv-art@ietf.org
Delivered-To: tsv-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 40A923A1016; Wed, 6 Jan 2021 08:48:39 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: Tommy Pauly via Datatracker <noreply@ietf.org>
To: tsv-art@ietf.org
Cc: draft-ietf-rift-applicability.all@ietf.org, last-call@ietf.org, rift@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.24.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <160995171921.18418.8487966634505695144@ietfa.amsl.com>
Reply-To: Tommy Pauly <tpauly@apple.com>
Date: Wed, 06 Jan 2021 08:48:39 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/_zVMcUrQoRf2JcPJOn8-e44lIfA>
Subject: [Tsv-art] Tsvart last call review of draft-ietf-rift-applicability-03
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Transport Area Review Team <tsv-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/tsv-art/>
List-Post: <mailto:tsv-art@ietf.org>
List-Help: <mailto:tsv-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/tsv-art>, <mailto:tsv-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 06 Jan 2021 16:48:39 -0000

Reviewer: Tommy Pauly
Review result: Not Ready

This document has been reviewed as part of the transport area review team's
ongoing effort to review key IETF documents. These comments were written
primarily for the transport area directors, but are copied to the document's
authors and WG to allow them to address any issues raised and also to the IETF
discussion list for information.

When done at the time of IETF Last Call, the authors should consider this
review as part of the last-call comments they receive. Please always CC
tsv-art@ietf.org if you reply to or forward this review.

>From a Transport Area perspective, this applicability document didn't raise any
concerns during my review. However, I found the document to be unclear and hard
to approach. Since applicability documents are particularly meant to be
explanatory for a potentially broader audience, I would recommend the document
going through a revision to make it more readable.

As someone not deeply familiar with the topics described in this document, I
found that it could benefit from more explanation of terms, or else references
to RFCs that define the terms used. This applies throughout sections 2 and 3.
For example, there should be at least some references or expansions for Clos,
fat-tree, SPF, PoD, TIEs, TORs, PDU, DC, ZTP, etc. Sometimes terms are
expanded/defined much later than when they are first used.

Editorially, the document needs some revisions to be grammatically clearer and
have a less awkward/colloquial tone. For example, this sentence:

"There are a bunch of more advantages unique to RIFT listed below which could
be understood if you read the details of [RIFT]."

Would read better as:

"RIFT provides many other unique advantages, which are listed below and
detailed further in [RIFT]."

There are many other similar examples that should be cleaned up before
publication.

I also found that the document made statements about how the industry would be
deploying this technology, such as “poised to deliver a majority of computation
and storage services in the future”. Whiale this may certainly be the case,
such statements don’t benefit a technical document, and make it too tied to
this moment in time (and can be harmful if the predictions ever change). Many
of these statements can be simply removed, and the document would be clearer
and more concise.