Re: [Tsv-art] [Last-Call] Tsvart last call review of draft-ietf-netconf-tcp-client-server-21

tuexen@fh-muenster.de Mon, 12 February 2024 22:35 UTC

Return-Path: <tuexen@fh-muenster.de>
X-Original-To: tsv-art@ietfa.amsl.com
Delivered-To: tsv-art@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id C44F5C1519B4; Mon, 12 Feb 2024 14:35:00 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.944
X-Spam-Level:
X-Spam-Status: No, score=-1.944 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_LOW=-0.7, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_SOFTFAIL=0.665, T_SCC_BODY_TEXT_LINE=-0.01] autolearn=no autolearn_force=no
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id lWzUln6Zs5aB; Mon, 12 Feb 2024 14:34:58 -0800 (PST)
Received: from drew.franken.de (mail-n.franken.de [193.175.24.27]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 9EB40C151980; Mon, 12 Feb 2024 14:34:53 -0800 (PST)
Received: from smtpclient.apple (unknown [IPv6:2a02:8109:1140:c3d:2000:826:171b:5496]) (Authenticated sender: micmac) by drew.franken.de (Postfix) with ESMTPSA id 9E962721E2830; Mon, 12 Feb 2024 23:34:49 +0100 (CET)
Content-Type: multipart/signed; boundary="Apple-Mail=_2FB1EA66-D064-48F5-8E38-61815687EC6A"; protocol="application/pkcs7-signature"; micalg="sha-256"
Mime-Version: 1.0 (Mac OS X Mail 16.0 \(3774.400.31\))
From: tuexen@fh-muenster.de
In-Reply-To: <0100018d9eeace8b-f182e99e-c65e-4b12-84cd-05830aa4fb82-000000@email.amazonses.com>
Date: Mon, 12 Feb 2024 23:34:48 +0100
Cc: tsv-art@ietf.org, draft-ietf-netconf-tcp-client-server.all@ietf.org, last-call@ietf.org, "netconf@ietf.org" <netconf@ietf.org>
Content-Transfer-Encoding: quoted-printable
Message-Id: <24163F75-3506-4A8D-B271-5ACF742F5FC4@fh-muenster.de>
References: <170773777422.36812.2729106686921670819@ietfa.amsl.com> <0100018d9eeace8b-f182e99e-c65e-4b12-84cd-05830aa4fb82-000000@email.amazonses.com>
To: Kent Watsen <kent+ietf@watsen.net>
X-Mailer: Apple Mail (2.3774.400.31)
Archived-At: <https://mailarchive.ietf.org/arch/msg/tsv-art/BYJrFJkRFEwHxql1SsLIn4794nM>
Subject: Re: [Tsv-art] [Last-Call] Tsvart last call review of draft-ietf-netconf-tcp-client-server-21
X-BeenThere: tsv-art@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
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: Mon, 12 Feb 2024 22:35:00 -0000

> On Feb 12, 2024, at 21:03, Kent Watsen <kent+ietf@watsen.net> wrote:
> 
> Hi Michael,
> 
> Thank you for your review,
> Responses  below.
> 
> Kent
> 
> 
>> On Feb 12, 2024, at 6:36 AM, Michael Tüxen via Datatracker <noreply@ietf.org> wrote:
>> 
>> Reviewer: Michael Tüxen
>> 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.
>> 
>> The document is well written, I only have the following two nits:
>> 
>> * 2.1.5. Guidelines for Configuring TCP Keep-Alives
>> It seems that the item list at the end of the subsection provides
>> information related to "idle-time", "max-probes", and "probe-interval".
>> One item for one leaf, in the given order.
>> What confuses me is that the first item contains:
>> "but keep-alive messages SHOULD NOT be transmitted more frequently than
>> once every 15 seconds"
>> For me, this applies to not only to "idle-time" but also to "probe-interval".
>> However, for "probe-interval" you provide lower values.
>> Using
>> "A lower value MAY be configured, but idle intervals SHOULD NOT be smaller
>> than 15 seconds. Longer idle intervals SHOULD be used when possible."
>> would have avoided this confusion.
> 
> I updated the document to use your text.  Thanks for the suggestion!
> 
> 
>> * Acknowledgements
>> The authors would like to thank for following for lively discussions
>> should possibly read
>> The authors would like to thank the following for lively discussions
> 
> Fixed (in all nine drafts)
Thanks for the quick response!

Best regards
Michael
> 
> 
>> Best regards
>> Michael Tüxen
> 
> 
> Thanks again,
> Kent
>