[Gen-art] Genart telechat review of draft-ietf-ntp-bcp-10

Robert Sparks <rjsparks@nostrum.com> Thu, 13 December 2018 23:12 UTC

Return-Path: <rjsparks@nostrum.com>
X-Original-To: gen-art@ietf.org
Delivered-To: gen-art@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 54BEE130EC2; Thu, 13 Dec 2018 15:12:12 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Robert Sparks <rjsparks@nostrum.com>
To: gen-art@ietf.org
Cc: ntp@ietf.org, draft-ietf-ntp-bcp.all@ietf.org, ietf@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.89.1
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <154474273232.32079.9486696704928473641@ietfa.amsl.com>
Date: Thu, 13 Dec 2018 15:12:12 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/gen-art/itw6MymIgGS5Has3KG11ZutphFk>
Subject: [Gen-art] Genart telechat review of draft-ietf-ntp-bcp-10
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.29
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 13 Dec 2018 23:12:12 -0000

Reviewer: Robert Sparks
Review result: Ready with Nits

I am the assigned Gen-ART reviewer for this draft. The General Area
Review Team (Gen-ART) reviews all IETF documents being processed
by the IESG for the IETF Chair. Please wait for direction from your
document shepherd or AD before posting a new version of the draft.

For more information, please see the FAQ at

<https://trac.ietf.org/trac/gen/wiki/GenArtfaq>.

Document: draft-ietf-ntp-bcp-10
Reviewer: Robert Sparks
Review Date: 2018-12-13
IETF LC End Date: 2018-10-08
IESG Telechat date: 2018-12-20

Summary: Ready (but with nits that should be considered) for publication as a
BCP RFC

Nits/editorial comments:

With a couple of exceptions, the changes between -07 and -10 are very helpful -
the document reads much more naturally.

One of the changes was to be more specific with actors - many uses of "you" or
"your" were replaced with "the operator" for example. But this wasn't done
throughout the document ("you" and "your" still appear frequently), and in at
least one place the change caused a sentence to stop making sense: "If the time
on your network has to be correct close to 100% of the time, then even if you
are using a satellite-based system, operators need to plan for those rare
instances when the system is unavailable (or wrong!)."

I strongly encourage yet another pass focusing on removing "you" and "your" to
the extent possible.

The changes also included using 2119 keywords much more often. Unfortunately
many of the new uses are not appropriate. "Vendors MUST" and several instances
of "It is RECOMMENDED" are particularly jarring. Moving 2119 to be an
Informational reference is also incorrect if you are going to use those terms
in this document.