[netmod] syslog-model-18 shepherd writeup issues

Kent Watsen <kwatsen@juniper.net> Fri, 05 January 2018 03:47 UTC

Return-Path: <kwatsen@juniper.net>
X-Original-To: netmod@ietfa.amsl.com
Delivered-To: netmod@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 260CD127333 for <netmod@ietfa.amsl.com>; Thu, 4 Jan 2018 19:47:28 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2
X-Spam-Level:
X-Spam-Status: No, score=-2 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_NONE=-0.0001, SPF_PASS=-0.001, URIBL_BLOCKED=0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=juniper.net
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 RI1sfejcifzz for <netmod@ietfa.amsl.com>; Thu, 4 Jan 2018 19:47:25 -0800 (PST)
Received: from mx0b-00273201.pphosted.com (mx0a-00273201.pphosted.com [208.84.65.16]) (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 A10BB127137 for <netmod@ietf.org>; Thu, 4 Jan 2018 19:47:25 -0800 (PST)
Received: from pps.filterd (m0108157.ppops.net [127.0.0.1]) by mx0a-00273201.pphosted.com (8.16.0.21/8.16.0.21) with SMTP id w053jPt2009501 for <netmod@ietf.org>; Thu, 4 Jan 2018 19:47:25 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=juniper.net; h=from : to : subject : date : message-id : content-type : content-id : content-transfer-encoding : mime-version; s=PPS1017; bh=oUaXXQSG3/kkhsiPHfion8WEqenHFLAxjFHH7M2EGdM=; b=Mwyxu/iqcyMNpZq4fSXWhbsmdp9/ttDky3r+Vv5A5q+chyVbdogx39n1eKeLeV7obmnu /oUXiEdQZJ0usU3PGNAYv0a4hA6wshYYEPv4bcTrODH1PeDGtTMFDomVuhWqA16QWbp0 /BKu28Mm53zBgyvj9gX2Je2Odn5904xwrW2wqJwn78psp0VAq//EBm+sewpJqarGsowG cmfw3HrH6gjElMGCl4eUyYqePQM8HI9oVpNe7QugSioEL0wjnWqS8UouO8/+EGZDlhP8 bWqJ1XxLcibLaUoPzeLrV3/NT/ltmaheIV8134F9+vzQNmZBwyFnnGnmd0qmo1/qaJFM Ag==
Received: from nam02-bl2-obe.outbound.protection.outlook.com (mail-bl2nam02lp0085.outbound.protection.outlook.com [207.46.163.85]) by mx0a-00273201.pphosted.com with ESMTP id 2fa1bhr11e-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT) for <netmod@ietf.org>; Thu, 04 Jan 2018 19:47:25 -0800
Received: from DM5PR05MB3484.namprd05.prod.outlook.com (10.174.240.147) by DM5PR05MB2793.namprd05.prod.outlook.com (10.168.175.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_CBC_SHA384_P256) id 15.20.407.1; Fri, 5 Jan 2018 03:47:23 +0000
Received: from DM5PR05MB3484.namprd05.prod.outlook.com ([10.174.240.147]) by DM5PR05MB3484.namprd05.prod.outlook.com ([10.174.240.147]) with mapi id 15.20.0407.000; Fri, 5 Jan 2018 03:47:23 +0000
From: Kent Watsen <kwatsen@juniper.net>
To: "netmod@ietf.org" <netmod@ietf.org>
Thread-Topic: syslog-model-18 shepherd writeup issues
Thread-Index: AQHThdfle+mgMO3MmESdKCsZf/n8Dw==
Date: Fri, 5 Jan 2018 03:47:23 +0000
Message-ID: <4F31BAEF-AAC8-43F5-93E8-13EFE1EBEF18@juniper.net>
Accept-Language: en-US
Content-Language: en-US
X-MS-Has-Attach:
X-MS-TNEF-Correlator:
user-agent: Microsoft-MacOutlook/f.20.0.170309
x-originating-ip: [66.129.241.10]
x-ms-publictraffictype: Email
x-microsoft-exchange-diagnostics: 1; DM5PR05MB2793; 7:MDX1emxIX1KPuWuk7Ia7S7uvFXqitWOzzCjd5yPz7EARJkU/sAmHrmH5jq5jAlnvKvZs90pwPpxUA23wXUigataOlwA0p4b8ypVrNXqqnfjqs18uKLOVi9voPellvcya0i546XSSpY/wGlmumSiRkibN0VB8+jnyhKrAGvWPFnzEDdz2a0167Vi/kuFyLXmX8lsXxEsEMltX7MM3e5OLJ4LHYDHqXlPILiL6M70oCfNGsAvMlz61B9WoCsu91fRe
x-ms-exchange-antispam-srfa-diagnostics: SSOS;
x-ms-office365-filtering-ht: Tenant
x-ms-office365-filtering-correlation-id: 17078e7c-08bb-4909-fb27-08d553ef079e
x-microsoft-antispam: UriScan:; BCL:0; PCL:0; RULEID:(48565401081)(4534020)(4602075)(4627115)(201703031133081)(201702281549075)(5600026)(4604075)(3008032)(2017052603307)(7153060); SRVR:DM5PR05MB2793;
x-ms-traffictypediagnostic: DM5PR05MB2793:
x-microsoft-antispam-prvs: <DM5PR05MB279305D4E5A25232BB2A7DBEA51C0@DM5PR05MB2793.namprd05.prod.outlook.com>
x-exchange-antispam-report-test: UriScan:;
x-exchange-antispam-report-cfa-test: BCL:0; PCL:0; RULEID:(6040470)(2401047)(5005006)(8121501046)(3231023)(944501075)(3002001)(10201501046)(93006095)(93001095)(6055026)(6041268)(20161123558120)(20161123560045)(20161123562045)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(6072148)(201708071742011); SRVR:DM5PR05MB2793; BCL:0; PCL:0; RULEID:(100000803101)(100110400095); SRVR:DM5PR05MB2793;
x-forefront-prvs: 05437568AA
x-forefront-antispam-report: SFV:NSPM; SFS:(10019020)(39860400002)(376002)(39380400002)(396003)(366004)(346002)(199004)(189003)(105586002)(6506007)(53376002)(2501003)(81166006)(81156014)(8676002)(6306002)(6512007)(6116002)(3846002)(59450400001)(1730700003)(5640700003)(478600001)(8936002)(33656002)(3660700001)(305945005)(36756003)(25786009)(7736002)(66066001)(6916009)(83506002)(106356001)(102836004)(53936002)(3280700002)(68736007)(2906002)(4001150100001)(5660300001)(6486002)(99286004)(2900100001)(2351001)(316002)(58126008)(97736004)(6436002)(966005)(86362001)(82746002)(77096006)(83716003)(14454004)(230783001); DIR:OUT; SFP:1102; SCL:1; SRVR:DM5PR05MB2793; H:DM5PR05MB3484.namprd05.prod.outlook.com; FPR:; SPF:None; PTR:InfoNoRecords; A:1; MX:1; LANG:en;
received-spf: None (protection.outlook.com: juniper.net does not designate permitted sender hosts)
x-microsoft-antispam-message-info: 7wwxU1FJJusXnVPmjRFx3x2YNJWBMU3pZgVU+3hAdlCqVyDlQu1b4EXF2ufUdZ8NV2Q2bpyitB0tKZxeX3PeIw==
spamdiagnosticoutput: 1:99
spamdiagnosticmetadata: NSPM
Content-Type: text/plain; charset="utf-8"
Content-ID: <D298A55F46D65441A47EACD483A74C4B@namprd05.prod.outlook.com>
Content-Transfer-Encoding: base64
MIME-Version: 1.0
X-OriginatorOrg: juniper.net
X-MS-Exchange-CrossTenant-Network-Message-Id: 17078e7c-08bb-4909-fb27-08d553ef079e
X-MS-Exchange-CrossTenant-originalarrivaltime: 05 Jan 2018 03:47:23.2629 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: bea78b3c-4cdb-4130-854a-1d193232e5f4
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR05MB2793
X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:, , definitions=2018-01-05_02:, , signatures=0
X-Proofpoint-Spam-Details: rule=outbound_spam_notspam policy=outbound_spam score=0 priorityscore=1501 malwarescore=0 suspectscore=0 phishscore=0 bulkscore=0 spamscore=0 clxscore=1011 lowpriorityscore=0 mlxscore=0 impostorscore=0 mlxlogscore=999 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.0.1-1711220000 definitions=main-1801050047
Archived-At: <https://mailarchive.ietf.org/arch/msg/netmod/wG8fKr8Q4KzSo0dOdOoCzzjkLm8>
Subject: [netmod] syslog-model-18 shepherd writeup issues
X-BeenThere: netmod@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: NETMOD WG list <netmod.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netmod>, <mailto:netmod-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netmod/>
List-Post: <mailto:netmod@ietf.org>
List-Help: <mailto:netmod-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netmod>, <mailto:netmod-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 05 Jan 2018 03:47:28 -0000

[Welcome to 2018!]


Hi Clyde,

In reviewing the -18 draft for Shepherd write-up, I found the following issues that I think need to be addressed before the document can be sent to Benoit for AD review:


1. I believe that there is a pending request from Tom on that has yet to be addressed: https://mailarchive.ietf.org/arch/msg/netmod/FkEzQeLJ31FiijFk-S-ojyafMdw

2. Idnits found the following.   Note, I'm aware that idnits, when run via the draft submission tool, doesn't show any issues, which is probably why you didn't fix these before. FWIW, I'm using the tool located at https://www.ietf.org/tools/idnits and do NOT select the "only do submission checks" box.

  Checking nits according to https://www.ietf.org/id-info/checklist :
  ----------------------------------------------------------------------------

  ** There is 1 instance of too long lines in the document, the longest one
     being 1 character in excess of 72.


  Checking references for intended status: Proposed Standard
  ----------------------------------------------------------------------------

     (See RFCs 3967 and 4897 for information about using normative references
     to lower-maturity documents in RFCs)

  == Missing Reference: 'RFC6021' is mentioned on line 374, but not defined

  ** Obsolete undefined reference: RFC 6021 (Obsoleted by RFC 6991)

  == Unused Reference: 'RFC6691' is defined on line 1446, but no explicit
     reference was found in the text

  == Unused Reference: 'RFC7895' is defined on line 1454, but no explicit
     reference was found in the text

  ** Downref: Normative reference to an Historic RFC: RFC 6587

  ** Downref: Normative reference to an Informational RFC: RFC 6691

3. In Section 4.1, [RFC xxxx] needs to be [I-D.ietf-netconf-keystore] and
   [RFC yyyy] needs to be [I-D.ietf-netconf-tls-client-server].

4. Also in Section 4.1, the reference statements for the ietf-keystore
   and ietf-tls-client modules are reversed.

5. This is more of a nit, but please fix the IANA Considerations section
   so it has two sections (i.e., 7.1 and 7.2), one for each registry that
   is updated.

6. This is definitely a nit, but in 7.1, "the the" --> "the".  also, something
   is wrong with your references in this section, as they're being rendered
   with XML bleeding thru (e.g., "]\>").  [PS: this is likely the root cause
   for the RFC7895 issue mentioned above].

7. In Section 7, the Registrant Contact should probably be "The IESG" (not 
   "The NETCONF WG of the IETF"), per RFC 3688, Section 4.

8. `rfcstrip` extracted "ietf-syslog.yang@2017-12-12.yang";, which has a
   superfluous ".yang" in it.  [check your <CODE BEGINS> line]

9. The examples in Section 5 do not need to define the "xmlns:syslog" prefix,
   as the prefix isn't used anywhere in the examples.

10. In Section 6, please remove the line-return after each name.  E.g., the entire
   section could be one paragraph...

11. In Section 9.1, [draft-ietf-netconf-keystore-04.txt] needs to be
    [I-D.ietf-netconf-keystore] and [draft-ietf-netconf-tls-client-server-05.txt]
    needs to be [I-D.ietf-netconf-tls-client-server].   BTW, all this is automatic
    if, in your XML, you put e.g. <?rfc include="reference.I-D.ietf-netconf-keystore"?>

12. In Appendix A.1, the module name should begin with "example-", per rfc6087bis
   Section 3.2.1.

13. Also in Appendix A.1, the namespace should follow one of the conventions
    defined in rfc6087bis Section 4.9 (e.g., http://example.com/ns/syslog-types)



Reviewing my comments from -17 (https://mailarchive.ietf.org/arch/msg/netmod/fail7CzcRCsIIY-CxhT3xXjkCvc)

1. mostly fixed, except the downref to RFC 6587 (see #1 above)
2. I see you made a change, but it's still not right (see #6 above)
3. fixed
4. fixed
5. fixed
6. not fixed (see #1 above)
7. not fixed, but this may be okay (what the tree diagrams draft say?)
8. fixed
9. not fixed (see #1 above)
10. fixed
11. not fixed (there MUST be normative references in Section 
12. not fixed (see #5 above)


Reviewing my comments from -15 (https://mailarchive.ietf.org/arch/msg/netmod/10lo41Ud4A3ZN11s-0gOfCe8NSE)

1. ok
2. better
3. fixed
4. better
5. not fixed (but I can live with it)
6. not fixed. from 8174, you should add the part "when, and only when, they appear in all capitals, as shown here."
7. fixed
8. fixed
9. fixed
10. not fixed
11. better
12. better
13. fixed
14. fixed
15. fixed
16. fixed
17. fine
18. not fixed
19. fixed
20. fixed
21. fixed
22. fine


Thanks,
Kent // Shepherd