[Int-dir] Intdir early review of draft-ietf-tictoc-1588v2-yang-07

Dave Thaler <dthaler@microsoft.com> Thu, 14 June 2018 21:59 UTC

Return-Path: <dthaler@microsoft.com>
X-Original-To: int-dir@ietf.org
Delivered-To: int-dir@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id EC4C0130F3C; Thu, 14 Jun 2018 14:59:17 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Dave Thaler <dthaler@microsoft.com>
To: int-dir@ietf.org
Cc: draft-ietf-tictoc-1588v2-yang.all@ietf.org, ietf@ietf.org, tictoc@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 6.81.2
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <152901355790.26571.11611749129899888335@ietfa.amsl.com>
Date: Thu, 14 Jun 2018 14:59:17 -0700
Archived-At: <https://mailarchive.ietf.org/arch/msg/int-dir/T2hC6ukm8bmSULDwlGmZhJHKvBg>
Subject: [Int-dir] Intdir early review of draft-ietf-tictoc-1588v2-yang-07
X-BeenThere: int-dir@ietf.org
X-Mailman-Version: 2.1.26
List-Id: "This list is for discussion between the members of the Internet Area directorate." <int-dir.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/int-dir>, <mailto:int-dir-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/int-dir/>
List-Post: <mailto:int-dir@ietf.org>
List-Help: <mailto:int-dir-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/int-dir>, <mailto:int-dir-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 14 Jun 2018 21:59:19 -0000

Reviewer: Dave Thaler
Review result: Ready with Nits

I have reviewed draft-ietf-tictoc-1588v2-yang-07 as assigned int-dir reviewer.
The draft was readable and technically good (though I am not an expert on
either YANG or IEEE 1588-2008. I do have a number of nits indicated in the
marked up copy at
https://www.microsoft.com/en-us/research/uploads/prod/2018/06/draft-ietf-tictoc-1588v2-yang-07.pdf

The most notable piece of feedback is that it's not clear what the units are
for several integer objects.

Dave