Re: [Ntp] Zaheduzzaman Sarker's No Objection on draft-ietf-ntp-yang-data-model-15: (with COMMENT)

Dhruv Dhody <dhruv.ietf@gmail.com> Thu, 09 December 2021 17:01 UTC

Return-Path: <dhruv.ietf@gmail.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 487463A05F8; Thu, 9 Dec 2021 09:01:30 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.097
X-Spam-Level:
X-Spam-Status: No, score=-2.097 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, HTML_MESSAGE=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 EHO6F98PkZNL; Thu, 9 Dec 2021 09:01:27 -0800 (PST)
Received: from mail-io1-xd36.google.com (mail-io1-xd36.google.com [IPv6:2607:f8b0:4864:20::d36]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 1829E3A0163; Thu, 9 Dec 2021 09:01:27 -0800 (PST)
Received: by mail-io1-xd36.google.com with SMTP id y16so7358168ioc.8; Thu, 09 Dec 2021 09:01:27 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=e8/VBKGkwWGi1E+9KWdfzbFb38sWOJd6PpHgSTzh/cw=; b=iH3Nj9PkHq29Q1t2lsNDcmUdlpCv9Dt8w8lfcT8eWI1LgJ5Jt8g0V29sM1xYmgCv6Y 0mXHMfSn+8V4LsF+QIYtnw9h0tI5LUbbEVweScAtjtcsLyiJBITZ37tfMQ1dJbyzhkEo cuFX+CWmvPGFQn4gS9usB2++J0fP7ptCGMUpKp195k/9JJDW+tuLEt3l9xoeO/BDpIuG 6vrqw8xIjoCkxfRRPvSYisVwv4ygDZSoSwMb5LnKu54/M7sCNVN9KIM7epI+eIY3WLi1 UqA6ztgHHOw5QMqa3H3lAWJBEpSyvfdk1z2tnUW0EogAxGbhegievO/l4Q20j4xfegXv kY+g==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=e8/VBKGkwWGi1E+9KWdfzbFb38sWOJd6PpHgSTzh/cw=; b=78HQEs3U3oGCqFLCD9nMJsS6pSlHExJ/MAwYbm7UmUpyG3uDTl0tkt+1TEjDp5KLPU pdYir4wfSn1c11gRjZo+lOce8ALDP5TkF8y3jCFhjtH/kxixqZnDntzdIXJdnkV3Leqx D2+MjOPS9583nQfPMgLn/XHsR8RweV6vNVE4YSRnmm2oUNXZnTe0YbrUn7tENInewL8b 9A1Blzyf2+tKaA/hz5J2R9xKbPmpWksxWjXE+wleqK9KOiPRHttrln7QCmksk2VGppk9 J1mBmf0eIMOvdUmZrseWkVCCQWMvw2jpRrYxdX3HtLV4woIVHfDCcqnC/odx0XZDw5H3 xZ0w==
X-Gm-Message-State: AOAM531uPSglWQMM9LZazmjygm48o1iUugs6n3ZvoRkHfVv+OO30zM5f cOaQJwBLQ9fFmPJqtpiW1lMkXdAwLpv0bBdDFqSuZVud
X-Google-Smtp-Source: ABdhPJyAPnWCTkLGW5KkVpiP7dlttU9+U1NHi7B24TRxAaaKIiKpefAuBAgOHZg3G99PLkpJF6kgKgDTwI1PprYnN2k=
X-Received: by 2002:a05:6638:32a2:: with SMTP id f34mr9783509jav.63.1639069283095; Thu, 09 Dec 2021 09:01:23 -0800 (PST)
MIME-Version: 1.0
References: <162506658420.19991.11377593316166696535@ietfa.amsl.com>
In-Reply-To: <162506658420.19991.11377593316166696535@ietfa.amsl.com>
From: Dhruv Dhody <dhruv.ietf@gmail.com>
Date: Thu, 09 Dec 2021 22:30:47 +0530
Message-ID: <CAB75xn5M8imdO_tf-2kzSeBtSyfZ3vPXvEzj250Jw0SB_=hxdQ@mail.gmail.com>
To: Zaheduzzaman Sarker <Zaheduzzaman.Sarker@ericsson.com>
Cc: The IESG <iesg@ietf.org>, draft-ietf-ntp-yang-data-model@ietf.org, ntp-chairs@ietf.org, NTP WG <ntp@ietf.org>, Dieter Sibold <dsibold.ietf@gmail.com>
Content-Type: multipart/alternative; boundary="00000000000061b69d05d2b98eb1"
Archived-At: <https://mailarchive.ietf.org/arch/msg/ntp/zDJkgJSsp2TQeIVFL2VH85WF7GE>
Subject: Re: [Ntp] Zaheduzzaman Sarker's No Objection on draft-ietf-ntp-yang-data-model-15: (with 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, 09 Dec 2021 17:01:31 -0000

Hi Zaheduzzaman,

Firstly let me apologize for this very late reply. Secondly thanks for your
review. I was finally able to make all the changes.

On Wed, Jun 30, 2021 at 8:53 PM Zaheduzzaman Sarker via Datatracker <
noreply@ietf.org> wrote:

> Zaheduzzaman Sarker has entered the following ballot position for
> draft-ietf-ntp-yang-data-model-15: No Objection
>
> 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/
>
>
>
> ----------------------------------------------------------------------
> COMMENT:
> ----------------------------------------------------------------------
>
> Thanks for the efforts on this document.
>
> I support Éric Vyncke's discuss.
>
> I also think adding reference to RFC 1305 in section 5 and section 6
> creates a
> bit of confusion when this document says it defines YANG model for RFC
> 5905.
> Also it is a fact that RFC 1305 is obsoleted by RFC 5905. So is this yang
> model
> also applicable to NTPv3? if not please mention, if so please mention and
> consider removing references to RFC 1305.
>
>
>
>
Dhruv: Since the YANG model allows the version to be set to 3, it can be
used to configure NTPv3. I have added text to make this clearer.

Thanks!
Dhruv

Diff:
https://www.ietf.org/rfcdiff?url1=draft-ietf-ntp-yang-data-model-15&url2=https://raw.githubusercontent.com/dhruvdhody/ietf/master/draft-ietf-ntp-yang-data-model-16.txt
Working Copy:
https://raw.githubusercontent.com/dhruvdhody/ietf/master/draft-ietf-ntp-yang-data-model-16.txt

Consolidated Review Status:
https://notes.ietf.org/draft-ietf-ntp-yang-data-model