Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-17.txt
tom petch <ietfc@btconnect.com> Thu, 29 November 2018 15:12 UTC
Return-Path: <ietfc@btconnect.com>
X-Original-To: ccamp@ietfa.amsl.com
Delivered-To: ccamp@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id EF0D7130E05 for <ccamp@ietfa.amsl.com>; Thu, 29 Nov 2018 07:12:24 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 1.738
X-Spam-Level: *
X-Spam-Status: No, score=1.738 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIMWL_WL_MED=-1.459, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, RATWARE_MS_HASH=2.148, RATWARE_OUTLOOK_NONAME=2.95, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=no 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 ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id rgD86babPE8A for <ccamp@ietfa.amsl.com>; Thu, 29 Nov 2018 07:12:22 -0800 (PST)
Received: from EUR01-DB5-obe.outbound.protection.outlook.com (mail-db5eur01on070f.outbound.protection.outlook.com [IPv6:2a01:111:f400:fe02::70f]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-SHA384 (256/256 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 3153C127598 for <ccamp@ietf.org>; Thu, 29 Nov 2018 07:12:22 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=btconnect.onmicrosoft.com; s=selector1-btconnect-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=580WZ2kUFmn8ynG9mOGaO8jCK2gcjoiqV6Ya2X2yCtM=; b=G6dkhYNxaIsbtpAtYH8Z4mN+7/5V42TCnAapafBAwyPM+5HUHv3Klki1gJ9Y6Sn3faOtMW6JsbyzKsdiEtQGpdoqQu+D4GhANXZM8ROboUDrhCKc6oQOpX01lqWS7KUHnr9MY7CY/tsRzH5P4L7V3UKon3dmpMmxe55ibj85qlY=
Received: from VI1PR07MB4717.eurprd07.prod.outlook.com (20.177.54.82) by VI1PR07MB4189.eurprd07.prod.outlook.com (20.176.6.26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.1361.9; Thu, 29 Nov 2018 15:12:20 +0000
Received: from VI1PR07MB4717.eurprd07.prod.outlook.com ([fe80::1575:d33b:33dd:c7c4]) by VI1PR07MB4717.eurprd07.prod.outlook.com ([fe80::1575:d33b:33dd:c7c4%5]) with mapi id 15.20.1382.017; Thu, 29 Nov 2018 15:12:20 +0000
From: tom petch <ietfc@btconnect.com>
To: Leeyoung <leeyoung@huawei.com>, "ccamp@ietf.org" <ccamp@ietf.org>
Thread-Topic: I-D Action: draft-ietf-ccamp-wson-yang-17.txt
Thread-Index: AQHUh/XsKq63D0+1m0+fvD418KDXhg==
Date: Thu, 29 Nov 2018 15:12:20 +0000
Message-ID: <034601d487f5$9ec5fde0$4001a8c0@gateway.2wire.net>
References: <154265045985.5307.326266587898119448@ietfa.amsl.com> <007501d480f6$2e8400a0$4001a8c0@gateway.2wire.net> <7AEB3D6833318045B4AE71C2C87E8E173D09EB90@SJCEML521-MBB.china.huawei.com>
Accept-Language: en-GB, en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
x-clientproxiedby: LO2P123CA0012.GBRP123.PROD.OUTLOOK.COM (2603:10a6:600:a6::24) To VI1PR07MB4717.eurprd07.prod.outlook.com (2603:10a6:803:69::18)
authentication-results: spf=none (sender IP is ) smtp.mailfrom=ietfc@btconnect.com;
x-ms-exchange-messagesentrepresentingtype: 1
x-originating-ip: [86.139.215.184]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; VI1PR07MB4189; 6:xa4Z5Eb2n6cAaxU3IZzNKjPi9pd1LwzDdmtdQADavCWy04sIV6FKlWii5OdAMNzvmEMRruL8LTfhS0EcYj8Z+0KbOFovXkolRlCvvTGYhdX1TyofE7zUpHkyn7WX0kPbfONBGG87BhYyDTw6uZswHHvs2mYvrN8uX4H2j8AR8IxlJjGimoAtvakYAp5rTEQQKEK8NN7HJUQ4KOJYeM1VJHd03kG36QaIaDniP8+E5X1jRZs846WqQLfT5Ey+MQT8KlsvxZG22PgAQcEuhp85GruZO3BQJ+oUbf1wyGJAmHqH+2LzFit+rTGrEuYbG3RZ/IhuYfU4jkYks2tTh87Q94SUzSK4mb8H+MHRp9VLkDssiE6E8XvBAg1tNrKPwvXm25RjqL0bVnQHGSlaASe+UibAl80otugc3q0qOH/SZOjG3ssoBjl3DsdYf0CawXTxRxJF44oSC5A25MmUnLIUnw==; 5:YH8qfVeIYXEoLEAkxDazaMLg+xtFlXGyTATgfVkQ0csixfQZASx+8qyxV+AQ4xldtIaRmF9ICtzBgRq94yEGs6VtPdfpeAnVczrzNILmO7pEftJsUO2IS59CziIRHCcL88rrKr/EzlsyawCGMff7pVwijkNs5m1bocu1erXLoWs=; 7:P1ONzB3DC6HUPH55q3TY9INJviCrBeyaxZ1O7WT+nogQNw0M9JIq4/2xmI9TjGWQdCsLej6oT2jb8y+5+vnzZRDLD8hYtzrhqNUsYc7SVUovBGhMMNEEasuIT9hshWpbjpRyn2MreG0mKslH1Bst/w==
x-ms-office365-filtering-correlation-id: b2a17b5c-7f63-442b-05a3-08d6560d0e79
x-microsoft-antispam: BCL:0; PCL:0; RULEID:(2390098)(7020095)(4652040)(8989299)(4534185)(4627221)(201703031133081)(201702281549075)(8990200)(5600074)(711020)(2017052603328)(7193020); SRVR:VI1PR07MB4189;
x-ms-traffictypediagnostic: VI1PR07MB4189:
x-microsoft-antispam-prvs: <VI1PR07MB41899AC6F500FEE107ECF1EDA0D20@VI1PR07MB4189.eurprd07.prod.outlook.com>
x-ms-exchange-senderadcheck: 1
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040522)(2401047)(8121501046)(5005006)(10201501046)(93006095)(93001095)(3231453)(999002)(944501410)(4982022)(52105112)(3002001)(6055026)(148016)(149066)(150057)(6041310)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(20161123558120)(20161123560045)(201708071742011)(7699051)(76991095); SRVR:VI1PR07MB4189; BCL:0; PCL:0; RULEID:; SRVR:VI1PR07MB4189;
x-forefront-prvs: 0871917CDA
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(39860400002)(376002)(346002)(366004)(136003)(396003)(189003)(13464003)(199004)(51444003)(8676002)(86362001)(99286004)(6506007)(4001150100001)(386003)(53546011)(106356001)(9686003)(97736004)(8936002)(316002)(81166006)(102836004)(81156014)(6512007)(2906002)(6246003)(229853002)(6306002)(14496001)(1556002)(68736007)(26005)(186003)(110136005)(14444005)(5024004)(446003)(76176011)(84392002)(486006)(6436002)(476003)(52116002)(44736005)(7736002)(305945005)(2501003)(3846002)(25786009)(5660300001)(256004)(71190400001)(71200400001)(53936002)(6116002)(86152003)(966005)(105586002)(6486002)(478600001)(66066001)(14454004)(33896004)(21314003); DIR:OUT; SFP:1102; SCL:1; SRVR:VI1PR07MB4189; H:VI1PR07MB4717.eurprd07.prod.outlook.com; FPR:; SPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:0;
received-spf: None (protection.outlook.com: btconnect.com does not designate permitted sender hosts)
x-microsoft-antispam-message-info: bzoMxS8ELgtCnR+YS3MWrSvAK9sFNetSMe6B0oy7WHKUSi2KFCYXTl5Ndl7jr7asbYQHHlTjJ8Ap5sd7uW5RrE0a6utN8/PoVR08PoWBGaAv4Zseypb+3BLJO7IXYb3hFYnRbDwLxiFnUmx9DyyOOJWsY8TZaVBppao1JpmC4NWpu7aSsMbXUuBXfEVZ9xsgW5WfJPYHuos+k5Q1g/WM3rtAnOm5nvpzaXI/h/Hn9anlea5ldUXbR5ivdrM+RYM29stAF3+8PQOAaWAIVDH2RjflEY0/HAMi7ELg1JfM8uYBG9SnQs30POgaRfSHK2ZfD0eAxTba2tHXMtHuOyoii+CvZ2cMV5AmSfaiV22CYmM=
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="iso-8859-1"
Content-ID: <F3BC00C96AB95F4D87BC7957FA25AEAD@eurprd07.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: btconnect.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b2a17b5c-7f63-442b-05a3-08d6560d0e79
X-MS-Exchange-CrossTenant-originalarrivaltime: 29 Nov 2018 15:12:20.0332 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: cf8853ed-96e5-465b-9185-806bfe185e30
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR07MB4189
Archived-At: <https://mailarchive.ietf.org/arch/msg/ccamp/AHqP2X88HRTDOgW0G9WCMVDQCUc>
Subject: Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-17.txt
X-BeenThere: ccamp@ietf.org
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Discussion list for the CCAMP working group <ccamp.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/ccamp>, <mailto:ccamp-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/ccamp/>
List-Post: <mailto:ccamp@ietf.org>
List-Help: <mailto:ccamp-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/ccamp>, <mailto:ccamp-request@ietf.org?subject=subscribe>
X-List-Received-Date: Thu, 29 Nov 2018 15:12:25 -0000
Looking better, still needs a few tweaks IMHO. Title; varies A YANG Data Model for WSON Optical Networks A YANG Data Model for WSON (Wavelenghth Switched Optical Networks) neither of which seem quite right; perhaps A YANG Data Model for WSON (Wavelength Switched Optical Networks) reference "RFC XXX: A YANG Data Model for... /XXX/XXXX/ OLD import ietf-te-topology { prefix "tet"; } reference "draft-ietf-teas-yang-te-topo-18: YANG Data Model for Traffic Engineering (TE) Topologies"; NEW import ietf-te-topology { prefix "tet"; } reference "RFC YYYY: YANG Data Model for Traffic Engineering (TE) Topologies"; -- Note to RFC Editor please replace YYYY with the number assigned to draft-ietf-teas-yang-te-topo OLD import ietf-layer0-types { prefix "layer0-types"; reference "draft-ietf-teas-yang-te-types-01: Traffic Engineering Common YANG Types"; NEW import ietf-layer0-types { prefix "layer0-types"; reference "RFC ZZZZ Traffic Engineering Common YANG Types"; -- Note to RFC Editor please replace ZZZZ with the number assigned to draft-ietf-teas-yang-te-types IANA Considerations references RFC7950 which just says go read RFC6020; readers better helped by Module Names registry [RFC6020]: I-D References now has 6163 6205 7698 which is good but you need to add them to the body of the I-D lest you get unused references. Suggest a Section 3.1or 1.4 " This module references [RFC6163], [RFC7205], [RFC7698] ...." I like the way that draft-ietf-netmod-syslog-model handles this, titles and all. I see that you found the Security Considerations that calls for TLS! Tom Petch ----- Original Message ----- From: "Leeyoung" <leeyoung@huawei.com> To: "tom petch" <ietfc@btconnect.com>; <ccamp@ietf.org> Sent: Tuesday, November 27, 2018 4:07 PM Hi Tom, A bit more editing -- Added RFC8446 in the Security Section per https://trac.ietf.org/trac/ops/wiki/yang-security-guidelines Please see the diff https://www.ietf.org/rfcdiff?url2=draft-ietf-ccamp-wson-yang-17 and let us know if there is any further fix needed. Thanks. Young -----Original Message----- From: Leeyoung Sent: Monday, November 26, 2018 4:40 PM To: 'tom petch' <ietfc@btconnect.com>; ccamp@ietf.org Subject: RE: I-D Action: draft-ietf-ccamp-wson-yang-16.txt Hi Tom, Thanks for your comments. Please see the attached diff file that compares v16 and v17. I believe all your comments have been incorporated except RFC8446 (TLS3.1) reference. Please also see inline for my response to each of your comments. Let me know if you have any further comments. Best regards, Young -----Original Message----- From: tom petch [mailto:ietfc@btconnect.com] Sent: Tuesday, November 20, 2018 11:28 AM I think that this I-D needs some more work on it. - I have never seen a YANG module with so few Normative References; in fact, you cannot validly have one with so few. Recall that Normative means you need to understand them in order to understand the I-D and at the very least RFC7950 is in that category and I suspect most of the others are too (but not RFC8340). YL>> Agree. - IANA Considerations references RFC7950; all this RFC does is tell the reader to go to RFC6020 so that should be the reference here. YL>> Added. - The Security Considerations is not current - it lacks any mention of RESTCONF. This will require Normative References to RFC6241, RFC6242, RFC8341 amd RFC8446. YL>> Added RESRCONF reference. Normative References to RFC6241, RFC6242, and RFC8341 were added. TLS1.3 reference, RFC8446 was not added. I was not sure how this one relates to the draft. RFC6536 is referenced but not in the I-D References YL>> Added. The Abstract and Introduction fail to mention whether or not the modules are NMDA-compliant. YL>> Added/ The YANG module has a lower case 'must' but no indication in the body of the I-D as to how this should be interpreted. YL>> Added Section 1.1 Requirements Language section. The YANG module contains " ...the generic TE topology module [TE-TOPO] based on the information model developed in [RFC7446] and the two encoding documents [RFC7581] and [RFC7579] that developed protocol independent encodings based on [RFC7446]. " AYANG module must be plain text so XML-style references in the module itself are not permitted; all those [....] look suspiciously like references in which case, they must go. YL>> Removed all the references and its related text. The YANG module 'import' statements import ietf-network { prefix "nw"; } import ietf-network-topology { prefix "nt"; } import ietf-te-topology { prefix "tet"; } import ietf-layer0-types { prefix "layer0-types"; need to reference the RFC where these modules can be found, e.g. import ietf-network { prefix "nw"; reference "RFC 8345: A YANG Data Model for Network Topologies"; } YL>> Added reference statement for each imported module. The YANG module references RFC6163 RFC6205 RFC7698 none of which are in the References. 'reference' statements in the module must be plain text but the document must be referenced from the body of the I-D with a suitable entry in the References of the I-D. YL>> Added all of these in the reference section. module ietf-layer0-types lacks a YANG version statement. YL>> Added. Tom Petch ----- Original Message ----- From: <internet-drafts@ietf.org> To: <i-d-announce@ietf.org> Cc: <ccamp@ietf.org> Sent: Monday, November 19, 2018 6:00 PM Subject: I-D Action: draft-ietf-ccamp-wson-yang-16.txt > > A New Internet-Draft is available from the on-line Internet-Drafts directories. > This draft is a work item of the Common Control and Measurement Plane WG of the IETF. > > Title : A YANG Data Model for WSON Optical Networks > Authors : Young Lee > Dhruv Dhody > Aihua Guo > Victor Lopez > Daniel King > Filename : draft-ietf-ccamp-wson-yang-16.txt > Pages : 74 > Date : 2018-11-19 > > Abstract: > This document provides a YANG data model for the routing and > wavelength assignment (RWA) TE topology in wavelength switched > optical networks (WSONs). > > > > > The IETF datatracker status page for this draft is: > https://datatracker.ietf.org/doc/draft-ietf-ccamp-wson-yang/ > > There are also htmlized versions available at: > https://tools.ietf.org/html/draft-ietf-ccamp-wson-yang-16 > https://datatracker.ietf.org/doc/html/draft-ietf-ccamp-wson-yang-16 > > A diff from the previous version is available at: > https://www.ietf.org/rfcdiff?url2=draft-ietf-ccamp-wson-yang-16 > > > Please note that it may take a couple of minutes from the time of submission > until the htmlized version and diff are available at tools.ietf.org. > > Internet-Drafts are also available by anonymous FTP at: > ftp://ftp.ietf.org/internet-drafts/ > > _______________________________________________ > I-D-Announce mailing list > I-D-Announce@ietf.org > https://www.ietf.org/mailman/listinfo/i-d-announce > Internet-Draft directories: http://www.ietf.org/shadow.html or > ftp://ftp.ietf.org/ietf/1shadow-sites.txt
- [CCAMP] I-D Action: draft-ietf-ccamp-wson-yang-16… internet-drafts
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… tom petch
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Leeyoung
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Leeyoung
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… tom petch
- Re: [CCAMP] I-D Action: draft-ietf-ccamp-wson-yan… Leeyoung