search

openconfig / public

Repository for publishing OpenConfig models, documentation, and other material for the community.
Apache License 2.0
905 stars 656 forks source link

Yang Issue with openconfig-mpls #51

Closed mksriram closed 3 months ago

mksriram commented 7 years ago

openconfig-mpls imports openconfig-segment-routing with a prefix of oc-sr openconfig-mpls includes the sub-package openconfig-mpls-te which imports openconfig-mpls-sr whose prefix is also defined as oc-sr

In openconfig-mpls-sr, the prefix is defined as oc-mpls-sr

So can we please have the openconfig-mpls-te be modified to change the following import openconfig-mpls-sr { prefix oc-sr; } to import openconfig-mpls-sr { prefix oc-mpls-sr; }

and the subsequent references of oc-sr to oc-mpls-sr?

robshakir commented 7 years ago

Hi,

I think this is valid YANG. Per Section 5.1 of 6020, then the prefix used in the import statement is only locally significant to the importing entity. We can look at changing this if it causes tooling issues -- could you confirm what toolchain you're using that doesn't support this please?

Thanks, r.

mksriram commented 7 years ago

Sorry. I didnt realize that there was a question posted.

I am using the Opendaylight yang parser.(Boron)

Thanks Sriram

github-actions[bot] commented 5 months ago

This issue is stale because it has been open 180 days with no activity. If you wish to keep this issue active, please remove the stale label or add a comment, otherwise will be closed in 14 days.