[I2nsf] YANG Doctors Working Group Last Call Review of draft-ietf-i2nsf-capability-data-model-04.txt
"Acee Lindem (acee)" <acee@cisco.com> Tue, 18 June 2019 19:09 UTC
Return-Path: <acee@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 289B912006A; Tue, 18 Jun 2019 12:09:00 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -14.5
X-Spam-Level:
X-Spam-Status: No, score=-14.5 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, RCVD_IN_DNSWL_HI=-5, SPF_PASS=-0.001, URIBL_BLOCKED=0.001, 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=LxPgfvDZ; dkim=pass (1024-bit key) header.d=cisco.onmicrosoft.com header.b=HEu+u6o3
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 DIMQP7Ww1fjm; Tue, 18 Jun 2019 12:08:58 -0700 (PDT)
Received: from alln-iport-5.cisco.com (alln-iport-5.cisco.com [173.37.142.92]) (using TLSv1.2 with cipher DHE-RSA-SEED-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 9554A1200B4; Tue, 18 Jun 2019 12:08:57 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=cisco.com; i=@cisco.com; l=3212; q=dns/txt; s=iport; t=1560884937; x=1562094537; h=from:to:cc:subject:date:message-id:content-id: content-transfer-encoding:mime-version; bh=SAZnJiE6zQiOjd7CcBHWOBPV8fCPT0KCwWSMQ3V4aq8=; b=LxPgfvDZcBrFt/0GjLumWw7rutMC/dVv2FMyOzuI30z4qiYUtqys9Udt Rqmx6V66eM5a/91MmQtd7naaOtskPdNdxZhph4Z7W8pf5aTDsCPwI8IZl WJih/q+mIFTbbGEtzyr4v+4iykDg4TsZP88pdPmHayaQkgiyGoVSYTDOs Y=;
IronPort-PHdr: 9a23:6UYbABTr0LhUvDzpimImomWLKNpsv++ubAcI9poqja5Pea2//pPkeVbS/uhpkESUANfA8/wRje3QvuigQmEG7Zub+FE6OJ1XH15NksAKh0olCc+BB1f8KavjZCE3NM9DT1RiuXq8NBsdFQ==
X-IronPort-Anti-Spam-Filtered: true
X-IronPort-Anti-Spam-Result: A0CDAAA9Ngld/4QNJK1mHgEGBwaBUQkLAYE9UANqVSAECyiEFoNHA45lgjKXW4EugSQDVAkBAQEMAQEjCgIBAYRAGYI6IzQJDgEDAQEEAQECAQRtHAELhU0WEREMAQE3AREBHAYCJgIEMBUSBA4ngwABgWoDHQEOnysCgTiIX3GBMYJ5AQEFhH8YghADBoEMKAGKMoErF4F/gTgME4Ieg00CgTqDMTKCBCKOP5slCQKCEIZIjQsbgieLEIl9jR2HJI9DAgQCBAUCDgEBBYFQOIFYcBVlAYJBgRZ5N4M5hRSFP3IMgR2MN4JSAQE
X-IronPort-AV: E=Sophos;i="5.63,390,1557187200"; d="scan'208";a="288923169"
Received: from alln-core-10.cisco.com ([173.36.13.132]) by alln-iport-5.cisco.com with ESMTP/TLS/DHE-RSA-SEED-SHA; 18 Jun 2019 19:08:55 +0000
Received: from XCH-RCD-001.cisco.com (xch-rcd-001.cisco.com [173.37.102.11]) by alln-core-10.cisco.com (8.15.2/8.15.2) with ESMTPS id x5IJ8td5018237 (version=TLSv1.2 cipher=AES256-SHA bits=256 verify=FAIL); Tue, 18 Jun 2019 19:08:55 GMT
Received: from xhs-rtp-003.cisco.com (64.101.210.230) by XCH-RCD-001.cisco.com (173.37.102.11) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Tue, 18 Jun 2019 14:08:54 -0500
Received: from xhs-aln-003.cisco.com (173.37.135.120) by xhs-rtp-003.cisco.com (64.101.210.230) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Tue, 18 Jun 2019 15:08:52 -0400
Received: from NAM01-BY2-obe.outbound.protection.outlook.com (173.37.151.57) by xhs-aln-003.cisco.com (173.37.135.120) with Microsoft SMTP Server (TLS) id 15.0.1473.3 via Frontend Transport; Tue, 18 Jun 2019 14:08:53 -0500
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=SAZnJiE6zQiOjd7CcBHWOBPV8fCPT0KCwWSMQ3V4aq8=; b=HEu+u6o3hQqFMO1Jey2eBq42wc4kHsJDvvFB6XMRqTyAz3Ra3aZZiHZDRY8gYWl3VpeM42S222b7v1cK/xxCY3JSH0hIj6nkjqy1fp1FeTuYcrcBWBJq8g1SxsykA7lzqFBtf0VJtTtfKss54Punez/Vmavv3kVmG8RzEpM3KDs=
Received: from MWHPR11MB1902.namprd11.prod.outlook.com (10.175.53.139) by MWHPR11MB1821.namprd11.prod.outlook.com (10.175.53.136) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1987.10; Tue, 18 Jun 2019 19:08:52 +0000
Received: from MWHPR11MB1902.namprd11.prod.outlook.com ([fe80::f1d4:41cf:84d6:ff73]) by MWHPR11MB1902.namprd11.prod.outlook.com ([fe80::f1d4:41cf:84d6:ff73%2]) with mapi id 15.20.1987.014; Tue, 18 Jun 2019 19:08:52 +0000
From: "Acee Lindem (acee)" <acee@cisco.com>
To: "draft-ietf-i2nsf-capability-data-model@ietf.org" <draft-ietf-i2nsf-capability-data-model@ietf.org>
CC: yang-doctors <yang-doctors-bounces@ietf.org>, "i2nsf-ads@ietf.org" <i2nsf-ads@ietf.org>, "i2nsf@ietf.org" <i2nsf@ietf.org>
Thread-Topic: YANG Doctors Working Group Last Call Review of draft-ietf-i2nsf-capability-data-model-04.txt
Thread-Index: AQHVJglEIwKHl04GPEqnj5R0hNyApA==
Date: Tue, 18 Jun 2019 19:08:51 +0000
Message-ID: <AB210DB8-0137-41D6-9495-0B3FBE07343F@cisco.com>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
authentication-results: spf=none (sender IP is ) smtp.mailfrom=acee@cisco.com;
x-originating-ip: [2001:420:c0c8:1003::c6]
x-ms-publictraffictype: Email
x-ms-office365-filtering-correlation-id: e21c8e86-0472-4c98-faf5-08d6f42066d2
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390118)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600148)(711020)(4605104)(1401327)(2017052603328)(7193020); SRVR:MWHPR11MB1821;
x-ms-traffictypediagnostic: MWHPR11MB1821:
x-ms-exchange-purlcount: 1
x-microsoft-antispam-prvs: <MWHPR11MB182152857DB702BC64C2DD3CC2EA0@MWHPR11MB1821.namprd11.prod.outlook.com>
x-ms-oob-tlc-oobclassifiers: OLM:9508;
x-forefront-prvs: 007271867D
x-forefront-antispam-report: SFV:NSPM; SFS:(10009020)(396003)(346002)(376002)(136003)(366004)(39860400002)(199004)(189003)(66476007)(6436002)(4326008)(81156014)(450100002)(5024004)(25786009)(91956017)(64756008)(256004)(478600001)(66946007)(76116006)(14444005)(46003)(33656002)(66556008)(316002)(53936002)(6506007)(86362001)(66446008)(66574012)(5660300002)(966005)(102836004)(99286004)(186003)(6486002)(305945005)(7736002)(2501003)(6512007)(6306002)(73956011)(54906003)(5640700003)(14454004)(8936002)(71200400001)(71190400001)(2616005)(81166006)(36756003)(68736007)(6116002)(486006)(476003)(6916009)(2906002)(2351001)(8676002); DIR:OUT; SFP:1101; SCL:1; SRVR:MWHPR11MB1821; H:MWHPR11MB1902.namprd11.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; A:1; MX:1;
received-spf: None (protection.outlook.com: cisco.com does not designate permitted sender hosts)
x-ms-exchange-senderadcheck: 1
x-microsoft-antispam-message-info: VA4nchtrAKYvdRd/Ip2agijV0Sv0J308K71aTBpYLZpe22XJXvCB47qWIgt+ogyvEM0CnC0f8sGZL7wxM+jsny8uCh+Xzvmwx3VOXKKKdRIH2HNrEb8DJxdDq+UQc+lDGNJoKHH/sWmdfGskv+Hd6bixBBKuNr4tAgen2QU6a9mH/+lCBi1p6XbKB4R7eC2SdubkpcKvggkSpUW1Vvpi2VxDpMmhvLUlrno+WcI5jhFUqOXjxIsFj6cJOVGJNDQzKG6T7zfThrmLWAoxTwV7co8K4F3o7a1iG/6a2flzgdI/l1cvWdlxY11PeJXdxkRhEWLzajH5eRhP8HpL24QPqVdivYJU1NIJGMtoTb1+mi8tpyeUho+Rm5KxKfamI5CXb1WrYQycokYt0wpOalGdrrPtC7tQ3LBVRxVmK5/83eI=
Content-Type: text/plain; charset="utf-8"
Content-ID: <30274AC118AA984BBFBD8165E5144FCE@namprd11.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-Network-Message-Id: e21c8e86-0472-4c98-faf5-08d6f42066d2
X-MS-Exchange-CrossTenant-originalarrivaltime: 18 Jun 2019 19:08:51.8555 (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: acee@cisco.com
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR11MB1821
X-OriginatorOrg: cisco.com
X-Outbound-SMTP-Client: 173.37.102.11, xch-rcd-001.cisco.com
X-Outbound-Node: alln-core-10.cisco.com
Archived-At: <https://mailarchive.ietf.org/arch/msg/i2nsf/zs4TQIV1nLmjLEJ4xbSl9dKSlk0>
Subject: [I2nsf] YANG Doctors Working Group Last Call Review of draft-ietf-i2nsf-capability-data-model-04.txt
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: Tue, 18 Jun 2019 19:09:00 -0000
Document: draft-ietf-i2nsf-capability-data-model-04.txt Reviewer: Acee Lindem Review Date: June 18, 2019 Review Type: Working Group Last Call Intended Status: Standards Track Summary: Not ready for publication Modules: "ietf-i2nsf-capability@2019-03-28.yang" Tech Summary: The model is logically structured and seems to fulfill its intended purpose. The "Overview" section defines the usage, context, and usage of the model, i.e., it is limited to the NSF capability registration interface. However, the draft/model is very rough and not ready for working group last call. It seems that it has not gotten adequate review by the chairs and other members of the I2NSF Working Group. Major Comments: 1. The "Security Considerations" in section 8 do not conform to the recommended template in https://trac.ietf.org/trac/ops/wiki/yang-security- guidelines> 2. The document is missing XML or JSON examples. Minor Comments: 1. Section 3.1 should reference RFC8340 rather than attempting to include tree diagram formatting semantics. 2. Much of the text is very hard to read and awkwardly worded. There are some instances of sentence fragments. I starting trying to remedy this but found I was rewriting the entire draft and, in many cases, I wasn't sure my edits matched the original intent. I'll send the attached diff with suggested edits to the authors - it was too big to send to IETF lists. 3. "iicapa" is a poor choice for default model prefix - I suggest "nsfcap". It is just as concise but actually expands to something meaningful. 4. Similar to the text in the narrative sections of the draft, the text in the YANG model description statements is very awkwardly worded. 5. What are the references for the ipv4-sameip and ipv4-geoip conditions? 6. Add reference for egress-action-capa. 7. RFC 2460 is obsoleted by RFC 8200. 8. Suggest hyphenation of identifiers ipv4-same-ip, ipv4-geo-ip, and ipv6-ip-opts. 9. Suggest hyphenation of anti-virus and anti-ddos both in identifiers and in the text. 10. Suggest providing a definitions for absolute and periodic time. 11. The References do not include all the RFCs referenced by YANG model reference statements. Thanks, Acee
- [I2nsf] YANG Doctors Working Group Last Call Revi… Acee Lindem (acee)
- Re: [I2nsf] YANG Doctors Working Group Last Call … Mr. Jaehoon Paul Jeong
- Re: [I2nsf] YANG Doctors Working Group Last Call … Acee Lindem (acee)
- Re: [I2nsf] YANG Doctors Working Group Last Call … Mr. Jaehoon Paul Jeong