Re: [Detnet] Benjamin Kaduk's No Objection on draft-ietf-detnet-mpls-over-udp-ip-07: (with COMMENT)

Balázs Varga A <balazs.a.varga@ericsson.com> Mon, 14 December 2020 16:10 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 34F7B3A1216; Mon, 14 Dec 2020 08:10:28 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.202
X-Spam-Level:
X-Spam-Status: No, score=-0.202 tagged_above=-999 required=5 tests=[DKIMWL_WL_HIGH=-0.001, 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 IQzTTDEtM21R; Mon, 14 Dec 2020 08:10:26 -0800 (PST)
Received: from EUR05-VI1-obe.outbound.protection.outlook.com (mail-vi1eur05on2045.outbound.protection.outlook.com [40.107.21.45]) (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 351103A1214; Mon, 14 Dec 2020 08:10:26 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=cjNbRV9ZwQIJZz5sFbMicPtKY7Z5ZMoHtypRa1XSy5hL68gd0xlysf7oWlqfv1fXcmDNBjyRPSa07Q8jgzurW6o+Sf707uQ5fTJOVxElYCN0OD5UhsbDjCt95W6zHR1Rlf0XC5XTOQ7G9LskxLepb+dgzbMvvt8BhBWPKyQsgRJqNjTb0qBBSY06ZT/buP/NT1KxyQws+saOcmXzcb+l6QJ3za4W/oeODGi/9IXSu8Dr49S3oWCus7jqHJFRuyWK3l+ks2NvsmvXmckKBqwV1KZIR2QyRNlq+mF3yldeh3iqmA28GCUQZlgNm7oxb1fxwdzBf5MKHJHSZ7oTY0C2sg==
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=z3a7IGWwPrKj3e4DewINRWATLvu9NDkSaNxcnWS9Dyo=; b=fVNK4FIi4FvXcEFwHelr5SORMffLzxzc4zlKRUnqr2bnYuRq6taXKip50Dz+rdcKYQwkGG3knlOFCUlSYCThW8ASmq+1kNJ8R8Ddd9n90D7UyuiABuREjS0ZHVHKGZlLpoE87ViiY6mAHRq0R11dUSA3kZhOKTzB1UEfA7O7sleR1PBf9xFeyFqhv6op225okSv9ftcWBLa7a+YO0LVGGKx0FK9ulmPY2ghdOTB3B2sd/dC9HRkioauyBredTd527QoiN0rk++MJnJ1o2rZ1jcBIqs4l7brRYL1M3PNE7deA7Kpp/08s0Q2rtTaRj4+ExsX0hPLGPZydQlfROkpKoA==
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=z3a7IGWwPrKj3e4DewINRWATLvu9NDkSaNxcnWS9Dyo=; b=uxBdmxkzRnSNDIGbVypxwyq26EaL/1Epgy1gidxowSHtfnNSrHB8f9cn6QWCAnPVd+PRpsQzaOaig6vc4NVKlqQZGseH7Sf3VVwxIWaf3gw5EIdpLPJpNUZjp79GPyRLu57/C1D35tyMOQL0IYAB7exyL5DwTT+dJ0XdAouKD18=
Received: from AM0PR0702MB3603.eurprd07.prod.outlook.com (2603:10a6:208:22::25) by AM0PR07MB6163.eurprd07.prod.outlook.com (2603:10a6:208:ee::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3676.15; Mon, 14 Dec 2020 16:10:23 +0000
Received: from AM0PR0702MB3603.eurprd07.prod.outlook.com ([fe80::e94e:dc9f:5924:3fda]) by AM0PR0702MB3603.eurprd07.prod.outlook.com ([fe80::e94e:dc9f:5924:3fda%2]) with mapi id 15.20.3676.013; Mon, 14 Dec 2020 16:10:23 +0000
From: Balázs Varga A <balazs.a.varga@ericsson.com>
To: Benjamin Kaduk <kaduk@mit.edu>, The IESG <iesg@ietf.org>
CC: "draft-ietf-detnet-mpls-over-udp-ip@ietf.org" <draft-ietf-detnet-mpls-over-udp-ip@ietf.org>, "detnet-chairs@ietf.org" <detnet-chairs@ietf.org>, "detnet@ietf.org" <detnet@ietf.org>, Ethan Grossman <eagros@dolby.com>
Thread-Topic: Benjamin Kaduk's No Objection on draft-ietf-detnet-mpls-over-udp-ip-07: (with COMMENT)
Thread-Index: AQHWyCeRsx5tdVkwj0ST54unJcxpeqn21SBA
Date: Mon, 14 Dec 2020 16:10:23 +0000
Message-ID: <AM0PR0702MB3603B3D162DCD942D37B5BF5ACC70@AM0PR0702MB3603.eurprd07.prod.outlook.com>
References: <160685752482.30350.17229104604967756401@ietfa.amsl.com>
In-Reply-To: <160685752482.30350.17229104604967756401@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: [178.164.141.167]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: b7f8ce13-2b2f-4fc3-91a9-08d8a04ac338
x-ms-traffictypediagnostic: AM0PR07MB6163:
x-microsoft-antispam-prvs: <AM0PR07MB61630F1B52A94A9B5C3C2955ACC70@AM0PR07MB6163.eurprd07.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:6790;
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: c5IkVDb7ivxS35rbMP2jOUev5RvKK8aShtGnWsact/ZEL4QRnSRlge7naz7pgw1ex2C5vIgY/7Cv7aZFyhJbsD/qqw+vw/9gqLYcLlMDAIjjZZyPhHAv/PRmmUKkG6tKX6mG7cKMGTjJj7DlzrlMOqO7Tfd+BEXRZN5M/DiS2OzZaqbGiwpOB3jtwSK2J1zk+jmZ7uhnQHv9CQf8d4xl31B4BNBEAUFBcHd0D2FbaI5mChE3j9skJ/v3ta2HXMkDG+mg27L4OwiksysJNShF8kTjdVl6QBoHUPyoSA8Ul8sKf5Gj29K0SxdsWMGWmR+WCl8DrQAbdomPvQz4266vagz6QPrQV3IMWKy/DZTBn6FBksf26uDZ6aj0ImZhrGWn0p9AAGUbWmUAXi3MHJxjOMX72Mp+ayMar1/0nqZ/hsW6LWcy4iJfqCGvd3U1Jp+r
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)(136003)(39860400002)(366004)(396003)(346002)(376002)(110136005)(5660300002)(54906003)(4326008)(85202003)(83380400001)(66574015)(6506007)(33656002)(55016002)(86362001)(7696005)(53546011)(186003)(8676002)(478600001)(66446008)(66476007)(9686003)(2906002)(66556008)(8936002)(26005)(66946007)(966005)(85182001)(64756008)(71200400001)(52536014)(316002)(76116006); DIR:OUT; SFP:1101;
x-ms-exchange-antispam-messagedata: g6u+xMpigSG4YR1y2TeGPZ+6GBMH0SBbqdnQwamn4uT36ql3ZVfJs8vPNZgV9pvIqSC6PiQDlJJ9BzmiK9xhe4ZHL5tAkXS6iUBstu1Jdbeg/AJyipeK0i/JIZVFMgK6AYfMghbeUJ6UCn3uRS3S1LLG7PU7ZT4z+Bhx6R25ytAbyqmDayzBrqBxxCldu9NtZHhUgBc9EhOj4uucE/e0KYywAK98to3JQ70RSiCo8vWaxscbFXV8EEqEpYc3DXccidGd/JuURVP720RA4lyTl4gaq4M3rBJi2MoC+gf7DaCg0aYICxuOKqgGLwh2EyYd4E+DgGKGDRWfdhVTxyn502gy/OoOPFjFTkYI3620S2O78Gr11bVlR5uFWROUenNZSFgo4jG52ZYeL9BctBmxgb7BUKyIsexCc52+BPzksw6pFNT09BxH53oMc0Y+lMBO+PRTsDsjCL8q4AlPYKbgj3SU5bBZtvNx5byi+h2UXiEY/5P/kVuoFVEWIZGg3g3V1nltNUSZoGcqCmtb1iHmYSGTZxNyQw8sA8cdX3oDEraEc0lELfSZXBcHXAJ4o2+oj/vYKoazVe4JliiMtRhR7hFDB2KPMlAds/iEBQYhONxMig97+9NKRmp2TuNuna1HiLLraW4xM7VK7TL7bajwTJadP0QYvXfqsVz3o/rW80zclnKEqx/Dd+o7rGbnJF4BB9TQ8/MoXPz82kBuBKuEJ2Cj/nCdcFhTGveekF9+Q5npEm7K7FqXsK+2mV9XTAu5RIfiiPoyOL0Giw/OMZQI6NcDh0GixveAeXb43N5OUOrA6hH8cDeCB7FYAVrK+e8/cut0F08W0QYEsJn/kFR7P4IBGvz+Fp1arcto0s/6q2SAwUeR5ZCJB/j24sM/U+pYkjqSGBViEOBHtLVjpuxbULAvqdQGmg84goa9jdXYnhzrTvgaXQrZ7aLwhlCpL12Lg2u/sCH+YPLvAc+XWo3umjB3l34U9Luj2qqvCCo5FZVqZCHso8pQ+NgT1sH4wlrx
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-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM0PR0702MB3603.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b7f8ce13-2b2f-4fc3-91a9-08d8a04ac338
X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Dec 2020 16:10:23.5793 (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: D0hkJwvP1AjKhm3KzqNeYnMwmVwi8hMajlewVLeJD7dDJp1l5P/M3hn2JPJTQSXbLwyoxSzxr7qJQHYHu4537VV4G9wDvuvQ3JRExnvHEXM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR07MB6163
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/85n3L6IllMdJg4vvrsmCFbmUJMM>
Subject: Re: [Detnet] Benjamin Kaduk's No Objection on draft-ietf-detnet-mpls-over-udp-ip-07: (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: Mon, 14 Dec 2020 16:10:28 -0000

Hi Benjamin,

Many thanks for your review and feedbacks. 
I will fix the nits. Please, see my replies inline marked <BV>

Thanks & Cheers
Bala'zs

-----Original Message-----
From: Benjamin Kaduk via Datatracker <noreply@ietf.org> 
Sent: Tuesday, December 1, 2020 10:19 PM
To: The IESG <iesg@ietf.org>
Cc: draft-ietf-detnet-mpls-over-udp-ip@ietf.org; detnet-chairs@ietf.org; detnet@ietf.org; Ethan Grossman <eagros@dolby.com>; eagros@dolby.com
Subject: Benjamin Kaduk's No Objection on draft-ietf-detnet-mpls-over-udp-ip-07: (with COMMENT)

Benjamin Kaduk has entered the following ballot position for
draft-ietf-detnet-mpls-over-udp-ip-07: 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-mpls-over-udp-ip/



----------------------------------------------------------------------
COMMENT:
----------------------------------------------------------------------

Basically just nits here; as the shepherd writeup notes, it's "pretty much just putting the pieces together".

Section 1

   2.  A method for carrying the DetNet sequence number.

   3.  A method for distinguishing DetNet OAM packets from DetNet data
       packets.

(nitty nit) these two are in the reverse order as they appear in draft-ietf-detnet-mpls

<BV> OK. Thanks.


Section 3

                                                    The UDP and IP
   header information is used to identify DetNet flows, including member
   flows, per [I-D.ietf-detnet-ip].  [...]

I couldn't find where draft-ietf-detnet-ip discussed member flows; could you give me a pointer?

<BV> Sure. https://tools.ietf.org/html/rfc8655


Section 4

   To support outgoing DetNet MPLS over UDP encapsulation, an
   implementation MUST support the provisioning of UDP and IP header
   information in addition or in place of F-Label(s).  Note, when PRF is

nit: s/in addition/in addition to/

<BV> OK. Thanks.


Section 5

   o  Label information (A-labels, S-labels and F-labels) to be mapped
      to UDP/IP flow.  Note that for example, a single S-Label can map

nit: s/flow/flows/ (singular/plural mismatch between "labels" and "flow")

<BV> OK. Thanks.


Section 6

The only potentially new consideration to the mpls-over-udp formulation of detnet is that the forwarding logic can be split across two places (IP+UDP headers and MPLS label stack), which makes implementation somewhat more complext and thus prone to error.  But that's probably more of an implementation issue than a protocol issue, so I don't feel very strongly that it must be documented here.

That said, I would also not be opposed to repeating the (still somewhat evolving, I guess) boilerplate from the other detnet RFCs/drafts about "security aspects which are unique to DetNet".  The reasoning is that we have a default Internet threat model, espoused in RFC 3552, and anything detnet fundamentally has to consider a weaker threat model in order to be able to do anything useful.  Since this document, in addition to the referenced ones, is also deviating from the default Internet threat model, that can be worth noting explicitly.

   [RFC8655] and [I-D.ietf-detnet-security].  Finally,MPLS and IP

nit: space after comma.

<BV> OK. Thanks.


Section 10.2

(draft-ietf-6man-segment-routing-header is RFC 8754, now, though I'm sure the RFC Editor will catch that.)

<BV> OK. Thanks.