[Teas] Rtgdir last call review of draft-ietf-teas-actn-vn-yang-28

Susan Hares via Datatracker <noreply@ietf.org> Tue, 11 June 2024 09:25 UTC

Return-Path: <noreply@ietf.org>
X-Original-To: teas@ietf.org
Delivered-To: teas@ietfa.amsl.com
Received: from ietfa.amsl.com (localhost [IPv6:::1]) by ietfa.amsl.com (Postfix) with ESMTP id 41A4FC1F7051; Tue, 11 Jun 2024 02:25:24 -0700 (PDT)
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit
From: Susan Hares via Datatracker <noreply@ietf.org>
To: rtg-dir@ietf.org
X-Test-IDTracker: no
X-IETF-IDTracker: 12.15.0
Auto-Submitted: auto-generated
Precedence: bulk
Message-ID: <171809792424.23886.3995614610449789087@ietfa.amsl.com>
Date: Tue, 11 Jun 2024 02:25:24 -0700
Message-ID-Hash: TIUHDBL7234YSV2M5VD2PF32CCQCKVYB
X-Message-ID-Hash: TIUHDBL7234YSV2M5VD2PF32CCQCKVYB
X-MailFrom: noreply@ietf.org
X-Mailman-Rule-Misses: dmarc-mitigation; no-senders; approved; emergency; loop; banned-address; member-moderation; header-match-teas.ietf.org-0; nonmember-moderation; administrivia; implicit-dest; max-recipients; max-size; news-moderation; no-subject; digests; suspicious-header
CC: draft-ietf-teas-actn-vn-yang.all@ietf.org, last-call@ietf.org, teas@ietf.org
X-Mailman-Version: 3.3.9rc4
Reply-To: Susan Hares <shares@ndzh.com>
Subject: [Teas] Rtgdir last call review of draft-ietf-teas-actn-vn-yang-28
List-Id: Traffic Engineering Architecture and Signaling working group discussion list <teas.ietf.org>
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/SfAZZaIBFUgeqY__HfauOGM7eO8>
List-Archive: <https://mailarchive.ietf.org/arch/browse/teas>
List-Help: <mailto:teas-request@ietf.org?subject=help>
List-Owner: <mailto:teas-owner@ietf.org>
List-Post: <mailto:teas@ietf.org>
List-Subscribe: <mailto:teas-join@ietf.org>
List-Unsubscribe: <mailto:teas-leave@ietf.org>

Reviewer: Susan Hares
Review result: Has Nits

Status: 
General Comment: Thank you for your careful work on this document. 
Ready with Editorial Nits: 

#1 page 25 - add period to sentence 

     identity vn-computation-error-not-ready {
       base vn-computation-error-reason;
       description
         "VN computation has failed because the MDSC is not
          ready";
     }

#2 Security section: Grammar 
Old/
   *  ap: this list includes a set of sensitive data that influences how
      the access points in the VN service are attached.   /
New:/ 
   *  ap: This list includes a set of sensitive data that influences how
      the access points in the VN service are attached.   /

Old:/
   *  vn-ap: this list includes a set of sensitive data that influences
      how the VN service is delivered. /
New:/
   *  vn-ap: This list includes a set of sensitive data that influences
      how the VN service is delivered.
/
Old:/
   *  vn: this list includes a set of sensitive data that influences how
      the VN service is delivered.  By accessing the following data
      nodes, an attacker may be able to manipulate the VN./
New:/
   *  vn: This list includes a set of sensitive data that influences how
      the VN service is delivered.  By accessing the following data
      nodes, an attacker may be able to manipulate the VN./
Old:/
   *  vn-member: this list includes a set of sensitive data that
      influences how the VN member in the VN service is delivered.  By
      accessing the following data nodes, an attacker may be able to
      manipulate the VN member./
New:/
   *  vn-member: This list includes a set of sensitive data that
      influences how the VN member in the VN service is delivered.  By
      accessing the following data nodes, an attacker may be able to
      manipulate the VN member./