The config loader should be updated to python3, and take advantage of configtree to pass set commands to cli-shell-api, instead of relying on Vyatta::Config. Note that the merge logic was separated out of the config loader script in T1397.
Description
Description
Details
Details
- Difficulty level
- Unknown (require assessment)
- Version
- -
- Why the issue appeared?
- Will be filled on close
Related Objects
Related Objects
- Mentioned Here
- T1397: Rewrite the config merge script
Event Timeline
Comment Actions
The rewrite has been complete in the fork for some time, but is on hold pending: (1) consider taking advantage of the functionality in https://github.com/vyos/vyos-1x/blob/current/python/vyos/configsession.py to avoid duplication of code (2) ongoing work on options to replace the python list manipulations in the current rewrite. Regarding the latter point, it was expected that there would be a performance hit in moving away from the legacy C++ backend (via Perl bindings), and the cost was considered worthwhile in order to move forward; one data point is a performance hit of .3 on a 17k line config file, which was not bad, considering.