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

OpenConfig Path Compression Support #274

Closed
wyltk-pfleming opened this issue Oct 9, 2021 · 3 comments
Closed

OpenConfig Path Compression Support #274

wyltk-pfleming opened this issue Oct 9, 2021 · 3 comments

Comments

@wyltk-pfleming
Copy link

wyltk-pfleming commented Oct 9, 2021

Requesting support for path compression as documented here

OpenConfig path are repetitive for human CLI consumption due to levels of hierarchy being introduced to the model.

Inefficient behavior:

  • Data values (leaf nodes) are mirrored in a config and state container
  • YANG list nodes are enclosed in a container, which they are the sole child of.

Utilizing the above referenced method
/interfaces/interface/subinterfaces/subinterface/ipv4/addresses/address becomes /interface/subinterface/ipv4/address. /interfaces/interface/subinterfaces/subinterface/config/enabled becomes /interface/subinterface/enabled. /interfaces/interface/subinterfaces/subinterface/state/oper-status becomes /interface/subinterface/oper-status.

I believe this should only apply at the CLI layer to allow compatibility with existing tools that expect OpenConfig paths.

@olofhagsand
Copy link
Member

using CLICON_CLI_GENMODEL=HIDE already takes care of the second requirement

@wyltk-pfleming
Copy link
Author

This looks complete with PR276 awaiting review from @olofhagsand

@olofhagsand
Copy link
Member

Complete closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants