Re: [Detnet] Transport sub-layer name change (Was Re: Tsvart last call review of draft-ietf-detnet-architecture-08)
"Grossman, Ethan A." <eagros@dolby.com> Tue, 20 November 2018 18:28 UTC
Return-Path: <eagros@dolby.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 11D49130DCF; Tue, 20 Nov 2018 10:28:57 -0800 (PST)
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, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, 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=dolby.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 4oa_gye1matL; Tue, 20 Nov 2018 10:28:52 -0800 (PST)
Received: from NAM05-DM3-obe.outbound.protection.outlook.com (mail-dm3nam05on0728.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe51::728]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id E76161277BB; Tue, 20 Nov 2018 10:28:51 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dolby.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=WfApsU/uwmucwxT6c0Sq0xweQQNYdQSvJyZKqPKJ3Lo=; b=XpYi+97YWtMgVz+k22CH6S0IPtd/nxe3tSKrcGymoms/F1srpfPvaod1BBqySkOKc2UFM65LYNkbO/+euu5TzVEwf4+WI5enLq271chowjlb5Up5JYisrRt4zQyC4cM3duwwZOnsmBDWbZwktRSiJgRaoquStuLesGJdvrKYBdw=
Received: from BL0PR06MB4548.namprd06.prod.outlook.com (20.177.145.145) by BL0PR06MB4308.namprd06.prod.outlook.com (10.167.179.225) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1339.26; Tue, 20 Nov 2018 18:28:47 +0000
Received: from BL0PR06MB4548.namprd06.prod.outlook.com ([fe80::a1e4:ef9a:133f:b991]) by BL0PR06MB4548.namprd06.prod.outlook.com ([fe80::a1e4:ef9a:133f:b991%4]) with mapi id 15.20.1339.027; Tue, 20 Nov 2018 18:28:47 +0000
From: "Grossman, Ethan A." <eagros@dolby.com>
To: "Pascal Thubert (pthubert)" <pthubert@cisco.com>, Lou Berger <lberger@labn.net>, "Scharf, Michael" <Michael.Scharf@hs-esslingen.de>
CC: "detnet@ietf.org" <detnet@ietf.org>, "draft-ietf-detnet-architecture.all@ietf.org" <draft-ietf-detnet-architecture.all@ietf.org>
Thread-Topic: Transport sub-layer name change (Was Re: [Detnet] Tsvart last call review of draft-ietf-detnet-architecture-08)
Thread-Index: AQHUgP3y0J9+EGvI5kmFt4kbuqY1w6VY+2YA//8S6RA=
Date: Tue, 20 Nov 2018 18:28:47 +0000
Message-ID: <BL0PR06MB4548D6E06909D74F227C84E6C4D90@BL0PR06MB4548.namprd06.prod.outlook.com>
References: <153817345967.27205.135001179751151278@ietfa.amsl.com> <fdf872d6-08a6-2c33-de21-9dd1506c1d21@labn.net> <6EC6417807D9754DA64F3087E2E2E03E2D16A4D3@rznt8114.rznt.rzdir.fht-esslingen.de> <e38ab4d6-0924-ab60-b1dc-4ac26600044c@labn.net> <16c050e436f342bb94b1ec9d1a38da3e@XCH-RCD-001.cisco.com> <3adfa63a-e6de-b899-f7ce-79d8f668d40f@labn.net> <dfea900c1cb54ee88a953f22a9c7e639@XCH-RCD-001.cisco.com>
In-Reply-To: <dfea900c1cb54ee88a953f22a9c7e639@XCH-RCD-001.cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-dg-ref: PG1ldGE+PGF0IG5tPSJib2R5LnR4dCIgcD0iYzpcdXNlcnNcZWFncm9zXGFwcGRhdGFccm9hbWluZ1wwOWQ4NDliNi0zMmQzLTRhNDAtODVlZS02Yjg0YmEyOWUzNWJcbXNnc1xtc2ctNjk4MTg1MTUtZWM3Yi0xMWU4LWI3NTYtYWNiYzMyN2E1MWM2XGFtZS10ZXN0XDY5ODE4NTE3LWVjN2ItMTFlOC1iNzU2LWFjYmMzMjdhNTFjNmJvZHkudHh0IiBzej0iMjg3MDQiIHQ9IjEzMTg3MTYxMTQ0MDIyODU5OSIgaD0idVRhOFpyTksrSXczekhLVUZiOW1qNFZXekhjPSIgaWQ9IiIgYmw9IjAiIGJvPSIxIi8+PC9tZXRhPg==
authentication-results: spf=none (sender IP is ) smtp.mailfrom=eagros@dolby.com;
x-originating-ip: [8.39.141.5]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; BL0PR06MB4308; 6:F6GVa9yrl1yRQu3F9xq0mphIiT3NarG62dbq8WohKyjfdmx58Meunuctdrygoyz39vct7TDxuh3QLhl277EzjmLRfXacYpcDus9Bx7UY+3qykte9ZHa2dkJi5bCkSSXHCSD7n2g2LmgTJd9knJa71gM10Iigi409phTQP9S4ueyi2BW+fTbn80BFWr+8CQornpth/xicHyUeYnUiSvJtime8sJqqRoHKGVz0w8QIjEIkLDYJXR7hDuftGp2GKcYRfv3vo4dFguuLztFHavRuSI1H2o8a941/N3naRb5k/tFPN4GGarhKoonzliS81lIiGrW+1ikFHnl5RZ0pIwY/4ANhVGS849cD8VosI6pJ3V9rbHtKq5Q56QEzqyBJt/ai3ZmllmDV/YEr6rdEkTJgc+TGykuAxC2ZUTm8Bx3zalfloxiHpEtL2+vxOMo7ud9KFoH5ag61ellxWAX1oq8kGA==; 5:wJVTgSERp3llVrml04fwTa6POYppq6I5oCw/T1DmvhPyenztacq/35d/Zjz52HxjQoedZAAFRannBRvB2V1OJLigHlKcj8b1AFwaIGQf+zWvuCrfKxwgPsmfDf1uRQoxsmJ57I2EKKVG+xIAb4pXhV0Rj1lzko2iYNktRUNx1yY=; 7:Jjr/A00+GfqN2hpU3Ta0s76bDgBHreB90LO7nW69jNQ2ayG9IUPMEa6bPLkHwAKKsLasMwxLk18NKMPYxFth3FWZdphrjVEAnJScvlnRvaJkFNCkj8H5M4uxwAXRo7uJ+wrxGoOKtR0fmly9COtuWA==
x-ms-exchange-antispam-srfa-diagnostics: SOS;
x-ms-office365-filtering-correlation-id: 4fa8004d-ff32-4a05-1ff3-08d64f16030e
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390098)(7020095)(4652040)(8989299)(5600074)(711020)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(2017052603328)(7153060)(7193020); SRVR:BL0PR06MB4308;
x-ms-traffictypediagnostic: BL0PR06MB4308:
x-microsoft-antispam-prvs: <BL0PR06MB43080C2C8B1DC755EFF29EFAC4D90@BL0PR06MB4308.namprd06.prod.outlook.com>
x-ms-exchange-senderadcheck: 1
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040522)(2401047)(5005006)(8121501046)(3231442)(944501410)(52105112)(10201501046)(3002001)(93006095)(93001095)(148016)(149066)(150057)(6041310)(20161123558120)(20161123560045)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(201708071742011)(7699051)(76991095); SRVR:BL0PR06MB4308; BCL:0; PCL:0; RULEID:; SRVR:BL0PR06MB4308;
x-forefront-prvs: 08626BE3A5
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(366004)(346002)(136003)(39860400002)(376002)(396003)(13464003)(54094003)(189003)(199004)(43544003)(105586002)(93886005)(6436002)(478600001)(8676002)(54906003)(53936002)(110136005)(53946003)(74316002)(316002)(6246003)(9686003)(229853002)(6306002)(4326008)(66066001)(99286004)(55016002)(305945005)(5660300001)(966005)(14454004)(186003)(6116002)(3846002)(6506007)(53546011)(68736007)(7696005)(76176011)(25786009)(26005)(81156014)(8936002)(102836004)(2906002)(7736002)(106356001)(33656002)(2900100001)(476003)(561944003)(71190400001)(486006)(256004)(81166006)(4744004)(11346002)(446003)(14444005)(97736004)(71200400001)(86362001)(579004)(559001); DIR:OUT; SFP:1102; SCL:1; SRVR:BL0PR06MB4308; H:BL0PR06MB4548.namprd06.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:3; A:1;
received-spf: None (protection.outlook.com: dolby.com does not designate permitted sender hosts)
x-microsoft-antispam-message-info: fhCfTQ7lsEyxlzjqZndz5X3/W6FuQl7s/jhGQOZmqFcuyv4+ZM+OsmheSWCISnoX1TiPCsi7ZpgHB4CnxbsAnibjXeiGbCDEJrc1IWYTkAIWjeWNBe6Miut4HEHhfOGsRyymtPVsFtjypHO1URAtjLQClTVbTpz2uhthYAj4xBblnAgjs1GICN7z6FBWMF6YRFIf3h81avDBFArwsjT0XsbmDmnhq8dA7XV1W394Z4TdyZgq1Ws3CM67QfkYW2DSxhvAy22inwMNzWPcooNaUWxowI55CoaZk/oJCOuVOcFQav+gZNxUmKFWKl7chkriVkbvzdEjIQgB0CHJBP/9wdNhJGNTd4dcE2wwoduBxRg=
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: dolby.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4fa8004d-ff32-4a05-1ff3-08d64f16030e
X-MS-Exchange-CrossTenant-originalarrivaltime: 20 Nov 2018 18:28:47.6494 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 05408d25-cd0d-40c8-8962-5462de64a318
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL0PR06MB4308
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/7LN5FKq6B7SXPWsiGwLoowMafgU>
Subject: Re: [Detnet] Transport sub-layer name change (Was Re: Tsvart last call review of draft-ietf-detnet-architecture-08)
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: Tue, 20 Nov 2018 18:28:57 -0000
I like it. Ethan. -----Original Message----- From: Pascal Thubert (pthubert) <pthubert@cisco.com> Sent: Tuesday, November 20, 2018 10:27 AM To: Lou Berger <lberger@labn.net>; Scharf, Michael <Michael.Scharf@hs-esslingen.de> Cc: detnet@ietf.org; draft-ietf-detnet-architecture.all@ietf.org Subject: RE: Transport sub-layer name change (Was Re: [Detnet] Tsvart last call review of draft-ietf-detnet-architecture-08) I support this change; Pascal > -----Original Message----- > From: Lou Berger <lberger@labn.net> > Sent: mardi 20 novembre 2018 19:19 > To: Pascal Thubert (pthubert) <pthubert@cisco.com>; Scharf, Michael > <Michael.Scharf@hs-esslingen.de> > Cc: detnet@ietf.org; draft-ietf-detnet-architecture.all@ietf.org > Subject: Transport sub-layer name change (Was Re: [Detnet] Tsvart last > call review of draft-ietf-detnet-architecture-08) > > ALL, > > There is a desire to replace the word "Transport" from the DetNet > Transport sub-layer to avoid confusion with L$ Transport protocols. > > In the TEAS WG we had a similar discussion and we replaced "Transport" > with "Traffic Engineered (TE) ". > > While a bit more verbose, what do people think about this change? > > To be clear, the suggestion is: > > OLD > > . > . > +----------------------------+ > | DetNet Service sub-layer | PW, UDP, GRE > +----------------------------+ > | DetNet Transport sub-layer | IPv6, IPv4, MPLS TE LSPs, MPLS SR > +----------------------------+ > . > . > > Figure 4: DetNet adaptation to data plane > > NEW > > . > . > +----------------------------+ > | DetNet Service sub-layer | PW, UDP, GRE > +----------------------------+ > | DetNet TE sub-layer | IPv6, IPv4, MPLS TE LSPs, MPLS SR > +----------------------------+ > . > . > > Figure 4: DetNet adaptation to data plane > > Lou > > On 11/20/2018 11:21 AM, Pascal Thubert (pthubert) wrote: > > Hello Lou: > > > > > > About ' discuss changing the name of the "DetNet Transport > > sub-layer" to > avoid the word "transport". ' > > > > For one I'd like to make that call. The unfortunate name collision > > has started > to hurt us quite a bit already and people are getting confused on very > active exchanges we have on the mailing list. > > > > I tend to agree that for the general IETF "transport" generally means "L4". > Even point one in your email uses "transport" that way I guess. Sadly > many alternate names are highly overloaded already (think "carrier" > for instance, or "bus"). I like the term "train" because of the > association with a schedule, but that's just me. > > > > Same goes actually for the complex path that we build. That complex > > path > can be an elongated DODAG with multiple PREOF points. Usual terms like > "circuit" or "path" fail to capture that complexity. 6TiSCH found the > term "Track" to refer to it. > > > > Would you push that discussion to the ML? > > > > Take care, > > > > Pascal > > > >> -----Original Message----- > >> From: Lou Berger <lberger@labn.net> > >> Sent: mardi 20 novembre 2018 13:11 > >> To: Scharf, Michael <Michael.Scharf@hs-esslingen.de> > >> Cc: tsv-art@ietf.org; detnet@ietf.org; draft-ietf-detnet- > >> architecture.all@ietf.org > >> Subject: Re: [Detnet] Tsvart last call review of > >> draft-ietf-detnet-architecture-08 > >> > >> Michael, > >> > >> I think we're getting somewhere and identifying where we have > >> disconnects and what may (and what may not) need to change in the > >> document. My takeaways are: > >> > >> - The document needs a good 'scrub' of the congestion related > >> references to ensure that the document only makes statements on > >> what is actually done within a DetNet and the relationship with > >> transport protocols that use detnet (which are in fact outside the > >> scope of the DetNet WG). I'll work with the authors and WG on this > >> -- I see this change as important, but editorial in nature. > >> > >> - We have a perception issue with at least one member of the TSV > >> area on the meaning and more importantly, implication, of the term > >> "DetNet > >> *Transport* sub-layer". While I don't disagree that a good portion > >> of the IETF thinks transport protocol (UDP/TCP) when they hear "transport" > >> there are plenty others, particularly in the routing area, who > >> understand that "transport" can refer to Transport Networks. And > >> Transport Network is a well understood general industry term. The > >> IETF even has a bunch of RFCs that relate to Transport networks. > >> This said, I think it reasonable to go back to the DetNet WG and > >> discuss changing the name of the "DetNet Transport sub- layer" to > >> avoid the word "transport". -- BTW we made a parallel change in > >> the TEAS > WG when producing RFC8453. > >> > >> See below for detail response in-line. > >> > >> On 11/19/2018 5:15 PM, Scharf, Michael wrote: > >>> Lou, > >>> > >>>> -- > >>>> I wanted to take a step back from the multiple discussions that > >>>> were spawned by your review -- from a doc shepherd perspective, > >>>> and see where we are. I know that the authors have sent a -09 > >>>> version that addresses some, but not all issues. > >>>> > >>>> From the exchanges I've seen, I think the key remaining issues > >>>> are related to: > >>>> > >>>> (a) possibly introduction of congestion in the general internet > >>>> if packets were somehow to escape a detnet domain. The source of > >>>> this congestion would be inelastic traffic using DetNet or due to > >>>> congestion loss that is masked by PREOF. > >>> These are two major issues that need to be addressed. Note that it > >>> may not > >> be sufficient just to add a section on operational and deployment > >> considerations. Also the existing text in the document will need to > >> get aligned to normative guidance on how to avoid a congestion collapse. > >>> In -09, one example would be Section 3.1. "Primary goals defining > >>> the > >> DetNet QoS" > >>> Congestion protection operates by allocating resources along the path > >>> of a DetNet flow, e.g., buffer space or link bandwidth. Congestion > >>> protection greatly reduces, or even eliminates entirely, packet loss > >>> due to output packet congestion within the network, but it can only > >>> be supplied to a DetNet flow that is limited at the source to a > >>> maximum packet size and transmission rate. Note that congestion > >>> protection provided via congestion detection and notification is > >>> explicitly excluded from consideration in DetNet, as it serves a > >>> different set of applications. > >>> At least the last sentence would contradict a better discussion of > >>> congestion > >> in the document. For instance, it could just be removed. In any > >> case, the current wording in the last sentence is not correct, as > >> the IETF term for what is described in the last sentence is "congestion control". > >>> Another example would be Section 3.2.1.1. "Eliminate congestion loss" > >>> > >>> The primary means by which DetNet achieves its QoS assurances is to > >>> reduce, or even completely eliminate, congestion within a DetNet node > >>> as a cause of packet loss. This can be achieved only by the > >>> provision of sufficient buffer storage at each node through the > >>> network to ensure that no packets are dropped due to a lack of buffer > >>> storage. Note that a DetNet flow cannot be throttled, i.e., its > >>> transmission rate cannot be reduced via explicit congestion > >>> notification. > >>> > >>> This section IMHO has to include a discussion of what happens in > >>> the (not > >> expected) case that packets get dropped or that ECN marks are > >> received. It is understood that this would not happen in normal > >> operation of a DetNet network, but I believe just considering the > >> error-free operation of a DetNet network is not sufficient for this > >> document. At least for the risk of traffic that may escape from a > >> DetNet network is inherently not sufficient to assume that the > >> DetNet > network is always error-free. > >> > >> I think these are examples of text that needs to be cleanup up and > >> to delineate what is done with a DetNet. > >> > >> > >>> As a result, addressing my concerns will most likely require > >>> editing several > >> parts of the document. > >>> In addition, I'd like to emphasize that my review comment "It is > >>> surprising > >> that there is hardly any discussion on network robustness and safety" > >> > >> I have no idea what you mean by safety here. Can you elaborate. > >> > >> > >>> covers more than just inelastic traffic that escapes from a DetNet > >>> network > >> and masking of packet loss. Given that DetNet traffic may be > >> extremely critical traffic, I really wonder why the document > >> doesn't emphasize more the required robustness against failures > >> *inside* the DetNet network as well as counter-measures. But this > >> is something the WG needs to decide. As TSV-ART reviewer, I will be > >> fine if the document clearly describes how the impact of failures > >> will be isolated inside the DetNet network and will not put the > >> general Internet at > risk. > >> > >> I agree - I think, the document should be clear on it's scope and > >> relationship to general internet usage. > >> > >> > >>>> (b) The use of the term 'transport' in DetNet to refer to what is > >>>> basically a Traffic Engineered sub-network layer, such as is > >>>> provided with MPLS-TE or Optical Transport Networks. > >>> In the Internet architecture, the term 'transport' refers to > >>> Internet transport > >> protocols. I doubt that the document can avoid discussing the > >> implications of and interactions with Internet transport protocols > >> such as UDP or TCP. As a result, I disagree that the document can > >> use the term 'transport' to refer to traffic engineered sub-network layers. > >> > >> I think this is covered by my comment above. > >> > >> > >>> From a TSV-ART point of view, the document can either only use > >>> the term > >> "transport" for Internet transport protocols and use another term > >> for > >> sub- network layers (as handled in the *routing* area of the IETF), > >> or the document has to clearly distinguish between the Internet > >> transport layer and other uses of the term "transport" and explain > >> the overlap. I believe the former would be less confusing, but I > >> will leave it up to the TSV ADs to discuss terminology overlap in > >> the IESG. As TSV-ART reviewer I insist that the document uses the > >> terms "transport layer" and "transport protocol" only when > >> referring to the > Internet transport layer. > >> > >> I'm personally okay with a name change and even willing to push > >> this discussion within the WG, but as said above, "Transport > >> Network" is a generally understood industry term that is also used > >> in RFCs -- so we'll have to see what where WG consensus ends up. > >> > >>>> Do you have any other issues that that are critical to be > >>>> addressed before this work moves forward? If so which? > >>> Regarding Section 4.4 I have already deferred the discussion to > >>> the IESG. The > >> TSV-ART review comment is that the IESG needs to carefully look at > >> the concepts, terminology, and references in section 4.4. > >>> Regarding my other comments, I acknowledge that -09 is a step > >>> forward. But > >> given the cross-dependencies e.g. regarding terminology and > >> definitions, I will need to read the text completely once there is > >> a proposal how to address my review. As noted in my review, I > >> believe the document must use terminology clearly and consistently. > >> As example, a statement in -09 such as "Network nodes supporting > >> DetNet flows have to implement some of the DetNet capabilities (not > >> necessarily all) in order to treat DetNet flows such that their QoS > >> requirements are met" is IMHO too vague. But in such cases it > >> depends whether there is precise normative guidance elsewhere. And > >> this requires > looking at the text as a whole. > >> > >> I think the next steps lie with me and the WG. We'll let you know > >> once there is a new version. Of course, you can also contribute to > >> the WG discussion on the topic. > >> > >> Thanks, > >> > >> Lou > >> > >>> Best regards > >>> > >>> Michael > >>> > >>> > >>> > >>>> Thank you, > >>>> > >>>> Lou > >>>> > >>>> On 9/28/2018 6:24 PM, Michael Scharf wrote: > >>>>> Reviewer: Michael Scharf > >>>>> Review result: Ready with Issues > >>>>> > >>>>> The document "Deterministic Networking Architecture" > >>>>> (draft-ietf-detnet-architecture-08) defines an overall framework > >>>>> for Deterministic Networking. > >>>>> > >>>>> As TSV-ART reviewer, I believe that this document has issues as > >>>> detailed below. > >>>>> Michael > >>>>> > >>>>> Major issues: > >>>>> > >>>>> * It seems that DetNet cannot easily be deployed in the Internet > >>>> without > >>>>> additional means. Thus, for a baseline document, one could > >>>>> expect > >>>> some > >>>>> explanation on the requirements of deploying DetNet in a network. > >>>> DetNet > >>>>> basically requires support in (almost) all network devices > >>>> transporting DetNet > >>>>> traffic. That assumption should be explicitly spelt out early in > >>>>> the > >>>> document, > >>>>> e.g., in the introduction. There also needs to be an explicit > >>>> discussion of the > >>>>> implications if not the whole network is aware of or supports DetNet. > >>>> There is > >>>>> some text in Section 4.2.2 and Section 4.3.3, but I believe > >>>> additional explicit > >>>>> discussion is needed at a prominant place. For instance, can use > >>>>> of > >>>> DetNet do > >>>>> harm to parts of a network not supporting DetNet? As a side > >>>>> note, > >>>> when TCPM > >>>>> published RFC 8257, the following disclaimer was added: "DCTCP, > >>>>> as > >>>> described in > >>>>> this specification, is applicable to deployments in controlled > >>>> environments > >>>>> like data centers, but it must not be deployed over the public > >>>> Internet without > >>>>> additional measures." I wonder if a similar disclaimer is needed > >>>>> for > >>>> DetNet. If > >>>>> there is an implicit assumption that DetNet will be used in > >>>> homogenous > >>>>> environments with mostly DetNet-aware devices within the same > >>>> organization, > >>>>> such an assumption should be made explicit. > >>>>> > >>>>> * It is surprising that there is hardly any discussion on > >>>>> network > >>>> robustness > >>>>> and safety; this probably also relates to security. For > >>>>> instance, misconfiguration or errors of functions performing > >>>>> packet replication > >>>> could > >>>>> severely and permantly congest a network and cause harm. How > >>>>> does the > >>>> DetNet > >>>>> architecture ensure that a network stays fully operational e.g. > >>>>> if > >>>> the topology > >>>>> changes or there are equipment failures? Probably this can be > >>>>> solved > >>>> by > >>>>> implementations (e.g., dynamic control plane), but why are > >>>> corresponding > >>>>> requirements not spelt out? Section 3.3.2 speculates that > >>>>> filters and > >>>> policers > >>>>> can help, and that may be true, but that probably still assumes > >>>> consistently > >>>>> and correctly configured (and well-behaving) devices. And > >>>>> Section > >>>> 3.3.2 is > >>>>> vague and mentions a "infinite variety of possible failures" > >>>>> without > >>>> stating > >>>>> any requirements or recommendations. There may be further > >>>>> solutions, > >>>> such as > >>>>> circuit breakers and the like. Why are such topics not discussed? > >>>>> > >>>>> * Somewhat related, the document only looks at impact of > >>>>> failures to > >>>> the QoS of > >>>>> DetNet traffic. What is missing is a discussion how to protect > >>>>> non- > >>>> DetNet parts > >>>>> of a network from any harm caused by DetNet mechanisms. > >>>>> Solutions to > >>>> this > >>>>> probably exist. But why is the impact on non-DetNet traffic > >>>>> (e.g., in > >>>> case of > >>>>> topology changes or failures of DetNet functions) not discussed > >>>>> at > >>>> all in the > >>>>> document? > >>>>> > >>>>> * Regarding security, an architecture like DetNet probably > >>>>> requires > >>>> that only > >>>>> authenticated and authorized end systems have access to the data > >>>> plane. The > >>>>> security considerations only briefly mention the control aspect > >>>>> ("the authentication and authorization of the controlling systems"). > >>>>> > >>>>> * For an architecture document, the lack of clarity and > >>>>> consistency > >>>> regarding > >>>>> terminology is concerning. This specifically applies to the case > >>>>> of > >>>> incomplete > >>>>> networks (as per Section 4.2.2 and 4.3.3) that include "DetNet- > >>>> unaware nodes". > >>>>> The document introduces terms such as "DetNet intermediate nodes" > >>>>> but > >>>> then > >>>>> repeatedly uses generic terms such as "node" or "hop" that may > >>>> include > >>>>> DetNet-unaware nodes. For instance, for incomplete networks, a > >>>> sentence such as > >>>>> "The primary means by which DetNet achieves its QoS assurances > >>>>> is to > >>>> reduce, or > >>>>> even completely eliminate, congestion within a node as a cause > >>>>> of > >>>> packet loss" > >>>>> seems to only apply to "DetNet transit nodes" but not > >>>>> "DetNet-unaware > >>>> nodes". > >>>>> Similar ambiguity exist for other use of the terms "hop" and > >>>>> "node", > >>>> which may > >>>>> or may not include DetNet-unaware nodes. It is unclear why the > >>>> document does > >>>>> not consistently use the terminology introduced in Section 2.1 > >>>>> in all > >>>> sections > >>>>> and clearly distinguishes cases with and without DetNet support. > >>>>> > >>>>> * Section 4.4 refers to RFC 7426, which is an informational RFC > >>>>> on > >>>> IRTF stream, > >>>>> and the document uses the concepts introduced there (e.g., "planes"). > >>>> This is > >>>>> very confusing. First, an IETF Proposed Standard should probably > >>>> refer to > >>>>> documents having IETF consensus. An example would be RFC 7491, > >>>>> albeit > >>>> there is > >>>>> other related work as well, e.g., in the TEAS WG. Second, > >>>>> Section > >>>>> 4.4 > >>>> is by and > >>>>> large decoupled from the rest of the document and not specific > >>>>> to > >>>> DetNet. > >>>>> Neither do other sections of the document refer to the concepts > >>>> introduced in > >>>>> Section 4.4, nor does Section 4.4 use the DetNet terminology or > >>>> discuss > >>>>> applicability to DetNet. Section 4.4 even mentions explicitly at > >>>>> the > >>>> end that > >>>>> it discusses aspects that are orthogonal to the DetNet architecture. > >>>> It is not > >>>>> at all clear why Section 4.4 is in this document. Section 4.4 > >>>>> could > >>>> be removed > >>>>> from the document without impacting the rest of the document. > >>>>> > >>>>> Minor issues: > >>>>> > >>>>> * Terminology "DetNet transport layer" > >>>>> > >>>>> The term "transport layer" has a well-defined meaning in > >>>>> the IETF, > >>>> e.g. > >>>>> originating from RFC 1122. While "transport" and e.g. > >>>>> "transport > >>>> network" is > >>>>> used in the IETF for different technologies in different > >>>>> areas, I > >>>> think the > >>>>> term "transport layer" is typically understood to refer to > >>>> transport > >>>>> protocols such as TCP and UDP. As such, I personally find > >>>>> the term > >>>> "DetNet > >>>>> transport layer" misleading and confusing. The confusion is > >>>>> easy > >>>> to see e.g. > >>>>> in Figure 4, where UDP (which is a transport protocol as > >>>>> per RFC > >>>> 1122) sits > >>>>> on top of "transport". > >>>>> > >>>>> Based on the document it also may be > >>>>> solution/implementation > >>>> specific whether > >>>>> the "DetNet transport layer" is actually a separate > >>>>> protocol layer > >>>> compared > >>>>> to the "DetNet service layer". Thus it is not clear to me > >>>>> why the > >>>> word > >>>>> "layer" has to be used, specifically in combination > >>>>> "transport > >>>> layer". > >>>>> To me as, the word "transport layer" (and "transport > >>>>> protocol") > >>>> should be > >>>>> used for protocols defined in TSV area, consistent with RFC 1122. > >>>> But this is > >>>>> probably a question to be sorted out by the IESG. > >>>>> > >>>>> * Page 9 > >>>>> > >>>>> A DetNet node may have other resources requiring > >>>>> allocation > >>>> and/or > >>>>> scheduling, > >>>>> > >>>>> This is just one of several examples for inconsistent use > >>>>> of > >>>> terminology. > >>>>> What is a "DetNet node"? That term is not introduced in > >>>>> Section > >>>> 2.1 > >>>>> * Page 14 > >>>>> > >>>>> A DetNet network supports the dedication of a high > >>>>> proportion > >>>> (e.g. > >>>>> 75%) of the network bandwidth to DetNet flows. > >>>>> > >>>>> The 75% value is not reasoned. What prevents using 99% of > >>>>> the > >>>> bandwidth for > >>>>> DetNet traffic? > >>>>> > >>>>> * Page 15: Figure 2 > >>>>> > >>>>> If the term "transport layer" cannot be avoided, the labels > >>>>> in > >>>> this figure > >>>>> should at least be expanded to "DetNet transport layer". > >>>>> > >>>>> * Page 18: Figure 4 > >>>>> > >>>>> As already mentioned earlier, Figure 4 is confusing. UDP is > >>>>> a > >>>> transport > >>>>> protocol. If the term "transport" cannot be avoided, the > >>>>> labels in > >>>> this > >>>>> figure should at least be expanded to "DetNet transport". > >>>>> > >>>>> * Page 23 > >>>>> > >>>>> If the source transmits less data than this limit > >>>>> allows, the unused resource such as link bandwidth can be made > >>>>> available by the system to non-DetNet packets. > >>>>> > >>>>> Could there be additional requirements on the use of unused > >>>> resources by > >>>>> non-DetNet packets, e.g., regarding preemption? I am just > >>>> wondering... If > >>>>> that was possible, a statement like "... can be made > >>>>> available by > >>>> the system > >>>>> to non-DetNet packets as long as all guarantees are fulfilled" > >>>> would be on > >>>>> the safe side, no? > >>>>> > >>>>> * Page 27: > >>>>> > >>>>> DetNet achieves congestion protection and bounded delivery > >>>> latency by > >>>>> reserving bandwidth and buffer resources at every hop > >>>>> along the > >>>> path > >>>>> of the DetNet flow. > >>>>> > >>>>> Why does this sentence use the word "hop"? As far as I > >>>>> understand, > >>>> in DetNet > >>>>> bandwidth and buffer resources are reserved in each DetNet > >>>> intermediate node. > >>>>> If there were hops over IP routers not being DetNet > >>>>> intermediate > >>>> nodes, no > >>>>> resources would be reserved there. As per Section 4.3.3, it > >>>>> is > >>>> possible to > >>>>> deploy DetNet this way. And obviously there can be resource > >>>> bottlenecks below > >>>>> IP, on devices that are not routers... So does "hop" here > >>>>> refer to > >>>> IP router > >>>>> hops or also to devices not processing IP (or IP/MPLS)? > >>>>> > >>>>> * Page 27: > >>>>> > >>>>> Standard queuing and transmission selection algorithms allow a > >>>>> central controller to compute the latency contribution of each > >>>>> transit node to the end-to-end latency, ... > >>>>> > >>>>> The text does not explain why a _central_ controller is > >>>>> needed for > >>>> this > >>>>> computation. Why would a distributed control plane not be > >>>>> able to > >>>> realize > >>>>> this computation. Isn't this implementation-specific? > >>>>> > >>>>> * Page 32 > >>>>> > >>>>> To somebody who is not deeply familiar with DetNet, it is > >>>> impossible to parse > >>>>> the description of the examples in Section 4.7.3. For > >>>>> instance, > >>>> "VID + > >>>>> multicast MAC address" is not introduced. I think this > >>>>> example > >>>> must be > >>>>> expaned with additional context and explanation to be > >>>>> useful to > >>>> readers. > >>>>> * Page 34 > >>>>> > >>>>> There are three classes of information that a central > >>>>> controller > >>>> or > >>>>> distributed control plane needs to know that can only be obtained > >>>>> from the end systems and/or nodes in the network. > >>>>> > >>>>> Wouldn't it be sufficient to state "Provisioning of DetNet > >>>> requires knowledge > >>>>> about ...". Does it matter in this context whether the > >>>> provisioning is done > >>>>> by a central controller or a distributed control plane? For > >>>> instance, could > >>>>> the same paragraph also apply to a network that uses > >>>>> _multiple_ > >>>> central > >>>>> controllers, or hybrid combinations of central controllers > >>>>> and > >>>> distributed > >>>>> control planes? In general, an architecture document should > >>>>> be > >>>> agnostic to > >>>>> implementation aspects unless there is a specific need. In > >>>>> this > >>>> specific > >>>>> case, I fail to see a need to discuss the realization of > >>>>> the > >>>> control plane of > >>>>> a network. > >>>>> > >>>>> Editorial nits: > >>>>> > >>>>> * Page 9: > >>>>> > >>>>> The low-level mechanisms described in Section 4.5 provide the > >>>>> necessary regulation of transmissions by an end system or > >>>>> intermediate node to provide congestion protection. The > >>>> allocation > >>>>> of the bandwidth and buffers for a DetNet flow requires > >>>> provisioning > >>>>> A DetNet node may have other resources requiring > >>>>> allocation > >>>> and/or > >>>>> scheduling, that might otherwise be over-subscribed and > >>>>> trigger > >>>> the > >>>>> rejection of a reservation. > >>>>> > >>>>> Probably a full stop is missing after "provisioning". > >>>>> > >>>>> * Page 11: "... along separate (disjoint non-SRLG) paths ..." > >>>>> > >>>>> I find this confusing. I would understand e.g. "along separate > >>>>> (SRLG-disjoint) paths". > >>>>> > >>>>> * Page 34: > >>>>> > >>>>> When using a peer- > >>>>> to-peer control plane, some of this information may be > >>>>> required > >>>> by a > >>>>> system's neighbors in the network. > >>>>> > >>>>> Would "acquired" be a better term? > >>>>> > >>>>> * Page 34: > >>>>> > >>>>> o The identity of the system's neighbors, and the > >>>> characteristics of > >>>>> the link(s) between the systems, including the length (in > >>>>> nanoseconds) of the link(s). > >>>>> > >>>>> "Latency" or "delay" would probably be a better terms if > >>>>> the value > >>>> is > >>>>> measured in nanoseconds. > >>>>> > >>>>> * Page 35: > >>>>> > >>>>> DetNet is provides a Quality of Service (QoS), and as > >>>>> such, does > >>>> not > >>>>> directly raise any new privacy considerations. > >>>>> > >>>>> Broken sentence > >>>>> > >>>>> * Please expand acronyms on first use (e.g., OTN) > >>>>> > >>>>> > >>> _______________________________________________ > >>> detnet mailing list > >>> detnet@ietf.org > >>> https://www.ietf.org/mailman/listinfo/detnet
- [Detnet] Tsvart last call review of draft-ietf-de… Michael Scharf
- Re: [Detnet] Tsvart last call review of draft-iet… János Farkas
- Re: [Detnet] Tsvart last call review of draft-iet… Scharf, Michael
- Re: [Detnet] Tsvart last call review of draft-iet… Black, David
- Re: [Detnet] Tsvart last call review of draft-iet… Lou Berger
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Scharf, Michael
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Black, David
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Black, David
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Lou Berger
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Scharf, Michael
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Lou Berger
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Toerless Eckert
- [Detnet] Fwd: Re: Tsvart last call review of draf… János Farkas
- Re: [Detnet] Fwd: Re: Tsvart last call review of … Lou Berger
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Scharf, Michael
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Toerless Eckert
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Scharf, Michael
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Toerless Eckert
- Re: [Detnet] Tsvart last call review of draft-iet… Lou Berger
- Re: [Detnet] Tsvart last call review of draft-iet… Scharf, Michael
- Re: [Detnet] Tsvart last call review of draft-iet… Lou Berger
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Scharf, Michael
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Black, David
- Re: [Detnet] Tsvart last call review of draft-iet… Pascal Thubert (pthubert)
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Pascal Thubert (pthubert)
- Re: [Detnet] Transport sub-layer name change (Was… Grossman, Ethan A.
- [Detnet] Transport sub-layer name change (Was Re:… Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Andrew G. Malis
- Re: [Detnet] Transport sub-layer name change (Was… Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Toerless Eckert
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Scharf, Michael
- Re: [Detnet] Transport sub-layer name change (Was… Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Mach Chen
- Re: [Detnet] Transport sub-layer name change (Was… Pascal Thubert (pthubert)
- Re: [Detnet] Transport sub-layer name change (Was… Loa Andersson
- Re: [Detnet] Transport sub-layer name change (Was… Toerless Eckert
- Re: [Detnet] Transport sub-layer name change (Was… Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Toerless Eckert
- Re: [Detnet] Transport sub-layer name change (Was… Norman Finn
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Norman Finn
- Re: [Detnet] [Tsv-art] Tsvart last call review of… Scharf, Michael
- Re: [Detnet] Transport sub-layer name change (Was… János Farkas
- Re: [Detnet] Transport sub-layer name change (Was… Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Andrew G. Malis
- Re: [Detnet] Transport sub-layer name change (Was… Greg Mirsky
- Re: [Detnet] Transport sub-layer name change (Was… Loa Andersson
- Re: [Detnet] Transport sub-layer name change (Was… S.V.R.Anand
- Re: [Detnet] Transport sub-layer name change (Was… Pascal Thubert (pthubert)
- Re: [Detnet] Transport sub-layer name change (Was… qiangli (D)
- Re: [Detnet] Transport sub-layer name change (Was… Balázs Varga A
- Re: [Detnet] Transport sub-layer name change (Was… Loa Andersson
- Re: [Detnet] Transport sub-layer name change (Was… Andrew G. Malis
- [Detnet] Congestion Protection name change (was: … János Farkas
- Re: [Detnet] Congestion Protection name change (w… Pascal Thubert (pthubert)
- Re: [Detnet] Congestion Protection name change (w… Andrew G. Malis
- Re: [Detnet] Congestion Protection name change (w… Lou Berger
- Re: [Detnet] Congestion Protection name change János Farkas
- Re: [Detnet] Congestion Protection name change Black, David
- Re: [Detnet] Congestion Protection name change Pascal Thubert (pthubert)
- Re: [Detnet] Congestion Protection name change (w… qiangli (D)
- Re: [Detnet] Congestion Protection name change qiangli (D)
- Re: [Detnet] Congestion Protection name change Balázs Varga A
- Re: [Detnet] Congestion Protection name change Lou Berger
- Re: [Detnet] Congestion Protection name change Lou Berger
- Re: [Detnet] Congestion Protection name change Pascal Thubert (pthubert)
- Re: [Detnet] Congestion Protection name change Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Stewart Bryant
- Re: [Detnet] Transport sub-layer name change (Was… Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Andrew G. Malis
- Re: [Detnet] Congestion Protection name change Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Lou Berger
- Re: [Detnet] Transport sub-layer name change (Was… Andrew G. Malis
- Re: [Detnet] Congestion Protection name change Andrew G. Malis
- Re: [Detnet] Congestion Protection name change John E Drake
- Re: [Detnet] Transport sub-layer name change (Was… Andrew G. Malis
- Re: [Detnet] Transport sub-layer name change (Was… Lou Berger
- Re: [Detnet] [Tsv-art] Tsvart last call review of… János Farkas