Re: [Teas] I-D Action: draft-ietf-teas-yang-path-computation-09.txt

Italo Busi <Italo.Busi@huawei.com> Sat, 11 July 2020 08:31 UTC

Return-Path: <Italo.Busi@huawei.com>
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 992973A0890 for <teas@ietfa.amsl.com>; Sat, 11 Jul 2020 01:31:20 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=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 bkD6_VoiEQZy for <teas@ietfa.amsl.com>; Sat, 11 Jul 2020 01:31:18 -0700 (PDT)
Received: from huawei.com (lhrrgout.huawei.com [185.176.76.210]) (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 3FA8B3A088C for <teas@ietf.org>; Sat, 11 Jul 2020 01:31:18 -0700 (PDT)
Received: from lhreml722-chm.china.huawei.com (unknown [172.18.7.106]) by Forcepoint Email with ESMTP id 2F821D1FE6D844CB4DCB; Sat, 11 Jul 2020 09:31:14 +0100 (IST)
Received: from fraeml710-chm.china.huawei.com (10.206.15.59) by lhreml722-chm.china.huawei.com (10.201.108.73) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1913.5; Sat, 11 Jul 2020 09:31:13 +0100
Received: from fraeml715-chm.china.huawei.com (10.206.15.34) by fraeml710-chm.china.huawei.com (10.206.15.59) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1913.5; Sat, 11 Jul 2020 10:31:13 +0200
Received: from fraeml715-chm.china.huawei.com ([10.206.15.34]) by fraeml715-chm.china.huawei.com ([10.206.15.34]) with mapi id 15.01.1913.007; Sat, 11 Jul 2020 10:31:12 +0200
From: Italo Busi <Italo.Busi@huawei.com>
To: tom petch <ietfa@btconnect.com>, "teas@ietf.org" <teas@ietf.org>
CC: "Belotti, Sergio (Nokia - IT/Vimercate)" <sergio.belotti@nokia.com>
Thread-Topic: I-D Action: draft-ietf-teas-yang-path-computation-09.txt
Thread-Index: AQHWQNW3/PO3N7NmdUe4cS/zY2X/gajZzklAgAEi5d2AIQra8IAAGXE2gATO80A=
Date: Sat, 11 Jul 2020 08:31:12 +0000
Message-ID: <532251d6a0a64eb4b4f1cb02cd63ed38@huawei.com>
References: <159197894856.11807.5837352993917365680@ietfa.amsl.com>, <AM6PR07MB5222C7C2ED957AD873A0C372919C0@AM6PR07MB5222.eurprd07.prod.outlook.com> <DB7PR07MB5340340D025E2A23C71EB5CDA29D0@DB7PR07MB5340.eurprd07.prod.outlook.com>, <3fba272cce3144fc9d5c24970297a128@huawei.com> <DB7PR07MB53406FC305C54E5ACCFB6010A2660@DB7PR07MB5340.eurprd07.prod.outlook.com>
In-Reply-To: <DB7PR07MB53406FC305C54E5ACCFB6010A2660@DB7PR07MB5340.eurprd07.prod.outlook.com>
Accept-Language: it-IT, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-originating-ip: [10.47.94.143]
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-CFilter-Loop: Reflected
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/Ujjmhb3MZg8QWo7mxpemD-o0fnU>
Subject: Re: [Teas] I-D Action: draft-ietf-teas-yang-path-computation-09.txt
X-BeenThere: teas@ietf.org
X-Mailman-Version: 2.1.29
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: Sat, 11 Jul 2020 08:31:21 -0000

Hi Tom,

Thanks a lot for your review and valuable comments.

We have just posted a new -10 revision addressing them.

Please find in-line below a couple of additional comments/questions that deserve further discussion.

Sergio and Italo

> -----Original Message-----
> From: tom petch [mailto:ietfa@btconnect.com]
> Sent: martedì 7 luglio 2020 13:01
> To: Italo Busi <Italo.Busi@huawei.com>om>; Belotti, Sergio (Nokia - IT/Vimercate)
> <sergio.belotti@nokia.com>om>; teas@ietf.org
> Cc: TEAS WG Chairs <teas-chairs@ietf.org>
> Subject: Re: I-D Action: draft-ietf-teas-yang-path-computation-09.txt
> 
> From: Italo Busi <Italo.Busi@huawei.com>
> Sent: 07 July 2020 10:34
> 
> Hi Tom,
> 
> The length and complexity of the document seems a bit unavoidable
> considering the complexity of the problem statement (computing paths for TE
> tunnels which can be unidir/bidir/co-routed/protected/restored/...)
> 
> Regarding the open issue depending on te-types, we can remove the
> comment pending on updated on te-types since it is already an RFC and leave
> the encoding-and-switching-type and tunnel-common-attributes groupings in
> this model.
> 
> We think that the content of the draft is quite stable and that few minor
> changes regarding the YANG code are needed to address the pending open
> issues.
> 
> We would like to get some feedbacks from the WG and YANG experts during
> YANG doctor review and/or WG LC to address them.
> 
> <tp>
> OK.  For a start
> 
> RFC 3945 needs adding to I-D References
> 
> ds or datastore?  I like consistency
> 
> RPC are validated against <operational>
> as NMDA says. RPC are never configuration.  I think that /te:input/ is required
> in the path.
> 

[[Italo Busi] ] With the /te:input/, the code will not pass compilation with pyang 2.1.
Removing it, the code will not pass compilation with pyang 1.7.5 and it seems that there is no way to constraints some data on the RPC output based on same attribute in the RPC input.

> Indentation of section headings  in the text is inconsistent second level more
> than third.
> 
> I-D authors and YANG authors are different
> 
> IANA URI needs a contact
> 
> Most of the I-D references lack a DOI
> 

[[Italo Busi] ] We do not know how to get the DOI for I-D references ...

> Tom Petch
> 
> Sergio and Italo
> 
> > -----Original Message-----
> > From: tom petch [mailto:ietfa@btconnect.com]
> > Sent: martedì 16 giugno 2020 10:42
> > To: Belotti, Sergio (Nokia - IT/Vimercate) <sergio.belotti@nokia.com>om>;
> > teas@ietf.org
> > Cc: TEAS WG Chairs <teas-chairs@ietf.org>rg>; Italo Busi
> > <Italo.Busi@huawei.com>
> > Subject: Re: I-D Action: draft-ietf-teas-yang-path-computation-09.txt
> >
> > From: Teas <teas-bounces@ietf.org> on behalf of Belotti, Sergio (Nokia
> > -
> > IT/Vimercate) <sergio.belotti@nokia.com>
> > Sent: 15 June 2020 16:23
> >
> > we have updated the path computation RPC model to:
> > - align with the latest version of the TE tunnel YANG model
> > (https://tools.ietf.org/html/draft-ietf-teas-yang-te-23)
> > - add support of path computation for protected paths (as discussed in
> > IETF
> > 104)
> > - add support of path computation for bidirectional paths
> > - returning errors in case path computation fails
> > - add support for multi-layer path computation
> >
> > The changes to the YANG model are described in section 5.3 and 5.4 of
> > the 09 version of the draft.
> >
> > <tp>
> > Try again.
> >
> > I had a look a the I-D (big, complicated:-) and see a number of open
> > issues, require-instance, RPC reference, waiting for a te-types update!
> >
> > What do you plan to do about those?
> >
> >  I have not tracked the progress of this I-D and would like it to be
> > +-stable before investing too  much time in it (having wasted an ocean
> > on the unstable teas-te:-(
> >
> > Tom Petch.
> >
> >
> >
> > We think that with these changes the model is now ready for YANG
> > doctor review, however due to the significant amount of changes we
> > would appreciate a review from the WG before asking YANG doctor review.
> >
> > Thanks
> > Italo and Sergio (as editors)
> >
> > -----Original Message-----
> > From: I-D-Announce <i-d-announce-bounces@ietf.org> On Behalf Of
> > internet- drafts@ietf.org
> > Sent: Friday, June 12, 2020 6:22 PM
> > To: i-d-announce@ietf.org
> > Cc: teas@ietf.org
> > Subject: I-D Action: draft-ietf-teas-yang-path-computation-09.txt
> >
> >
> > A New Internet-Draft is available from the on-line Internet-Drafts directories.
> > This draft is a work item of the Traffic Engineering Architecture and
> > Signaling WG of the IETF.
> >
> >         Title           : Yang model for requesting Path Computation
> >         Authors         : Italo Busi
> >                           Sergio Belotti
> >                           Victor Lopez
> >                           Anurag Sharma
> >                           Yan Shi
> >         Filename        : draft-ietf-teas-yang-path-computation-09.txt
> >         Pages           : 94
> >         Date            : 2020-06-12
> >
> > Abstract:
> >    There are scenarios, typically in a hierarchical SDN context, where
> >    the topology information provided by a TE network provider may not
> >    be sufficient for its client to perform end-to-end path computation.
> >    In these cases the client would need to request the provider to
> >    calculate some (partial) feasible paths.
> >
> >    This document defines a YANG data model for an RPC to request path
> >    computation. This model complements the solution, defined in
> >    RFCXXXX, to configure a TE Tunnel path in "compute-only" mode.
> >
> >    [RFC EDITOR NOTE: Please replace RFC XXXX with the RFC number of
> >    draft-ietf-teas-yang-te once it has been published.
> >
> >    Moreover this document describes some use cases where a path
> >    computation request, via YANG-based protocols (e.g., NETCONF or
> >    RESTCONF), can be needed.
> >
> >
> > The IETF datatracker status page for this draft is:
> > https://datatracker.ietf.org/doc/draft-ietf-teas-yang-path-computation
> > /
> >
> > There are also htmlized versions available at:
> > https://tools.ietf.org/html/draft-ietf-teas-yang-path-computation-09
> > https://datatracker.ietf.org/doc/html/draft-ietf-teas-yang-path-comput
> > ation-
> > 09
> >
> > A diff from the previous version is available at:
> > https://www.ietf.org/rfcdiff?url2=draft-ietf-teas-yang-path-computatio
> > n-09
> >
> >
> > Please note that it may take a couple of minutes from the time of
> > submission until the htmlized version and diff are available at tools.ietf.org.
> >
> > Internet-Drafts are also available by anonymous FTP at:
> > ftp://ftp.ietf.org/internet-drafts/
> >
> >
> > _______________________________________________
> > I-D-Announce mailing list
> > I-D-Announce@ietf.org
> > https://www.ietf.org/mailman/listinfo/i-d-announce
> > Internet-Draft directories: http://www.ietf.org/shadow.html or
> > ftp://ftp.ietf.org/ietf/1shadow-sites.txt
> >
> > _______________________________________________
> > Teas mailing list
> > Teas@ietf.org
> > https://www.ietf.org/mailman/listinfo/teas