Page MenuHomeVyOS Platform
Create Task
Maniphest T5089

Add unit test of config_diff
Closed, ResolvedPublicENHANCEMENT
Actions

Description

Add nosetest of config_diff function as sanity check. This is a missing useful check, and is needed before implementing planned extensions and simplifications to config_diff.

Details

Version
vyos-1.4
Is it a breaking change?
Perfectly compatible

Related Objects
Search...

StatusSubtypeAssignedTask
 ResolvedENHANCEMENTjestabroT5087 Add support for lexical ordering of nodes in config_tree
 ResolvedENHANCEMENTjestabroT5089 Add unit test of config_diff

Event Timeline

jestabro created this task.Mar 14 2023, 6:17 PM
jestabro triaged this task as Normal priority.
jestabro created this object in space S1 VyOS Public.
jestabro added a parent task: T5087: Add support for lexical ordering of nodes in config_tree.
jestabro added a comment.Mar 14 2023, 7:35 PM

PR for vyos1x-config:
https://github.com/vyos/vyos1x-config/pull/15

pasik subscribed.Mar 15 2023, 1:15 PM
jestabro added a comment.Mar 20 2023, 7:55 PM

Draft PR:
https://github.com/vyos/vyos-1x/pull/1900

jestabro mentioned this in rVYOSONEXc57494027bd3: configtree: T5089: sorting of nodes is now implemented on parsing config.Mar 29 2023, 3:26 PM
Restricted Repository Identity mentioned this in rVYOSONEX623dfc9d87d5: Merge pull request #1900 from jestabro/diff-test.Mar 29 2023, 7:12 PM
jestabro mentioned this in rVYOSONEXdccc689e0e66: configdiff: T5089: add unit test.Mar 29 2023, 7:12 PM
jestabro mentioned this in rVYOSONEXda43c4cbd495: configdiff: T5089: add optional arg ordered_values for unit tests.
jestabro mentioned this in rVYOSONEXe5758160c8a2: configdiff: T5089: add union of configtrees for unit test.
jestabro closed this task as Resolved.Mar 29 2023, 7:17 PM
jestabro moved this task from Open to Finished on the VyOS 1.4 Sagitta board.
jestabro mentioned this in rVYOSONEX4c1583cd1343: T5089: binding for configtree tree_union.Sep 10 2025, 1:16 PM
jestabro mentioned this in rVYOSONEXd2f1a6514300: T5089: add support for optional arg 'ordered_values' (ord_val).
Restricted Repository Identity mentioned this in rVYOSONEX137d7c3f977f: Merge pull request #9 from jestabro/tree-union.Sep 10 2025, 1:16 PM