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

Support for module example in leaf description #5

Closed
bclaise opened this issue Jun 29, 2016 · 2 comments
Closed

Support for module example in leaf description #5

bclaise opened this issue Jun 29, 2016 · 2 comments

Comments

@bclaise
Copy link

bclaise commented Jun 29, 2016

I found a bug with the latest restconf draft https://tools.ietf.org/html/draft-ietf-netconf-restconf-14

There is an issue with the YANG module extraction
xym.py draft-ietf-netconf-restconf-14.txt
ERROR: 'draft-ietf-netconf-restconf-14.txt', Line 3930 - 'module' statement within another module
Created the following models::
example-ops.yang
example-actions.yang
example-jukebox.yang
example-mod.yang

Not an issue with the draft, but with the xym.py that can't deal with a module definition in a description:

container operations {
           description
             "Container for all operation resources
              (application/yang-operation),

              Each resource is represented as an empty leaf with the
              name of the RPC operation from the YANG rpc statement.
              For example, the 'system-restart' RPC operation defined
              in the 'ietf-system' module would be represented as
              an empty leaf in the 'ietf-system' namespace. This is
              a conceptual leaf, and will not actually be found in
              the module:

                 module ietf-system {                        <=====
                   leaf system-reset {
                     type empty;
                   }
                 }
@VinodSKumar
Copy link
Contributor

Fix is merged in the below commit

5c2d5a6

@bclaise
Copy link
Author

bclaise commented Oct 20, 2016

Fix works fine. Thanks.

@bclaise bclaise closed this as completed Oct 20, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants