Re: [Detnet] Benjamin Kaduk's No Objection on draft-ietf-detnet-tsn-vpn-over-mpls-06: (with COMMENT)
Balázs Varga A <balazs.a.varga@ericsson.com> Thu, 18 February 2021 09:26 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 5B64B3A0E5C; Thu, 18 Feb 2021 01:26:00 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.671
X-Spam-Level:
X-Spam-Status: No, score=-2.671 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.57, 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] 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 OiNdSWewED8D; Thu, 18 Feb 2021 01:25:58 -0800 (PST)
Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05on2085.outbound.protection.outlook.com [40.107.22.85]) (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 640E73A0E60; Thu, 18 Feb 2021 01:25:57 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Mhuns5XTdICTbpb1uU24puZ93S97wi3FOFJXCwWhuZcdNN+qVP5FTVFNOKAoAXQ0WOIF+i8FQz/Vb6scdPqRrH2v9t6uO1PRWL+kcuD2PZqhvrrOqC47dfo0iLCzc2LmxxI3UqKxCMkXfRG71JUk/M/J69SEfZepc6dIFoaZ2eeKysTLT8I7v32ujKHo4AO8J56PTmvT/SDznZ3rdQi0cm0ELkx2MkcPTAv7TaODIrBvAm8puIzwdIqzZsQLW7/SbwCWwQjuxc2IYxOhsvdsjFRUAM+v6pHUY1/y53EE20JZPJZZGJAPTTAJlYXC9rawZDyKa/QQaowRVRvlkxuGBw==
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=0BnMVj300DSwg3UafsFsH0ZqXVyHtIR+n2RJul/0ffk=; b=CA+oGnhhMWBNplIQO//YQ/j+jOK4Pn7hF1LaoDVcgAHfMvEtLEi4EWnyU7uqzwTQVOTZmnuGArZ0Pc9G44xx5+4bC+tj5ffDWZlyhQR0tMZrhsan57YQyhVXVVnCZCLMApPyskBuKL046jUV07imrUUe3GXCwl7uh4oJjNp2NRJsTzbeWZqA9DVx/PMeDe69r74qSSpF/b7Z5joZWBXaGjp4Z94FWwzTFY69+Pch8b9ySUllGFxutiqXdRBEEliGPJEmaIuo2CKzyO5zRwNp7By8Na2+OGAA65tQe7tTJZU89U6pOghQmC1sw4tX268ImKbn/dZ49ojyJ3ePOscxwA==
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=0BnMVj300DSwg3UafsFsH0ZqXVyHtIR+n2RJul/0ffk=; b=N+q/GU1O3hJehsS+de+kb4C/Wsl45JQaHGPDdiJ4TrQm/LAFn4H3k2hqFEiCOhutFMGV80Q8Mo0ifo/0P/qTa0lI0VTgRkGWk+1adMy+9WRYv90mXg3VTepePaZNgFzi+9GyfkdzhByDuF/p5reob7WE2CkW14ACpIvXc9CtM48=
Received: from AM0PR0702MB3603.eurprd07.prod.outlook.com (2603:10a6:208:22::25) by AM9PR07MB7700.eurprd07.prod.outlook.com (2603:10a6:20b:2c2::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3868.12; Thu, 18 Feb 2021 09:25:49 +0000
Received: from AM0PR0702MB3603.eurprd07.prod.outlook.com ([fe80::3851:5264:9c3e:a251]) by AM0PR0702MB3603.eurprd07.prod.outlook.com ([fe80::3851:5264:9c3e:a251%6]) with mapi id 15.20.3868.019; Thu, 18 Feb 2021 09:25:49 +0000
From: Balázs Varga A <balazs.a.varga@ericsson.com>
To: Benjamin Kaduk <kaduk@mit.edu>, The IESG <iesg@ietf.org>
CC: "detnet@ietf.org" <detnet@ietf.org>, "draft-ietf-detnet-tsn-vpn-over-mpls@ietf.org" <draft-ietf-detnet-tsn-vpn-over-mpls@ietf.org>, "lberger@labn.net" <lberger@labn.net>, "detnet-chairs@ietf.org" <detnet-chairs@ietf.org>
Thread-Topic: [Detnet] Benjamin Kaduk's No Objection on draft-ietf-detnet-tsn-vpn-over-mpls-06: (with COMMENT)
Thread-Index: AQHXBVvYOl4uVdKXPUye9Up9Ulhqe6pdodSw
Date: Thu, 18 Feb 2021 09:25:49 +0000
Message-ID: <AM0PR0702MB36030B0F5A17D21F39770CEBAC859@AM0PR0702MB3603.eurprd07.prod.outlook.com>
References: <161358698867.12626.1261967054256046021@ietfa.amsl.com>
In-Reply-To: <161358698867.12626.1261967054256046021@ietfa.amsl.com>
Accept-Language: hu-HU, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: mit.edu; dkim=none (message not signed) header.d=none;mit.edu; dmarc=none action=none header.from=ericsson.com;
x-originating-ip: [188.143.101.39]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: ce39614e-894b-488c-974d-08d8d3ef2de9
x-ms-traffictypediagnostic: AM9PR07MB7700:
x-microsoft-antispam-prvs: <AM9PR07MB7700459BEB9CD7631B50C2CFAC859@AM9PR07MB7700.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:7219;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: xjQmRfytwG1tqeIwGBXRow4wIbORLK5shPZI9ei/qmnM5Z9qXM/aQxYEBFFYwySgWWF8QpNqp7v+rm1BQgNXwnZ6aOVIk1Rw9q/C4qMr+nfQl1H7CUAruIEstlWth/5cOHGrwfrpX8g3yiPBxuALAgVe0QByMDS6wbjh7igDwpehmwOVcbXslkZlR6Rt/P9+HYFjUnXT7xrsTnBIHJ1IwT282s6OyvyVPY1y8gOKjwHmLMCmRuxQzJkbo/5RAn7tf6iU21x+xkg82wVls2WTVoCcs7iPcOUXG+YJaEPDUsq14VaTB0EqNbn95UKEY2jO+MNGB6qlHS7Wl3iFauLgaZUgxtO1SQ61jgjJjx+pU4ERbW4sMORwchmeG82kF21CQ6vygEkZy0gSEsWZXZAnPlp2oDolPEa/9MsPaMgz+xfGHMN6MVERJUmE50nYKqQXJiEKHQRfc7N1McbpY/6yqhSIbqf5/p1+zNKmm8wPZXH33JJyBopPOBPHImWMuY35RmTLBDFag9HBplZqyAnrxS2N9c8+Bc41iYnS2uPNBT2X4muTlmjIhryUREGbi5RGKDMQPLsnENzqDnRkEGldcdJwS+ETH/GApwVv+ZWHUUQ=
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:AM0PR0702MB3603.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(136003)(376002)(39860400002)(346002)(396003)(54906003)(5660300002)(83380400001)(966005)(2906002)(316002)(71200400001)(478600001)(110136005)(4326008)(8936002)(7696005)(86362001)(33656002)(186003)(8676002)(66446008)(76116006)(52536014)(53546011)(6506007)(55016002)(66556008)(66946007)(66476007)(64756008)(26005)(9686003); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: seiYV7aA58bHIU5NsQnZyHxLKqhwHFP48Z4R9aLtv6YZmnNn/DV/Ch96c1ofkvUdpGrVimb5wPEf8U87JTORQARclK+Ja5mKD8bMatka9mMVVgfHJHD2sMiKNRzFEZLdzFkYZVbYaX3hT3KBew0uZfvMS/GsZIEsTmiUNhN/JX2lYRoHv9zgzBngo8WXxHXF0AbuwP34xA4kgzOnaz71DsmFJjICPJ4N/7aWaHDPtX87baYOnNnW10rqyud/KjeXGtyd+bj4yJBgoGLxdKojEaUbFopC1MwqXX18D9JtwGaI8nhDl369xb8yHjl349EVpFOKkqmsEcskFSpCliu+md86QMzSkNMEeT2EPPc8ulmh3daj99rHJgN3AYSDzLoE39cfigDSt4KD5cxwEuWRrZJiBixcfUrV9zqQkGHwW+Bkvlv8ytnb9HiLqnE1f6jlp8fZmpkNJR4deKJmGw5mtDpTYP995fcmx2C1fgzw7UQkwYVtHi4xfv4zK3PyVnjwfgFUfQN6dm8j5uzhrISTfcXmNx6dZxCPA5k8c7Z8ys1qKUqoiouX48MUtMWgGKNmn80lfE1MgYdbaoRjyb4mal5/p0ijJTcXpJwMnJR8V2wwk6DUjlibUikQ8S3t68T13dM5u5P0T0nll0cYINuzDi5JrbU2aMXFJUhHooMw2EadtHSTjRlqycUdzVEXPqvhvFruo2mU3VmRxX1wl60hLlUjpwxh8Zc6g2chFQsf2R1Yan6FODF/JI/o1pHeB/ATPiDdL8EVN1gISYxzg8idubFtf4nWzKufN0L0FsZPcsDgra1NVc3oT5Ykt0pVSwPKiC6JV9xyhNMzQknsPakcJt9ndVraJPP4wLqlycG7r/P0KdHONnEcUoW7FFa/hCeRmplaQz1OgddmqxjMu/eDhSZNywrZRvhRYAF2QqS4VfvH8jEZ2d25hSWCBaz0KWpE1G6qgPigtxuZdOIYoXBU+dBITbXaGy98EFL+WNHD/WqP8hiVhxIKk3Wsq7qB9wA9rPAEbk5q+9NBBYlXFNNANqqmcSZG4LkTDmUp05osL6hL7KCvh+pVH9HPbMHmgeAGXGacau+XCGRAo2GaLfZipYPY1QO2iq7XgE3DLcsk0AU6V5UjwJZ6xBUHtQX4LWnP8KfOmt8t+QXeMLfFljEG0l/tMCfTKmW8rp1FSBWCxXksPWt7EZ29nFUEth7RMOxQ2nLpL0RSBkeJkxT+yKW53BibKjT+j7QmjNxn6VEJUGoGtu5ZOOX01oZ3eVBwTk0AEQgGVgGcEICzbkQdQa2grLivMz4OPkPjlX+zEg+3j++YPSIIOzxsypkOXjYiPcJk
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: ericsson.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM0PR0702MB3603.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ce39614e-894b-488c-974d-08d8d3ef2de9
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Feb 2021 09:25:49.3233 (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: yTWNX5N9ILNRx8XQCm+TLp4R6T8UarClnoZXY4zvUsOifrjMtZkPkhAg/Zissqz6k2HfxRLufrYC78Gd/mMo6gpdXtOY+Gi3cc6AsdGV6MA=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM9PR07MB7700
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/fFMJi5q2EP4I3-aKFbjQZdKlEPM>
Subject: Re: [Detnet] Benjamin Kaduk's No Objection on draft-ietf-detnet-tsn-vpn-over-mpls-06: (with COMMENT)
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: Thu, 18 Feb 2021 09:26:00 -0000
Hi Benjamin, Many thanks for your review. Reaction inline, nits will be fixed. Thanks Bala'zs -----Original Message----- From: detnet <detnet-bounces@ietf.org> On Behalf Of Benjamin Kaduk via Datatracker Sent: Wednesday, February 17, 2021 7:36 PM To: The IESG <iesg@ietf.org> Cc: detnet@ietf.org; draft-ietf-detnet-tsn-vpn-over-mpls@ietf.org; lberger@labn.net; detnet-chairs@ietf.org Subject: [Detnet] Benjamin Kaduk's No Objection on draft-ietf-detnet-tsn-vpn-over-mpls-06: (with COMMENT) Benjamin Kaduk has entered the following ballot position for draft-ietf-detnet-tsn-vpn-over-mpls-06: No Objection When responding, please keep the subject line intact and reply to all email addresses included in the To and CC lines. (Feel free to cut this introductory paragraph, however.) Please refer to https://www.ietf.org/iesg/statement/discuss-criteria.html for more information about IESG DISCUSS and COMMENT positions. The document, along with other ballot positions, can be found here: https://datatracker.ietf.org/doc/draft-ietf-detnet-tsn-vpn-over-mpls/ ---------------------------------------------------------------------- COMMENT: ---------------------------------------------------------------------- While I could easily imagine a document like this being an Informational document, I don't object to the requested Proposed Standard status. (FWIW, the only strong new normative requirement I remember seeing is that aggregation of TSN flows into DetNet flows "SHALL be supported".) I didn't take the time to sign up for an IEEE account to use the "IEEE Get" program and pull the external references, but what is described from them seems to make sense just from the standalone descriptions here. I just have a couple of (barely) substantive comments, followed by some editorial/nit-level remarks that need no reply. Section 7 Given that this documents calls for flooding of TSN frames in some scenarios, it seems like a caution against the risk of L2 loops would not be out of place (though I would hope no one would be surprised about it). <Bala'zs> Yes. TSN Streams are engineered to avoid loops. Section 10.2 I think the IEEE references need to be normative; we say that DetNet Edge Node acts as a TSN entity and so compliance with the TSN specs is mandatory. <Bala'zs> OK. And the editorial stuff: Section 4.1 MPLS DetNet nodes and transit nodes include DetNet forwarding sub-layer functions, support for notably explicit routes, and resources allocation to eliminate (or reduce) congestion loss and jitter. nits: I think this is better as "notably, support for explicit routes and resource allocation to eliminate (or reduce) congestion loss and jitter". <Bala'zs> OK, thanks. I will update. Section 4.2 My understanding is that each column in figure 3 is intended to represent a distinct TSN Stream/App-flow; it might be useful to confirm that somewhere (e.g., with a note about showing three example flows in the figure). In the figure, "Application" indicates the application payload carried by the TSN network. "MPLS App-Flow" indicates that the TSN Stream is the payload from the perspective of the DetNet MPLS data plane defined in [RFC8964]. [...] nit: the figure seems to say "App-Flow for MPLS" rather than the "MPLS App-Flow" in the quoted prose. <Bala'zs> OK, thanks. I will update. Section 5 Description of Edge Nodes procedures and functions for TSN over DetNet MPLS scenario follows the concept of [RFC3985] and covers the Edge Nodes components shown on Figure 1. In this section the following procedures of DetNet Edge Nodes are described: some nits here; maybe NEW: > The description of Edge Node procedures and functions for TSN over > DetNet MPLS scenarios follows the concepts from [RFC3985], and covers > the Edge Node components shown in Figure 1. In this section the > following procedures of DetNet Edge Nodes are described: <Bala'zs> OK, thanks. I will update. Section 5.1 TSN specific functions are executed on the data received by the DetNet Edge Node from the connected CE before forwarded to connected CE(s) or presentation to the DetNet Service Proxy function for transmission across the DetNet domain. [...] nit: there's a type of speech mismatch between "forwarded" and "presentation" (and "forwarded" would need to be "being forwarded" in order to be grammatical), so the list doesn't have a parallel structure. I'd suggest "before being forwarded to connected CE(s) or presented to the DetNet Service Proxy function", but there are plenty of other valid options. <Bala'zs> OK, thanks. I will update. When a TSN entity of the PE receives a packet from the DetNet Service Proxy, it first checks via Stream Identification (see Clause 6. of IEEE 802.1CB [IEEE8021CB] and IEEE P802.1CBdb [IEEEP8021CBdb]) whether the packet belongs to a configured TSN Stream. If no Stream ID is matched, then packet is dropped. [...] nit: s/then packet/then the packet/ <Bala'zs> OK, thanks. I will update. Section 5.2 When a DetNet Service Proxy receives a packet from the TSN Entity it MUST check whether such an App-flow is present in its mapping table. If present it associates the internal DetNet flow-ID to the packet and MUST forward it to the DetNet Service and Forwarding sub-layers. If no matching statement is present it MUST drop the packet. (nit) this seems to be the only place I could find that uses the term "statement" to refer to an entry in the DetNet flow mapping table; I'd suggest using a phrasing like "if no match is found". <Bala'zs> OK, thanks. I will update. The management or control function that provisions flow mapping SHALL ensure that adequate resources are allocated and configured to provide proper service requirements of the mapped flows. nit: I think we'd say "provider proper service that meets the requirements" or "to fulfil the service requirements"; just "providing service requirements" doesn't seem to match up to the intended meaning. <Bala'zs> OK, thanks. I will update. Due to the (intentional) similarities of the DetNet PREOF and TSN FRER functions service protection function interworking is possible between the TSN and the DetNet domains. Such service protection interworking scenarios MAY require to copy sequence number fields from TSN (L2) to PW (MPLS) encapsulation. However, such interworking is out-of-scope in this document and left for further study. This feels more like a descriptive "may" or "might" than a normative "MAY" to me. <Bala'zs> OK, thanks. I will update. Section 5.3 sequence number are not valid outside the DetNet network. MPLS (DetNet) Edge node terminates all related information elements encoded in the MPLS labels. nit: missing article for "MPLS Edge node", but I'd suggest converting to the plural "Edge nodes terminate" to avoid the issue. <Bala'zs> OK, thanks. I will update. Section 6 For example, it may be not trivial to locate the egress point/interface of a TSN Streams with a multicast destination MAC address. nit: singular/plural mismatch "a TSN Streams" (it looks like using the singular "Stream" is the minimal change to resolve). <Bala'zs> OK, thanks. I will update. _______________________________________________ detnet mailing list detnet@ietf.org https://www.ietf.org/mailman/listinfo/detnet
- [Detnet] Benjamin Kaduk's No Objection on draft-i… Benjamin Kaduk via Datatracker
- Re: [Detnet] Benjamin Kaduk's No Objection on dra… Balázs Varga A