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

prevent clixon-restconf@2021-05-20.yang module from loading #318

Closed
eduardpo opened this issue Mar 21, 2022 · 5 comments
Closed

prevent clixon-restconf@2021-05-20.yang module from loading #318

eduardpo opened this issue Mar 21, 2022 · 5 comments

Comments

@eduardpo
Copy link

Hi @olofhagsand,

We are trying to prevent clixon-restconf module from loading by clixon at all,
Which will presumably allow us to completely hide restconf/netconf access of restconf configuration from user,
In this regard i would like to ask you few questions if I may:

  1. does the effort make sense and what could be the impact on the restconf/netconf system?
  2. what's the prefered way to accomplish the goal?

Please advise,
Ed

@olofhagsand
Copy link
Member

Should work by just:

./configure --without-restconf

See also https://clixon-docs.readthedocs.io/en/latest/restconf.html#installation
If that does not work it is an error.

@eduardpo
Copy link
Author

Hi @olofhagsand,

Thanks for reply,
According to the manual ./configure --without-restconf means - # Do not build restconf,
Which is not that we want to achieve, we do use restconf protocol in our setup.
As I understand, clixon-restconf@2021-05-20.yang module is responsible for configuration of restconf daemon/protocol functionality?
Does it make sense not to load this particular module and continue to fully use restconf functionality?

Thanks,
Ed

@eduardpo eduardpo changed the title prevent clixon-restconf module from loading prevent clixon-restconf@2021-05-20.yang module from loading Mar 22, 2022
@olofhagsand
Copy link
Member

I understand, makes sense.
Is it only in the CLI or do you want to hide it also from netconf/restconf for configuration?

@olofhagsand
Copy link
Member

Changing it to a change request: make it optional to load clixon-restconf.yang

@eduardpo
Copy link
Author

I understand, makes sense. Is it only in the CLI or do you want to hide it also from netconf/restconf for configuration?

We want to hide it from all frontend's - cli, netconf, restconf...

olofhagsand added a commit that referenced this issue Mar 29, 2022
  * Instead of always loading it, load it to datastore YANGs only if `CLICON_BACKEND_RESTCONF_PROCESS` is `true`
*
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