[Ntp] Antw: [EXT] Francesca Palombini's Discuss on draft-ietf-ntp-yang-data-model-15: (with DISCUSS and COMMENT)

Ulrich Windl <Ulrich.Windl@rz.uni-regensburg.de> Thu, 01 July 2021 06:26 UTC

Return-Path: <Ulrich.Windl@rz.uni-regensburg.de>
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 69E473A1635; Wed, 30 Jun 2021 23:26:44 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id kDiCqKpe-PuD; Wed, 30 Jun 2021 23:26:41 -0700 (PDT)
Received: from mx2.uni-regensburg.de (mx2.uni-regensburg.de [194.94.157.147]) (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 4C8213A1636; Wed, 30 Jun 2021 23:26:41 -0700 (PDT)
Received: from mx2.uni-regensburg.de (localhost [127.0.0.1]) by localhost (Postfix) with SMTP id 0A8026000051; Thu, 1 Jul 2021 08:26:38 +0200 (CEST)
Received: from gwsmtp.uni-regensburg.de (gwsmtp1.uni-regensburg.de [132.199.5.51]) by mx2.uni-regensburg.de (Postfix) with ESMTP id 30AF7600004D; Thu, 1 Jul 2021 08:26:36 +0200 (CEST)
Received: from uni-regensburg-smtp1-MTA by gwsmtp.uni-regensburg.de with Novell_GroupWise; Thu, 01 Jul 2021 08:26:35 +0200
Message-Id: <60DD6019020000A100042190@gwsmtp.uni-regensburg.de>
X-Mailer: Novell GroupWise Internet Agent 18.3.1
Date: Thu, 01 Jul 2021 08:26:33 +0200
From: "Ulrich Windl" <Ulrich.Windl@rz.uni-regensburg.de>
To: <francesca.palombini@ericsson.com>,"The IESG" <iesg@ietf.org>
Cc: "Dieter Sibold" <dsibold.ietf@gmail.com>, <draft-ietf-ntp-yang-data-model@ietf.org>, "ntp@ietf.org" <ntp@ietf.org>, "ntp-chairs@ietf.org" <ntp-chairs@ietf.org>
References: <162506975525.16899.10820299795831774115@ietfa.amsl.com>
In-Reply-To: <162506975525.16899.10820299795831774115@ietfa.amsl.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Content-Disposition: inline
Archived-At: <https://mailarchive.ietf.org/arch/msg/ntp/H972FcTSj8l-q1E_9kJgYXYuYJg>
Subject: [Ntp] Antw: [EXT] Francesca Palombini's Discuss on draft-ietf-ntp-yang-data-model-15: (with DISCUSS and COMMENT)
X-BeenThere: ntp@ietf.org
X-Mailman-Version: 2.1.29
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: Thu, 01 Jul 2021 06:26:45 -0000

I fully agree to the discussion items, i.e.: the items need discussion.

>>> Francesca Palombini via Datatracker <noreply@ietf.org> schrieb am
30.06.2021 um
18:15 in Nachricht <162506975525.16899.10820299795831774115@ietfa.amsl.com>om>:
> Francesca Palombini has entered the following ballot position for
> draft‑ietf‑ntp‑yang‑data‑model‑15: Discuss
> 
> When responding, please keep the subject line intact and reply to all
> email addresses included in the To and CC lines. (Feel free to cut this
> introductory paragraph, however.)
> 
> 
> Please refer to https://www.ietf.org/iesg/statement/discuss‑criteria.html 
> for more information about DISCUSS and COMMENT positions.
> 
> 
> The document, along with other ballot positions, can be found here:
> https://datatracker.ietf.org/doc/draft‑ietf‑ntp‑yang‑data‑model/ 
> 
> 
> 
> ‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑
> DISCUSS:
> ‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑
> 
> Thank you for the work on this document.
> 
> I have a simple‑to‑solve DISCUSS point, and some non blocking comments.
> 
> Francesca
> 
> 
> 1. ‑‑‑‑‑
> 
>         leaf clock‑precision {
>           type int8;
>           units "Hz";
> 
> FP: I believe the units should be seconds here.
> 
> 
> ‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑
> COMMENT:
> ‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑‑
> 
> 2. ‑‑‑‑‑
> 
>   typedef ntp‑date‑and‑time {
>     type union {
>       type yang:date‑and‑time;
>       type uint8;
>     }
>     description
>       "Follows the normal date‑and‑time format when valid value
>        exist, otherwise allows for setting special value such as
>        zero.";
> 
> FP: I'd rather the document avoided the term "normal", which could be just
> removed. Also, could a 'reference' be added, pointing to RFC 6991?
> 
> 3. ‑‑‑‑‑
> 
>         "The association was configured or dynamic
>          which result in clock synchronization.";
> 
> FP: This sentence doesn't parse correctly for me. Is there any text missing?

> Or
> maybe a wrong copy paste?
> 
> 4. ‑‑‑‑‑
> 
>       description
>         "Configuration to control access to NTP service
>          by using NTP access‑group feature.
>          The access‑mode identifies how the acl is
>          applied with NTP.";
> 
> FP: (here and other places) please make sure the capitalization of ACL is
> consistent in the descriptions and in the text of the doc.
> 
> 
> 
> _______________________________________________
> ntp mailing list
> ntp@ietf.org 
> https://www.ietf.org/mailman/listinfo/ntp