Re: [Ntp] Quick review of WGLC for status change for draft-ietf-ntp-update-registries

Miroslav Lichvar <mlichvar@redhat.com> Mon, 08 August 2022 09:23 UTC

Return-Path: <mlichvar@redhat.com>
X-Original-To: ntp@ietfa.amsl.com
Delivered-To: ntp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 7A513C15948B for <ntp@ietfa.amsl.com>; Mon, 8 Aug 2022 02:23:30 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.689
X-Spam-Level:
X-Spam-Status: No, score=-2.689 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.582, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_HELO_NONE=0.001, SPF_NONE=0.001, T_SCC_BODY_TEXT_LINE=-0.01] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=redhat.com
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 tyZgkTt-ajFd for <ntp@ietfa.amsl.com>; Mon, 8 Aug 2022 02:23:28 -0700 (PDT)
Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [170.10.129.124]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id DF5B0C14F748 for <ntp@ietf.org>; Mon, 8 Aug 2022 02:23:27 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1659950606; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=1J6ByYp07r6QrQRQ2CevjKCsJl9ZeKj2VX3jO8HG/7o=; b=D7DFI4UXhBpgwTujjxilEa9jcmmdW5BW5OWq5g4wjjaHZkF3P7ZwkAePjfvxfkuMMAeoj7 Um5cbcdRqkv2wxqYR583SiXXFO4qHrAe/2ieMqUlg6LG7OQZ7nZoiE2/NN+vQOwmO+o5Dq +ru9aEIvSagJ2iyfkQ4586tPzkpSaoE=
Received: from mimecast-mx02.redhat.com (mimecast-mx02.redhat.com [66.187.233.88]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id us-mta-625-PM-0C2IsO2muUTtxdo588Q-1; Mon, 08 Aug 2022 05:23:25 -0400
X-MC-Unique: PM-0C2IsO2muUTtxdo588Q-1
Received: from smtp.corp.redhat.com (int-mx09.intmail.prod.int.rdu2.redhat.com [10.11.54.9]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id C319618A6522 for <ntp@ietf.org>; Mon, 8 Aug 2022 09:23:24 +0000 (UTC)
Received: from localhost (unknown [10.43.135.229]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 6D934492C3B for <ntp@ietf.org>; Mon, 8 Aug 2022 09:23:24 +0000 (UTC)
Date: Mon, 08 Aug 2022 11:23:23 +0200
From: Miroslav Lichvar <mlichvar@redhat.com>
To: ntp@ietf.org
Message-ID: <YvDWC27qKnODlD52@localhost>
References: <PH0PR06MB7061FA7A5B338D262B3A2963C2999@PH0PR06MB7061.namprd06.prod.outlook.com> <6a187a2f-9883-2fb5-1f51-1593591ddebb@nwtime.org> <PH0PR06MB706126984E4442EF32F8242AC2999@PH0PR06MB7061.namprd06.prod.outlook.com> <da155c84-2c70-2e3b-59eb-03e380806cf2@nwtime.org> <PH0PR06MB70611F2331D8255F7E2B6604C2999@PH0PR06MB7061.namprd06.prod.outlook.com> <0b4c7efa-3977-b588-0974-33b6a9437e52@nwtime.org>
MIME-Version: 1.0
In-Reply-To: <0b4c7efa-3977-b588-0974-33b6a9437e52@nwtime.org>
X-Scanned-By: MIMEDefang 2.85 on 10.11.54.9
X-Mimecast-Spam-Score: 0
X-Mimecast-Originator: redhat.com
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
Archived-At: <https://mailarchive.ietf.org/arch/msg/ntp/rhVOgkoCCqv3kpKcbaxy9D4SnTg>
Subject: Re: [Ntp] Quick review of WGLC for status change for draft-ietf-ntp-update-registries
X-BeenThere: ntp@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: Network Time Protocol <ntp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ntp>, <mailto:ntp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ntp/>
List-Post: <mailto:ntp@ietf.org>
List-Help: <mailto:ntp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ntp>, <mailto:ntp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 08 Aug 2022 09:23:30 -0000

On Mon, Aug 08, 2022 at 02:00:21AM -0700, Harlan Stenn wrote:
> The problems I (and the NTP Project) have with this document include, but
> are not limited to:
> 
> - it uses far too many words to say far too little.  In other places it uses
> far too few words to say even less, except where it says nothing about
> various significant aspects of the registries.

Which significant aspects of the registries are you referring to?

> - Table 1 is incomplete.  It does not include the value used by RFC 7821,
> and it frequently refers to values that are "reserved for historic reasons"
> without further explanation.  This leads me to:

The Checksum Complement EF (0x2005) is present in the table 1, but it
is incorrectly described as "Reserved for historic reasons". That
should be changed to refer to RFC 7821. Only the Autokey-related
entries should be "Reserved for historic reasons".

> - The document does not describe (ignores?) 22 years of conscious design
> decisions around the two versions (revisions?) of extension field syntax,
> which Dave Mills and I have previous spent 3+ years' time and effort (10
> document revisions) in documenting, in draft-stenn-ntp-extension-fields.

As was discussed when the draft was submitted, it was making changes
incompatible with RFC7822, e.g. decreasing the minimum EF length. We
can do that in NTPv5, but not in NTPv4.

-- 
Miroslav Lichvar