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

provide support for load config of cli format along with json and xml format as save config is supported for all 3 formats #320

Closed
DeekshaBhandary opened this issue Mar 29, 2022 · 3 comments

Comments

@DeekshaBhandary
Copy link

No description provided.

olofhagsand added a commit that referenced this issue Apr 1, 2022
…l format as save config is supported for all 3 formats](#320)
@olofhagsand
Copy link
Member

Added, please verify

@pheller
Copy link
Contributor

pheller commented Apr 1, 2022

I've just tested the load replace method for cli and while it does work, sizable configurations take quite some time to load.

As an example, I have a ~ 6200 line configuration file (using OpenConfig network-instances, interfaces, system schemas) that takes ~ 9 minutes to complete, and during which clixon_backend consumes the majority of the CPU.

13990 root      20   0  145244  52796   4020 R  94.0  2.6   4:32.22 clixon_backend

@olofhagsand
Copy link
Member

I take this as a verification of the function and therefore close this.
Regarding the performance 6200 line / 9 minutes, can you please open a separate (non-functional) issue on that?

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

3 participants