Re: FW: New Version Notification for draft-rvelucha-bfd-offload-yang-05.txt

t petch <ietfa@btconnect.com> Wed, 05 April 2023 09:22 UTC

Return-Path: <ietfa@btconnect.com>
X-Original-To: rtg-bfd@ietfa.amsl.com
Delivered-To: rtg-bfd@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id CE463C151548 for <rtg-bfd@ietfa.amsl.com>; Wed, 5 Apr 2023 02:22:51 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.899
X-Spam-Level:
X-Spam-Status: No, score=-1.899 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, NICE_REPLY_A=-0.001, RCVD_IN_MSPIKE_H2=-0.001, RCVD_IN_ZEN_BLOCKED_OPENDNS=0.001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, 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=btconnect.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 345Ha763ffhC for <rtg-bfd@ietfa.amsl.com>; Wed, 5 Apr 2023 02:22:48 -0700 (PDT)
Received: from EUR04-VI1-obe.outbound.protection.outlook.com (mail-vi1eur04on2123.outbound.protection.outlook.com [40.107.8.123]) (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 B3F47C151535 for <rtg-bfd@ietf.org>; Wed, 5 Apr 2023 02:22:47 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=hjGGmANwUHhWL3M7FZKJBtJPwLbh/mA3vSiSg8XO+8EDbivK0gIOUNUktIbRfCR2vXI6PW11LHMv0b54dUvap/iBDuBgpq1l38jiMc40NFFOZduH+Md/cSQfVJsG0qvqxU5h1VYWLl5dGUQDlkvKwRo42dRhX3+iHaQd+meG4v9G9/5pd+DXdymwdJ6fS2eD0RYZ1K5qgDfCN8mgvyPMNVxQ282ljVm5zxgMN42fQkIPdCTNHa6XRm4AxDXGTIR9DlEax968pPuN61Ta3Chkq2zSYbYY3UW8BlYYMhObWQ40TZcQ3K69/bi6P+Ho2eWLG/4m3ouIQOz0F5au49+pvg==
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=ukqhMd9aBppsdSbqjC6WcCzpk9uOkqOFip4zr7byDmw=; b=Hi+YQogNbnsSbX/Ds08NRi9AusJpFyEL7qMjjHFHl/MMKbQJGAfTSHfKbMRqUhRO672GuLLFSyrvd+iKu4f/VaE/FSEN8+A7lUl+UFpXSIcjr7FAlI7n9Ypvg13pjxEjtJw6FpacRBlUYi7jJYM4FJGp8cOHoRzlnwqyLBT35JHfy+5keeShDKbyA2i+G1JE9uN+pFPNB2CtYV0766o45tq4IZG9XISlICaZg4M1m+zFMaKbdRuCkrO3Iu0si1eJrvp5p68DW8v2WzwbMAiyo7ZrG7UPji6CBpoPL6ZFWJfzNSAu44A5Sp0jHxoR+lCv0eq2cJmlb7s/4IAK8njzsQ==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=btconnect.com; dmarc=pass action=none header.from=btconnect.com; dkim=pass header.d=btconnect.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btconnect.onmicrosoft.com; s=selector2-btconnect-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=ukqhMd9aBppsdSbqjC6WcCzpk9uOkqOFip4zr7byDmw=; b=Qgcy33TJL5sjBL87FtCeueSmdiv/kvyw7pKiIe4/uUDKngPi2ZJueiE6y89mURRLQwBLcBKZPWl3DROx7qCDONBhqZ1XRjO4py4bHyXjsTtiNHY8xOfOJ90GO83GWUAUdG42I9xk78CLc120UlfMjn8H8w8BAVvBBm+DYGVo5ik=
Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=btconnect.com;
Received: from DB7PR07MB5546.eurprd07.prod.outlook.com (2603:10a6:10:73::23) by DB9PR07MB7257.eurprd07.prod.outlook.com (2603:10a6:10:217::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6254.35; Wed, 5 Apr 2023 09:22:44 +0000
Received: from DB7PR07MB5546.eurprd07.prod.outlook.com ([fe80::4265:3d58:4dba:2db4]) by DB7PR07MB5546.eurprd07.prod.outlook.com ([fe80::4265:3d58:4dba:2db4%3]) with mapi id 15.20.6254.028; Wed, 5 Apr 2023 09:22:44 +0000
Subject: Re: FW: New Version Notification for draft-rvelucha-bfd-offload-yang-05.txt
To: "Rajaguru Veluchamy (rvelucha)" <rvelucha@cisco.com>, "Rajaguru Veluchamy (rvelucha)" <rvelucha=40cisco.com@dmarc.ietf.org>, "rtg-bfd@ietf.org" <rtg-bfd@ietf.org>
References: <168018517638.26385.151594919232783582@ietfa.amsl.com> <DM4PR11MB5294B5CEBD7D55AF6091CE87D28E9@DM4PR11MB5294.namprd11.prod.outlook.com> <6425B988.3040206@btconnect.com> <DM4PR11MB52940BD4B8305ABC32A85774D2939@DM4PR11MB5294.namprd11.prod.outlook.com>
Cc: "reshad@yahoo.com" <reshad@yahoo.com>
From: t petch <ietfa@btconnect.com>
Message-ID: <642D3D71.4060708@btconnect.com>
Date: Wed, 05 Apr 2023 10:20:49 +0100
User-Agent: Mozilla/5.0 (Windows NT 5.1; rv:38.0) Gecko/20100101 Thunderbird/38.5.0
In-Reply-To: <DM4PR11MB52940BD4B8305ABC32A85774D2939@DM4PR11MB5294.namprd11.prod.outlook.com>
Content-Type: text/plain; charset="utf-8"; format="flowed"
Content-Transfer-Encoding: 7bit
X-ClientProxiedBy: LO4P302CA0035.GBRP302.PROD.OUTLOOK.COM (2603:10a6:600:317::11) To DB7PR07MB5546.eurprd07.prod.outlook.com (2603:10a6:10:73::23)
MIME-Version: 1.0
X-MS-PublicTrafficType: Email
X-MS-TrafficTypeDiagnostic: DB7PR07MB5546:EE_|DB9PR07MB7257:EE_
X-MS-Office365-Filtering-Correlation-Id: 9522c1f8-fa53-4b5a-8bc2-08db35b74fe1
X-MS-Exchange-SenderADCheck: 1
X-MS-Exchange-AntiSpam-Relay: 0
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: sK2vxCwdoZ7kZ4SOSbHUnJGip+ZftyPyn/NIu0ZkPxfGS8by5xyz1uBJNlzndUwJ+I5nD6kocF4g5Mzj1CaVFG/38eW/WaA8dw20KUnOZo1e1EwbRG6CU+4xHeitsg6J0UjXP6K2YBaSIjEfsjSlEcBNEonSSKcETHlRQW7Q9ec3Rajq0JCQrxqb1nXLZuuWZlfHRDSLXdnlD3tRqQl7O8w5REvUIeyWfnZKeLUvfWgrYoOPOSHGX+98CDbBFDPnzrK98dDj3vZMRvMUJdp9+MUM0F6xwXXfGC7LhZJexet+HJFURpXgCf15VNJhZaz2l1SZmdetnByUjwV0KJsS5shj8FPckXttKRreVIwS7McpvFqskDdXFWrHDBO75/iGQ+pSGMNcnCtxnPQPf/LiFi6zF5BaHhutIikhGTsRN7oj7EAqc0v0J5elSkkACin15nFCu+gZuz0QwVHS/6N16uMK2b685FFbRhDKai5jZqBtbdHEMVKgXGPoTCJ0jpzoYzftdFw01/B7rWgIqkXxZizM4g7iztQD65UyvpKclxzr1N7+jGWy0weuG97iU12D4sgkIdFepKwmmk5I1neD0dm04WZytFoU2olWZU2pG6z/9gAWIpvY31Nna8k0Dle5Ho9j2G4b8nBxJwHE6bKPAw==
X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DB7PR07MB5546.eurprd07.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(376002)(346002)(366004)(396003)(39860400002)(136003)(451199021)(15650500001)(6486002)(38350700002)(966005)(82960400001)(2616005)(6512007)(53546011)(6506007)(186003)(38100700002)(8936002)(26005)(66574015)(2906002)(33656002)(5660300002)(83380400001)(478600001)(36756003)(66476007)(66946007)(66556008)(87266011)(41300700001)(8676002)(110136005)(86362001)(4326008)(316002)(52116002)(66899021); DIR:OUT; SFP:1102;
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: uybW5m/+UwWTjsBNZnRFGMaXWNolsZrE49Zitp6e1vmYGqQiY1N8siBqoMkK6AtWfoKUUAaXqJDsE0rNq7aSlXMtf0f0L8DlmnI2tFROisjuNb8ZAVP3sLv6dtF73pYtw712D9ENPXz8uspsoJksL99Qs2LwZNFr0cY2k77O0817OftFTQB8/mwEPfMgeIqVwLiNGIzaPx6qoweHcNuJcYtjbq0cTSUt5Z2alrE5/mknrDtG94oOLEv+AzlY9eyUxkQXJGE0WsAohrM4AfIwKu0BrlfbwB6Og88rIlE480eBZfuHynaVpvYOvjdy4iQsIynxFoeFWMgTSqFhIVDnRVLXHbZsZEgqg4kz9g7M4bevBrnJgfFuMEp3OnAtLfLzjyLDlftK+ktGde1dpNiVKruBr6xH70NFuK+Y7aIbbJSg3m9nmsXg2Fg/LwFWlzmvKySi9iOqECBWmSX6yBZGknkqLxkFnb151cN+MiG54rB1zmj+AyDBOOuvJBwfrGDV1m3gk7H3khJvdMcsDd2dGSw+o4DMtMyD2iMH6uLvRkJxhuNfy5uuoJMF96B9AkvsUj9FBOydz7XAjdHBMJa4x6bt5SXAmpw+MHcRFuFMurH1isoSkIFdG4bmshXddXs8l66ueHoSwsYuTxNwTDEmQ+yVXfBd/9aM4d9qkLV+usb+9h9BpUvIAw4HhtMJ8OMn/0eVw3xLC3YuMbxnTbWrZefn1BB6EPVqxAmrImTLQQKtHMoP4nWgKDOWUPBwzTb51oGUXPmqzq/ClUGna8YGGxvN8ZzIvxPu5HQE/+yVSI2Z1SreQIeh8dkERG+O3QLB1OjkX5CB1/XX6VEIxllsRMMRaD54ZzjEOuLgn0lB4nJkwDNw8mn2/0UOJk1pm1JUkmh1LywitXNpj5KOnOSnRmECQWBrI27E/cCVNQMm9PH87hwYtIecOwlZQInApQtBKUfHeA/dbBpst2FychYTBWi1EeQy4RlYz9V9JspCfAS0XCrXFlHY7wHmnZxL+KzAXTTS3kifyXOZCHl9aTMx/CP6+iHdVVDe9cdYgr2bgnqpt/NmXaACriNYfnK9zN0Y3h8lpGV7mG5pYbyCUt0sfBmjLFXg5xcqIu/SLuzkxcT3778ENKQU6wIaXdQmJRHJkhOPxJ+VWvkKv5cimLfDo+5qQsgLwNI5DZMSYgwiiiEjn7DYFTkaR4rE3ssCJf+IjNdzpIhrmJLG3gk/zmeHwZ9EWU/dfAvHB6QKNeR+Rhn/oD6965C5sQJXJb7r1JrbDZi4KN52Lnjo9JczNSkR7C5ryaLdkxGoOUkU1Kpf+sw8Ru8USj/535z1dD4GmslfmYAZeR7x7l5dViMlJafzTBhJHgsDPvIHzagIgG73/Xs+4UMVX+CQXcmwRoKRZ2QSU/eZeFxCuVNahvA9zNU9L8BOBe2SOOFOa7tOIeieV9HmEyjqdaxjkKOHDzoFrBScSGKyPMNGb8x55RIcG49HhfNMesDo652snxLPadnOWhWRASqcdHZdYw8CRhp4Ti77ySrI0vDopiz3Jmm+OkOOYIdXHZwebeapBX7HomNDQYovcwQdNDGRBIHRCxHkskfP
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 9522c1f8-fa53-4b5a-8bc2-08db35b74fe1
X-MS-Exchange-CrossTenant-AuthSource: DB7PR07MB5546.eurprd07.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Apr 2023 09:22:43.9524 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: cf8853ed-96e5-465b-9185-806bfe185e30
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 9ueq/CFRxPZv2W8PD+Nf6/C1PGU4B0klheWzi7Eimml4wAjSo+1yxWFYSDdow76k0dWnm4s0LM8S5qjK9aBIqQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR07MB7257
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-bfd/XmTimzDx8hfIj6snrsAEO22hT6w>
X-BeenThere: rtg-bfd@ietf.org
X-Mailman-Version: 2.1.39
Precedence: list
List-Id: "RTG Area: Bidirectional Forwarding Detection DT" <rtg-bfd.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/rtg-bfd/>
List-Post: <mailto:rtg-bfd@ietf.org>
List-Help: <mailto:rtg-bfd-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/rtg-bfd>, <mailto:rtg-bfd-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 05 Apr 2023 09:22:51 -0000

On 04/04/2023 06:20, Rajaguru Veluchamy (rvelucha) wrote:
> Thanks, Tom for review/comments.
>
> Version been updated for comments. Can u please check.
>
> https://www.ietf.org/archive/id/draft-rvelucha-bfd-offload-yang-06.txt

Mmmm, no not really.

For a YANG module to exist it must be registered.  You register the name 
spaces, which looks ok, except that I think that the choice of names 
needs thinking about.

You do not register the YANG modules.  This requires a reference to the 
document in which the YANG module appears with name and prefix.  Where 
this is yet to be published, the usual convention is to reference
RFC XXXX <title of I-D>

The prefix in IANA Considerations you specify do not match those in the 
YANG modules.

I think the choice of names unfortunate.  Yes, these are extensions to 
the existing modules but what happens when the next extensions to these 
modules comes along?  .extext, .ext2, .next?  Better to use some 
abbreviation of what these modules are about IMHO as a suffix in the 
name and in the prefix; hardware offload, reduced to three characters 
ideally but 'hw' seems ok - I avoid the letter 'o' because of the 
numerous times it gets confused with zero but you might have a better idea.

Security considerations must use the template from YANG Guidelines even 
it says there appear to be no exposures.

'contact; ' must be HTTPS: and to the datatracker not tools

BSD licence is 'Revised' not 'Simplified;

Fundamentally, while these are largely editorial I would oppose adoption 
until there is some reason why this should exist.  Who cares if there is 
hardware offload?  And what is the boolean for?  Can it be set and reset 
to turn hardware offload on and off or is it read only to reflect what 
the hardware is currenly up to?  I expect to learn this from the 
Introduction and perhaps from the Abstract.

These thoughts were triggered by my recent experience with an up-market 
PC which refused to shutdown when I told it to and led to me pulling the 
plug from the wall.  I eventually found a customisation option which 
specified what should happen when I pressed the off button.  To me it is 
obvious what should happen when I press the off button, but seemingly 
not to leading PC manufacturers and suppliers of leading operating 
systems; they need to be told that pressing the off button is an 
instruction to power off the PC.

Thus you know exactly what is meant here.  I do not:-(

Tom Petch


>
> Regards
> V.Rajaguru
> XR/BFD/PI
>
>
> -----Original Message-----
> From: t petch <ietfa@btconnect.com>
> Sent: Thursday, March 30, 2023 10:02 PM
> To: Rajaguru Veluchamy (rvelucha) <rvelucha=40cisco.com@dmarc.ietf.org>; rtg-bfd@ietf.org
> Cc: reshad@yahoo.com; Rajaguru Veluchamy (rvelucha) <rvelucha@cisco.com>
> Subject: Re: FW: New Version Notification for draft-rvelucha-bfd-offload-yang-05.txt
>
> On 30/03/2023 15:09, Rajaguru Veluchamy (rvelucha) wrote:
>> HI Team,
>>
>> Can we have WG adoption for this draft please.
>>
>> https://datatracker.ietf.org/doc/draft-rvelucha-bfd-offload-yang/
>
> Since it imports RFC8349, that must be a Normative Reference in the7950 I-D.
>
> There is a template for Security Considerations in the YANG Guidelines RFC which there needs to be a good reason not to follow. Those guidelines drag in some more references.
>
> A YANG module does not exist until it is registered via IANA so the IANA Considerations need supplying, registering the three modules, registered the three name spaces.
>
> YANG would benefit from a reference, RFC7950, in the Introduction.
>
> The WG Web URL are out of date as are the trust URL.
>
> The BSD License reference is out of date.  (In fact, that reference was never valid in the first place:-(
>
> RFC9127 gives some ideas of what is wanted.
>
> Tom Petch
>
>
>
>>
>> Regards
>> V.Rajaguru
>> XR/BFD/PI
>>
>> -----Original Message-----
>> From: internet-drafts@ietf.org<mailto:internet-drafts@ietf.org> <internet-drafts@ietf.org<mailto:internet-drafts@ietf.org>>
>> Sent: Thursday, March 30, 2023 7:36 PM
>> To: Rajaguru Veluchamy (rvelucha) <rvelucha@cisco.com<mailto:rvelucha@cisco.com>>; Rajaguru
>> Veluchamy (rvelucha) <rvelucha@cisco.com<mailto:rvelucha@cisco.com>>
>> Subject: New Version Notification for
>> draft-rvelucha-bfd-offload-yang-05.txt
>>
>>
>> A new version of I-D, draft-rvelucha-bfd-offload-yang-05.txt
>> has been successfully submitted by RAJAGURU VELUCHAMY and posted to the IETF repository.
>>
>> Name:           draft-rvelucha-bfd-offload-yang
>> Revision:       05
>> Title:          YANG Data Model for Bidirectional Forwarding Detection (BFD) Hardware Offloaded Session
>> Document date:  2023-03-30
>> Group:          Individual Submission
>> Pages:          10
>> URL:            https://www.ietf.org/archive/id/draft-rvelucha-bfd-offload-yang-05.txt
>> Status:         https://datatracker.ietf.org/doc/draft-rvelucha-bfd-offload-yang/
>> Htmlized:       https://datatracker.ietf.org/doc/html/draft-rvelucha-bfd-offload-yang
>> Diff:           https://author-tools.ietf.org/iddiff?url2=draft-rvelucha-bfd-offload-yang-05
>>
>> Abstract:
>>      This document defines a extension YANG data model that can be used to
>>      manage Hardware Offloaded Bidirectional Forwarding Detection (BFD).
>>
>>      This document specially talks about BFD sessions that are offloaded
>>      to hardware.
>>
>>      The YANG modules in this document conform to the Network Management
>>      Datastore Architecture (NMDA).
>>
>>
>>
>>
>> The IETF Secretariat
>>
>>
>>
>