Re: [Teas] teas-yang-rsvp wasRe: TEAS WG Document Status Reports

Tarek Saad <tsaad.net@gmail.com> Fri, 04 September 2020 02:20 UTC

Return-Path: <tsaad.net@gmail.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 1D3493A1501; Thu, 3 Sep 2020 19:20:02 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.098
X-Spam-Level:
X-Spam-Status: No, score=-2.098 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=unavailable autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com
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 QF8MzUcn9pdW; Thu, 3 Sep 2020 19:19:59 -0700 (PDT)
Received: from mail-io1-xd30.google.com (mail-io1-xd30.google.com [IPv6:2607:f8b0:4864:20::d30]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id B59933A14FE; Thu, 3 Sep 2020 19:19:56 -0700 (PDT)
Received: by mail-io1-xd30.google.com with SMTP id b6so5641644iof.6; Thu, 03 Sep 2020 19:19:56 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:thread-topic:thread-index:date:message-id :references:in-reply-to:accept-language:content-language :content-transfer-encoding:mime-version; bh=3HfIt7adxQYw4QuX4bk8V0+KXMiMbDpp3K92pMe2jR8=; b=fgQWuZ4uBxywd8TDJHYQ+AFJgb+Skn1JnaxcYsUbNu6CjkCKVeJ497yBb63BVayP8x +ft1RspWp6pvp04DGGK+Ph1i1v/R1Vuy/JfZ0vbOLV1t3BAXdxpSYAeCvD53G7rt7mz7 NXg0j07xmIBV7h5xEV2JOm5TCrizE83FHXGO6QC1FucDhKE5hp6mV7LKFjTYbUgPYfOM HkAZxaFMVRDc4oBUGnnG+pcQW72KZYCMDhEhr8ssbDNB2PIoyweqGFnI3EinBmlVEVMe U//yglUTMIjN07Rpkk8K8TMppCeWXCiZZV0fuAJsUTE3QibFYdBclTZf7M4hWjNUAmcs 6kag==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:thread-topic:thread-index :date:message-id:references:in-reply-to:accept-language :content-language:content-transfer-encoding:mime-version; bh=3HfIt7adxQYw4QuX4bk8V0+KXMiMbDpp3K92pMe2jR8=; b=fUUI7H4FmZ0Pkd3jGaPp0SOj2aShu6ZOv5B2rxQizNECne0nLWjj9Z6J0NoGR7TKL7 3+xyvsqzafdxrLsXMtfcLHiyw7vQl7L/OxyTKPK92v2ht8yQGesC0VQ/pPuf77HC5eIY M3DsWkOA2HisXUAV5YBtwpf/2GQUoEIg9ErLD1QS3byW0Ap0focrHaddQzlBsBxKUZJ9 LR7ru6xKFXO2itNPAGfMdJM34xNipBplQpqRlEov6GRnYeUJAY3ao0F4e7jfPL6IlqvN 3UcspwIwCmrsC62LhHHexoyiwJRkqdZ3n2gwL1Ov2AASHqVueV3cY6kSsDhF7Y2RmPKs XUXw==
X-Gm-Message-State: AOAM533Xyt1UOllCZqgALTZZ4utHIN89Yn8KTnk0ouKUOtOSJxzakA2E 59T+OxCR+Y2S1myR94qM+8A=
X-Google-Smtp-Source: ABdhPJyqb3iNTwhfRcQM5VKsBjUKswMlqyTy1O6G479nctMUV+TyNoCMpO+UboZdm0caMrJ2A6TcUw==
X-Received: by 2002:a5e:930d:: with SMTP id k13mr5645077iom.40.1599185995835; Thu, 03 Sep 2020 19:19:55 -0700 (PDT)
Received: from DM5PR1901MB2150.namprd19.prod.outlook.com ([2603:1036:4:9e::5]) by smtp.gmail.com with ESMTPSA id z4sm1466838iol.52.2020.09.03.19.19.54 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Thu, 03 Sep 2020 19:19:54 -0700 (PDT)
From: Tarek Saad <tsaad.net@gmail.com>
To: tom petch <ietfa@btconnect.com>, Tarek Saad <tsaad=40juniper.net@dmarc.ietf.org>, "teas@ietf.org" <teas@ietf.org>
CC: "draft-ietf-teas-yang-rsvp@ietf.org" <draft-ietf-teas-yang-rsvp@ietf.org>
Thread-Topic: [Teas] teas-yang-rsvp wasRe: TEAS WG Document Status Reports
Thread-Index: ATA3NzE14Czfa0u6BRcZUxOQg2EcJzAyQjkwytsyOgCAOZERfw==
X-MS-Exchange-MessageSentRepresentingType: 1
Date: Fri, 4 Sep 2020 02:19:54 +0000
Message-ID: <DM5PR1901MB2150E95EFE5266A9711036BCFC2D0@DM5PR1901MB2150.namprd19.prod.outlook.com>
References: <B2787AE6-31CA-4DDA-88CC-7C3B6EF6B72F@juniper.net> <DB7PR07MB5340D7119E71B374A6E9D75CA2A70@DB7PR07MB5340.eurprd07.prod.outlook.com> <DM5PR1901MB21509E29E0A9B13A45FE2325FC720@DM5PR1901MB2150.namprd19.prod.outlook.com> <DB7PR07MB53406341349D4FCFD08A7399A2700@DB7PR07MB5340.eurprd07.prod.outlook.com>
In-Reply-To: <DB7PR07MB53406341349D4FCFD08A7399A2700@DB7PR07MB5340.eurprd07.prod.outlook.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-Exchange-Organization-SCL: -1
X-MS-TNEF-Correlator:
X-MS-Exchange-Organization-RecordReviewCfmType: 0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/svgji2sZumtk17VGOFJUREDH1sU>
Subject: Re: [Teas] teas-yang-rsvp wasRe: TEAS WG Document Status Reports
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: Fri, 04 Sep 2020 02:20:02 -0000

Hi Tom,

Thanks. We posted version -15 that addresses the new comments. Please see inline below.

On 7/29/20, 7:13 AM, "tom petch" <ietfa@btconnect.com> wrote:

    From: Tarek Saad <tsaad.net@gmail.com>
    Sent: 27 July 2020 13:29

    Hi Tom,

    Thanks for your review of ID: draft-ietf-teas-yang-rsvp and for the your comments. We have posted version https://tools.ietf.org/html/draft-ietf-teas-yang-rsvp-14
    Which addresses those comments. 

    <tp>
    Mostly:-)
    You still have  a prefix mismatch between module and IANA.
[TS]: I corrected the typo:
OLD:
      prefix:     rsvp-extendeed
NEW:
      prefix:     rsvp-extended

    I suspect you will get asked for examples at some stage 

    s.3.1 The augments diagrams use 'o' to show augmentation.  Other I-D use '>' but use it in different senses.  My take is 
    A----->B
    means A augments B.
[TS]: OK, replaced 'o' with '>'

    3.3 I think that filters need references.  Subtree is 6241 but XPATH is probably 8639 since that specifies the context thereof which is not obvious.
[TS]: I added the references above.

Regards,
Tarek

    Tom Petch









    Please see inline (look for TS) for more details.

    >>>>>>>>>>
    
        2. draft-ietf-teas-yang-rsvp-11
        Current Status:

        <tp>
        teas-yang-rsvp-12 would benefit from some tweaks
        Abstract confused me into thinking that the Extended model was elsewhere - I think that this needs an explicit mention.
    TS: revised abstract to address this.


        Introduction ditto.
    TS: revised Introduction to address this.

        Introduction should mention RFC7950 and RESTCONF

        Introduction should mention NMDA support
    TS: Addressed.

        1 is wrong
        rt-type I am not familiar with but rt-types is in RFC8294
        key-chain is RFC8177
    TS: corrected.

        2.4 lists two RPC but tree diagram lists three, one of which is authentication but authentication is a feature of extended not base - this confuses me
    TS: corrected and added the third.

        revision description
        should be 'Initial version' in both modules

        the rpc should probably be NACM default deny all
    TS: added.

        IANA Considerations
        URI lack registrant contact
        prefix are not those used in the modules
    TS: updated.

        Security
        alsocontrols
    TS: corrected.

        RFC5495 needs adding to I-D References
    TS: updated references.

    Again, thanks much for the comments.

    Regards,
    Tarek

        Tom Petch






           *   Version -10 was reviewed by YANG Dr. and received comments

          *   Authors published version -12 which addressed all outstanding comments
          *   Authors followed up with YANG Dr. reviewer on closing comments
        Open Issues:

          *   None
        Next Steps:

          *   Proceed to WGLC


        3. draft-ietf-teas-yang-rsvp-te-07
        Current Status:

          *   Version -07 was reviewed by YANG Dr. and received comments
        Open Issues:

          *   Authors working on addressing YANG Dr. comments
        Next Steps:

          *   Folllow-up with YANG Dr. reviewer to close on comments
          *   Proceed to WGLC after addressing

        4. draft-ietf-teas-yang-te-mpls
        Current Status:

          *   Authors discussing augmentation of this module for MPLS-TP
          *   A new module and model is expected
        Open Issues:

          *   Close on MPLS-TP modeling
        Next Steps:

          *   Publish a new revision
          *   Ask for YANG Dr. review



        _______________________________________________
        Teas mailing list
        Teas@ietf.org
        https://www.ietf.org/mailman/listinfo/teas