[RTG-DIR]Re: Rtgdir last call review of draft-ietf-teas-actn-vn-yang-28

Dhruv Dhody <dhruv.ietf@gmail.com> Thu, 13 June 2024 07:17 UTC

MIME-Version: 1.0
References: <171809792424.23886.3995614610449789087@ietfa.amsl.com>
In-Reply-To: <171809792424.23886.3995614610449789087@ietfa.amsl.com>
From: Dhruv Dhody <dhruv.ietf@gmail.com>
Date: Thu, 13 Jun 2024 12:46:51 +0530
Message-ID: <CAB75xn5nXd867Kvh8=uEVpbTY4GHrfbzZeSvgtoT4i0Z4cQy_g@mail.gmail.com>
To: Susan Hares <shares@ndzh.com>
Content-Type: multipart/alternative; boundary="000000000000a122bf061ac04a46"
Message-ID-Hash: 5X3XM4JR54GFXVVTUZMYAA5DWQHZTPHK
CC: rtg-dir@ietf.org, draft-ietf-teas-actn-vn-yang.all@ietf.org, last-call@ietf.org, teas@ietf.org
Precedence: list
Subject: [RTG-DIR]Re: Rtgdir last call review of draft-ietf-teas-actn-vn-yang-28
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-dir/7jGWxzFfUXWN5D7UHP0adzu3GAM>

Hi Sue,

Thanks for your review.

On Tue, Jun 11, 2024 at 2:55 PM Susan Hares via Datatracker <
noreply@ietf.org> wrote:

> 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";
>      }
>
>
Dhruv: Added '.' to all description clauses.



> #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./
>
>
Dhruv: Ack, updated in my local copy!

Thanks!
Dhruv

[RTG-DIR]Rtgdir last call review of draft-ietf-te… Susan Hares via Datatracker
[RTG-DIR]Re: Rtgdir last call review of draft-iet… Dhruv Dhody