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

[Francesca Palombini via Datatracker <[e-mail]>]

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.