Page MenuHomeVyOS Platform

Use empty string for internal name of root node of config_tree
Closed, ResolvedPublicBUG

Description

There had been a convention of using 'root' as the internal name of the root node of the config_tree data structure, however, the config diff algorithm does a naive check against node name, which will collide with any actual node named 'root' (for example 'set pki ca root ...'). Instead, we will use the empty string as the name entry of the root node, as no 'actual' node can have an empty name.

Details

Version
vyos-1.4
Is it a breaking change?
Perfectly compatible
Issue type
Internal change (not visible to end users)

Revisions and Commits

Event Timeline

jestabro changed the task status from Open to In progress.
jestabro triaged this task as Normal priority.
jestabro created this object in space S1 VyOS Public.
jestabro renamed this task from Use empty string for internal name of root of config_tree to Use empty string for internal name of root node of config_tree.Jun 28 2022, 8:17 PM
jestabro moved this task from Open to Finished on the VyOS 1.4 Sagitta board.
Restricted Repository Identity added a commit: Restricted Diffusion Commit.Jul 1 2022, 8:09 PM
jestabro changed the task status from Unknown Status to Resolved.Jul 14 2022, 12:55 PM
jestabro moved this task from Need Triage to Finished on the VyOS 1.3 Equuleus board.