Re: [Detnet] Rtgdir last call review of draft-ietf-detnet-mpls-04

Balázs Varga A <balazs.a.varga@ericsson.com> Fri, 10 January 2020 13:56 UTC

Return-Path: <balazs.a.varga@ericsson.com>
X-Original-To: detnet@ietfa.amsl.com
Delivered-To: detnet@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 34CB21208D2; Fri, 10 Jan 2020 05:56:32 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.001
X-Spam-Level:
X-Spam-Status: No, score=-2.001 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, 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 WhyfZSc9Ke4m; Fri, 10 Jan 2020 05:56:27 -0800 (PST)
Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-db5eur01on0614.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe02::614]) (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 472E01208DB; Fri, 10 Jan 2020 05:56:26 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=nBd0rMT6KXZLhMZJ1XH+Vh8RRaboZL2l9Fc/tlY1ZXnQZhR82LTgu0My1TI3/MEWRZ4Q8jGGvZXVlhAoMCtI+E7wSs7Bcvb7UorQZA0SC2rfdsh3dvo6WzRsUqCKKKUGQixa/XxL09VO5dPthG6XTU+HhiZK1Mjn+yLEbvoCYqbSfFIlMFo5qtg1jhnZ7yljYwVpNsxCnxKhOrZmd6Bb05GceTdK370zhULHnWe9DKQOznfhV2T3WDQTJkNpHRZQFQURMRHyrgNk1/p7FfxSRIK9+tgY5BkWEt/gwOCtZUFkq+9E38FXrKoN29GoBXzf2f2+IFhJG+XryHI9691onQ==
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=EB0XEg2NNgjlxjEo0BVe0ybTe+JdfXjqr4DLcxiyFv4=; b=i4tRyiZfxvFLCJrP/k6xYS9PNyHntJQkQXT9y6k1Jch+szEqw+mxTDD6tplPVjr9S3Jj2i0FHKAWMk3cAEAqBvXvRwvj3bIxg4MWiCThI93hzZK6WnYXTbglSkaon/6PpNGkPHZOVn8bOBGuGaT8jX50Hcyg4B/eKMf0osXq5lfV+UwAah/PrcO3zMFG/M/TwuXuo7bIdPYo/0V5YlwjdL4DwGWHOUvOgv1a5048X12io6Mr7hEBqlTLdTg21Crs2528Q7SUipme/L2ERkjMPbskz0XDAl1t6db0aoh/2fnTbMt5sK70q5cUC6lDDsYPshQ4fVh9UiHgSAnaU4wUXA==
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=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=EB0XEg2NNgjlxjEo0BVe0ybTe+JdfXjqr4DLcxiyFv4=; b=cOjsQZIsJ5CiL7Arykiufmx2VjSB8xKSMveFss1uxkZY7dksSNntHSHIlHvrnWhL9usXktaPBy8EsvbIhXfKyddYNSTV5O9r61bzSlyT3yw+Vsj0w6I30jn/AKb0uTaHe8BWYaU9Wcxx6+ILVt50z09VEMc/PKmUjU8n+KEb2ic=
Received: from VI1PR07MB5389.eurprd07.prod.outlook.com (20.178.80.18) by VI1PR07MB5151.eurprd07.prod.outlook.com (20.178.12.84) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2623.4; Fri, 10 Jan 2020 13:56:22 +0000
Received: from VI1PR07MB5389.eurprd07.prod.outlook.com ([fe80::d552:3f2a:78ee:60f2]) by VI1PR07MB5389.eurprd07.prod.outlook.com ([fe80::d552:3f2a:78ee:60f2%7]) with mapi id 15.20.2644.010; Fri, 10 Jan 2020 13:56:22 +0000
From: Balázs Varga A <balazs.a.varga@ericsson.com>
To: Carlos Pignataro <cpignata@cisco.com>, "rtg-dir@ietf.org" <rtg-dir@ietf.org>
CC: "last-call@ietf.org" <last-call@ietf.org>, "draft-ietf-detnet-mpls.all@ietf.org" <draft-ietf-detnet-mpls.all@ietf.org>, "detnet@ietf.org" <detnet@ietf.org>
Thread-Topic: Rtgdir last call review of draft-ietf-detnet-mpls-04
Thread-Index: AQHVue2QpbeVfYykfUO/Wtl6FHPRZ6fjsWMA
Date: Fri, 10 Jan 2020 13:56:22 +0000
Message-ID: <VI1PR07MB5389DFAE0BC5593CC2FC258BAC380@VI1PR07MB5389.eurprd07.prod.outlook.com>
References: <157714579482.2458.7370182245915799132@ietfa.amsl.com>
In-Reply-To: <157714579482.2458.7370182245915799132@ietfa.amsl.com>
Accept-Language: hu-HU, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=balazs.a.varga@ericsson.com;
x-originating-ip: [178.164.178.34]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: e77d47e3-c2bb-472c-2501-08d795d4e069
x-ms-traffictypediagnostic: VI1PR07MB5151:
x-microsoft-antispam-prvs: <VI1PR07MB5151423A388197FCB0DF5080AC380@VI1PR07MB5151.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 02788FF38E
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(376002)(136003)(366004)(39860400002)(396003)(346002)(13464003)(199004)(15374003)(51444003)(189003)(51914003)(81166006)(8676002)(81156014)(478600001)(2906002)(316002)(52536014)(64756008)(55016002)(66556008)(186003)(66476007)(8936002)(76116006)(66446008)(66946007)(9686003)(85202003)(7696005)(110136005)(53546011)(86362001)(54906003)(26005)(33656002)(71200400001)(5660300002)(4326008)(85182001)(6506007); DIR:OUT; SFP:1101; SCL:1; SRVR:VI1PR07MB5151; H:VI1PR07MB5389.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: BCL:0;
x-microsoft-antispam-message-info: v4uBBzt1twYAZj8PfSPaIBl99M0PW6xPftg6g9LwOBO0/hrKvEc1G1UkL0q1qjZ6iO8XMXzMdqi2ZmEBWQwjxkKSPlAhrvEhpdv0YW/Xv8WizHPSsNgIzK6RrUSk0fn75vSFKvlCyNJNBn5tf3tsuexx9mVos4RYDZ3uiXCyPvualrzKPay+umJULIWMnsvSE3wJdoe98EAFmAgO3KsHxgTMVPoPso3HmpbZYYy1RlaFNqEy8KZZq6XpgkH3FkAxXx+lvV8W8eLNyiTpjrq5NV3FIkI3WRi/zPWUeuvwnMjQTNsUhohFXStcK22L53OpnUm2klK5Uym0Ef6MnlY6oVR408Ee+N5HUWvBMR10+AJUJbwEQ+tznCLh1Vh/havPIYU9Y3DXxH3xwNlN7qn/5NdRfGsNP7zEyhMFhQmZU4Q4O03b5j0L3fiMg6Fq62DtF8msNdO3a/mM+8tg+L+LprmZik0BrWFrdE678TAv5lEuNHKNAvYWqS3pBtgwMO9CQDi+FoKygGJvx8OkAohDz8oZf5aTE16+9oMrtIGtQX7R9RVTgkIoCQZnEoN+TT7nmREtyq3VJ2fUqk6annfND6G8xSDtYGUxXrkjn45YmD0=
x-ms-exchange-transport-forked: True
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: ericsson.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e77d47e3-c2bb-472c-2501-08d795d4e069
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Jan 2020 13:56:22.6117 (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: bTG0DbSY7FwzHiLFl1dvevcd/fCD5aQHe/+XWKs+ZQnp/GYe0pwGc+6Y45yAXbUGZiOd5M/q1Tro7xVYhZ/FN4RjRSKS5hNcKnNWyOUTmJ4=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR07MB5151
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/oeU9Imq1rZcsncjfksiVXhfnjNs>
Subject: Re: [Detnet] Rtgdir last call review of draft-ietf-detnet-mpls-04
X-BeenThere: detnet@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Discussions on Deterministic Networking BoF and Proposed WG <detnet.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/detnet>, <mailto:detnet-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/detnet/>
List-Post: <mailto:detnet@ietf.org>
List-Help: <mailto:detnet-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/detnet>, <mailto:detnet-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 10 Jan 2020 13:56:33 -0000

Hi Carlos,

Many thanks for the review and your mailing with Stewart. 
Your former discussion points I tried to incorporate in this mail. 
Proposed resolutions are described below (marked with 
"<Balazs> and/or <Stewart>").

We will start to update the text if reactions are OK with You.

Thanks & Cheers
Bala'zs

-----Original Message-----
From: Carlos Pignataro via Datatracker <noreply@ietf.org> 
Sent: Tuesday, December 24, 2019 1:03 AM
To: rtg-dir@ietf.org
Cc: last-call@ietf.org; draft-ietf-detnet-mpls.all@ietf.org; detnet@ietf.org
Subject: Rtgdir last call review of draft-ietf-detnet-mpls-04

Reviewer: Carlos Pignataro
Review result: Has Issues

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

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-detnet-mpls-04
Reviewer: Carlos Pignataro
Review Date: December 2019
Intended Status: Proposed Standard

Summary:

I have some minor concerns about this document that I think should be resolved before publication.

[Resolving, in this context, means discussing and responding to, some of these would and others would not imply changes]

This document specifies the Deterministic Networking data plane when operating over an MPLS Packet Switched Networks.

Major Issues:

Minor Issues:

The document lists 7 (i.e, more than 5) authors.

A am somewhat confused regarding how some of these DETNET dataplane documents interact, interface, and intersect with each other. Where are requirements ultimately coming from for any variety of dataplanes? Specifically: *
draft-ietf-detnet-data-plane-framework-03 targets Informational (should be
STD?) but seems to include foundational Reqs. * draft-ietf-detnet-ip for IP * draft-ietf-detnet-ip-over-mpls for IP over MPLS * draft-ietf-detnet-ip-over-tsn and draft-ietf-detnet-mpls-over-tsn for X over TSN * draft-ietf-detnet-mpls-over-udp-ip and draft-ietf-detnet-mpls for MPLS Among all of these, where are really the requirements for MPLS? and for IP? Is there a "Roadmap" or Rosetta Stone to understand these? There are many permutations of X-over-Y. I appreciate that, not having followed DETNET discussions, I am likely missing something.

<Balazs> One can group the drafts in three sets:
1, General overview
The "draft-ietf-detnet-data-plane-framework" is intended to be used as a "Rosetta Stone".
It provides an overall framework for the Deterministic Networking data plane. Framework covers 
concepts and considerations that are generally common to any Deterministic Networking data 
plane specification.
2, DetNet Data Plane specifications
Two drafts are dedicated to IP and to MPLS based DetNet data planes. These drafts specifies
the requirements for MPLS and for IP.  "draft-ietf-detnet-mpls" specifies the Deterministic 
Networking data plane when operating over an MPLS Packet Switched Networks. 
"draft-ietf-detnet-ip" specifies the Deterministic Networking data plane when operating 
in an IP Packet Switched Networks.
3, Specifications for X-over-Y scenarios
Remaining drafts cover the X-over-Y scenarios. Some are called as "sub-network" scenarios
for DetNet. For example, if DetNet MPLS nodes are interconnected by and IP sub-network,
what is described "draft-ietf-detnet-mpls-over-udp-ip". 


3.1.  Layers of DetNet Data Plane

   The DetNet control word (d-CW)
   conforms to the Generic PW MPLS Control Word (PWMCW) defined in
   [RFC4385].

Yes, but why not the Preferred CW?

<Balazs>/<Stewart> Sum of mailing + proposed fixing:
The PCW only supports a 16bit sequence number and it has the skip zero auto-signaling of active S/N feature.
This was a problem for DetNet because:
- We were worried about S/N rollover frequency in some applications and so we wanted the option of a larger S/N.
- We wanted to have the option to propagate the S/N from the payload to the transport to simplify the implementation 
in some cases. These applications have a non-skip zero S/N. Skip zero is an irritation to implement and we should probably 
have signaled in in PWs.
As you note in is only a preferred design for PWs, DetNet is not constrained by that and there were good reasons to adopt 
this alternate approach.
We assume to fix this with adding above information to the text.
NEW text to be added in section 4.2.1:
    "This format of the d-CW was created in order (1) to allow larger S/N space to 
    avoid S/N rollover frequency in some applications and (2) to allow non-skip 
    zero S/N what simplifies implementation."

4.1.  DetNet Over MPLS Encapsulation Components

   The LSP used to forward the DetNet packet may be of any type (MPLS-
   LDP, MPLS-TE, MPLS-TP [RFC5921], or MPLS-SR
   [I-D.ietf-spring-segment-routing-mpls]).

I am not sure of the value of this statement for an "MPLS Dataplane" document.
Further, are these "LSP Types" and if so where are the different types ennumerated? Does this mean that static binding LSPs and BGP signaled cannto be used? T-LDP does not work? "SDN Assigned"? I recommend removing this, since it can confuse and does not add much.

<Balazs>/<Stewart> Text intends to say that any type of LSP can be used below DetNet. 
DetNet design is not restricted to a single method of establishing an LSP nor to the 
characteristics and constraints that go with those LSPs.
We assume to fix this with adding “for example” to the text and avoid the term "type".
CHANGE text to:
   "The LSP used to forward the DetNet packet is not restricted regarding any method 
   used for establishing that LSP (for example, MPLS-LDP, MPLS-TE, MPLS-TP [RFC5921],
   MPLS-SR [RFC8660], etc.)."


4.3.  OAM Indication

It is important to have the OAM Indication, but what type of OAM packets can run on top of this AcH? I found it interesting that for example there is not reference or citation to RFC 8029.

<Balazs>/<Stewart> Sum of former mailing + proposed fixing
Work has started on OAM for DetNet. We have two drafts dealing with IP and with MPLS based DetNet data planes (draft-mirsky-detnet-ip-oam-01, 
draft-mirsky-detnet-mpls-oam-01). We expect to cover OAM for DetNet with these two OAM documents.
The OAM for DetNet will be more complex than the OAM for a classical P2P or P2MP LSP (or PW) because of the PREOF function. There is nothing in the data 
plane that precludes us using on of the existing OAM indicators (GAL or 0001 ACH), but I think that it is important to thing through the subtleties. Thus I think 
it is OK to make progress on the elements of the data plane that we can nail down, and leave. The OAM as follow-up work.
We assume to fix issue this with adding reference to DetNet OAM drafts.
Add text to section 4.3:
   "DetNet specific OAM functions for MPLS based DetNet data plane are discussed 
     in [draft-mirsky-detnet-mpls-oam-01]."


Nits:

2.2.  Abbreviations

   The following abbreviations are used in this document:

Many of these abbreviations are well-known, many others have authoritative definitions and expansions. I believe this section should point to the appropriate RFCs for LSR, CW, PE, OAM, etc...
Also, interesting, this section does not expand LSE, and I think it should use it.

<Balazs> OK. We will update.

4.6.1.  Class of Service

Should this include "and TTL" in the title? I was looking for TTL specs in the Table of Contents but could not find it.

<Balazs> There are no new TTL related procedures defined for DetNet. 
Regarding TTL the section just intends to say that existing stuff can be used.
So, I think we can leave the title as it is (no reference to TTL)

9.2.  Informative References

Are all the "draft-ietf-detnet-*" really Informative?
<Balazs> No, we will fix and update.

Also, outdated reference: draft-ietf-spring-segment-routing-mpls has been published as RFC 8660
<Balazs>/<Stewart> OK, We will update.

   [RFC8655]  Finn, N., Thubert, P., Varga, B., and J. Farkas,
              "Deterministic Networking Architecture", RFC 8655,
              DOI 10.17487/RFC8655, October 2019,
              <https://protect2.fireeye.com/v1/url?k=7a40605f-26946939-7a4020c4-8610d8a762ca-97dd584868114772&q=1&e=42fa17eb-44f5-42d1-9898-4d6ba7365c47&u=https%3A%2F%2Fwww.rfc-editor.org%2Finfo%2Frfc8655>.

And is this also Informational?
<Balazs> No, We will update.

Thank you!

Carlos Pignataro.