Page MenuHomeVyOS Platform
Create Task
Maniphest T4235

Add config tree diff algorithm
Closed, ResolvedPublic
Actions

Description

The PRs below add a replacement for the legacy config diff algorithm in vyatta-cfg. Immediate applications will include an update to the 'load config' script that does not rely on the legacy 'loadFile'; extensions of that script in line with T3441; and python utils extending those of configdiff.py. In addition this adds a necessary step in the path towards replacing vyatta-cfg.

https://github.com/vyos/vyos1x-config/pull/2
https://github.com/vyos/libvyosconfig/pull/3
https://github.com/vyos/vyos-1x/pull/1213

Details

Difficulty level
Unknown (require assessment)
Version
vyos-1.4
Why the issue appeared?
Will be filled on close
Is it a breaking change?
Perfectly compatible
Issue type
Internal change (not visible to end users)

Related Objects
Search...

Event Timeline

jestabro triaged this task as Normal priority.Feb 9 2022, 3:04 AM
jestabro created this task.
jestabro created this object in space S1 VyOS Public.
jestabro updated the task description. (Show Details)Feb 9 2022, 3:20 AM
jestabro added a subtask: T3441: More intelligent config loading scripts.
jestabro changed the task status from Open to Needs testing.Feb 9 2022, 3:27 AM
jestabro closed this task as Resolved.Feb 10 2022, 3:58 PM
jestabro moved this task from Need Triage to Finished on the VyOS 1.4 Sagitta board.
jestabro reopened this task as Open.Feb 23 2022, 2:59 PM

Reopen to add some simplifications to the interface, inspired by T4260.

jestabro added a subtask: T4260: Extend vyos.configdict.node_changed() to support recursiveness.Feb 23 2022, 2:59 PM
pasik added a subscriber: pasik.Feb 23 2022, 3:40 PM
jestabro changed the task status from Open to Backport candidate.Mar 2 2022, 3:51 PM
jestabro changed the status of subtask T4260: Extend vyos.configdict.node_changed() to support recursiveness from Open to Needs testing.
jestabro changed the status of subtask T4260: Extend vyos.configdict.node_changed() to support recursiveness from Needs testing to Backport candidate.Mar 3 2022, 2:19 PM
jestabro added a project: VyOS 1.3 Equuleus.
jestabro added a subtask: T4295: Use config_tree instead of legacy loadFile in vyos-load-config.py.Mar 9 2022, 5:23 PM
jestabro changed the status of subtask T4295: Use config_tree instead of legacy loadFile in vyos-load-config.py from Backport candidate to Open.Mar 24 2022, 8:06 PM
jestabro mentioned this in T4382: Replacing legacy loadFile exposes missing steps in migration scripts and other errors.Apr 20 2022, 8:25 PM
jestabro closed this task as Resolved.Apr 26 2022, 11:23 AM
jestabro moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus board.
jestabro changed the status of subtask T4295: Use config_tree instead of legacy loadFile in vyos-load-config.py from Open to On hold.Jun 28 2022, 7:57 PM
jestabro added a subtask: T4491: Use empty string for internal name of root node of config_tree.Jun 28 2022, 8:17 PM
jestabro changed the status of subtask T4491: Use empty string for internal name of root node of config_tree from In progress to Backport candidate.Jun 29 2022, 2:47 PM
jestabro closed subtask T4491: Use empty string for internal name of root node of config_tree as Resolved.Jul 14 2022, 12:55 PM
dmbaturin closed subtask T4260: Extend vyos.configdict.node_changed() to support recursiveness as Resolved.Aug 14 2022, 6:16 PM
syncer edited projects, added VyOS 1.3 Equuleus (1.3.0); removed VyOS 1.3 Equuleus.Aug 29 2022, 12:15 PM
syncer moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus (1.3.0) board.