[Tsv-art] Tsvart last call review of draft-ietf-opsawg-sdi-08

Mirja Kühlewind via Datatracker <noreply@ietf.org> Tue, 05 May 2020 15:27 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 7AE663A08AD; Tue, 5 May 2020 08:27:59 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 8bit
From: =?utf-8?q?Mirja_K=C3=BChlewind_via_Datatracker?= <noreply@ietf.org>
To: <tsv-art@ietf.org>
Cc: opsawg@ietf.org, draft-ietf-opsawg-sdi.all@ietf.org, last-call@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.129.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <158869247940.11245.12621022136416122709@ietfa.amsl.com>
Reply-To: =?utf-8?q?Mirja_K=C3=BChlewind?= <ietf@kuehlewind.net>
Date: Tue, 05 May 2020 08:27:59 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/JpeiYa8hDItfbp5n0XclCSo2fUg>
Subject: [Tsv-art] Tsvart last call review of draft-ietf-opsawg-sdi-08
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: Tue, 05 May 2020 15:28:00 -0000

Reviewer: Mirja Kühlewind
Review result: 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.

This document specifies a process to encrypt an initial configuration file. The
process of fetching the config file is not altered and as such there are no new
transport related issue.

However, one quick question/comment regarding the following sentence in section
4.3.: "if parsing the
   configurations fails, the device will either abort the auto-install
   process, or will repeat this process until it succeeds."
Is this supposed to indicate that the whole process, including fetching the
file should be repeated? If so, there needs to be guidance that one should not
immediately fetch again but wait for a period of maybe seconds (or minutes?)
and a limit for maximum number of retries must be implemented. Yes, this is not
necessarily part of the process that is altered in this document but if this
guidance is given it should be correct. If similar guidance is already provides
in other documents, a pointer to those docs might work as well.

Editorial comment:
I would recommend to use more generic company names than Sirius Cybernetics
Corp and Acme Network Widgets to avoid that these names can be mistaken as real
companies. I know it's boring but Vendor A and Operator B would probably work
just fine.