Re: [Netconf] Shepherd review of draft-ietf-netconf-rfc7895bis
Mahesh Jethanandani <mjethanandani@gmail.com> Fri, 06 April 2018 16:21 UTC
Return-Path: <mjethanandani@gmail.com>
X-Original-To: netconf@ietfa.amsl.com
Delivered-To: netconf@ietfa.amsl.com
Received: from localhost (localhost [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id A3090124D37 for <netconf@ietfa.amsl.com>; Fri, 6 Apr 2018 09:21:12 -0700 (PDT)
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, FREEMAIL_FROM=0.001, RCVD_IN_DNSWL_NONE=-0.0001, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
Authentication-Results: ietfa.amsl.com (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.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 hYS1ZipLtGSB for <netconf@ietfa.amsl.com>; Fri, 6 Apr 2018 09:21:10 -0700 (PDT)
Received: from mail-pf0-x22b.google.com (mail-pf0-x22b.google.com [IPv6:2607:f8b0:400e:c00::22b]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id F34931200C5 for <netconf@ietf.org>; Fri, 6 Apr 2018 09:21:09 -0700 (PDT)
Received: by mail-pf0-x22b.google.com with SMTP id j2so1090618pff.10 for <netconf@ietf.org>; Fri, 06 Apr 2018 09:21:09 -0700 (PDT)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=1hiUKhK+TFprTTE2MuA6OuHXGZSFvXrbW/utaCunBzU=; b=gQucp0ntJvF//cq7IG1W64XyyWNSEmsjxPVVKSM8yRevVn9s8NSYDHT1lSUhADlRkk I4M3ySIVxj6E6Xf2mU9aE726cKZJjlhEiEvRIMo6l1Bi0V0QUPYknEF4cmBKOU+UbR4Z 0lfMPzPD2nNmw/JjdoelqrLpF12M/If2zOqu92tO1O9OpEEfs6Yoxg+7J9Qu43h+0DXP LFov+bTAi9MNGu513y3DRDA3KEyvwzBP6SeAieFZisWpWGYCmJ1RiWyXVPTvI2WWJcGT tqpR3fpmmWtO97e8MHOfaEi6LGno+PXeZSy3vjadn8aeayVeNqk0XdLh6ifLT314z4W1 Avug==
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:subject:from:in-reply-to:date:cc :content-transfer-encoding:message-id:references:to; bh=1hiUKhK+TFprTTE2MuA6OuHXGZSFvXrbW/utaCunBzU=; b=C0y5Go29Dj9+uvo3IQgNrwJ0MdO2tgRn/cBJwSu4AjfDNE+fUgYTRWy+rFbsVviHpw /1OiptkEvkjAADDa1C9NEWIc1OlHoJFIW3Yq6foC8IzLULp5hLQuKcgtIUUw7dmgg1eB vOiTlGG3nz9Llj8u/UcVeJfGX8/UcO6qLYSZFP+h4QIxW8q62PeP1t/Ux9B5oskowAud xjEARTkc+7ffCPgt6S8eaWjU6QpCBOrCUCKJqbVczMgdmgkgjDq1jiscAYVukgBfmTBU 01pdMimWhxIDPMD77hSrwCuvtljzTt8dsd7i4Nw83fXx/x0KRbBmdg8p9C+G26RSM9FH u+Ng==
X-Gm-Message-State: AElRT7EtvcgI4JdfZSZQKk+T75O5jyrxCbjQTgwG1FRRkhucUaVX8qUE 52frPEU4BBMhf0WHAvqn8oI=
X-Google-Smtp-Source: AIpwx4+6Nf9EFAZcn9ViLKGY+9ybPccB59Ssv/qAg0rZnjL9NZDYu37nYwbj00nlKiqEjX/U+8Vv+Q==
X-Received: by 10.101.78.131 with SMTP id b3mr18318408pgs.8.1523031669473; Fri, 06 Apr 2018 09:21:09 -0700 (PDT)
Received: from ?IPv6:2601:647:4700:1280:59c2:6e72:fd86:4fbf? ([2601:647:4700:1280:59c2:6e72:fd86:4fbf]) by smtp.gmail.com with ESMTPSA id y20sm19107172pfe.20.2018.04.06.09.21.08 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 06 Apr 2018 09:21:08 -0700 (PDT)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 11.3 \(3445.6.18\))
From: Mahesh Jethanandani <mjethanandani@gmail.com>
In-Reply-To: <20180406.145439.1617449037676400603.mbj@tail-f.com>
Date: Fri, 06 Apr 2018 09:21:07 -0700
Cc: netconf@ietf.org
Content-Transfer-Encoding: quoted-printable
Message-Id: <4B6A29A6-0396-4E14-A4CA-6CA107ED5A97@gmail.com>
References: <03CB4BEB-EAFC-45B3-BE40-B40720DF047D@gmail.com> <20180406.145439.1617449037676400603.mbj@tail-f.com>
To: Martin Bjorklund <mbj@tail-f.com>
X-Mailer: Apple Mail (2.3445.6.18)
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/vyWTRGQsQL11534njNLcPd9QQCI>
Subject: Re: [Netconf] Shepherd review of draft-ietf-netconf-rfc7895bis
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.22
Precedence: list
List-Id: Network Configuration WG mailing list <netconf.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/netconf>, <mailto:netconf-request@ietf.org?subject=unsubscribe>
List-Archive: <https://mailarchive.ietf.org/arch/browse/netconf/>
List-Post: <mailto:netconf@ietf.org>
List-Help: <mailto:netconf-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/netconf>, <mailto:netconf-request@ietf.org?subject=subscribe>
X-List-Received-Date: Fri, 06 Apr 2018 16:21:12 -0000
Martin, > On Apr 6, 2018, at 5:54 AM, Martin Bjorklund <mbj@tail-f.com> wrote: > > Hi, > > Mahesh Jethanandani <mjethanandani@gmail.com> wrote: >> Minor comments: >> >> Should the revision statement in the YANG module be updated to reflect >> the actual date of publication of the RFC, or will it remain >> 2018-02-21? > > Yes, the module has the usual comment: > > // RFC Ed.: update the date below with the date of RFC publication > // and remove this note. Ok. I got a note from one of the ADs, on one of my other drafts, that it helps the RFC editor if the authors identify where all in the draft the particular edit needs to be applied. > >> Not so minor a comment: >> >> 1) I note at least one use of the lower case (“must”) in the >> document. Suggest either shifting any lower-case requirements to upper >> case or using https://tools.ietf.org/html/rfc8174 >> <https://tools.ietf.org/html/rfc8174> for the definition of >> requirements language. A Gen-ART review will look at some of the RFC >> 2119 language more critically. > > I will update the 2119/8174 boilerplate text, and use "MUST" instead > of "must" where appropriate. > > But, in the Objective section we use a mix of "MUST" and "must". I > think the correct thing to do is use "must" in this section. Agree. The point was to include the text from RFC8174 is to clarify the use of lower case “must” in the document. > >> 2) A run to validate the example in the back of the document using >> yanglint revealed the following error. I could not validate if this >> was a tool issue, a tool use issue or a bug that needs looking at, as >> the error message is very cryptic and there are no references to any >> line numbers. >> >> err : Module "ietf-yang-library" in another revision already >> implemented. >> err : Module "ietf-yang-library" parsing failed. > > This is a well-known yanglint problem. The yanglint-people are aware > of this. Ok. > >> 3) A run of idnits reveals the following: >> >> Checking boilerplate required by RFC 5378 and the IETF Trust (see >> https://trustee.ietf.org/license-info): >> ---------------------------------------------------------------------------- >> >> No issues found here. >> >> Checking nits according to >> https://www.ietf.org/id-info/1id-guidelines.txt: >> ---------------------------------------------------------------------------- >> >> No issues found here. >> >> Checking nits according to https://www.ietf.org/id-info/checklist : >> ---------------------------------------------------------------------------- >> >> No issues found here. >> >> Miscellaneous warnings: >> ---------------------------------------------------------------------------- >> >> == Line 282 has weird spacing: '...mespace ine...' >> >> == Line 293 has weird spacing: '...mespace ine…' >> >> <I generally ignore this> >> >> == The document seems to lack the recommended RFC 2119 boilerplate, >> even if >> it appears to use RFC 2119 keywords. >> >> (The document does seem to have the reference to RFC 2119 which the >> ID-Checklist requires). >> >> <I see that you do reference RFC 2119, so do not know why this >> message. Either ways, see my other comment on the use of lower case >> “must” statement.> >> >> -- The document date (February 27, 2018) is 33 days in the past. Is this >> intentional? >> >> <Ignore> >> >> >> Checking references for intended status: Proposed Standard >> ---------------------------------------------------------------------------- >> >> (See RFCs 3967 and 4897 for information about using normative >> references >> to lower-maturity documents in RFCs) >> >> <If I am updating the document, I would update some of these >> references> >> <But check one reference in particular with (**)> >> >> == Outdated reference: draft-ietf-netmod-revised-datastores has been >> published as RFC 8342 >> >> ** Obsolete normative reference: RFC 6536 (Obsoleted by RFC 8341) >> >> <Any reason we are referencing RFC 6536 still?> >> >> == Outdated reference: draft-ietf-i2rs-yang-network-topo has been >> published >> as RFC 8345 >> >> == Outdated reference: A later version (-04) exists of >> draft-ietf-netconf-nmda-netconf-03 >> >> == Outdated reference: A later version (-03) exists of >> draft-ietf-netconf-nmda-restconf-02 >> >> == Outdated reference: draft-ietf-netmod-entity has been published as >> RFC 8348 >> >> == Outdated reference: draft-ietf-netmod-rfc7223bis has been published >> as RFC >> 8343 >> >> == Outdated reference: draft-ietf-netmod-rfc7277bis has been published >> as RFC >> 8344 >> >> == Outdated reference: draft-ietf-netmod-rfc8022bis has been published >> as RFC >> 8349 >> >> == Outdated reference: A later version (-09) exists of >> draft-ietf-netmod-schema-mount-08 >> >> == Outdated reference: draft-ietf-netmod-yang-tree-diagrams has been >> published as RFC 8340 >> >> >> Summary: 1 error (**), 0 flaws (~~), 13 warnings (==), 1 comment (--). > > I will update the references. > > Let me know if you want me to post an updated version of this document. Please do. Thanks > > > > /martin Mahesh Jethanandani mjethanandani@gmail.com
- [Netconf] Shepherd review of draft-ietf-netconf-r… Mahesh Jethanandani
- Re: [Netconf] Shepherd review of draft-ietf-netco… Martin Bjorklund
- Re: [Netconf] Shepherd review of draft-ietf-netco… Mahesh Jethanandani
- Re: [Netconf] Shepherd review of draft-ietf-netco… Martin Bjorklund