[Ntp] Genart last call review of draft-ietf-ntp-yang-data-model-12
Tim Evens via Datatracker <noreply@ietf.org> Fri, 12 February 2021 23:51 UTC
Return-Path: <noreply@ietf.org>
X-Original-To: ntp@ietf.org
Delivered-To: ntp@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id F3AF53A10F6; Fri, 12 Feb 2021 15:51:14 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Tim Evens via Datatracker <noreply@ietf.org>
To: gen-art@ietf.org
Cc: draft-ietf-ntp-yang-data-model.all@ietf.org, last-call@ietf.org, ntp@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.25.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <161317387496.31377.7416127213693172983@ietfa.amsl.com>
Reply-To: Tim Evens <tievens@cisco.com>
Date: Fri, 12 Feb 2021 15:51:14 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/ntp/CO8jc3SEY8LEO84v9cdvPz3l0t4>
Subject: [Ntp] Genart last call review of draft-ietf-ntp-yang-data-model-12
X-BeenThere: ntp@ietf.org
X-Mailman-Version: 2.1.29
List-Id: <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: Fri, 12 Feb 2021 23:51:15 -0000
Reviewer: Tim Evens 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 treat these comments just like any other last call comments. For more information, please see the FAQ at <https://trac.ietf.org/trac/gen/wiki/GenArtfaq>. Document: draft-ietf-ntp-yang-data-model-?? Reviewer: Tim Evens Review Date: 2021-02-12 IETF LC End Date: 2021-02-12 IESG Telechat date: Not scheduled for a telechat Summary: Overall, nice document. Major issues: Minor issues: Nits/editorial comments: Spelling correction; s/retriving/retrieving/g It appears that "bb1d6929 e9593728 7fa37d12 9b756746" is not correctly enclosed in xml. The key/hexadecimal-string declares it correctly, but the space delimited string seems out of place. """ <config> <ntp xmlns="urn:ietf:params:xml:ns:yang:ietf-ntp"> <authentication> <auth-enabled>true</auth-enabled> <authentication-keys> <key-id>10</key-id> <algorithm>aes-cmac</algorithm> bb1d6929 e9593728 7fa37d12 9b756746 <key> <hexadecimal-string> bb1d6929e95937287fa37d129b756746 </hexadecimal-string> </key> <istrusted>true</istrusted> </authentication-keys> </authentication> </ntp> </config> """ Are the backslashes required here? It seems that many examples have them when there is a value and the closing tag is not on the same line. For example: """ <data xmlns="urn:ietf:params:xml:ns:netconf:base:1.0"> <ntp xmlns="urn:ietf:params:xml:ns:yang:ietf-ntp"> <clock-state> <system-status> <clock-state>synchronized</clock-state> <clock-stratum>7</clock-stratum> <clock-refid>192.0.2.1</clock-refid> <associations-address>192.0.2.1\ </associations-address> <associations-local-mode>client\ </associations-local-mode> <associations-isconfigured>yes\ </associations-isconfigured> <nominal-freq>100.0</nominal-freq> <actual-freq>100.0</actual-freq> <clock-precision>18</clock-precision> <clock-offset>0.025</clock-offset> <root-delay>0.5</root-delay> <root-dispersion>0.8</root-dispersion> <reference-time>10-10-2017 07:33:55.258 Z+05:30\ </reference-time> <sync-state>clock-synchronized</sync-state> </system-status> </clock-state> </ntp> </data> """
- [Ntp] Genart last call review of draft-ietf-ntp-y… Tim Evens via Datatracker
- [Ntp] Antw: [EXT] Genart last call review of draf… Ulrich Windl
- Re: [Ntp] [Last-Call] Antw: [EXT] Genart last cal… tom petch
- Re: [Ntp] [Last-Call] Antw: [EXT] Genart last cal… Tim Evens (tievens)
- Re: [Ntp] [Last-Call] Antw: [EXT] Genart last cal… tom petch