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

openconfig-isis-routing.yang: missing rt-admin-config in multi-topology config container #78

Closed nikhil2553 closed 2 weeks ago

nikhil2553 commented 7 years ago

under isis-mt-list we have multi-topology container which contains config and state container. Looks like we are missing rt-admin-config (enabled) in config container. It is present in state container though.

nikhil2553 commented 6 years ago

what is the need of afi-name and safi-name under multi-topology container as multi-topology container itself is under afi-safi list.

robshakir commented 6 years ago

Thanks for catching the missing rt-admin-config issue. This will be fixed in a pending release.

I also agree that the dual-specification of the AFI and SAFI identifier is a bug. I'll amend this and suggest a fix to the team.

Thanks! r.

nikhil2553 commented 6 years ago

Thanks

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.