Re: [I2nsf] Yangdoctors last call review of draft-ietf-i2nsf-registration-interface-dm-04

"Reshad Rahman (rrahman)" <rrahman@cisco.com> Fri, 07 February 2020 23:38 UTC

Return-Path: <rrahman@cisco.com>
X-Original-To: i2nsf@ietfa.amsl.com
Delivered-To: i2nsf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 69BD712010D; Fri, 7 Feb 2020 15:38:06 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.49
X-Spam-Level:
X-Spam-Status: No, score=-14.49 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HTML_MESSAGE=0.001, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, T_FILL_THIS_FORM_SHORT=0.01, USER_IN_DEF_DKIM_WL=-7.5] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (1024-bit key) header.d=cisco.com header.b=j1UHKwVo; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=VT6aaXyz
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 TCvtimX6uLzQ; Fri, 7 Feb 2020 15:38:03 -0800 (PST)
Received: from alln-iport-1.cisco.com (alln-iport-1.cisco.com [173.37.142.88]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 299DD1200A3; Fri, 7 Feb 2020 15:38:03 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=50689; q=dns/txt; s=iport; t=1581118683; x=1582328283; h=from:to:cc:subject:date:message-id:references: in-reply-to:mime-version; bh=TbCJLw/Vw4fNjk1kBr89Xx2hK9Yr1ElOJbddvhfT7NY=; b=j1UHKwVomSepb9DB7XK7DCqaNU+La3DqkiXqeh2QrMBptfqD3DV4LOfL WQtCeArPPrAJTAsiBCAACPt5CcJCTWzB39m3pAGZbnd006ZUMLURYAjUC TfhfO05AW2KFYXpKWVp33NKq69yPrDpYrGbVGltuFG1ZsdWMTYTY4wjGJ k=;
IronPort-PHdr: =?us-ascii?q?9a23=3Avr+lkRXCl/UzqvleEHRjuljdnTfV8LGuZFwc94?= =?us-ascii?q?YnhrRSc6+q45XlOgnF6O5wiEPSA92J8OpK3uzRta2oGXcN55qMqjgjSNRNTF?= =?us-ascii?q?dE7KdehAk8GIiAAEz/IuTtankgBs1CUVZj13q6KkNSXs35Yg6arw=3D=3D?=
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: =?us-ascii?q?A0BLDgDC8z1e/4YNJK1MGh0BAQEJARE?= =?us-ascii?q?FBQGBe4ElL1AFbFggBAsqCoQLg0YDiwCCX4lijjCBQoEQA1AECQEBAQwBARg?= =?us-ascii?q?BDgYCAQGBTIE+gTYCF4IsJDgTAgMNAQEEAQEBAgEFBG2FNwyFZgEBAQQBARA?= =?us-ascii?q?RBBkBASkDCwENAgIBCBEDAQIhAQYDAgICFAsGCxQJCAIEDgUigwQBgX1NAy4?= =?us-ascii?q?BAgw8ozcCgTmIYnV/M4J/AQEFgS8BAwIBCwJAAYMiDQuCDAkFgTOMIxqBQT+?= =?us-ascii?q?BEScggh4uPoIbSQEBAQIBgSQIAQsBBgE4CQ0JgloyggoijVSCTDuFZCSHAoJ?= =?us-ascii?q?ZjhlbRAqCOodMhUiFDASEKBuCSHmHGIEGjzCQL4U4gWmCKJARAgQCBAUCDgE?= =?us-ascii?q?BBYE/KiJncXAVOyoBgkEJRxgNjXkkOIM7aoQqhT90AoEninwBAQ8XB4EEAYE?= =?us-ascii?q?PAQE?=
X-IronPort-AV: E=Sophos;i="5.70,415,1574121600"; d="scan'208,217";a="424173451"
Received: from alln-core-12.cisco.com ([173.36.13.134]) by alln-iport-1.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 07 Feb 2020 23:38:01 +0000
Received: from XCH-RCD-005.cisco.com (xch-rcd-005.cisco.com [173.37.102.15]) by alln-core-12.cisco.com (8.15.2/8.15.2) with ESMTPS id 017Nc1Qh021701 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Fri, 7 Feb 2020 23:38:01 GMT
Received: from xhs-aln-001.cisco.com (173.37.135.118) by XCH-RCD-005.cisco.com (173.37.102.15) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 7 Feb 2020 17:38:00 -0600
Received: from xhs-aln-001.cisco.com (173.37.135.118) by xhs-aln-001.cisco.com (173.37.135.118) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Fri, 7 Feb 2020 17:38:00 -0600
Received: from NAM10-DM6-obe.outbound.protection.outlook.com (173.37.151.57) by xhs-aln-001.cisco.com (173.37.135.118) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Fri, 7 Feb 2020 17:38:00 -0600
ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=g79Ka+sH3JjqlSjqSX5U5XWDhp3X1FfgxWOCXMnOXaAN/F9MOwK95HuScy+hiLmS+N3SFUtfM645ZS3JNp6S/OGmWqaviwgwhaTmydMHR7cinzekSltzb/oL9gGrX2yXYDu5HZUGa+hw+Ak4D7amZiXkr4ksI7H6qZeeTZqJ3zFAgTyIiUd1oONxkcyXS7jiKw7AtOtSXMfbZQ29HW7rn7+P9gAd8faPKwUzj7d/b8JHyxFeN8U1Vl6jFJhQJ6iUhYwnGD7/E5CVHhC2CkY2amFCV/teMWMIFwAHPE+tpDOKjqI2LAPaSNNSHUKHPAHE7nm08QVxP07AlKZnV1+w/w==
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=TbCJLw/Vw4fNjk1kBr89Xx2hK9Yr1ElOJbddvhfT7NY=; b=A7sHn8VRysYz3/fNHQTRUTzzcO9PEx1dHWCZOqH/jWJusPzDseS6hvPRMXhV1x4yEpb3teHqisjw8J2NLpV2cl4VzxcNvgz1M72xyZvUyFX3K7x5nSfjiFCDrqAXhSh+Ku1auBIIyI8asIahQBJkVZU9TqYMU0BAMDLY+2/cZAlAaJj+UC48+ZO9YzAToXt4V/7j0pWlQnGghTfv/obaRiTJUr9okz47aMKAuQZuVV0V9nq+QxZkGE5SdUsaO4wHfJ/JyPzJzxeCEAdR9NiI3OMFIbtPnzLVl1M56BjatLYe7sucvMroQPmR2kKGOHRcNdfu3Px/TrLWa7C6IhoIJA==
ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=cisco.com; dmarc=pass action=none header.from=cisco.com; dkim=pass header.d=cisco.com; arc=none
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=cisco.onmicrosoft.com; s=selector2-cisco-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=TbCJLw/Vw4fNjk1kBr89Xx2hK9Yr1ElOJbddvhfT7NY=; b=VT6aaXyzu+bj9uOq0ptuxlez59cyhnMaNke25eOx9uwRsnC8KCrqj3AyBOy4idct87Zpot7hZPTZ6lIi2rv6ojdSINdJmsC+cHdye6FEwD3r8sXwSRQB4aZciUjU45OH0ph8xQdxRTrsfT6gJSzuzE+0GM1DCLxZjzERljlL+l4=
Received: from DM6PR11MB3420.namprd11.prod.outlook.com (20.177.218.95) by DM6PR11MB3818.namprd11.prod.outlook.com (20.179.17.86) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.2707.21; Fri, 7 Feb 2020 23:37:58 +0000
Received: from DM6PR11MB3420.namprd11.prod.outlook.com ([fe80::d887:b11e:eab5:5476]) by DM6PR11MB3420.namprd11.prod.outlook.com ([fe80::d887:b11e:eab5:5476%3]) with mapi id 15.20.2707.024; Fri, 7 Feb 2020 23:37:58 +0000
From: "Reshad Rahman (rrahman)" <rrahman@cisco.com>
To: "Mr. Jaehoon Paul Jeong" <jaehoon.paul@gmail.com>
CC: YANG Doctors <yang-doctors@ietf.org>, "i2nsf@ietf.org" <i2nsf@ietf.org>, Sangwon Hyun <swhyun77@gmail.com>, skku-iotlab-members <skku-iotlab-members@googlegroups.com>, Patrick Lingga <patricklink888@gmail.com>
Thread-Topic: [I2nsf] Yangdoctors last call review of draft-ietf-i2nsf-registration-interface-dm-04
Thread-Index: AQHVQvTAkkfTX09q5USrQyd4LCPXc6d+Bl8AgAiCPgCAb4ewAIAObmKAgAzG1AA=
Date: Fri, 7 Feb 2020 23:37:58 +0000
Message-ID: <9B0042BE-DF30-4EAD-9B7F-5E4FB196F980@cisco.com>
References: <156175664267.21931.10329030969352120108@ietfa.amsl.com> <CAPK2DezmXhBA39GFMrU6tYFte_rDK+DE=WyWpzSNFMvnNU+xGQ@mail.gmail.com> <CAPK2Deym5jVmi4JKekRVznpNCkO_x8PN7jz7DLXkgUNZ9V4uUw@mail.gmail.com> <47E6993E-1045-4B29-B814-E7185516B272@cisco.com> <CAPK2DexseZBxXZthOU3wjyDnXPGe3PYrNvbcC-JO8+8JjypM4w@mail.gmail.com> <CAPK2DeycWH3k1wsHKADhAQnHXsDsB6A0QNAf-2EkFyWoSppWOQ@mail.gmail.com>
In-Reply-To: <CAPK2DeycWH3k1wsHKADhAQnHXsDsB6A0QNAf-2EkFyWoSppWOQ@mail.gmail.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/10.21.0.200113
authentication-results: spf=none (sender IP is ) smtp.mailfrom=rrahman@cisco.com;
x-originating-ip: [173.38.117.70]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: cab17610-fca4-4406-4577-08d7ac26c360
x-ms-traffictypediagnostic: DM6PR11MB3818:|DM6PR11MB3818:
x-microsoft-antispam-prvs: <DM6PR11MB3818D0DA611FDC392E63CC05AB1C0@DM6PR11MB3818.namprd11.prod.outlook.com>
x-ms-exchange-transport-forked: True
x-ms-oob-tlc-oobclassifiers: OLM:10000;
x-forefront-prvs: 0306EE2ED4
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(4636009)(39860400002)(136003)(366004)(396003)(376002)(346002)(189003)(199004)(53546011)(6506007)(81166006)(81156014)(9326002)(8936002)(316002)(91956017)(26005)(186003)(54906003)(6916009)(2906002)(8676002)(66446008)(4326008)(64756008)(66556008)(66476007)(66946007)(6486002)(71200400001)(33656002)(36756003)(966005)(2616005)(16799955002)(478600001)(6512007)(15188155005)(86362001)(76116006)(5660300002); DIR:OUT; SFP:1101; SCL:1; SRVR:DM6PR11MB3818; H:DM6PR11MB3420.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam: BCL:0;
x-microsoft-antispam-message-info: PQsejd+wM8jRmj4BBIqzxV8W2x4/2Ie+10i8Om06Bs8ZCZDHBIXHTkTWjXKqN3uSsxuQUkqsd7VMM/orF9t0vlQ8EgexzYk8d2Fp1DujI2QLzn9cc2zp2r5OUkMBsPeGIVT9gTbFxGSfWrZVM2/3faD0FViEz5zuFnzvrBduc34uWMy4Z2Z62ShcrHgerChK5hyL3AzxHFmP5zhGWR35WT8XINYvuNCpILwZudq0iCn24svSWGJGNJX72RKbkFhhX1ohCJ0Dvxt7qx7C2y9AEArZimeUnsrv55o/lxbKtXa+ZeVMO3v2rlFAJ9fLJFFgVvsKulEG5WekPmiMlI7qErBKKFOiHLDkYYZFT6RSEEKBjvO0u/C0kGtORrEVic3Z39EtpCVAH+sdolZ/6eT/BwApS+MX8z7zITYQUIqKG8HqZpeaV86sa0I0UCabSipvgLv7kOAVb6INeuHJaxU6TcjcsMd9kZpQO3SJXc56XTcE8ZIk3IMEwxEIo62JZVSLBBL8GC4fngAzfwhJScTMUg==
x-ms-exchange-antispam-messagedata: BjOwmfrO1pJDojkX6oqHq9wnJrPL/Gch++zKCpcCrQZz9j6RCFI7Ez3dToRqA+hHnEMTPQI0Rx+hEm+q3lJo727NOPWV7h1tqh5wkmXH8TRDQp3Etmdexz84riIlt4qgkHHsjnERXXwdrfA07UJhDg==
Content-Type: multipart/alternative; boundary="_000_9B0042BEDF304EAD9B7F5E4FB196F980ciscocom_"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: cab17610-fca4-4406-4577-08d7ac26c360
X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Feb 2020 23:37:58.1656 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 5ae1af62-9505-4097-a69a-c1553ef7840e
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: nSs8URYwgWtk1BHaH+bMqi2bWSDm4s44nsR7YIWMYttWpcI3niwqPNwO9XUP7gJ7/votJUgrT05me+QGXhNWZw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM6PR11MB3818
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.37.102.15, xch-rcd-005.cisco.com
X-Outbound-Node: alln-core-12.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/i2nsf/98ppQ3Z9sn3kBJQKFMw6VsY50aU>
Subject: Re: [I2nsf] Yangdoctors last call review of draft-ietf-i2nsf-registration-interface-dm-04
X-BeenThere: i2nsf@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: "*I2NSF: Interface to Network Security Functions mailing list*" <i2nsf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/i2nsf>, <mailto:i2nsf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/i2nsf/>
List-Post: <mailto:i2nsf@ietf.org>
List-Help: <mailto:i2nsf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/i2nsf>, <mailto:i2nsf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 07 Feb 2020 23:38:06 -0000

Hi Paul,

I have verified that you’ve addressed all comments which were provided on 05.

Regards,
Reshad.

From: "Mr. Jaehoon Paul Jeong" <jaehoon.paul@gmail.com>
Date: Thursday, January 30, 2020 at 10:32 AM
To: "Reshad Rahman (rrahman)" <rrahman@cisco.com>
Cc: YANG Doctors <yang-doctors@ietf.org>rg>, "i2nsf@ietf.org" <i2nsf@ietf.org>rg>, Sangwon Hyun <swhyun77@gmail.com>om>, skku-iotlab-members <skku-iotlab-members@googlegroups.com>om>, Patrick Lingga <patricklink888@gmail.com>om>, "Mr. Jaehoon Paul Jeong" <jaehoon.paul@gmail.com>
Subject: Re: [I2nsf] Yangdoctors last call review of draft-ietf-i2nsf-registration-interface-dm-04

Reshad,
Could you respond to my previous email about the latest revision of the I2NSF Registration Interface YANF Data Model draft?
https://tools.ietf.org/html/draft-ietf-i2nsf-registration-interface-dm-06

If you are satisfied with my revision, could you update the YANG Doctor's review result in the following link?
https://datatracker.ietf.org/doc/review-ietf-i2nsf-registration-interface-dm-04-yangdoctors-lc-rahman-2019-06-28/

After getting your review update state, I can ask the I2NSF WG chairs to make the WG Last Call on
our I2NSF Registration Interface draft.

Thanks for your efforts and help.

Best Regards,
Paul

On Tue, Jan 21, 2020 at 8:08 PM Mr. Jaehoon Paul Jeong <jaehoon.paul@gmail.com<mailto:jaehoon.paul@gmail.com>> wrote:
Hi Reshad,
We authors have addressed your comments with the following revision:
https://tools.ietf.org/html/draft-ietf-i2nsf-registration-interface-dm-06

I attach the revision letter to show how to address your comments.

Could your check this revision and proceed with the YANG review update?

Thanks.

Best Regards,
Paul


On Tue, Nov 12, 2019 at 1:58 AM Reshad Rahman (rrahman) <rrahman@cisco.com<mailto:rrahman@cisco.com>> wrote:
Hi Paul,

I have done another review (I don’t think an automated email got sent out).
https://datatracker.ietf.org/doc/review-ietf-i2nsf-registration-interface-dm-04-yangdoctors-lc-rahman-2019-06-28/

YANG Doctor review of draft-ietf-i2nsf-registration-interface-dm-05 (by Reshad Rahman)

Thank you for addressing comments from my earlier review @ https://datatracker.ietf.org/doc/review-ietf-i2nsf-registration-interface-dm-04-yangdoctors-lc-rahman-2019-06-28/

Major comments/questions:
- There is a YANG warning on the datatracker page:
ietf-i2nsf-reg-interface@2019-07-24.yang:54: warning: RFC 8407: 3.1: The IETF Trust Copyright statement seems to be missing (see pyang --ietf-help for details).
To fix this, in the YANG module remove the <> around 2019: Copyright (c) <2019>

- For contact in YANG module, please remove WG chair info (see RFC8407 appendix B for an example)

- For the revision in YANg module, put "Initial version" (even though it's the 5th revision)

- Why define a union of ipv4-address and ipv6-address in typedef nsf-address, why not reuse existing ip-address type from RFC6021?

- For bandwidth, is there a reason why it's limited to uint16? Even though 65Tbps is a lot, I wouldn't limit it to uint16. And aren't there any use-cases for bandwidth smaller than 1 Gbps? If yes, use e.g Mbps as unit and use uint32 instead of uint16? Please use units statement.
- It is not clear to me what’s the distinction between nsf-name and nsf-instance-name. In Examples 4 and 5, they have the same value, but not in Example 3.  Might be worth clarifying or giving the same name.

- Having nsf or i2nsf in many node names is redundant, since NSF or I2NSF is in the higher level container name.  e.g, in NSF Capability Registration all nodes seem to have i2nsf or nsf in their name.

- There seems to be some indentation issues in the YANG  module (e.g. P16)

- Abide by order in RFC8407 Appendix B. e.g. RPC statements should be after groupings.

Nits:

- Appendix B: Managmenet -> Management

- Section 6.2: capailities -> capabilities

- Example 5: space in "http_and_h ttps_flood_mitigation_capability"

Regards,
Reshad.


Regards,
Reshad.

From: "Mr. Jaehoon Paul Jeong" <jaehoon.paul@gmail.com<mailto:jaehoon.paul@gmail.com>>
Date: Tuesday, November 5, 2019 at 9:02 PM
To: "Reshad Rahman (rrahman)" <rrahman@cisco.com<mailto:rrahman@cisco.com>>
Cc: YANG Doctors <yang-doctors@ietf.org<mailto:yang-doctors@ietf.org>>, "i2nsf@ietf.org<mailto:i2nsf@ietf.org>" <i2nsf@ietf.org<mailto:i2nsf@ietf.org>>, "draft-ietf-i2nsf-registration-interface-dm.all@ietf.org<mailto:draft-ietf-i2nsf-registration-interface-dm.all@ietf.org>" <draft-ietf-i2nsf-registration-interface-dm.all@ietf.org<mailto:draft-ietf-i2nsf-registration-interface-dm.all@ietf.org>>, Sangwon Hyun <swhyun77@gmail.com<mailto:swhyun77@gmail.com>>, "skku-iotlab-members@googlegroups.com<mailto:skku-iotlab-members@googlegroups.com>" <skku-iotlab-members@googlegroups.com<mailto:skku-iotlab-members@googlegroups.com>>, "Mr. Jaehoon Paul Jeong" <jaehoon.paul@gmail.com<mailto:jaehoon.paul@gmail.com>>
Subject: Re: [I2nsf] Yangdoctors last call review of draft-ietf-i2nsf-registration-interface-dm-04

Hi Reshad,
I believe that I have addressed your comments on I2NSF Registration Interface Data Model:
https://tools.ietf.org/html/draft-ietf-i2nsf-registration-interface-dm-05

If you are satisfied with the revision, could you update the Review result in the following page?
https://datatracker.ietf.org/doc/review-ietf-i2nsf-registration-interface-dm-04-yangdoctors-lc-rahman-2019-06-28/

Thanks.

Best Regards,
Paul

On Thu, Jul 25, 2019 at 11:23 PM Mr. Jaehoon Paul Jeong <jaehoon.paul@gmail.com<mailto:jaehoon.paul@gmail.com>> wrote:
Hi Reshad,
Here is the revision letter for the revised draft, reflecting your comments along with the revised draft:
https://tools.ietf.org/html/draft-ietf-i2nsf-registration-interface-dm-05

If you have further comments and questions, please let me know.

Thanks.

Best Regards,
Paul

On Fri, Jun 28, 2019 at 5:17 PM Reshad Rahman via Datatracker <noreply@ietf.org<mailto:noreply@ietf.org>> wrote:
Reviewer: Reshad Rahman
Review result: On the Right Track

YANG Doctor review of draft-ietf-i2nsf-registration-interface-dm-04 (by Reshad
Rahman)

Major comments:
- Look at appendix B of RFC8407 for an example of how a YANG module should be
structured. This document does not abide to that. - Poor descriptions e.g.
"nsf-name" for leaf "nsf-name" etc - prefix "iiregi" doesn't seem right. What
about "nsfreg"? Probably needs coordination with the other I2NSF YANG modules
to have consistency between the prefixes. I see that YD Acee suggested
"nsfintf" for draft-ietf-i2nsf-nsf-facing-interface-dm-06 - No unit specified
for bandwidth, processing (performance) - nsf-address is IPv4 specific -
Security considerations should list the nodes as per section 3.7 of RFC8407. -
Should this document be informational since 8329 is informational? - Section 2
should use RFC8174 also - Refer to RFC8407 instead of 6807 (YANG Guidelines) -
Examples should use IPv6 as examples (use the range from RFC3849). Kudos for
all the examples.

Minor comments and questions:
- The YANG trees such as Figure 6, 7 etc don't show the contents of the
groupings. So they don't help much. - nsf-port-address should be nsf-port? -
Section 4, last bullet. I am not an expert on I2NSF so not clear to me why this
query is needed, is it because NSF may not re-register after their capabilities
have been updated? Might be worth adding some explanation. - Have the examples
been validated?

_______________________________________________
I2nsf mailing list
I2nsf@ietf.org<mailto:I2nsf@ietf.org>
https://www.ietf.org/mailman/listinfo/i2nsf


--
===========================
Mr. Jaehoon (Paul) Jeong, Ph.D.
Associate Professor
Department of Software
Sungkyunkwan University
Office: +82-31-299-4957
Email: jaehoon.paul@gmail.com<mailto:jaehoon.paul@gmail.com>, pauljeong@skku.edu<mailto:pauljeong@skku.edu>
Personal Homepage: http://iotlab.skku.edu/people-jaehoon-jeong.php<http://cpslab.skku.edu/people-jaehoon-jeong.php>


--
===========================
Mr. Jaehoon (Paul) Jeong, Ph.D.
Associate Professor
Department of Software
Sungkyunkwan University
Office: +82-31-299-4957
Email: jaehoon.paul@gmail.com<mailto:jaehoon.paul@gmail.com>, pauljeong@skku.edu<mailto:pauljeong@skku.edu>
Personal Homepage: http://iotlab.skku.edu/people-jaehoon-jeong.php<http://cpslab.skku.edu/people-jaehoon-jeong.php>


--
===========================
Mr. Jaehoon (Paul) Jeong, Ph.D.
Associate Professor
Department of Software
Sungkyunkwan University
Office: +82-31-299-4957
Email: jaehoon.paul@gmail.com<mailto:jaehoon.paul@gmail.com>, pauljeong@skku.edu<mailto:pauljeong@skku.edu>
Personal Homepage: http://iotlab.skku.edu/people-jaehoon-jeong.php<http://cpslab.skku.edu/people-jaehoon-jeong.php>


--
===========================
Mr. Jaehoon (Paul) Jeong, Ph.D.
Associate Professor
Department of Software
Sungkyunkwan University
Office: +82-31-299-4957
Email: jaehoon.paul@gmail.com<mailto:jaehoon.paul@gmail.com>, pauljeong@skku.edu<mailto:pauljeong@skku.edu>
Personal Homepage: http://iotlab.skku.edu/people-jaehoon-jeong.php<http://cpslab.skku.edu/people-jaehoon-jeong.php>