Re: [Teas] Benjamin Kaduk's No Objection on draft-ietf-teas-actn-info-model-09: (with COMMENT)

Benjamin Kaduk <kaduk@mit.edu> Fri, 22 June 2018 14:28 UTC

Return-Path: <kaduk@mit.edu>
X-Original-To: teas@ietfa.amsl.com
Delivered-To: teas@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id B4CAB130E86; Fri, 22 Jun 2018 07:28:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -4.201
X-Spam-Level:
X-Spam-Status: No, score=-4.201 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_DNSWL_MED=-2.3, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id MlDwuf5vMIYF; Fri, 22 Jun 2018 07:28:05 -0700 (PDT)
Received: from dmz-mailsec-scanner-2.mit.edu (dmz-mailsec-scanner-2.mit.edu [18.9.25.13]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 0DEBD130E74; Fri, 22 Jun 2018 07:28:04 -0700 (PDT)
X-AuditID: 1209190d-209ff700000059f6-9d-5b2d0773f4d0
Received: from mailhub-auth-2.mit.edu ( [18.7.62.36]) (using TLS with cipher DHE-RSA-AES256-SHA (256/256 bits)) (Client did not present a certificate) by dmz-mailsec-scanner-2.mit.edu (Symantec Messaging Gateway) with SMTP id 15.8C.23030.3770D2B5; Fri, 22 Jun 2018 10:28:03 -0400 (EDT)
Received: from outgoing.mit.edu (OUTGOING-AUTH-1.MIT.EDU [18.9.28.11]) by mailhub-auth-2.mit.edu (8.13.8/8.9.2) with ESMTP id w5MES2uC018518; Fri, 22 Jun 2018 10:28:03 -0400
Received: from kduck.kaduk.org (24-107-191-124.dhcp.stls.mo.charter.com [24.107.191.124]) (authenticated bits=56) (User authenticated as kaduk@ATHENA.MIT.EDU) by outgoing.mit.edu (8.13.8/8.12.4) with ESMTP id w5MERvdP019855 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NOT); Fri, 22 Jun 2018 10:28:00 -0400
Date: Fri, 22 Jun 2018 09:27:57 -0500
From: Benjamin Kaduk <kaduk@mit.edu>
To: Leeyoung <leeyoung@huawei.com>
Cc: The IESG <iesg@ietf.org>, "draft-ietf-teas-actn-info-model@ietf.org" <draft-ietf-teas-actn-info-model@ietf.org>, Vishnu Beeram <vbeeram@juniper.net>, "teas-chairs@ietf.org" <teas-chairs@ietf.org>, "teas@ietf.org" <teas@ietf.org>
Message-ID: <20180622142757.GD64617@kduck.kaduk.org>
References: <152952816841.28616.13487599041543688657.idtracker@ietfa.amsl.com> <7AEB3D6833318045B4AE71C2C87E8E173D01A803@sjceml521-mbx.china.huawei.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <7AEB3D6833318045B4AE71C2C87E8E173D01A803@sjceml521-mbx.china.huawei.com>
User-Agent: Mutt/1.9.1 (2017-09-22)
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFupjleLIzCtJLcpLzFFi42IRYrdT0S1m1402OP/MwuLGnelMFjP+TGS2 mDbP1aJp7i4mi9YfO1gsFqyZyezA5tFy5C2rx5IlP5k8rjddZQ9gjuKySUnNySxLLdK3S+DK uLt8GlPBQ92KySfOsTYwHlDqYuTkkBAwkTg65zQjiC0ksJhJ4uUbuy5GLiB7I6PExIkv2SCc q0wSM/ccZAapYhFQldj+/AgriM0moCLR0H0ZLC4ioCQxbcccsAZmgR4mif3bG8HGCgskScw5 fZCpi5GDgxdk3WttiKGzGCVuNz1nAanhFRCUODnzCZjNLKAlcePfS7B6ZgFpieX/OEBMToEw iYW9QSAVogLKEnv7DrFPYASagdA8C0nzLITmBYzMqxhlU3KrdHMTM3OKU5N1i5MT8/JSi3SN 9HIzS/RSU0o3MYKCmlOSdwfjv7tehxgFOBiVeHg1vupEC7EmlhVX5h5ilORgUhLlvVkBFOJL yk+pzEgszogvKs1JLT7EKMHBrCTC6/YUKMebklhZlVqUD5OS5mBREufNXsQYLSSQnliSmp2a WpBaBJOV4eBQkuCdzKYbLSRYlJqeWpGWmVOCkGbi4AQZzgM0PAukhre4IDG3ODMdIn+KUZej 4/2UHmYhlrz8vFQpcV4LkCIBkKKM0jy4OaBkJJG9v+YVozjQW8K8m0CqeICJDG7SK6AlTEBL qpu1QJaUJCKkpBoYFfnb116446DZulP3gDTT3cKdE+4e1/nS0d6495v2trtxBemsRVUTy8Qf t+Xol5y26PZwu5rQEtF8baVj0Lypdvd4XaO+PhJbNP3gW7ULtb12UWz/fyYssgycw/dg0tG3 F2Wu861OX5z++fuz8pcCbd9aH9c/Xya2qFRMTTGvVGNSdYeWxwU2JZbijERDLeai4kQAWLwc viEDAAA=
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/VpNPiuwXgOzvkZs7tkYFSdJpyys>
Subject: Re: [Teas] Benjamin Kaduk's No Objection on draft-ietf-teas-actn-info-model-09: (with COMMENT)
X-BeenThere: teas@ietf.org
X-Mailman-Version: 2.1.26
Precedence: list
List-Id: Traffic Engineering Architecture and Signaling working group discussion list <teas.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/teas>, <mailto:teas-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/teas/>
List-Post: <mailto:teas@ietf.org>
List-Help: <mailto:teas-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/teas>, <mailto:teas-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 22 Jun 2018 14:28:08 -0000

Hi Young,

These proposed changes look good to me.

Thanks,

Benjamin

On Wed, Jun 20, 2018 at 10:43:10PM +0000, Leeyoung wrote:
> Hi Benjamin,
> 
> 
> 
> Thanks for proving valuable comments and suggestions.
> 
> 
> 
> Please see inline for my response.
> 
> 
> 
> Thanks.
> 
> Young
> 
> 
> 
> -----Original Message-----
> From: Benjamin Kaduk [mailto:kaduk@mit.edu]
> Sent: Wednesday, June 20, 2018 3:56 PM
> To: The IESG <iesg@ietf.org>;
> Cc: draft-ietf-teas-actn-info-model@ietf.org; Vishnu Beeram <vbeeram@juniper.net>;; teas-chairs@ietf.org; vbeeram@juniper.net; teas@ietf.org
> Subject: Benjamin Kaduk's No Objection on draft-ietf-teas-actn-info-model-09: (with COMMENT)
> 
> 
> 
> Benjamin Kaduk has entered the following ballot position for
> 
> draft-ietf-teas-actn-info-model-09: No Objection
> 
> 
> 
> When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.)
> 
> 
> 
> 
> 
> Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html
> 
> for more information about IESG DISCUSS and COMMENT positions.
> 
> 
> 
> 
> 
> The document, along with other ballot positions, can be found here:
> 
> https://datatracker.ietf.org/doc/draft-ietf-teas-actn-info-model/
> 
> 
> 
> 
> 
> 
> 
> ----------------------------------------------------------------------
> 
> COMMENT:
> 
> ----------------------------------------------------------------------
> 
> 
> 
> I echo the comments questioning the archival value for publishing a document like this.
> 
> I think it's a good intermediate point to have, including the security considerations text (thank you for the quick update after the secdir review!), but am unsure what portion(s) of it could not be easily folded into successor documents.
> 
> 
> 
> YL>> Please see my comment to Ben on the archival value of this document.
> 
> 
> 
> Section 2
> 
> 
> 
>   This section provides an ACTN common interface information model to
> 
>    describe in terms of primitives, objects, their properties
> 
>    (represented as attributes), their relationships, and the resources
> 
>    for the service applications needed in the ACTN context.
> 
> 
> 
> nit: The grammar here is a bit odd -- as-is, it says we have a model "to describe in terms of [a list of things]", but don't say what is being described.  I'm not entirely sure what the intended text would be, though.
> 
> 
> 
> YL>> I agree. I would change as follows:
> 
> 
> 
> NEW: This section provides an ACTN common interface information model to
> 
>    describe primitives, objects, their properties
> 
>    (represented as attributes), their relationships, and the resources
> 
>    for the service applications needed in the ACTN context.
> 
> 
> 
> 
> 
> Section 3.2, 3.4
> 
> 
> 
> Do I understand correctly that Modify is for CNC-to-MDSC commmunication and Update is for MDSC-to-CNC communication?  Perhaps this could be made more explicit.
> 
> 
> 
> YL>> Yes, your understanding is correct. We will add explicit text that says: Modify is for CNC-to-MDSC communication in Section 3.2 and Update is for MDSC-to-CNC communication in Section 3.4.
> 
> 
> 
> Section 5.7.2
> 
> 
> 
>       [...] For example, permission
> 
>       the correct selection from the network of the destination related
> 
>       to the indicated VNF It is e.g. the case of VM migration among
> 
>       data center and CNC can enforce specific policy that can permit
> 
>       MDSC/PNC to calculate the correct path for the connectivity
> 
>       supporting the data center interconnection required by
> 
>       application.
> 
> 
> 
> There seems to be an editing error around "indicated VNF It is e.g.", maybe just a missing full stop and commas around "e.g."?
> 
> 
> 
> YL>> Yes, there is an editing error here.
> 
> OLD: For example, permission
> 
>       the correct selection from the network of the destination related
> 
>       to the indicated VNF It is e.g. the case of VM migration among
> 
>       data center and CNC can enforce specific policy that can permit
> 
>       MDSC/PNC to calculate the correct path for the connectivity
> 
>       supporting the data center interconnection required by
> 
>       application.
> 
> 
> 
> NEW: For example, CNC can enforce client-specific preferences, e.g.,
> 
>       selection of a destination data center from the set of candidate data
> 
>       centers based on some criteria in the context of VM migration.
> 
>       MSDC/PNC should then provide the data center interconnection that
> 
>       supports the client-specific preference.
> 
> 
> 
> Section 9
> 
> 
> 
>    The ACTN information model does not directly introduce security
> 
>    issues. Rather, it defines a set of interfaces for traffic
> 
> 
> 
> I would hope that no product of the IETF directly introduces security
> 
> *issues* (problems)!  Maybe "is not directly relevant when considering potential security issues" is some better wordsmithing.
> 
> 
> 
> YL>> Agree. Will change the sentence as you suggest.
> 
> 
> 
>    Implementations of the ACTN framework will have distributed
> 
>    functional components that will exchange this information model.
> 
> 
> 
> nit: Perhaps this is overly pedantic, but are they exchanging this information model, or a concrete instantiation that adheres to this model?
> 
> 
> 
> YL>> Agree. How about:
> 
> 
> 
> NEW:
> 
>    Implementations of the ACTN framework will have distributed
> 
>    functional components that will exchange a concrete instantiation
> 
>    that adheres to this information model.
> 
> 
> 
>