summaryrefslogtreecommitdiff
path: root/interface-definitions/include
diff options
context:
space:
mode:
authorChristian Poessinger <christian@poessinger.com>2022-04-18 19:24:52 +0200
committerChristian Poessinger <christian@poessinger.com>2022-04-18 19:24:52 +0200
commit1a1094c28e32c3d6d072cf14a38aa631d51b8aee (patch)
treefd652d63f5d5ee0d7dedb087e6255c8b224eb699 /interface-definitions/include
parentc8da5866a79189244549d20837089924f0355e6e (diff)
downloadvyos-1x-1a1094c28e32c3d6d072cf14a38aa631d51b8aee.tar.gz
vyos-1x-1a1094c28e32c3d6d072cf14a38aa631d51b8aee.zip
vyos.configdict(): T4369: bugfix - execution order in leaf_node_changed()
Commit c685c0f7 ("vyos.configdict(): T4369: leaf_node_changed() must return True when node is added") added a code path then a node was newly added to the CLI. Unfortunately it turned out that this introduced a regression: File "/usr/lib/python3/dist-packages/vyos/ifconfig/wireguard.py", line 230, in update super().update(config) File "/usr/lib/python3/dist-packages/vyos/ifconfig/interface.py", line 1428, in update for addr in list_diff(config['address_old'], new_addr): File "/usr/lib/python3/dist-packages/vyos/configdict.py", line 105, in list_diff return [item for item in first if item not in second] TypeError: 'bool' object is not iterable The execution order of the if statements is essential and the new check was moved to the bottom to not interfere with the existing logic.
Diffstat (limited to 'interface-definitions/include')
0 files changed, 0 insertions, 0 deletions