[Lsr] RtgDir review: draft-ietf-ospf-te-link-attr-reuse-08

Daniele Ceccarelli <daniele.ceccarelli@ericsson.com> Wed, 18 September 2019 09:01 UTC

Return-Path: <daniele.ceccarelli@ericsson.com>
X-Original-To: lsr@ietfa.amsl.com
Delivered-To: lsr@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 715EA1200F7; Wed, 18 Sep 2019 02:01:18 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2
X-Spam-Level:
X-Spam-Status: No, score=-2 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_NONE=-0.0001, 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=ericsson.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 ruDgh1hpYqJ8; Wed, 18 Sep 2019 02:01:15 -0700 (PDT)
Received: from EUR01-VE1-obe.outbound.protection.outlook.com (mail-eopbgr140075.outbound.protection.outlook.com [40.107.14.75]) (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 9E663120137; Wed, 18 Sep 2019 02:01:11 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=R3y6MkzG3QuTQeYCN7YRpac3RWm+vlgTvkL+XYyr71AaHCfNrfiw/JxtTJwlciKKXIq7iEhagV+4FGAG35Klw1DOQOmysdN7rojm2XQoEgD9GN3N7gJt1w1TszAQK7dTh95FKYHp8EerC5dmZV8KE9qu3m4JoKjrYsxNff6JnJhh5Bf0GugeBJTxx7B/LOCSUcl3Cgam+Ej36n+Xu1nLJTKkiq5vuxHXXfhmfgNxVlZplJ396z9qtLw/S7TCd43q8TrnXo1JT4xI5tVKsKvzz4KwBUAbYw5BShZ0hFTxf8qA1+ZTWCyCwQ5lq2mZOb7gqIpzskXT2SyB/JvfDqWXVg==
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=A7K0FKmo33ybBFdVjmiU2AsAbSqJoK46Y5e79bi01Ys=; b=Qr0egIwDx6OY3Pl2CRPy/pL/DALYoBors8fQEd7QiKHOU0NlMHcTsEeDntbLnYDH2PZD2fiKAr+OJK0ry77TlflgWa6Em2lrUr4e8n1PyuNJXfxtRS3/eZ0wlGSFXCJaL4Rv8+1goy86yUbSoTZeGjF/q9Nvl5EEJAFtSEDz00fCdf/UPrnZM9nFAtCtyrLipvvQHtM1+fwxmQGMMGZTQ2IQsjB1qq4Fp62JN+R4FsMsrNrQWUvheQuKdXjE5ynwSFrJ9DU1F4hnGSiMb46JDN25ineilaQE0DSYfjkfFWC006VxlrIHpLYmad0rfhMHTCSSlvdhh6E8Hud48JpzuA==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=ericsson.com; dmarc=pass action=none header.from=ericsson.com; dkim=pass header.d=ericsson.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ericsson.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=A7K0FKmo33ybBFdVjmiU2AsAbSqJoK46Y5e79bi01Ys=; b=EhJ7AFq9AThtI215Ufcit7i3LppJK2ZhniALbyvbayCHcv1757r+uje9xwnyDnmvL8X/MtEa5Yg/ZtGGYov1dWQ7Q1bU7BMKwz6RTZcNZXEXMow5/ce+aumryXtn6yLPRdLKTtlzfS7pyhSucyhMSVG8SbNRAO0x/U1gqrRuoTs=
Received: from AM0PR07MB6098.eurprd07.prod.outlook.com (20.178.112.202) by AM0PR07MB4531.eurprd07.prod.outlook.com (52.135.146.15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2284.15; Wed, 18 Sep 2019 09:01:08 +0000
Received: from AM0PR07MB6098.eurprd07.prod.outlook.com ([fe80::806:8b63:dceb:6ccf]) by AM0PR07MB6098.eurprd07.prod.outlook.com ([fe80::806:8b63:dceb:6ccf%2]) with mapi id 15.20.2284.009; Wed, 18 Sep 2019 09:01:08 +0000
From: Daniele Ceccarelli <daniele.ceccarelli@ericsson.com>
To: "<rtg-ads@ietf.org> (rtg-ads@ietf.org)" <rtg-ads@ietf.org>
CC: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "draft-ietf-ospf-te-link-attr-reuse.all@ietf.org" <draft-ietf-ospf-te-link-attr-reuse.all@ietf.org>, "lsr@ietf.org" <lsr@ietf.org>
Thread-Topic: RtgDir review: draft-ietf-ospf-te-link-attr-reuse-08
Thread-Index: AdVt/5sGAKOxj6NwT/GaTs7ycHNSbw==
Date: Wed, 18 Sep 2019 09:01:08 +0000
Message-ID: <AM0PR07MB6098E9C3DB72E9BDF3316B74F08E0@AM0PR07MB6098.eurprd07.prod.outlook.com>
Accept-Language: it-IT, en-US
Content-Language: en-US
X-MS-Has-Attach: yes
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=daniele.ceccarelli@ericsson.com;
x-originating-ip: [192.176.1.84]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: bb2c2f6b-6795-44b7-5970-08d73c16bec7
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600167)(711020)(4605104)(1401327)(2017052603328)(49563074)(7193020); SRVR:AM0PR07MB4531;
x-ms-traffictypediagnostic: AM0PR07MB4531:
x-microsoft-antispam-prvs: <AM0PR07MB4531637EEBB6BAC4F01D4FCAF08E0@AM0PR07MB4531.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 01644DCF4A
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(366004)(199004)(189003)(450100002)(54906003)(7736002)(81166006)(4326008)(66066001)(606006)(44832011)(5660300002)(81156014)(6436002)(76116006)(102836004)(6506007)(66476007)(66616009)(66556008)(8676002)(8936002)(52536014)(64756008)(66946007)(66446008)(236005)(54896002)(71190400001)(55016002)(99286004)(486006)(25786009)(74316002)(7696005)(9686003)(498600001)(6116002)(790700001)(3846002)(14454004)(14444005)(6306002)(26005)(2906002)(256004)(86362001)(99936001)(186003)(33656002)(476003)(71200400001); DIR:OUT; SFP:1101; SCL:1; SRVR:AM0PR07MB4531; H:AM0PR07MB6098.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: ericsson.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: 4Nvd9mx4GVcKA+ffMkicflsymQJ1y1vJC3knK4U6GN18xI8VmpEAJGfyiMfWYHJYO0XDsH344mrGMnjCPogt4Ca4vabWhWCwlXH+Mu91m/YEf9milzo4nSQndf9hCpCiLmhIltI1a9Hvo8lyxhFxUyKNz70TJg6BlxKrFIfVYV4lUPe21kha9mZStQ4zhWMdVALLSLB6zQ71JMLe1jfnGfaIuYe6M7oudU2NCyFMdCoaRaVibGEBuN8pOfsQcPJbhL6sw77QmPV2ZGzlpBhP25fLzPRWGMcA/RmzY50O0HIxJLvIbacQ6p248NjzgKYgQftJ/qhOTyH+0V+Wf1Rpy2eUQ9+6Ljp7+wd2cDgGYz+RTHG36AamkELVMS0uaekNoLVpz75irPUFXuduv+48MTtDxk0vzDuhTXrRt1yGWzg=
x-ms-exchange-transport-forked: True
Content-Type: multipart/signed; protocol="application/x-pkcs7-signature"; micalg=SHA1; boundary="----=_NextPart_000_0081_01D56E10.5F1B62A0"
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: bb2c2f6b-6795-44b7-5970-08d73c16bec7
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Sep 2019 09:01:08.3647 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 92e84ceb-fbfd-47ab-be52-080c6b87953f
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: AjVxMZe6tFCTUjFbX+rpLoNln7boA7+Xcji46AttvguAhIkcohxiPXrc1CqSlHrukfhL8C77Jhk98l6rT6Gp9coyoJ7Wy9hZNsacvoLxQI0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR07MB4531
Archived-At: <https://mailarchive.ietf.org/arch/msg/lsr/q0rO8ZTsFCyKlVdH5ZICpkRc3kc>
Subject: [Lsr] RtgDir review: draft-ietf-ospf-te-link-attr-reuse-08
X-BeenThere: lsr@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Link State Routing Working Group <lsr.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/lsr>, <mailto:lsr-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/lsr/>
List-Post: <mailto:lsr@ietf.org>
List-Help: <mailto:lsr-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/lsr>, <mailto:lsr-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 18 Sep 2019 09:01:19 -0000

Hello,

I have been selected as the Routing Directorate reviewer for this draft. The Routing Directorate seeks to review all routing or routing-related drafts as they pass through IETF last call and IESG review, and sometimes on special request. The purpose of the review is to provide assistance to the Routing ADs. For more information about the Routing Directorate, please see  <http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir> ​http://trac.tools.ietf.org/area/rtg/trac/wiki/RtgDir

Although these comments are primarily for the use of the Routing ADs, it would be helpful if you could consider them along with any other IETF Last Call comments that you receive, and strive to resolve them through discussion or by updating the draft.

Document: draft-ietf-ospf-te-link-attr-reuse-08.txt
Reviewer: Daniele Ceccarelli
Review Date: 2019-09-13
IETF LC End Date: date-if-known
Intended Status: Standard Track

Summary:
I have significant concerns about this document and recommend that the Routing ADs discuss these issues further with the authors.

Comments:

The drafts needs some improvement to be clear and easy to read. It is outside the scope of the RTG-Directorate review to consider consensus on it, but the it is not possible to ignore comments received from a WG member of its usefulness. Implementations on ISIS segment routing and OSPF segment routing (publicly available) prove that applications like Flexible Algorithm, TI-LFA and R-LFA can be implemented using TE parameters compliant with RFC3630 and RFC5305 without the need for these extensions.

That said the rest of the review will be limited only to the quality of the document.

Major Issues:

*	No major issue in addition to the one described in the comments.

Minor Issues:

*	Abstract: it would be nice to have an overview of what is the purpose of distributing the attributes (in addition to MPLS-TE and GMPLS). The document starts with a very generic scope but then focuses on segment routing. It could be stated at the beginning.
*	Section 2: what does this sentence mean?: “Additionally, there will be additional standardization effort. Additionally, there will be additional standardization effort.  However, this could also be viewed as an advantage as the non-TE use cases for the TE link attributes are documented and validated by the LSR working group”
*	It is not clear the usage of RFC2119 language (RECOMMENDED) in section 2.1, is section 2.1 defining a new procedure? My understanding is that section 2 is the actual solution while section 3 is the newly defined one. Am I wrong? If so it should be made a bit more clear and I would expect to see RFC2119 language only in section 3.
*	Section 3: “This situation SHOULD be logged as an error” how? Should a notification be sent? Logging an error is not part of the protocol definition but rather an implementation issue.
*	Section 4: the title is misleading. It is defining how to encode the list of attributed defined at the end of section 3 (some of them are reused, some others are TBD), why the title of the section is Reused TE link attributes?
*	Sections 5-6-7: Section 3 describes the procedure and TLV format, section 4 the encoding of the attributes…what is defined in section 5-6-7. If I search for e.g. Maximum link bandwidth (the title of section 5), the first occurrence is the title of section 5. Maybe gouping sections 5-6-7 into a single one with an intro of what is defined could improve the reading. 

Nits:

*	MPLS TE is sometimes in capital letters and sometimes not.
*	SRTE expand on first use.

 

BR

Daniele