Re: [Teas] Status update on draft-ietf-teas-yang-sr-te-topo

tom petch <ietfa@btconnect.com> Wed, 15 July 2020 11:14 UTC

Return-Path: <ietfa@btconnect.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 9B5BC3A0C1A for <teas@ietfa.amsl.com>; Wed, 15 Jul 2020 04:14:47 -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, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=btconnect.onmicrosoft.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 fI-kOL9PxYdh for <teas@ietfa.amsl.com>; Wed, 15 Jul 2020 04:14:45 -0700 (PDT)
Received: from EUR04-HE1-obe.outbound.protection.outlook.com (mail-eopbgr70112.outbound.protection.outlook.com [40.107.7.112]) (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 D5B3F3A0C11 for <teas@ietf.org>; Wed, 15 Jul 2020 04:14:44 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=RumJjOs1TntG1KvSCEBHd/Nou6MCLYPIQh5wyQgpR3ocvphcYsRlKp6dJgH+IaGRlsBVnzrwZ0HuEt3vrdXkaNWcV0M52g/xXFelOZ3/emcYLljN4makGmCF6OfAJmVqWa9FkN6eW26v2EkEkODbAS6Ra6xEGqnyZBJ8fHioxu2RN4aVHPRUB+8B75C3uBDa9uLAJul3Q1RC7YlArH7FmI+QkK5cTKSIDBhjg4uTGyuJ1YJ1a5mTG3DunnjjNaBc/o5iKDs1YGwtP76jQCSJNmQ9hX/MNU5fPwwCV2yBArMXaU08teOa2pGbhdoiJlD6/n3zcX9DZJbzuigoiwuLqw==
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=E4V+HZVjWRgL5evIaK1W5yACdVUXgHZ+1V4jAk/RRao=; b=Ovy+jkH6Jfmp7aRc9leWLMbA0OcvB+gxI+FMh2sYPNBSk/h0xHi8BM8PsxwG2muGloNU4/WVPh7cXC2egk48E1sp9g9tUgTqEdnKLrkb+oTpaDOJrZBLjQRNNI9DJXBk72SFCYfGvwi29hfu88KRdJXdRalkXZIg22Tlg/7RValoe4/GEXCw9jry/vga4r7kHoz691Py2S6GdZvULfr/5zSIo87pM29QzlfRQYtADg8igmt98PM73OLYtpcudkoNEEbW+Vy/d7x5I+CPU/qgnubHl1RJqp0ptJSzXrpyeARgz1zSzlNz0mRui9tb+N/A+GwE/r7Ew3p3U9ORVZ2RLQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=btconnect.com; dmarc=pass action=none header.from=btconnect.com; dkim=pass header.d=btconnect.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btconnect.onmicrosoft.com; s=selector2-btconnect-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=E4V+HZVjWRgL5evIaK1W5yACdVUXgHZ+1V4jAk/RRao=; b=fbaI4E9SVUuE4w1PFxnQyw15Ayil5Nnax7WsmjqnqHB/QWMPrDrO3LUKvpaMP1KhtXQhyMEMAVDGIdcJxL4EqGdxvS9tVx1Y1vNjfBadlkgWvSzDdDSu/kOLPtFhBEJoCmBy5XCln4f3ykXogmDmjhTcg13Ob/XRHyT+ut0aD1k=
Received: from AM0PR07MB5329.eurprd07.prod.outlook.com (2603:10a6:208:f5::20) by AM0PR07MB4100.eurprd07.prod.outlook.com (2603:10a6:208:4d::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3195.13; Wed, 15 Jul 2020 11:14:42 +0000
Received: from AM0PR07MB5329.eurprd07.prod.outlook.com ([fe80::107:1c69:8e89:f670]) by AM0PR07MB5329.eurprd07.prod.outlook.com ([fe80::107:1c69:8e89:f670%5]) with mapi id 15.20.3195.018; Wed, 15 Jul 2020 11:14:42 +0000
From: tom petch <ietfa@btconnect.com>
To: Xufeng Liu <xufeng.liu.ietf@gmail.com>
CC: TEAS WG <teas@ietf.org>
Thread-Topic: [Teas] Status update on draft-ietf-teas-yang-sr-te-topo
Thread-Index: AQHWGBkGzUvWFr3pNESfdfWaueckeKiUtcTMgAHXrzaAbqd9AIADyZMy
Date: Wed, 15 Jul 2020 11:14:42 +0000
Message-ID: <AM0PR07MB53297327553E9048BC697554A27E0@AM0PR07MB5329.eurprd07.prod.outlook.com>
References: <CAEz6PPTm0vW+6HLjTv-dQQu7ccuRPcTN5jP4D5HtZt5nQwROYw@mail.gmail.com> <DB7PR07MB53401963CB478EA84CE0FEE1A2A80@DB7PR07MB5340.eurprd07.prod.outlook.com> <DB7PR07MB53400D95E243D7ACE86352DAA2A90@DB7PR07MB5340.eurprd07.prod.outlook.com>, <CAEz6PPRpsnO_eJiKUmCP5u8o9WwVKj4xkx9K2BsOkZsZMFscKg@mail.gmail.com>
In-Reply-To: <CAEz6PPRpsnO_eJiKUmCP5u8o9WwVKj4xkx9K2BsOkZsZMFscKg@mail.gmail.com>
Accept-Language: en-GB, en-US
Content-Language: en-GB
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=btconnect.com;
x-originating-ip: [81.131.229.35]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: 464a69f5-2fbe-4a96-2324-08d828b045cb
x-ms-traffictypediagnostic: AM0PR07MB4100:
x-microsoft-antispam-prvs: <AM0PR07MB41000268034854C26B9AD6B6A27E0@AM0PR07MB4100.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: K+fKTYy4egVYMzsp8gn8FurRdmHqW8oGyDXPv/HQqHfozqr1Rlqpz44/V4wz3xwBmjQS1bzRUlgT5WHJQOtRZy/wSVkihPVQaqIIwkQMIqAnrXXE4BV+jTXQSnZNxGwh0OirU45XRVUWucRUrdtDIMAidMdmKszThp2MvN0McRIt/eZM6+aRsFX+9L1AT7dvDtIzYSJuIk9ObIMQSfIVAFJVDCHsOOJz6AY2eQJnjtFNKrz5Yyp8QDcgZyO72awRi/6UcLxWjIXDxGzFY3K9sFGx0CxCui5DlvB+1MH0i3ALQbUcrIkuvs3KMwvL0dnpb1kOUoXV1gVoIKRzTxTIHEPU9y7AaREWFLAs5AXypc+0V+0KFFjEmMhFMBX4CF9XV/4odWXnNXT1BOXTZijOgA==
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM0PR07MB5329.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFTY:; SFS:(396003)(136003)(346002)(376002)(366004)(39860400002)(71200400001)(26005)(53546011)(7696005)(186003)(6506007)(966005)(33656002)(6916009)(316002)(478600001)(8676002)(83380400001)(66476007)(66946007)(66556008)(64756008)(55016002)(5660300002)(66446008)(86362001)(8936002)(91956017)(4326008)(52536014)(76116006)(2906002)(15650500001)(9686003); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata: um++JlKZGoea28mD/Np+WnNc3JwhdPK8pD0ZHbV3D4jqClVK20y5+1BcfC3M6dO/3a1pJIkm+F1o6XTJsiKwelfYuUzoyhqnme87PDsldGN7ZGLmUIgZOUcfqcpBhxRyHSRCrqMelCC1o4ebc+fLJNIE/U9uNR5AEqHSDsAUn2kFZyZv/Psfu75Ys7qwJ/FbShfR9fLp7R2IvntAvFXj7ZTjgggHZPLLbwq6aeCM/3solvV0CyNjKL3y+pNEv7v5FULUrG+f/RcGGNGjQS+MvGac/I+eCYemYHiHWpOgtZutdjQBZDOsKmn+Na7mZT8cPmL6PAThd2nUjr9T6oRh1xQaPW2Ixqa5S5EFIU8fSNEp3bBwkjdcNSfW3tJfMJlAHKA83wLxo5iOEgD6uS4qWkyrU64ywOzDN3GLkLpM+0PZqQmO5RA1y+mr76VVdqhKrtHp43YWCSuxRFxdqTl8tdbFRR5eZSvLpXNC1TTo6cU=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM0PR07MB5329.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 464a69f5-2fbe-4a96-2324-08d828b045cb
X-MS-Exchange-CrossTenant-originalarrivaltime: 15 Jul 2020 11:14:42.1671 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: cf8853ed-96e5-465b-9185-806bfe185e30
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: t5mN6JXqUmq39tGPJrSyRc1kXoV2CvHpo2GlxopWdloOezwnjqdJJ7DTT2fUCYd/0X0U/XwnoJwNok/Z/T9qRA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR07MB4100
Archived-At: <https://mailarchive.ietf.org/arch/msg/teas/RiFTFRt-qoJkRPOv4BX_sNsDiEU>
Subject: Re: [Teas] Status update on draft-ietf-teas-yang-sr-te-topo
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: Wed, 15 Jul 2020 11:14:48 -0000

From: Xufeng Liu <xufeng.liu.ietf@gmail.com>
Sent: 13 July 2020 02:11

Hi Tom,

Thank you much for the much-needed review. We have posted the updated revision https://datatracker.ietf.org/doc/html/draft-ietf-teas-yang-sr-te-topo-07, trying to address these comments. Please also see the replies in-line below.
Best regards,

<tp>
Yes, clearer.

IANA needs updating with the amended prefix

A recent YANG doctor review said that all feature should have reference which sounds right.  Perhaps RFC8476, RFC8491 along with RFC7752 for msd

Diagrams are useful but ..
these seem the opposite to e.g.  yang-l3-te-topo
Does A---> B mean A augments B or B augments A?  Would be nice to be consistent.  Perhaps one for the YANG Doctors list.

Another common practice in most TEAS Yang I-D is to have a table in the text of prefix used and the corresponding module and reference.

Tom Petch

- Xufeng

On Sun, May 3, 2020 at 11:46 AM tom petch <ietfa@btconnect.com<mailto:ietfa@btconnect.com>> wrote:
And while you are at it ..

I don't understand quite a lot of this I-D.

In places it is explicit - MPLS data plane only.  Elsewhere it seems to claim to be for all SR as with the choice of prefix srt module name or with container SR or with the Abstract.  If this is MPLS data plane only. then I think that that needs spelling out in more places starting with the Abstract
[Xufeng]: As suggested, used MPLS throughout the document and the model, including module name, prefixes, and container names.

2.1 references unicast-igp-topology - I know of no such module
[Xufeng]: Fixed the typo.

2.7 Both IGP and BGP are supported ...is this BGP or BGP-LS?
[Xufeng]: Right. This is BGP-LS. Fixed the term and added some references.

feature msd
what if one IGP supports msd and another does not, say OSPFv2 does and OSPFv3 does not?  I don't know if that will happen or if it should be supported but, for once, YANG does have good support for such an approach
[Xufeng] msd is a device feature, independent of the protocols. In case that one particular protocol does not support msd, the corresponding leaf msd in the operational datastore will not have a value. Since the user knows which protocol provides the information based on the information-source, he will understand the reason for it.

grouping sr topology type
a presence container for SR-MPLS - good
But how about
sr node prefix attributes
a presence container which indicates SR is enabled and
[Xufeng]: Since a prefix may or may not be configured with a Prefix-SID, the "presence" statement will allow such an option.
sr link attributes
a presence container which indicates SR is enabled.
[Xufeng]:  Since an L3 link may or may not be configured with an Adj-SID, the "presence" statement will allow such an option.
That is eight possible combinations of presence containers with eight different interpretations - I do not understand what they mean.  I do realise that the groupings have different roles.
[Xufeng]: Two "presence" side effects relevant here are: (1) to assign a semantic meaning (non-presence container can be ignored); (2) to stop propagating the "mandatory" requirement to the parent container.


RFC 7752 is the reference for BGP-LS but
needs to be in the I-D references.
[Xufeng]: Added.
And how about references for OSPFv2 etc?
[Xufeng]: Also added.
Currently BGP-LS may be new and different and people want to know where to find it but in a year or two they may be wondering what OSPFv2 is or was.

Since you import sr-routing-cmn I cannot see how that can fail to be a Normative Reference - I cannot understand the I-D without it!
[Xufeng]: Fixed.

Tom Petch
________________________________________
From: Teas <teas-bounces@ietf.org<mailto:teas-bounces@ietf.org>> on behalf of tom petch <ietfa@btconnect.com<mailto:ietfa@btconnect.com>>
Sent: 02 May 2020 12:16
To: Xufeng Liu; TEAS WG
Subject: Re: [Teas] Status update on draft-ietf-teas-yang-sr-te-topo

Xufeng

I suggest you respin this I-D lest it expires in four days time.

Tom Petch

________________________________________
From: Teas <teas-bounces@ietf.org<mailto:teas-bounces@ietf.org>> on behalf of Xufeng Liu <xufeng.liu.ietf@gmail.com<mailto:xufeng.liu.ietf@gmail.com>>
Sent: 21 April 2020 21:10

Current Status:

  *  No update since IETF106 with last post on November 3, 2019.
  *  Answered YANG doctor's review comments.

Open Issues:

  *   None.

Next Steps:

  *  Update the model to sync with the referenced models like draft-ietf-spring-sr-yang if there are any changes.
  *  Welcome further reviews and suggestions.
  *  Working Group Last Call after completing the above.

Thanks,
- Xufeng


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