[Tsv-art] Tsvart last call review of draft-ietf-detnet-yang-18

Joerg Ott via Datatracker <noreply@ietf.org> Wed, 20 December 2023 14:28 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 79AAEC151099; Wed, 20 Dec 2023 06:28:15 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Joerg Ott via Datatracker <noreply@ietf.org>
To: tsv-art@ietf.org
Cc: detnet@ietf.org, draft-ietf-detnet-yang.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 12.1.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <170308249548.55336.14416176973188830718@ietfa.amsl.com>
Reply-To: Joerg Ott <jo@acm.org>
Date: Wed, 20 Dec 2023 06:28:15 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/76lces6mcGBsdA88RWqHhrNOBVA>
Subject: [Tsv-art] Tsvart last call review of draft-ietf-detnet-yang-18
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.39
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, 20 Dec 2023 14:28:15 -0000

Reviewer: Joerg Ott
Review result: Ready with Nits

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.

This document describes a YANG model for Detnet nodes and layers, addressing
configuration and monitoring of the different entities.  The Detnet YANG
model has limited extra bearing on transport (beyond what Detnet already
does): the model just allows accessing and controlling Detnet nodes, the
protocols for doing are also separately specified.

With this, assuming that the underlying Detnet mechanisms function properly,
the YANG model should not be able to introduce additional transport layer
issues that would need to be considered.  Detnet also defines its own flow
specification for traffic description. Detnet itself resides below the
transport layer and may use service differentiation by various means so that
the traffic inside a Detnet class is separated from that on the outside,
but this should not affect the various traffic flows within a given class.

Again, the YANG model only assists configuration.  I trust the YANG expert
reviews on the details of the model, which seem to have been discussed before.

Nits:

One may argue if section 4 could offer a bit more context; but, then, one may
expect that those who would read this document would be intimately familiar
with the Detnet architecture and terminology in the first place.  If more
stand-alone usage was intended, section 4 could be expanded a bit.

The draft ,misses quite a few articles "the" in front of nouns, quite prominently
in the last bullet list in the beginning of each bullet item, but also 
occasionally in other places.

So, nothing major, but I wanted to still point out these two bits.