openconfig / public

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

Some yang modules import other modules with a wrong prefix #168

Open jean-christophe-manciot opened 5 years ago

jean-christophe-manciot commented 5 years ago

in openconfig-acl.yang:

  import openconfig-packet-match { prefix oc-match; }

when openconfig-packet-match real prefix is oc-pkt-match:

module openconfig-packet-match {

  yang-version "1";

  // namespace
  namespace "http://openconfig.net/yang/header-fields";

  prefix "oc-pkt-match";
jean-christophe-manciot commented 5 years ago

Same issue with openconfig-pf-forwarding-policies.yang which imports openconfig-packet-match with yet another prefix oc-pmatch.

jean-christophe-manciot commented 5 years ago

As a reminder from RFC 6020:

When used inside the "module" statement, the "prefix" statement
   defines the prefix to be used when this module is imported.
jean-christophe-manciot commented 5 years ago

Same issue with some yang modules which import "openconfig-network-instance" with the prefix "oc-ni" instead of the real prefix "oc-netinst". For instance, in openconfig-bfd.yang:

  import openconfig-network-instance { prefix "oc-ni"; }

which should be:

  import openconfig-network-instance { prefix "oc-netinst"; }
github-actions[bot] commented 2 weeks 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.