search

clicon / clixon

YANG-based toolchain including NETCONF and RESTCONF interfaces and an interactive CLI
http://www.clicon.org/
Other
216 stars 71 forks source link

Yang schema mount fixes #515

Closed colinhusu closed 5 months ago

colinhusu commented 5 months ago

While trying out clixon's Yang schema mount support, I discovered and fixed some issues. See the individual commit messages for details.

olofhagsand commented 5 months ago

Good initiative, the yang schema mount as far as I know, is only used in the clixon controller project. It would be good with more usage.

colinhusu commented 5 months ago

The "cli_dbxml(): Fix handling of mountpoints" patch had an error, it didn't handle the case where more than one api_path_fmt argument was passed in via argv, which made test_autocli_grouping.sh fail. I have adjusted the patch to properly handle that case.

Good initiative, the yang schema mount as far as I know, is only used in the clixon controller project. It would be good with more usage.

FYI, we have a relatively simple use-case for yang schema mount right now. We mount an external yang module inside of an empty presence container from our own yang module. The yang module to be mounted is known at build time and not determined dynamically.

olofhagsand commented 5 months ago

Looks good, merged, thanks.

FYI, we have a relatively simple use-case for yang schema mount right now. We mount an external yang module inside of an empty presence container from our own yang module. The yang module to be mounted is known at build time and not determined dynamically.

Do you have more info available on your project / how it uses mount-points?