Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Newer version of yanglint needed for YANG validation #2667

Closed
ietf-svn-bot opened this issue Dec 19, 2018 · 11 comments
Closed

Newer version of yanglint needed for YANG validation #2667

ietf-svn-bot opened this issue Dec 19, 2018 · 11 comments

Comments

@ietf-svn-bot
Copy link

keyword_sprint resolution_overtaken by events type_enhancement | by rrahman@cisco.com


So datatracker still has yanglint showing these errors for https://datatracker.ietf.org/doc/draft-ietf-netconf-subscribed-notifications:
yanglint 0.14.80: yanglint --verbose -p {rfclib} -p {draftlib} -p {tmplib} {model} -i:
err : The leafref leaf is config but refers to a non-config leaf. (/ietf-subscribed-notifications:subscriptions/subscription/target/stream/stream)

This is due to issue below which was fixed in yanglint in 0.16.59
CESNET/libyang#644

So we need datatracker to use a more recent version of yanglint.

Regards,
Reshad.


Issue migrated from trac:2667 at 2022-03-04 07:06:19 +0000

@ietf-svn-bot
Copy link
Author

@henrik@levkowetz.com commented


Libyang (which provides yanglint) is updated on the server during system maintenance when a new version is available in the OpenSUSE package repositories.

Currently, the version available under OpenSUSE is 0.14.80-1.1:

Information for package libyang:
--------------------------------
Repository     : @System                                 
Name           : libyang                                 
Version        : 0.14.80-1.1                             
Arch           : x86_64                                  
Vendor         : obs://build.opensuse.org/home:liberouter
Installed Size : 976.5 KiB                               
Installed      : Yes                                     
Status         : up-to-date                              
Source package : libyang-0.14.80-1.1.src                 
Summary        : Libyang library                         
Description    :                                         
    Libyang is YANG data modelling language parser and toolkit written (and providing API) in C.

Has 0.16.59 or later been packaged for OpenSUSE?

@ietf-svn-bot
Copy link
Author

@rrahman@cisco.com commented


I'm not very familiar with OpenSUSE but I see:

  1. https://software.opensuse.org/package/libyang has 0.16r3: https://github.com/CESNET/libyang/releases/tag/v0.16-1d6f2aa81b29043390617e6d02496b63bf78f1ab, but this is tumbleweed.

  2. https://software.opensuse.org/package/libyang0_16, same as 1 above.

Regards,
Reshad.

@ietf-svn-bot
Copy link
Author

@henrik@levkowetz.com commented


Umm. Ok, I'll ask the server admins, but in any case it seems to me that 0.16r3 << 0.16.59, so that upgrade would not help here in any case, no?

@ietf-svn-bot
Copy link
Author

@rrahman@cisco.com commented


Looking at CESNET/libyang@v0.16-1d6f2aa81b29043390617e6d02496b63bf78f1ab...master, it looks like the v0.16-1d6f2aa81b29043390617e6d02496b63bf78f1ab release contains everything up to and excluding 0.16.106. So 0.16.59 should be in that release.

@ietf-svn-bot
Copy link
Author

@henrik@levkowetz.com commented


Huh. That's one weird version numbering scheme.

Anyway, I've sent an email to the admins; will see what comes back.

@ietf-svn-bot
Copy link
Author

@henrik@levkowetz.com commented


I've got an answer, and in essence it says:

"Tubleweed" is the unstable distro, so it's not really relevant for the version we're on (42.3).

There is no libyang available in the official distro; the package we're running was built outside the official distro by "liberouter" (I don't know who that is, but you probably do). There has however not been built anything later than 0.14.81 for OpenSUSE 42.3, so at the moment we don't have an upgrade path to a later version of libyang.

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com changed priority from major to waiting

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com changed status from new to waiting

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com changed status from waiting to closed

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com set resolution to overtaken by events

@ietf-svn-bot
Copy link
Author

@rjsparks@nostrum.com commented


yanglint --version
yanglint SO 1.6.7

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant