Re: [Netconf] I-D Action: draft-ietf-netconf-subscribed-notifications-19.txt

Carl Moberg <calle@tail-f.com> Thu, 20 December 2018 00:13 UTC

Return-Path: <calle@tail-f.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 E4F75129AB8 for <netconf@ietfa.amsl.com>; Wed, 19 Dec 2018 16:13:17 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -1.901
X-Spam-Level:
X-Spam-Status: No, score=-1.901 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, SPF_PASS=-0.001] autolearn=ham autolearn_force=no
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 Ggvzr1erzGTH for <netconf@ietfa.amsl.com>; Wed, 19 Dec 2018 16:13:16 -0800 (PST)
Received: from mail.tail-f.com (mail.tail-f.com [46.21.102.45]) by ietfa.amsl.com (Postfix) with ESMTP id E9F041200D7 for <netconf@ietf.org>; Wed, 19 Dec 2018 16:13:15 -0800 (PST)
Received: from ams-camoberg-nitro7.cisco.com (unknown [173.38.220.46]) by mail.tail-f.com (Postfix) with ESMTPSA id BF5261AE0446; Thu, 20 Dec 2018 01:13:11 +0100 (CET)
Content-Type: text/plain; charset="utf-8"
Mime-Version: 1.0 (Mac OS X Mail 12.2 \(3445.102.3\))
From: Carl Moberg <calle@tail-f.com>
In-Reply-To: <B6F8FE35-B713-4EB1-9E9A-88873126908D@tail-f.com>
Date: Thu, 20 Dec 2018 01:13:10 +0100
Cc: "netconf@ietf.org" <netconf@ietf.org>, Martin Björklund <mbj@tail-f.com>, lhotka@nic.cz
Content-Transfer-Encoding: quoted-printable
Message-Id: <3E762C43-7F3A-4EB3-A6CB-22006950D57B@tail-f.com>
References: <154515596361.5411.7153657725124836233@ietfa.amsl.com> <20181219.145202.1027346732573212384.mbj@tail-f.com> <91e0bc7bee2e4b56bc968565fa43d78c@XCH-RTP-013.cisco.com> <20181219.204027.1134533539828819405.mbj@tail-f.com> <3C140E32-5CD1-4F13-A025-F22ABAE54CBB@tail-f.com> <7D59A993-EE20-429C-B5A0-C18F8B405EE8@cisco.com> <B6F8FE35-B713-4EB1-9E9A-88873126908D@tail-f.com>
To: "Reshad Rahman (rrahman)" <rrahman@cisco.com>
X-Mailer: Apple Mail (2.3445.102.3)
Archived-At: <https://mailarchive.ietf.org/arch/msg/netconf/KL3cPZK4XDsAsOoAUBIkn9aWC14>
Subject: Re: [Netconf] I-D Action: draft-ietf-netconf-subscribed-notifications-19.txt
X-BeenThere: netconf@ietf.org
X-Mailman-Version: 2.1.29
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: Thu, 20 Dec 2018 00:13:18 -0000

 Actually, and to put a finer point on the below. I do extract YANG modules from drafts, but only store those that pass extraction and validation. And it turns out that validation of the module extracted from https://www.ietf.org/id/draft-ietf-netmod-schema-mount-12.txt fails because the file name in the CODE BEGINS marker is missing the “.yang” suffix which violates https://tools.ietf.org/html/rfc6087#section-3.1.


> On Dec 20, 2018, at 12:57 AM, Carl Moberg <calle@tail-f.com> wrote:
> 
> Reshad,
> 
> Yes, so the YANG modules in the draft tries to import the 'ietf-yang-schema-mount’ and 'ietf-network-instance' YANG modules which are not available in the repo in the validator. The reason why we don’t keep them in the repo is that the specification that defines those modules (draft-ietf-netmod-schema-mount, and draft-ietf-rtgwg-ni-model respectively) are not published yet and we only keep published YANG modules.
> 
> The easiest way forward is probably to extract the YANG modules from your draft (perhaps using the https://github.com/xym-tool/xym tool), zip up your module with the imported modules and validate them using the “Upload multiple YANG files or a zip archive” option.
> 
> I am considering storing draft-level YANG files (they are many and broken :-) in the validator repo as well. Maybe time to revisit that.
> 
>> On Dec 20, 2018, at 12:46 AM, Reshad Rahman (rrahman) <rrahman@cisco.com> wrote:
>> 
>> Hi Carl,
>> 
>> Yangvalidator.org has the following errors for -20, pyang is fine though.
>> 
>> Regards,
>> Reshad.
>> 
>> Confdc Output
>> ietf-network-instance@2018-03-20.yang:18: error: module 'ietf-yang-schema-mount' not found
>> yanglint Validation
>> err : Data model "ietf-yang-schema-mount" not found.
>> err : Importing "ietf-yang-schema-mount" module into "ietf-network-instance" failed.
>> err : Module "ietf-network-instance" parsing failed.
>> err : Importing "ietf-network-instance" module into "ietf-subscribed-notifications" failed.
>> err : Module "ietf-subscribed-notifications" parsing failed.
>> 
>> 
>> On 2018-12-19, 6:37 PM, "Netconf on behalf of Carl Moberg" <netconf-bounces@ietf.org on behalf of calle@tail-f.com> wrote:
>> 
>> 
>> 
>>> On Dec 19, 2018, at 8:40 PM, Martin Bjorklund <mbj@tail-f.com> wrote:
>>> 
>>> "Eric Voit (evoit)" <evoit@cisco.com> wrote:
>>>> Thanks Martin,
>>>> 
>>>> 
>>>> 
>>>> I have the version addressing the double quotes here:
>>>> 
>>>> https://github.com/netconf-wg/rfc5277bis/blob/master/draft-ietf-netconf-subscribed-notifications-20.xml
>>>> 
>>>> 
>>>> 
>>>> Before I upload this -20, I want to see if you have concerns with the Pyang Validation issue shown below.   Reshad mentioned that other users have seen it.  Is this something which should stop me from posting?
>>>> 
>>> 
>>> You get this error b/c you are running an old version of pyang that
>>> isn't compatible with the (newer) version of pip on your system.  I
>>> suggest you upgrade pyang.
>>> 
>> 
>>    Yup, thanks. Upgraded and yangvalidator.org now works (better).
>> 
>>   _______________________________________________
>>   Netconf mailing list
>>   Netconf@ietf.org
>>   https://www.ietf.org/mailman/listinfo/netconf
>> 
>> 
> 
> _______________________________________________
> Netconf mailing list
> Netconf@ietf.org
> https://www.ietf.org/mailman/listinfo/netconf