search

openconfig / public

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

wrong prefix for import mpls types - should be oc-mpls-types #102

Closed sgrazian closed 2 weeks ago

sgrazian commented 6 years ago

line 10 of openconfig-isis-routing.yang - version 0.3.3

import openconfig-mpls-types { prefix oc-mplst; }

and line 11 of openconfig-segment-routing.yang - version 0.0.3

import openconfig-mpls-types { prefix "oc-mpls-t"; }

are not consistent with prefix defined in module openconfig-mpls-types - version 2.4.1 - line 8

prefix "oc-mpls-types";

Problem possibly apply for other modules as well

aashaikh commented 6 years ago

As we've noted in other similar issues, having the same prefix is not required in YANG -- this is likely a problem in the tooling you're using to parse / compile YANG modules.

3alexb commented 6 years ago

@robshakir pls comment

robshakir commented 6 years ago

Thanks Alex. Sorry I missed the response -- this is something that should be supported by YANG tooling. @sgrazian -- can you confirm what toolchain does not support this please?

github-actions[bot] commented 2 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.