Re: [spring] Last Call: <draft-ietf-spring-sr-yang-23.txt> (YANG Data Model for Segment Routing) to Proposed Standard

"Acee Lindem (acee)" <acee@cisco.com> Mon, 23 November 2020 17:27 UTC

Return-Path: <acee@cisco.com>
X-Original-To: spring@ietfa.amsl.com
Delivered-To: spring@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id D3C3C3A098A; Mon, 23 Nov 2020 09:27:45 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -7.701
X-Spam-Level:
X-Spam-Status: No, score=-7.701 tagged_above=-999 required=5 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=g4YBUa3q; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=whNwOo+4
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 tyhSKp2bSGn8; Mon, 23 Nov 2020 09:27:43 -0800 (PST)
Received: from alln-iport-6.cisco.com (alln-iport-6.cisco.com [173.37.142.93]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 850E43A08C0; Mon, 23 Nov 2020 09:27:43 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=6926; q=dns/txt; s=iport; t=1606152463; x=1607362063; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-id:content-transfer-encoding: mime-version; bh=+lvEWiaW+pMKbhIH91B1ZWkRJ9XvceOtRMpnDQf2L4I=; b=g4YBUa3q2+E7qqZVXQrLc3SFlXzsquvT8WTaWpnCUtdtay1iA16RtCvk M6XGVVDLPidt0UVZwzejPjvOemb3kUFt/Uf727CugqEyzsBpXpDeaFU86 lxnzXUNHDJoeZt7BoXi7jucdb6/ZNSd7xxbhc7oD6/kMLshngxHVpQSqb I=;
X-IPAS-Result: =?us-ascii?q?A0DiCADV77tffZNdJa1iHgEBCxIMQIMhUXtZLy4KhDODS?= =?us-ascii?q?QONWZkEglMDVAsBAQENAQEYCwoCBAEBhEoCF4IUAiU4EwIDAQEBAwIDAQEBA?= =?us-ascii?q?QUBAQECAQYEFAEBhjwMhXMBAQEDAQEQEREMAQEsCwEPAgEIGAICHwcCAgIlC?= =?us-ascii?q?xUQAgQBDQUigwQBglUDLgEOpHkCgTyIaHaBMoMEAQEFgUdBgxkYghADBoEOK?= =?us-ascii?q?oJzg3aGVxuCAIERJxyCTz6CXQEBAwGBFR4/gwAzgiyTcYdInREKgm6JE5ILA?= =?us-ascii?q?x+DGooZlFIdkz+LBJVcAgQCBAUCDgEBBYFrIYFZcBUaISoBgj5QFwINjh8MF?= =?us-ascii?q?xSDOoUUhUR0NwIGAQkBAQMJfIsHgTQBgRABAQ?=
IronPort-PHdr: =?us-ascii?q?9a23=3ASZmFXRL8vXDv58kL29mcpTVXNCE6p7X5OBIU4Z?= =?us-ascii?q?M7irVIN76u5InmIFeGv68/hkPCWoPd5vlYzeHRtvOoVW8B5MOHt3YPONxJWg?= =?us-ascii?q?QegMob1wonHIaeCEL9IfKrCk5yHMlLWFJ/uX3uN09TFZX0e1bVpHu/5iJUER?= =?us-ascii?q?L6ZkJ5I+3vEdvUiMK6n+m555zUZVBOgzywKbN/JRm7t0PfrM4T1IBjMa02jB?= =?us-ascii?q?DOpyhF?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-AV: E=Sophos;i="5.78,364,1599523200"; d="scan'208";a="634966063"
Received: from rcdn-core-11.cisco.com ([173.37.93.147]) by alln-iport-6.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 23 Nov 2020 17:27:42 +0000
Received: from XCH-RCD-003.cisco.com (xch-rcd-003.cisco.com [173.37.102.13]) by rcdn-core-11.cisco.com (8.15.2/8.15.2) with ESMTPS id 0ANHRgRl028961 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Mon, 23 Nov 2020 17:27:42 GMT
Received: from xhs-rcd-001.cisco.com (173.37.227.246) by XCH-RCD-003.cisco.com (173.37.102.13) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 23 Nov 2020 11:27:42 -0600
Received: from xhs-aln-002.cisco.com (173.37.135.119) by xhs-rcd-001.cisco.com (173.37.227.246) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Mon, 23 Nov 2020 11:27:41 -0600
Received: from NAM10-BN7-obe.outbound.protection.outlook.com (173.37.151.57) by xhs-aln-002.cisco.com (173.37.135.119) with Microsoft SMTP Server (TLS) id 15.0.1497.2 via Frontend Transport; Mon, 23 Nov 2020 11:27:41 -0600
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=A3njAVUbXmvwaRRXPNLS4dETK6Vo5zgN2UKz848MUL2Kwxp9tDJGriUXP3EfAuBPXEHnqco3Ol8KCol8+CfPYoXSLngbi3VnGgRSDmO24X6t4zZcOCDcktQTTC7hPnEXnlT7jKhrB4i5GkhgPOgAcNhMKhZWXEOMR7EojKjREv76c1QkzLayBBazLZOkm/5b5LZ79GBB751edMQ+touf1CumcgvUpeBbd8AM3nZD5tFUPfB0xwZPCWS4JBDf7H6N+qTjWfwMrlC8UjDeDMAeIwbojp4pVTmwuoj9vALJil+GbF6chLT/CYZh2hE3UQ4u0H4zYw4R6EyQKBn5OL6QYw==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+lvEWiaW+pMKbhIH91B1ZWkRJ9XvceOtRMpnDQf2L4I=; b=M1QjDrc/zoBx23ig1j8X32McJUzKyNkN4S02JS+aTB2HX+83ZibPxwdIK+zeX034vC6XJbCofp193MwU6jAyHSsC7Ebl6uqRaA6FqEgCDcAOqMsg8/eNub9H2dTZt3iy0IAWm6OSDeATAqHbRUYQdXmH1wgqK3ZZcQgejlumEzspca0NJGXhj8TaPpGDrpt282+m7LbII0pQOB/h4dnmN8BEWt+uacjhw6jkPVa5HhrcIlI2QRN7a2eRh04xUhLDo6y1Bsg2fd2HFO3LtaESsFvBJAAqOv9oFIxPeDhng6HU/NcUTLVAfx4RJxvG3WzW2AkA22uJOh/dLAODsGPwxg==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+lvEWiaW+pMKbhIH91B1ZWkRJ9XvceOtRMpnDQf2L4I=; b=whNwOo+4xqid2VYqxddGNovyS5FMMfYN6F4wVQWd6BCD4H0Sh8QzXOONqBQglIyuVO3ZjfPJkSqpJ7RotOlFjwgz7LKXQA7fbkSSZUbQd/KAaMopTUiqq5cTPmyM+hgeERT1LbgLA+G390SXHP65N6S4vaGRIl0+7sSWxmMYttU=
Received: from BYAPR11MB2887.namprd11.prod.outlook.com (2603:10b6:a03:89::27) by SJ0PR11MB5152.namprd11.prod.outlook.com (2603:10b6:a03:2ae::22) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3589.22; Mon, 23 Nov 2020 17:27:40 +0000
Received: from BYAPR11MB2887.namprd11.prod.outlook.com ([fe80::1ddc:cdb4:32cc:f078]) by BYAPR11MB2887.namprd11.prod.outlook.com ([fe80::1ddc:cdb4:32cc:f078%3]) with mapi id 15.20.3589.025; Mon, 23 Nov 2020 17:27:40 +0000
From: "Acee Lindem (acee)" <acee@cisco.com>
To: tom petch <daedulus@btconnect.com>, "last-call@ietf.org" <last-call@ietf.org>
CC: "spring-chairs@ietf.org" <spring-chairs@ietf.org>, "spring@ietf.org" <spring@ietf.org>, Joel Halpern <jmh@joelhalpern.com>, "draft-ietf-spring-sr-yang@ietf.org" <draft-ietf-spring-sr-yang@ietf.org>
Thread-Topic: Last Call: <draft-ietf-spring-sr-yang-23.txt> (YANG Data Model for Segment Routing) to Proposed Standard
Thread-Index: AQHWvE9TnHz4xC0KAECDY9QT1Wf1oanN1wuAgAfYZQA=
Date: Mon, 23 Nov 2020 17:27:40 +0000
Message-ID: <FE952290-CC70-4F30-8F22-6BC20D676FAB@cisco.com>
References: <160555515848.16672.7178345983262697681@ietfa.amsl.com> <5FB515F7.1020306@btconnect.com>
In-Reply-To: <5FB515F7.1020306@btconnect.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/16.43.20110804
authentication-results: btconnect.com; dkim=none (message not signed) header.d=none;btconnect.com; dmarc=none action=none header.from=cisco.com;
x-originating-ip: [136.56.133.70]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 9d776592-51ea-4799-8dc2-08d88fd5147e
x-ms-traffictypediagnostic: SJ0PR11MB5152:
x-microsoft-antispam-prvs: <SJ0PR11MB5152B1141A359038B0C287D2C2FC0@SJ0PR11MB5152.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:4303;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: modGgmSIXrpAdTV25xP6nq8S74nQuT49AXYWpfgh0H/M40pcIlPijyGcJzK2oxhueArgLyYVk7HQyCNFGNNqofRtDm+BA79HgdNRltFp78qCO7y0xsY+zjOL6CS1v0uF8n9V4HQ9oBjcJg7JiQ6b7u7Aj3y2BRnD/fq9KlYwQAj6y8wA8eOCmZkT5jNK72/9u7lo6EAh6RcijLjvqAG5/JYvssiN+ertH03CZxITmte+ATsKNruJvVvSXrKx9LdjNd+2jUBUjFrd5IUOpEwtctxiDlGLduVwhE+eEQwY0ebXYODo86saXNmGVoeFdRXDPsENsULPmE6uNAINRvrHEWINs60Bsu5X5XhhaXdaLUq5qCB0L61smP4HtemCq6p9fSQLdfZ6mbb6ysl61ovTiw==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BYAPR11MB2887.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(346002)(376002)(39860400002)(136003)(366004)(396003)(6512007)(71200400001)(76116006)(36756003)(5660300002)(66946007)(6486002)(4001150100001)(33656002)(966005)(8936002)(64756008)(66476007)(8676002)(478600001)(66446008)(4326008)(66556008)(2906002)(296002)(2616005)(86362001)(316002)(54906003)(6506007)(66574015)(186003)(110136005)(83380400001)(26005)(53546011); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: qJ04vQkVt24qvaFv4UYlULepNGmgkdEPW83d56EkORx7AY5XJNOgCIjp9t/iYEFHKvxfucrrS+cW0MUxI1JEmmK5fvhnBuIFqaSrUcORTowYKU0dVR5ar0vv/FP5s2/MTcrtVCSsyEcPlEaj+6vury1Gh6tLgcrEdTBDXI7m82P99aP/aI9Sr9cuxqJEQR4WiNqOSMQYzGWUlGD5NZ0agxU1PuS7XtDQ2JlzjSSeQ7Oee3WeajYP50MEGM8OrY4+/MCPqq0q3vhdt1KH6YbpaAsqv34qUhgWqFZGxI6vrNnw0ddQObLo8kC9v9wW8YfBT09E1zLhEP+pSDL52tUmkIzos7esUF3mAjIYQdn67UI5ZE5ljBX84RgvWX9lhIAhYi6qKiUhJX5nVXmIjQdWf0GLySRIZ51yhc76a/nz6adP8VsQJvqMd448V9LxnLS2o08nTpYAHwmLBY3ySu5w9eaYXkgKQQOgArw6I0HCwvJfRdBwxjGxC8TEbyYi4nUafh9We9NsCy3qJ/hB10+QUPYjUMgCy1BZdUOpyqTckZNxTeIzUsWDMAeZzK4hY+EY/JrpGUm2S2RlVDeRwfDMvQkJnd/TMUSB5hA4yNSMnMXEjkjNrXOCVXWzsls+hA88XAie24IW2twgilswmFTDwQ==
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-ID: <AF4996107F481A45A33AB0DF39EA310F@namprd11.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB2887.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9d776592-51ea-4799-8dc2-08d88fd5147e
X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Nov 2020 17:27:40.6290 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: NyTz0MP8mMCVaJ5ubMBZO517O0nSBc4H1WjGghRbfBmGIBeeryVTyuJapq1En02s
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR11MB5152
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.37.102.13, xch-rcd-003.cisco.com
X-Outbound-Node: rcdn-core-11.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/spring/fKZUtqKt40aYnXNtFrL5Rowa70M>
Subject: Re: [spring] Last Call: <draft-ietf-spring-sr-yang-23.txt> (YANG Data Model for Segment Routing) to Proposed Standard
X-BeenThere: spring@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "Source Packet Routing in NetworkinG \(SPRING\)" <spring.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/spring>, <mailto:spring-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/spring/>
List-Post: <mailto:spring@ietf.org>
List-Help: <mailto:spring-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/spring>, <mailto:spring-request@ietf.org?subject=subscribe>
X-List-Received-Date: Mon, 23 Nov 2020 17:27:50 -0000

Hi Tom, 

See a couple responses inline enclosed in <acee> and </acee>. We are addressing the rest of your comments. 

On 11/18/20, 7:39 AM, "tom petch" <daedulus@btconnect.com> wrote:

    IANA Considerations does not register the module names used in the modules

<acee>
This is in the IANA considerations... 

   This document registers a YANG module in the YANG Module Names
   registry [RFC6020].

      name: ietf-segment-routing-common
      namespace: urn:ietf:params:xml:ns:yang:ietf-segment-routing-common
      prefix: sr-cmn
      reference: RFC XXXX

      name: ietf-segment-routing
      namespace: urn:ietf:params:xml:ns:yang:ietf-segment-routing
      prefix: sr
      reference: RFC XXXX

      name: ietf-segment-routing
      namespace: urn:ietf:params:xml:ns:yang:ietf-segment-routing-mpls
      prefix: sr-mpls
      reference: RFC XXXX
</acee>

    Examples are IPv4 only, IPv6 would be good

    BGP is included when it comes to defining a router-id but is ignored 
    everywhere else, such as signalling MSD, protocol extensions etc

    reference "RFC XXXX" would be improved by including the title in all 
    cases not just some

    the scheme http: appears in many places.  It would be lovely if this 
    really was the scheme but I fear that it is not
<acee>
This is directly from the RFC 8407 template in Appendix B. What would you suggest? 
</acee>

    module srcmn
      the upper bound must be larger
      the value must be greater
    consistency is good - I think greater is better

    8.3
    operation states
    usually operational

    two imports lack references

    typedef router-id
    this is a well known type from RFC8394; it seems likely to confuse to 
    redefine it with a related but different meaning

    leaf enabled
    enables protocol extensions
    which protocols?

    leaf protected
    it is used to protect
    how does it do that:-)

    enum dual
    ... In this case will be advertised with backup flag set
    What is the backup flag?  It does not feature in RFC8660.  Needs an 
    explanation and reference

    container link-msd
      list link-msds
        leaf msd
    The usual YANG convention is for a list to be plural and the leaf 
    singular.  You have the plural list but not the leaf. 
<acee> 
So you are asking for a change from "leaf msd" to "leaf link-msd"? 
</acee>


 And who needs the 
    container?  This is mpls not a common module that might be augmented so 
    what does the container give apart from complexity?

    list policy
      leaf string
    YANG string caters for very large items of very complex character sets. 
      Is that desirable?
<acee>
IETF models normally do not limit identifiers. An individual implementation could do this with a deviation. 
</acee> 

Thanks,
Acee

    leaf used
    will used plus free equal size?

    Indicates if the binding is /instal/installed/

    notification-segment-routing-global-srgb-collision
    a mix of conflict and collision;  consistency is good and I prefer the 
    latter which is the name of the notification

    containing /s/a/ mapping

    ... sid collision
    again consistency good, prefer collision to conflicting

    s.9
    I would have thought the srgb worthy of mention under sensitive nodes

    Tom Petch


    On 16/11/2020 19:32, The IESG wrote:
    >
    > The IESG has received a request from the Source Packet Routing in Networking
    > WG (spring) to consider the following document: - 'YANG Data Model for
    > Segment Routing'
    >    <draft-ietf-spring-sr-yang-23.txt> as Proposed Standard
    >
    > The IESG plans to make a decision in the next few weeks, and solicits final
    > comments on this action. Please send substantive comments to the
    > last-call@ietf.org mailing lists by 2020-11-30. Exceptionally, comments may
    > be sent to iesg@ietf.org instead. In either case, please retain the beginning
    > of the Subject line to allow automated sorting.
    >
    > Abstract
    >
    >
    >     This document defines a YANG data model for segment routing
    >     configuration and operation, which is to be augmented by different
    >     segment routing data planes.  The document also defines a YANG model
    >     that is intended to be used on network elements to configure or
    >     operate segment routing MPLS data plane, as well as some generic
    >     containers to be reused by IGP protocol modules to support segment
    >     routing.
    >
    >
    >
    >
    > The file can be obtained via
    > https://datatracker.ietf.org/doc/draft-ietf-spring-sr-yang/
    >
    >
    >
    > No IPR declarations have been submitted directly on this I-D.
    >
    >
    >
    >
    >
    > _______________________________________________
    > IETF-Announce mailing list
    > IETF-Announce@ietf.org
    > https://www.ietf.org/mailman/listinfo/ietf-announce
    > .
    >