[Lsr] Rtgdir last call review of draft-ietf-lsr-yang-isis-reverse-metric-01

Michael Richardson via Datatracker <noreply@ietf.org> Mon, 07 December 2020 02:49 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: lsr@ietf.org
Delivered-To: lsr@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id BA8483A0E7F; Sun, 6 Dec 2020 18:49:50 -0800 (PST)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Michael Richardson via Datatracker <noreply@ietf.org>
To: rtg-dir@ietf.org
Cc: draft-ietf-lsr-yang-isis-reverse-metric.all@ietf.org, last-call@ietf.org, lsr@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 7.23.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <160730939072.4448.14205081033305726081@ietfa.amsl.com>
Reply-To: Michael Richardson <mcr+ietf@sandelman.ca>
Date: Sun, 06 Dec 2020 18:49:50 -0800
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsr/JKNghwKgpM0qAT4Buh-ZfoD_dAc>
Subject: [Lsr] Rtgdir last call review of draft-ietf-lsr-yang-isis-reverse-metric-01
X-BeenThere: lsr@ietf.org
X-Mailman-Version: 2.1.29
List-Id: Link State Routing Working Group <lsr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lsr>, <mailto:lsr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lsr/>
List-Post: <mailto:lsr@ietf.org>
List-Help: <mailto:lsr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lsr>, <mailto:lsr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 07 Dec 2020 02:49:51 -0000

Reviewer: Michael Richardson
Review result: Ready

This document is a short YANG module relating to RFC8500, and expertise in that
RFC are required to be sure if all the right control bits are required. It has
reasonable security consideration, although the words, "The lowest NETCONF
layer is the secure transport layer" seems a bit awkward. I would instead
write, "All [NETCONF] transactions run over a secure transport layer, which is
SSH"...