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: ietf@ietf.org
Delivered-To: ietf@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
Subject: Intdir early review of draft-ietf-tictoc-1588v2-yang-07
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/ietf/RoX7s6l3K3eNotMADj-LHRvOVIc>
X-BeenThere: ietf@ietf.org
X-Mailman-Version: 2.1.26
List-Id: IETF-Discussion <ietf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ietf>, <mailto:ietf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ietf/>
List-Post: <mailto:ietf@ietf.org>
List-Help: <mailto:ietf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ietf>, <mailto:ietf-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