The possibility of replacing loadFile with config_diff was mentioned in the description in T4295. The generalization of 'diff functions' in the course of T5317 support the plan of defining batch set/delete operations of a cstore instance within a call to diff. One expects performance to be comparable, and the lack of useful error reporting in loadFile can be addressed.
Description
Description
Details
Details
- Difficulty level
- Hard (possibly days)
- Version
- vyos-1.4
- Why the issue appeared?
- Will be filled on close
- Is it a breaking change?
- Perfectly compatible
- Issue type
- Feature (new functionality)
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| In progress | FEATURE REQUEST | None | T3355 Remove all remaining legacy Vyatta code | ||
| Open | jestabro | T4316 Update save-config/load-config | |||
| Resolved | jestabro | T4235 Add config tree diff algorithm | |||
| Resolved N/A | FEATURE REQUEST | jestabro | T4295 Use config_tree instead of legacy loadFile in vyos-load-config.py | ||
| Resolved | ENHANCEMENT | jestabro | T5316 configtree: use a single pass of the diff algorithm | ||
| Resolved | ENHANCEMENT | jestabro | T5317 configtree: remove mutable references | ||
| In progress | ENHANCEMENT | jestabro | T5528 Replace legacy loadFile with config_diff | ||
| Resolved | ENHANCEMENT | jestabro | T5815 Add load_config module |
Event Timeline
Comment Actions
The working implementation is linked below; performance is not yet comparable to the legacy loadFile, though there are obvious points of optimization to be made. The general framework will be submitted for PR to support other development, with load_config as an illustrative example, but not yet a replacement for loadFile.
https://github.com/jestabro/vyos1x-adapter
https://github.com/vyos/vyos1x-config/compare/master...jestabro:vy-adapter
https://github.com/vyos/vyos-1x/compare/current...jestabro:vy-adapter
https://github.com/vyos/vyatta-cfg/compare/current...jestabro:vy-adapter
https://github.com/vyos/libvyosconfig/compare/master...jestabro:vy-adapter