Re: Yangdoctors last call review of draft-ietf-bfd-stability-08

Ebben Aries <ebben.aries@nokia.com> Mon, 29 March 2021 23:44 UTC

Return-Path: <ebben.aries@nokia.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 C6C443A25BF; Mon, 29 Mar 2021 16:44:15 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.001, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, MSGID_FROM_MTA_HEADER=0.001, 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=nokia.onmicrosoft.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 p-oN5yxWUwzz; Mon, 29 Mar 2021 16:44:11 -0700 (PDT)
Received: from NAM12-BN8-obe.outbound.protection.outlook.com (mail-bn8nam12on2114.outbound.protection.outlook.com [40.107.237.114]) (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 9B68D3A25A1; Mon, 29 Mar 2021 16:43:36 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=LBtZUjFoCP86arxkN36Ge0fcN3IT3hcc9ekrgXK4VXV+2xxdN15R4iZHloAlM7PRq+OySmClMRJUtIxs+hpYz9dJTnTMRJstKqB0rhGyUVdsQsA5pCvyNe9CeBpXJNM1/rywfYr21f/SyXLbo+ch3kPcu9bTFVu0xU5AqJZHymkeevRrlf2i1zbzxds77zf6Fs0MsCq7DBVFYvmOCf6sVi3h3FE4QG7jBqIgPiwxghTKu6gfaoyI0cJBom65qVAVxV1u51YNlDuWWF9V860j4S5UwlT55azOJxthDEegam53XNnVqeASmUTXlUFhjKMFpNRcT1WxFtM1YtQyDujMLw==
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=4Lvl53/LoHqs9JXHt7OesCIHavlx/kLZx2A7Hk3+fAs=; b=BFOGyejzHW2/SyqIrwTG4AKSciaEhZx4/AA2eQKn+MQiZY7ilyLIVqGiB2p5CTUCHv3kLWGK/1X/m9ZIc7SUYjIjkGox9dkDiuiSblsqF/5WzU/aA8CiXDL8dyd9zdvOggvVo5fqY4ITLoUicaskChwo2XA3BInNlnk0hPJw8Mkps80e0jiUk0X4E62M+FSM/+1jiexYOJZphlNtQiZlK2ztGuGqU46r+1q0dDftWyTQ10EHX3B65sxH2hModzoyAN636HHk61wu5vqOAdVYLrTUuAMjqVGz2Q1YVwiKPAYF1uStNQvvVgHzeXa4rK4PMGQqojpnKcXNe8Vl2LGwCw==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nokia.com; dmarc=pass action=none header.from=nokia.com; dkim=pass header.d=nokia.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=nokia.onmicrosoft.com; s=selector1-nokia-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=4Lvl53/LoHqs9JXHt7OesCIHavlx/kLZx2A7Hk3+fAs=; b=lWxS6LT1UIbLjeni2r7qrMglKatUZnunJhMkbSrAe5gUNzDsSxTGatTGrOsjjNHi1tdM/k5xMQhkOwstWFp5AOHz94Da1hStYHnN1TulB25QItfrqyXUAKewB9hkKD+DOSIOa8ponTEGUWXNaUqBCa7cF5i1mO1Pk2bXy+75Lc8=
Authentication-Results: gmail.com; dkim=none (message not signed) header.d=none;gmail.com; dmarc=none action=none header.from=nokia.com;
Received: from CY4PR08MB3400.namprd08.prod.outlook.com (2603:10b6:910:78::12) by CY4PR0801MB3795.namprd08.prod.outlook.com (2603:10b6:910:95::24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3955.18; Mon, 29 Mar 2021 23:43:34 +0000
Received: from CY4PR08MB3400.namprd08.prod.outlook.com ([fe80::14ef:d84b:636e:7153]) by CY4PR08MB3400.namprd08.prod.outlook.com ([fe80::14ef:d84b:636e:7153%5]) with mapi id 15.20.3977.033; Mon, 29 Mar 2021 23:43:33 +0000
Date: Mon, 29 Mar 2021 17:43:31 -0600
From: Ebben Aries <ebben.aries@nokia.com>
To: Mahesh Jethanandani <mjethanandani@gmail.com>
Cc: YANG Doctors <yang-doctors@ietf.org>, draft-ietf-bfd-stability.all@ietf.org, "rtg-bfd@ietf. org" <rtg-bfd@ietf.org>
Subject: Re: Yangdoctors last call review of draft-ietf-bfd-stability-08
Message-ID: <YGJmI3OJMM1vdG4L@localhost>
References: <161637181289.17673.15670328442520728650@ietfa.amsl.com> <D021E185-3703-450A-B70D-1C679252F600@gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <D021E185-3703-450A-B70D-1C679252F600@gmail.com>
X-Originating-IP: [2601:283:4602:43e0:1cad:e995:8263:2173]
X-ClientProxiedBy: SA0PR11CA0198.namprd11.prod.outlook.com (2603:10b6:806:1bc::23) To CY4PR08MB3400.namprd08.prod.outlook.com (2603:10b6:910:78::12)
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from localhost (2601:283:4602:43e0:1cad:e995:8263:2173) by SA0PR11CA0198.namprd11.prod.outlook.com (2603:10b6:806:1bc::23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.3977.26 via Frontend Transport; Mon, 29 Mar 2021 23:43:33 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-HT: Tenant
X-MS-Office365-Filtering-Correlation-Id: 6d964e3f-8200-43b3-5a5f-08d8f30c7706
X-MS-TrafficTypeDiagnostic: CY4PR0801MB3795:
X-Microsoft-Antispam-PRVS: <CY4PR0801MB3795DABB3EA79AC57CA7AA0DFB7E9@CY4PR0801MB3795.namprd08.prod.outlook.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:8273;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: LQwUyM2N94qyLYjRQAMWhCBGkca1GiejCyvjwvVK9DWzpmgf0kSXU4L1JsrKJ96zjbgJsLbPZdbulyHSge7l3ArvwsWlQD8D7TeWBufWsLrYcCsQSqEOCQm8cDKpwIqG39JqwW1I5ze+AGEuvjga8PPD97phpGFHP1oTol6uZ6HKKqXrlv+XevMz0+wK3LDmojsHdiePsCoZLt5V1oCFFBZf7OYb6VxkYztbDHPXzYD4LSAZYsbdSaC0lQGwaPGmdDtpzzbx9Gyb0NpHbfDnBkY2ZgTx7MTFVz8TTLdVSm57mWOAH8u6lHd/eKSWGXdmuNgtxBAHgJEOP0mCocOwOac0ur9fOWSsH0aju7oZQQE5vGxtTX6xf3mlV8ejz3I/eJRZmshaGqvQyZnZOJOqLaF2OQsHu/suWu3gIeH2rLElQJRkhwrqQHoBU9N4QdmeO/JWIGbVnEJw7fZIymnVBKZePPQHGX0vxQOIXDJ1vKYhC2Cu8M+1xG2LlDdJSCFdMKmQmqQ8ruDR6HQIPvGYsW/qdVSBDH7+bEg1otY8lI/9Hh7HY5WuhIHirZ5ZVvYvEvoSjsROpIYZhLtJnpnt3ablSF4GB33SRZGv39v4KXrmO+5+Q8QMDrKIVdytpIzMk5lIneWoJCbjPSZcXIJJJc9DmkT8RUF7iUgvWAmumM5y4USMVdhXRCxPjq5yIqS3M+sniJy6NevzyOeIBP1Yu1eS9vSFym42dM4Yt+wvFes=
X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CY4PR08MB3400.namprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(136003)(376002)(346002)(396003)(39860400002)(16526019)(6916009)(8936002)(9576002)(66946007)(55016002)(33716001)(9686003)(86362001)(8676002)(38100700001)(54906003)(2906002)(4326008)(83380400001)(6496006)(44832011)(186003)(66556008)(53546011)(478600001)(66476007)(5660300002)(316002)(966005); DIR:OUT; SFP:1102;
X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?bTJsaStLWjFocFQxeTM2Z1J6MzUrRlhZTk41K3p5MmlQd2d5Y3JLVGdFV3lN?= =?utf-8?B?TFpkVnVwb3I2TWpxQmVMOGJiN2duUG4rakc2c0t0bm9oVXljY1RnMkZBODlr?= =?utf-8?B?VzlQdjNQVWIwMVF5Yzk5WEo4K242TGRhSjBEZStuMWRFZDNKSHJ3d3hMdHgx?= =?utf-8?B?ckVVVUtJcXFNV05UWWdVamZSR0prZlhXbjk1dWNCby92SHpNTTNNbFQ3T3ow?= =?utf-8?B?eTEwSlRPbENneEJMMW56MTdYeTdOay81VHpOeiswVFYxMEtqWHI4WS9FcjNF?= =?utf-8?B?bFZuVnZhMG4vN2Z0MmhCeE9GUHpLR3hSV0ZRNWh5cHltK0pUdnVGclNpenhE?= =?utf-8?B?OVp0K3dGM1V6dUh0N1JtWEVKTXBmTSttbUd5TUk5a1FpN2VGZFhJL0pJWDdj?= =?utf-8?B?WFl2NVNvL0M0ZHNzbFBPUTh5U0dGVkROYmJsejVBcThHbVpsS0lITldMTWty?= =?utf-8?B?bjlsNUU2T2lueUQ2SDE2bDBmOHBJZnFLMkU1Y2UvM3R3TVZ2UjNyUm9uWjBS?= =?utf-8?B?TFFyTzFtaUwwOTNGR0U4bXRFdkpRUld6OEdvYnBQUlI0N2p5TlFJSGRVcjRk?= =?utf-8?B?cHNWbGVPa080VFN3VTZQOWpQT2FBT0EwNGJlazRjNjU1QXR2RHkzQm1xaDFC?= =?utf-8?B?L0RvbXdKRktyRjBYUDlSZ091WllvQTBLVklqeVFlblA4dDNIVzNhVXVITVU0?= =?utf-8?B?aVJ1NWlIa0Z0ZDdQang0bTZJallCTitTTENkSXZoWUljQ2FQMFJmZ2lHQThy?= =?utf-8?B?RnBuZlE4VnBzTTNic3U4MjBLdHNRTm9SaHptcDY1MmdmbjIxSXBNb1haNFlF?= =?utf-8?B?VmpBczF6eEp2ckRHZEZScEFFKzVQZlNzR0JwaVVXalp3TGxwTVJ6WXBKN2ZD?= =?utf-8?B?M252QndSNXRadjg0cUUzTmt3TktzVE9XT0FZaHlaRGsyUmp2dExSdjNZeGV2?= =?utf-8?B?YzhRb3Y2cHdDbnNYWmJSWit4bWFsUjZLbXBRVDhTLytxejhtY281eGFPeENW?= =?utf-8?B?Sjh4d1d1N05KdUI3UzJ2MVc2ZTAzdmxPSXlPL0tTc3pHV0laZTdxMWI5Wkpi?= =?utf-8?B?U1ZCZ1hLd1I0cThlamtzM01HbERYRlhRVHRDZ2t3ZW9xcEJMSTUyR0xkSXp1?= =?utf-8?B?eW8vV3Z1dVVVNkhYNmhnU3JybnlLVmk0YzRncDJObEcvenA2b2huYnNERkJN?= =?utf-8?B?TVFCakY5eU1pVmJUQ1VHYUNJVU1VU3l4T05WMlByVHJpWmVuazJGdmx4OVBh?= =?utf-8?B?bCswQk1jWFphV29yNnBhZU5DeWFlZTNUbzkyS2RLUE9CaXJaYkdFT0JGckx4?= =?utf-8?B?TXZEckpLUmozOXRVZS9EUTJGTjdRaWhZZTh5d1pTbndHRXhuSDdzL2d3WnhE?= =?utf-8?B?RmZocDQvc3V2c0hLQnRXckZqOTlKMzdmeGIyZHlRQ1VaVXNLOGRTRGtOTWUx?= =?utf-8?B?bFJuajdNUDlubi9IOWVvbU5ZN3JnRnZBN2REcHZDWE4vWVNtTVA5a3RhSjFs?= =?utf-8?B?RmpsdkpVSFhudlJLZ3pSSitMOGNVaG5wNXkyTVFIR3pFak9RS1lGZzdCNmF5?= =?utf-8?B?UmFtMlpkeUsxazg3Rll6d1FJaUo4cXZzQjJRWUkrZlRNaHd5aUYreWptV2dH?= =?utf-8?B?Y0JadE9CT1JpajhTZllBdjUyeENpWjQ5czI0K3lhZmJqZjgyZzF4L1ZmN2to?= =?utf-8?B?WWpRY0ZFZkFNaHE5RWhyRXVhQk9ZTWVBaEdJdHVUWXRRcTY2VnRseXpndmJt?= =?utf-8?B?SU5KazJIT3ZBZFlUWkdpenVsWjNvQkVNQ2ptZlZsV2Fia2xsZnE5WmVodmFz?= =?utf-8?B?YnNZeEs2QXJrVEJidVRtbWxoMC9NZWxheWNQWVg2TWx1OGQ1T0c1SGZtSkR5?= =?utf-8?Q?g0QLP6H02HfUY?=
X-OriginatorOrg: nokia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 6d964e3f-8200-43b3-5a5f-08d8f30c7706
X-MS-Exchange-CrossTenant-AuthSource: CY4PR08MB3400.namprd08.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Mar 2021 23:43:33.6536 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 5d471751-9675-428d-917b-70f44f9630b0
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: cOUD/+iw8vVtFMyCTAhlC2LkClFQMeHgBMSROBHtPxyv19+a4FmAzGt0cu721599Ce2UxcXgR2+s3PxxcyMCEQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR0801MB3795
Archived-At: <https://mailarchive.ietf.org/arch/msg/rtg-bfd/MX2oBrik3-9J30-9wsDLiyJhLT8>
X-BeenThere: rtg-bfd@ietf.org
X-Mailman-Version: 2.1.29
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: Mon, 29 Mar 2021 23:44:21 -0000

On Mar 25 10:11 AM, Mahesh Jethanandani wrote:
> Hi Ebben,
> 
> Thanks first of all for the review.
> 
> 
>     On Mar 21, 2021, at 2:10 PM, Ebben Aries via Datatracker <noreply@ietf.org>
>     wrote:
> 
>     Reviewer: Ebben Aries
>     Review result: Ready with Nits
> 
>     One of the simpler YD reviews since this draft contains only a single
>     module
>     with a single r/o leaf augment to various other BFD module nodes.
> 
>     1 module in this draft:
>     - ietf-bfd-stability@2021-02-24.yang
> 
>     YANG compiler errors or warnings (pyang 2.4.0, yanglint 1.10.17, confdc
>     7.2.3.4)
>     - warning: RFC 8407: 3.1: The IETF Trust Copyright statement seems to be
>     missing
>      - Minor issue just due to slight wording adjustment from copyright info
>        normally used in the draft/RFC itself
>        https://trustee.ietf.org/documents/trust-legal-provisions/tlp-5/
> 
> 
> I cut and pasted the text from ’pyang —ietf-help’ and it still gives me the
> error. Go figure!

The following should be the correct wording to pass the ietf checks:

--- yang/ietf-bfd-stability@2021-02-24.yang	2021-03-29 17:31:29.494394915 -0600
+++ yang/ietf-bfd-stability@2021-02-24.yang	2021-03-29 17:31:23.152419684 -0600
@@ -79,7 +79,7 @@
      a per session count for BFD packets that are lost.

      Copyright (c) 2021 IETF Trust and the persons identified as
-     the document authors.  All rights reserved.
+     authors of the code.  All rights reserved.
      Redistribution and use in source and binary forms, with or
      without modification, is permitted pursuant to, and subject
      to the license terms contained in, the Simplified BSD

$ pyang --ietf-help

Validates the module or submodule according to the IETF rules found
in RFC 8407.

The module's or submodule's description statement must contain the
following text:

     Copyright (c) <year> IETF Trust and the persons identified as
     authors of the code.  All rights reserved.
...


> 
> 
> 
>     Module ietf-bfd-stability@2021-02-24.yang:
>     - Contact information should follow formatting seen in other published
>      modules.  Just some minor cleanup/readjustments here.
> 
> 
> Do you e-mail addresses of the authors? If so, they were written as such to
> prevent online email extractors from extracting email addresses.
> 

Following RFC8407 includes the 'WG List' as <mailto:bfd@ietf.org>
followed by an 'Editor:' block with Names/emails listed in the above
format as well.  Obfuscating email addresses as you have it (not sure
I've seen any other modules attempt this) is unlikely to provide any
benefit (These same email addresses are also listed in the draft/RFC
document itself unobfuscated)

Thx

/ebben

> 
>     - Various augments will augment the respective 'session-statistics'
>     container
>      however for 'bfd-lag:member-links/micro-bfd-ipv4',
>      'bfd-lag:member-links/micro-bfd-ipv6' and 'bfd-mpls:sessions' these
>     augments
>      sit in parallel to that container.  Is there any reason this is not
>      consistent?
> 
> 
> 
> Good catch. Have added a path to “session-statistics” container.
> 
>   augment "/rt:routing/rt:control-plane-protocols/" +
>           "rt:control-plane-protocol/bfd:bfd/bfd-lag:lag/" +
>           "bfd-lag:sessions/bfd-lag:session/bfd-lag:member-links/" +
>           "bfd-lag:micro-bfd-ipv4/bfd-lag:session-statistics" {
> 
> 
>     General comments on the draft/modules:
>     - Section 7.2: It is stated the document registers "2" YANG modules when it
>     is
>      infact only a single module
> 
> 
> Fixed.
> 
> 
> 
>     Overall, just minor nits and view as ready from a YANG doctor review
>     standpoint once the above is addressed
> 
> 
> Thanks
> 
> Mahesh Jethanandani
> mjethanandani@gmail.com
> 
> 
> 
> 
>