Re: [spring] WG LC https://datatracker.ietf.org/doc/draft-ietf-spring-sr-yang/

Yingzhen Qu <yingzhen.qu@futurewei.com> Fri, 10 July 2020 06:50 UTC

Return-Path: <yingzhen.qu@futurewei.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 BC4773A0DCD; Thu, 9 Jul 2020 23:50:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.09
X-Spam-Level:
X-Spam-Status: No, score=-2.09 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, RCVD_IN_MSPIKE_H2=-0.001, T_SPF_PERMERROR=0.01, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=futurewei.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 Gjt3kxd5pNuM; Thu, 9 Jul 2020 23:50:06 -0700 (PDT)
Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2090.outbound.protection.outlook.com [40.107.237.90]) (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 CE3AA3A0D22; Thu, 9 Jul 2020 23:50:05 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=lljCUoPE4EXUKPtumzzpdAQ1FdfGewIagvOV5+ti3mcdYVvues6ZbXNNC3nAE/l6lenjSrlH7pNddK3T7HXFLw+EtYx7cDQmTwnGBmadTCDBMOl7iJS1DX3Q9P1eClK0Y3/lv6HQq8/oHnC3/Ktm8NK6FR5rBVJlTgulPFH/F++6r5HKIMJpacZzAC+TTn6iBvfqCDhp22MDfL+iwIEq2IX87d/QSgfhruRxzWfapXPSWoUPIin+JKqrvAIE/GhsjvnghlRgAdOsr7srAwylGNC5dE4hXHjYYaC135GVkBvbhq+O1Skco12rFcjxFoSXDXyLq5CBxdofidJKYynAyQ==
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=us6T0xEgY+qOexaJhDYuebRvgJPBonrs3ykdduAcPFY=; b=e2Z07kjGy+1gQEsR769mLYLsps+WK4ezf46MvEm1/Ez3nE7Tmt3padLIMjLutt9+uqRcUPlh/dCypWRBdSuZZ3dB6vqzCYQMexEKdCIIh1m3MezrHoVLzVPC5eOjtTJyv/RO4RfZ91BHsD+ydb2GkST46v/+m5RZAJh65t2SeoS/TPEiU01TofJnZNVzW2eS6lP86Z30oBZlj4gahZGKKD2JSudxZaXPn5ijXpY0kr5/5jflhC9upB40neUXwBdNhQSVeScGM95+kD4yaOqm+voDXZmBNEgaJ+NZg6pjuVP+4q0gTzD7YJSghwvbZFv1HHirb94+/iKR55bL1vZoMQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=futurewei.com; dmarc=pass action=none header.from=futurewei.com; dkim=pass header.d=futurewei.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Futurewei.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=us6T0xEgY+qOexaJhDYuebRvgJPBonrs3ykdduAcPFY=; b=FcHeXj7IBZtkr9bsLo9YorwLge1yDLQt58uAck0VKK4nCwhbM1rBr8xgX5UTv05Ipj8bE8BImImalFMgDGRI+IfiEmAxnc8mObUZ+z+LtrZrqxOYDoX4MMzxFyIN6LXdYEOadt19WW0iQYeKqBLZoI7o5brOe0AabkkRL6KO3fk=
Received: from BY5PR13MB3048.namprd13.prod.outlook.com (2603:10b6:a03:188::21) by BYAPR13MB2615.namprd13.prod.outlook.com (2603:10b6:a03:ff::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3174.16; Fri, 10 Jul 2020 06:49:59 +0000
Received: from BY5PR13MB3048.namprd13.prod.outlook.com ([fe80::28f0:8a33:3418:b39b]) by BY5PR13MB3048.namprd13.prod.outlook.com ([fe80::28f0:8a33:3418:b39b%4]) with mapi id 15.20.3195.010; Fri, 10 Jul 2020 06:49:59 +0000
From: Yingzhen Qu <yingzhen.qu@futurewei.com>
To: Dhruv Dhody <dhruv.ietf@gmail.com>, James Guichard <james.n.guichard@futurewei.com>
CC: "spring@ietf.org" <spring@ietf.org>, "spring-chairs@ietf.org" <spring-chairs@ietf.org>
Thread-Topic: [spring] WG LC https://datatracker.ietf.org/doc/draft-ietf-spring-sr-yang/
Thread-Index: AdZJhkNnJdHpUwcGRJuhHUzvqEDvhwLa6NmAAFC8WwA=
Date: Fri, 10 Jul 2020 06:49:58 +0000
Message-ID: <952978AA-F633-4102-9104-F0D9742A4021@futurewei.com>
References: <DM6PR13MB3066AF7A192440EBA63E11DBD2940@DM6PR13MB3066.namprd13.prod.outlook.com> <CAB75xn6t_dPeXuh2O8gdc4KWUm7FjZ3N1WVUPEvX2xng=4_yRA@mail.gmail.com>
In-Reply-To: <CAB75xn6t_dPeXuh2O8gdc4KWUm7FjZ3N1WVUPEvX2xng=4_yRA@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.1e.0.191013
authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=futurewei.com;
x-originating-ip: [2601:646:9500:c900:4592:765d:d738:365c]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 3e9885ed-c0e4-442a-c526-08d8249d76ab
x-ms-traffictypediagnostic: BYAPR13MB2615:
x-ms-exchange-transport-forked: True
x-microsoft-antispam-prvs: <BYAPR13MB261521AF3289D2186C0DB960E1650@BYAPR13MB2615.namprd13.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: BLM3Q5mjqcjKdtjMkGLueIxq81uHMB9p6zEPtRo4WWXJlLlcIQZkVE7/ZfbEgUAec/bgh/ckqNjfndDSFF6rPLR+ADb9kc3hJs2FXSH1tyWdDlKKhwTiqChSf0S86xckgTiBxiOrD7/2sRWLby7PilitRitv1noRIbDWNYXR5PfC/k1jG9pq+to0h00yCBht7/Uw5HVWA/9MzJx+ijLBcUg29Fy0w0GMI4HsmfS4MXUkBqcZMInuITxWusUHIUGXrPTYbHdXrL/E/2pNon570S2SJD31LkUG1LMoTlmF+J7irTILW0/rZuWfQqj7zSxvAopQPXhWFj+LiQqAuB6eaWZRFEVd0sxL8paV4xXGoXyH05vG63WKlistkpUy+a89Rs2my6TuXG6n5sjXOyoS5w==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BY5PR13MB3048.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(4636009)(136003)(366004)(396003)(346002)(376002)(39840400004)(66946007)(66446008)(66556008)(66476007)(64756008)(478600001)(6486002)(316002)(76116006)(8936002)(966005)(45080400002)(36756003)(33656002)(54906003)(2906002)(5660300002)(2616005)(53546011)(6512007)(83380400001)(6636002)(71200400001)(4326008)(44832011)(6506007)(86362001)(110136005)(8676002)(186003); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata: HOfMyHOJwOKDHKgTtuqcdqHeiZKJkZSc95v1z9lVXBNnhd6zLztB/7fuXgu09Qdr+v3y1WRyAWgpCouGiodRTydPiJ8iys703q+Q0PJk5OZLdjq0oGlWUyv+bypJN7KEF4dPSeaWYDmvLTZh+36FcRTIYfiHycf+ZDBj9cH9M/b8dpwc0+aSMPQGZ4JDG6oDt0DEeQF2WK65Hr5r7V4LgfDTF9RtU2jKyf1+me84NbQX0hl/V4NiD19KgHQM6g0Ru7Ki9ENpcx064fqofnzKr9C2rS7VMdqC0/rxdngrL1j2s4YHE3OSCFgWvxVg75AMDa7vrf6AHfU07AmibeSS4swGDj8nsRhaKlMY/d96xIJxBd4Ykwc7HVIU5WodYH+uWPR421Bp4h2Iih1cLOTYnsI06bCixqUBj2HwbgqxIGGgRdvhc6EzZkliQSgvvI0h0Nf7HCwIPrR3FZC4HnYbhVaYSzmFjwR/BzaocQyk9BYv2kbi790zO01E9vGNxO2pIiFgrJ8PtWHNwmGUKvZU0KADrAyNrLhkfVPEIRnu7UtuhMhh342k59KstSV02NA+
Content-Type: text/plain; charset="utf-8"
Content-ID: <60336B1887A38349B024B42FA3E290B8@namprd13.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: Futurewei.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: BY5PR13MB3048.namprd13.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 3e9885ed-c0e4-442a-c526-08d8249d76ab
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Jul 2020 06:49:58.8111 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 0fee8ff2-a3b2-4018-9c75-3a1d5591fedc
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: X2uvwxkRGG3GhuAr6xEtB+7YNJ0T/gZvRytZIUZc0tQ2TiRl0Joo4hmxVgtXbN5cci4hIZq1oUGCjv9yANekww==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR13MB2615
Archived-At: <https://mailarchive.ietf.org/arch/msg/spring/BzvcsuOqit1uSAi2m8ydpUq9qrU>
Subject: Re: [spring] WG LC https://datatracker.ietf.org/doc/draft-ietf-spring-sr-yang/
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: Fri, 10 Jul 2020 06:50:08 -0000

Hi Dhruv,

Thank you for your review and comments.

You're correct that SRv6 YANG should be augmenting segment-routing like sr-mpls. This was the agreement between SRv6 and SR-MPLS YANG authors.  Hopefully SRv6 YANG authors will update so in the next revision.

I've uploaded version -17 to address your comments. Detailed answers please see inline with [YQ].

Thanks,
Yingzhen

Other detailed comments please see answer inline.

On 7/7/20, 11:40 PM, "spring on behalf of Dhruv Dhody" <spring-bounces@ietf.org on behalf of dhruv.ietf@gmail.com> wrote:

    Hi WG,
    
    I guess it is still 7th in some part of the earth and hoping I am not
    too late :)
    
    I find it weird that the SRv6 Yang
    [https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftools.ietf.org%2Fhtml%2Fdraft-raza-spring-srv6-yang-05&amp;data=02%7C01%7Cyingzhen.qu%40futurewei.com%7Caf6a379099eb41ab642d08d82309bfec%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637297872081216770&amp;sdata=1fiXJq6aFeKyAVgFN7gSlHRdqINQgoKfCcyIzp5AWe4%3D&amp;reserved=0] does not
    use/augment ietf-segment-routing or ietf-segment-routing-common. A
    sync up between these modules will be useful to understand the overall
    design! A section describing this would also be quite useful.
    
    Few comments -
    
    - Avoid references in Abstract, See
    https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftools.ietf.org%2Fhtml%2Frfc7322%23section-4.3&amp;data=02%7C01%7Cyingzhen.qu%40futurewei.com%7Caf6a379099eb41ab642d08d82309bfec%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637297872081226764&amp;sdata=jD%2FCEGQKug86nyaE1bx2N4i5x4l0gkdy3XNvZ4oD3lI%3D&amp;reserved=0
    
       Similarly, the Abstract should be complete in itself.  It will appear
       in isolation in publication announcements and in the online index of
       RFCs.  Therefore, the Abstract must not contain citations.
    
       You could replace those with straight textual mentions of the documents.
    
       Also, the use of SHOULD in the abstract does not seem right.

[YQ]: fixed abstract.

    - Section 4, you say -
    
       The sr-mpls configuration is split in global configuration and
       interface configuration.
    
       But there is no interface configuration mentioned.
    
       s/lcoal/local/
    
    - Section 5.1.1.1,  you should remove this -
    
       This section is a first proposal on how to use S-bit in Adj-SID to
       create bundles.  Authors would like to trigger discussion based on
       this first proposal.
    
       Please also add references for S-bit and B-Flag etc
 [YQ]: fixed
   
    - Section 8
    
       - The yang model lacks a "reference" statement when importing
    external modules or designing key concepts. They are useful while
    reviewing.
[YQ]: added references

       - Can we rename ipv4-sid and ipv6-sid to ipv4-prefix-sid and
    ipv6-prefix-sid respectively?
[YQ]: not sure whether there are other modules using these groupings. Will check to make sure not breaking other YANG modules importing it.

       - It could be useful to add a description of why ERLD is read-only
    in the YANG.
[YQ]: ERLD?

       - Why is the grouping srgb defined in ietf-segment-routing-common,
    but the feature protocol-srgb is in ietf-segment-routing-mpls?
[YQ]: ietf-segment-routing-common is also used by other modules, while protocol-srgb is specific to sr-mpls.

       - Can the ISIS YANG typedef be used instead of redefining the
    system-id in SR yang.
[YQ]: ISIS-SR YANG will be importing both SR-MPLS and ISIS YANG, so the typedef from ISIS YANG can't be reused since it will cause circular reference.

       - In the grouping sr-controlplane, is it not better to have a
    reference to the policy rather than a string?
    
          +--rw segment-routing
          |  +--rw enabled?    boolean
          |  +--rw bindings
          |     +--rw advertise
          |     |  +--rw policies*   string
          |     +--rw receive?     Boolean

[YQ]: Do you mean the policy in mapping server? Which is a string.
    
       - Target is defined as a string in the yang. You do say that they
    are IPv4/IPv6 prefix in the context of the I-D. I want to confirm that
    the string is the right choice in such a case.
 [YQ]: this is intended.
   
    - Section 11, ietf-segment-routing-mpls is missing from IANA section
   [YQ]: fixed
 
    Thanks!
    Dhruv
    
    
    On Tue, Jun 23, 2020 at 11:29 PM James Guichard
    <james.n.guichard@futurewei.com> wrote:
    >
    > Dear SPRING WG:
    >
    >
    >
    > This email starts a two week WG LC for https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdatatracker.ietf.org%2Fdoc%2Fdraft-ietf-spring-sr-yang%2F&amp;data=02%7C01%7Cyingzhen.qu%40futurewei.com%7Caf6a379099eb41ab642d08d82309bfec%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637297872081226764&amp;sdata=kyQNCDzQZFMpA6ypqu%2Bfg8X6vH4iZPDSqrpHrUNiELU%3D&amp;reserved=0p;reserved=0.
    >
    >
    >
    > Substantive comments should be directed to the mailing list no later than July 7th. Editorial suggestions can be sent to the authors.
    >
    >
    >
    > Thanks!
    >
    >
    >
    > Jim, Joel & Bruno
    >
    >
    >
    > _______________________________________________
    > spring mailing list
    > spring@ietf.org
    > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.ietf.org%2Fmailman%2Flistinfo%2Fspring&amp;data=02%7C01%7Cyingzhen.qu%40futurewei.com%7Caf6a379099eb41ab642d08d82309bfec%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637297872081226764&amp;sdata=I0BaAyU%2BI9uvcRW8Suj4%2BN7XPsd%2FbupIkAeg%2F6QknyA%3D&amp;reserved=0
    
    _______________________________________________
    spring mailing list
    spring@ietf.org
    https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fwww.ietf.org%2Fmailman%2Flistinfo%2Fspring&amp;data=02%7C01%7Cyingzhen.qu%40futurewei.com%7Caf6a379099eb41ab642d08d82309bfec%7C0fee8ff2a3b240189c753a1d5591fedc%7C1%7C0%7C637297872081226764&amp;sdata=I0BaAyU%2BI9uvcRW8Suj4%2BN7XPsd%2FbupIkAeg%2F6QknyA%3D&amp;reserved=0