Re: [Detnet] RtgDir Review: draft-ietf-detnet-yang-16
Don Fedyk <dfedyk@labn.net> Wed, 21 September 2022 18:42 UTC
Return-Path: <dfedyk@labn.net>
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 8895CC14CE28; Wed, 21 Sep 2022 11:42:43 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -6.91
X-Spam-Level:
X-Spam-Status: No, score=-6.91 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RCVD_IN_DNSWL_HI=-5, RCVD_IN_MSPIKE_H2=-0.001, SPF_PASS=-0.001, T_SCC_BODY_TEXT_LINE=-0.01, URIBL_DBL_BLOCKED_OPENDNS=0.001, URIBL_ZEN_BLOCKED_OPENDNS=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=labn.onmicrosoft.com
Received: from mail.ietf.org ([50.223.129.194]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id JD6YHCEzTxey; Wed, 21 Sep 2022 11:42:39 -0700 (PDT)
Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2124.outbound.protection.outlook.com [40.107.237.124]) (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 C9E6DC15256B; Wed, 21 Sep 2022 11:42:38 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hXKrjVkwPrWavc550PLkvc+SUlwTr1CwGq2DNsx+Su1MyEccawQ/SBbODdGQsTfwzTOF42XE8XI0tEn7vaMhMVX+dwg2BQcyU2GeUJIK1AriszDgNajcM0/SsAopNi/WLK9VxT7JzyJQGHViO2jFmsE7KOaL7Tj8otmZVAAsbeCWuFYRWqVDPEcHTBkc1di5AgxBBdazfzQwzxC9GLN3/gGY87q5LU7vSmiWW0rTxpJIWKEZB4+9WIc+H9kxTQHNOwHtx4Cb4IAA4oPxKbdxy3CA2JfZkF8WdoBzsDZVPAgOqrwsTQODawZv8ff+dkak1L2hDMQ9LLtB0PVrQJt/Kw==
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-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=xQ6aXiPWmReXgzROOL7c09nQkvD4qf/yfgx0BjYlPK4=; b=XCOWZulI06MYIikDVKjTfx4YE2fJLWK1GxRtOv15442tna+fsPrjYHs7ZihIgW47eUv6Pxc+0nIGPMythcQecxVHElGEcwEYkDXH6dVrK4cTeEDVZmB8lKSmOMqKItK6EHEHWVgUdQoVoCXTlkSyYmULOmyRFU0qQsyqpAIulu1/pI9HDac6YKu3Zfj7srS08WqL1xpPLEyK+jBS2kDe781PWFuHvKUQuDm6h5XhAFCxx5jTE1TPSCEzYXikE9GBIFpemVUm0pRVFc9HGrYeJom0lI69PQHZfaLUoHAg8RW11j46e3rj7V/DvVAwKPjBenSpCn1UdJT1+POpBhmB8w==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=labn.net; dmarc=pass action=none header.from=labn.net; dkim=pass header.d=labn.net; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=labn.onmicrosoft.com; s=selector2-labn-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=xQ6aXiPWmReXgzROOL7c09nQkvD4qf/yfgx0BjYlPK4=; b=PjfYFDV/zz8KRhMlpYh8SMh4wl5mAq2Sk3XrnZYuFXVGy/uoxERJ0KAUxAhqpbGyA3NiFaWaorZRcDRxgP+nUDv+6ZAaaKsBVfTWsRjFyfv1LnkbEX1zM/Y+BkVuQo+TsDHN2t68+IhMI7fT0ji9w5pV4t/eT00eMMcD5hTXCNY=
Received: from MN2PR14MB4030.namprd14.prod.outlook.com (2603:10b6:208:1dc::14) by SJ0PR14MB5372.namprd14.prod.outlook.com (2603:10b6:a03:42b::12) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.14; Wed, 21 Sep 2022 18:42:35 +0000
Received: from MN2PR14MB4030.namprd14.prod.outlook.com ([fe80::1518:9586:d5c6:81]) by MN2PR14MB4030.namprd14.prod.outlook.com ([fe80::1518:9586:d5c6:81%5]) with mapi id 15.20.5654.017; Wed, 21 Sep 2022 18:42:35 +0000
From: Don Fedyk <dfedyk@labn.net>
To: "julien.meuric@orange.com" <julien.meuric@orange.com>, "<rtg-ads@ietf.org>" <rtg-ads@ietf.org>, "draft-ietf-detnet-yang.all@ietf.org" <draft-ietf-detnet-yang.all@ietf.org>
CC: "rtg-dir@ietf.org" <rtg-dir@ietf.org>, "detnet@ietf.org" <detnet@ietf.org>
Thread-Topic: RtgDir Review: draft-ietf-detnet-yang-16
Thread-Index: AQHYx3mKklNZ7M4Cp0aO2Zfkbd3Rxq3p7kUg
Date: Wed, 21 Sep 2022 18:42:35 +0000
Message-ID: <MN2PR14MB4030869219F77B6E853E2973BB4F9@MN2PR14MB4030.namprd14.prod.outlook.com>
References: <f8842b8f-bbb5-cf31-90de-24fa061bc3f9@orange.com>
In-Reply-To: <f8842b8f-bbb5-cf31-90de-24fa061bc3f9@orange.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=labn.net;
x-ms-publictraffictype: Email
x-ms-traffictypediagnostic: MN2PR14MB4030:EE_|SJ0PR14MB5372:EE_
x-ms-office365-filtering-correlation-id: 8fc99a38-6ac8-4554-12b3-08da9c010d12
x-ms-exchange-senderadcheck: 1
x-ms-exchange-antispam-relay: 0
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: x2MyBZYZ4001ljtwUL9g2ZBj7vxGdVTWG303AJzH2KMnTYSGk2ztsA6NvH76/IigoP6+0nS6KoJ3VEUAdotZcI0QyLKZnpPQjr9cD/UJmA1vQgOY/8Xs+sm8zbxxTw+3KlicCYmJl5KV+cPzkbWEFwqRucifosvnuoxpjHaBa/x0q60qPcjGLXTjKulpg+d1sUoWwUHbyHz/yBuOPa7/Gcx72UoxlCA1XmXsrupxqg91sP/30FBBOxlzUs30VKl97JxQoYw2A81MQWTZyKM2m5UL4dQ44HmPyCPIUzfuLkzAinOZlncPAYH8Z93VJ/VtnF47AwMdrKKAiT9KAGdfzOBipiGBjrj2+ATJIK4fTTGgjrwz+OH6sbmPW0YMtBtleaZGERyuv9MBFW/yPMJsuYe1Ecw8j5/YXd3jXb69EjAVSOvoGKU8yvvqTHFnJiGWFgc5kV3kJOAUEL5i9CIwB1asiIJcLBPd1hXeh/Ti4/ScroN15GnIWb9tiMRHFJRH9SgxXqYt3/SjxKJ4DfB5BwJiISn3adYxpWv5uNXHbGiBbxbfTrL2jWQaZf0oh3AQAC1s5dmHvqL1wc7tiowz5A3AEcqg5/HXCKzz2wgUV+IXdpuASE0yn6i6WoXVxi2CJMWhOLJQUf12UH5g1CT/jiIAtHvht8pegxJI/TEOU8qH2DAlvsRm9d9fMru2JSylz4eLlPUcFj8QpxY0m28agD7a6dNPf5+LIX0yx3k2k8kX1G+j2OhnYLkoflRH22TZDLmpTKllLB0LPGHczO5OzqSsRHwNNFSxY9I4wyroNHx2D3X5Fv7dRlSpB07cUnBs
x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MN2PR14MB4030.namprd14.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(136003)(366004)(376002)(396003)(39830400003)(346002)(451199015)(478600001)(33656002)(66556008)(54906003)(76116006)(66946007)(110136005)(66446008)(71200400001)(8676002)(41300700001)(450100002)(6506007)(66476007)(86362001)(64756008)(4326008)(966005)(7696005)(186003)(38070700005)(8936002)(52536014)(55016003)(122000001)(316002)(9686003)(38100700002)(5660300002)(83380400001)(2906002)(491001); DIR:OUT; SFP:1102;
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 05S1VUYIEhsGGvUNF59QxXZ5U3sKEgGcP6BqY/9AKqjHZr6ggq6fzjUnIhqnQIt/yxtKOm+C8fREKsBj59yJBvXmHv5SXB+xXkU6LbSE2YLDn2MvNv0+hXMkYHHozFYucyfJ5Hw+x1CMpxmwtOrCc4bx14lLdcoVMOmK+1wwnzjxgjZidcQSM9qGqVCYoS00a6oeywytCzwR6+/FTDcpb84aMn1YAJ50bIxiS//1WN83uZussSys5MixBc1OflBpjYql4f/7LxhuXKDPeb8oYeG0AZ72ant3eA9M3m5IrIbModt0EYG1JPggclXd+qJB11sfCJIXHMIHfLDbksVSAeF77N6J191t/EVGs225CNrHgXHKAsjHNa7hp8O3xKBNShj9XromarzVuVdObN7M2QlX/AZvZo5hRaeyBEm418M3ie/NwD6O20Wrfwy9Bw+8KYreNjYk2oiuD8ZHnwwDvDdInG/oNAVIY2MBxX0JTVPmSNpvGdUCmdySMjJk6k5ZbxaaWPtwGNFpziPSYo+XNx/ACBXLMKNPyMw6ukknzPnMuWq6eeBYFtrNzrL601+P6pBeBmIxgMI95bh7arbsAASUzD7tCSN0bZ5eGryJMlOTO4GyK4Axnn+9haZBfweLchjd6+GlU6rC+XHihloxAqo2fHBsMUW+22j5/sSjI9HQEH0VTqBvM2jazJIb3XXQ/8MNOUIT5ulboOQC00nX6MR0Hi6bM4+WdwsiqGUK39qrDeZBGUt0ROXlZFOA1zrzgd3+xC6l9EZZuzDegwPkihDYaHDgm+JQv7vQ41ECVEQpRYc6K0jiM/CpBWqNOix2NaVsH6M8RyySEBBwfVdfOPwuIACyUbOrhDHCWzN2+uju13wVK0DfBIbIPp9Voxdt/ZvegIiePyp6mfwZG2lUSjmiCxk4JsE1g7GGpxlP5dl++UWXFDKF5fc0WUaYLbsccJGTCqTQ7NfHLOa3/xTJe9GUlTlZa3soJw8Anfhjy7ZvrsgKhZPjYi8BYSqev84NeQTz7m1Gn2mRbvL7+3mPiY1uV311eoT4z22IMDp3rwEPu250exbapv/KuyS+lgh9CIwVU88yHNU/nea7Q+jgrBcX6uQ4xfDyH+RsYzPpTOLX6EQBDw5+w8j9KiuJIr/bsW3MtbatD7DN31gVNIP50EN4kjT131JHJ4DwsP8BqiiZRICsJ+yJNb1LZIaqRw2Y4b2U1kEtoWMYH6YhpDU18O4sPDn0y7VzCxh7/5LXFZTjoc1F20TYGhBgwRFSooq+nk3P+S7CzsrrmwLj1jVaivK9ao1BbAaPnlneUzQ494Ca6ABJC/gupCcuTwAdXA+xMPnugLcg4aprdGFudlz5tq1fg038miR7j5B/IaonuPM+AJlEz0PxTssRJHTRc4mtH76uhuBeVn7g+6K5B/2QSEztYvk7D7M8SKTUZ7IsBCFNTmVOMG3EWN6Pu7Iv0XkPdiRQkNFWUjfzfAa/7cRduNeSJdkmvbYYJhHXxV5ObVv6TLc0NzDa9JPEsoR+0erirdd9gIT5C8SMCvH9HoCWO581TisrgFyo7WomkaF2ovrBy7m/AE1Luvs202rBiskcW620O4f3VWgV2zhW/kS7Unu/cbZxNW2UKdN1YjQd5mOE/rOp382pm4aqPA0IzSTz
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: labn.net
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MN2PR14MB4030.namprd14.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8fc99a38-6ac8-4554-12b3-08da9c010d12
X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Sep 2022 18:42:35.4378 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eb60ac54-2184-4344-9b60-40c8b2b72561
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: PWKSC9NCDj+xmyyKduwOWerqMgkbWOjtCyfYR0KgE7Zb0sm1brYHrNmYUsFGPIfDbbgT2U/ysmvFM0XNIAzshA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SJ0PR14MB5372
Archived-At: <https://mailarchive.ietf.org/arch/msg/detnet/GCM6XQzSkU4DG13B4BN2LhD1m8E>
Subject: Re: [Detnet] RtgDir Review: draft-ietf-detnet-yang-16
X-BeenThere: detnet@ietf.org
X-Mailman-Version: 2.1.39
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: Wed, 21 Sep 2022 18:42:43 -0000
Hi Julien Thanks for the review. I have addressed the issues below where noted: Inline [don] I have placed the pending changes in the public https://github.com/detnet-wg/draft-ietf-detnet-yang for my co authors to look at and comment. The easy way to see the changes is to copy the xml file form git hub and go to the IETF author tools an see the diff. Coauthors, please have a look or say that I can push the changes as version 17. I will push if my coauthors agree or next week if there is no feedback. Thanks Don Subject: RtgDir Review: draft-ietf-detnet-yang-16 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-yang Reviewer: Julien Meuric Review Date: 2022-09-13 Intended Status: Standards Track _Summary_ I have some minor concerns about this document that I think should be resolved before publication. _Comments_ The YANG module itself seems almost ready but the text that introduces it needs a few clarification or rewording. _Minor Issues_ - First of all, IdNits points out 3 normative references on informational RFCs 8938, 9016 and 9055. Are you sure the 3 of them are mandatory to implement the YANG module? [Don] I think that 8938, 9016 are normative. I have moved 9055. - In the abstract, I don't really understand the point of the sentence "An operator or network controller programs the configuration of the devices" since service provisioning on devices along the path is previously mentioned. If the intent was to say: " The configuration of the devices can be programmed by an operator or a network controller.", that feels like stating the obvious for a device-embeded YANG module. [don] the point was this is a Configured Datapath model not a singling path model. I've removed the operator /network controller part. - In section 4, the wording of "ingress" and "egress" definitions feel odd. Is it meant to say "Ingress refers to entering the DetNet application layer and egress to leaving the application layer."? [don] Old: All of the layers have ingress/incoming and egress/outgoing operations but any instance may be configured as only unidirectional. This means that each unidirectional flow identifier configuration is programmed starting at the ingress and flow status is reported at ingress on each end. In the MPLS cases once encapsulated, the IP 6-tuple parameters may not be required to be programmed again. In the IP case, without encapsulation, various IP flow id parameters must be configured along the flow path. New: All of the layers have ingress/incoming and egress/outgoing operations, but any instance may be configured as only unidirectional. Ingress refers to any DetNet layer where a DetNet context is applied. Ingress allows functions such as switching, aggregation and encapsulation. Likewise, egress refers to any DetNet layer where a Detnet context is removed. Egress allows functions such as switching, disaggregation and decapsulation. This means that each unidirectional flow identifier configuration is programmed starting at the ingress and flow status is reported at ingress on each end. In the MPLS cases once encapsulated, the IP 6-tuple parameters may not be required to be programmed again. In the IP case, without encapsulation, various IP flow id parameters must be configured along the flow path. - The described aggregation cases are scoped either as layer N to layer N or as layer N to layer N-1. However, there's a relay node case where aggregation is described as layer N (forwarding) to layer N+1 (service). Since there's no forwarding to forwarding relay case, I suspect a mismatch... [Later note: in the model itself, one can find "forwarding-to-forwarding aggregation at the ingress node or relay node or transit node", so it looks like an issue in the text part.] [don] Not sure there is a transit node which is forwarding to forwarding and there is a relay node that is forwarding to forwarding. I struggle to word the missing case? - In section 8, the max-loss leaf is an uint32 but is defined as a "ratio". Considering the value in the examples (2), it seems that the description text (and units?) should be adjusted. [Don] I think this should be packet loss rate from RFC 9016 and I have adjusted it. Packet loss rate is a percentage. I have allowed from 0 to 100. With 0.0000000001 being the smallest non zero value. _Nits_ ------ Abstract --- s/operational data for DetNet Flows/operational data of DetNet Flows/ [already 2 "for"s in the phrase] [Don] Done ------ Section 4. --- OLD Node types typically are logical per DetNet service and one DetNet service can be one node type while another is another node type on same device. NEW Node types are logical roles per DetNet service: a device along one DetNet service can be of one node type, while another service may use the same device with a different node type. [don] Done. s/edge node(egress/edge node (egress/ s/These may used/These may be used/ s/the configuration need to/the configuration needs to/ s/IP based path/IP-based path/ s/parameters for aggregated flow/parameters for aggregated flow/ [Don] Done ------ Section 10. --- OLD o this also coudl be considered moer sensitive. The trafic profiles liked to NEW so this also could be considered more sensitive. The traffic profiles linked to ------ [Don] Done Regards, Julien
- [Detnet] RtgDir Review: draft-ietf-detnet-yang-16 julien.meuric
- Re: [Detnet] RtgDir Review: draft-ietf-detnet-yan… Don Fedyk
- Re: [Detnet] RtgDir Review: draft-ietf-detnet-yan… John Scudder
- Re: [Detnet] RtgDir Review: draft-ietf-detnet-yan… Don Fedyk
- Re: [Detnet] RtgDir Review: draft-ietf-detnet-yan… Don Fedyk