Instead of generating the interface configuration commands by hand, make use of vyos.ifconfig like it is done in bridge, loopback, dummy and soon bonding interface.
While doing this transition it should also be made use of the VYOS_TAGNODE_VALUE environment variable so the interface-wireguard.py script is only run for the specified tag node and not on all tag nodes.
This results in only the interface which had changes in the CLI will be changed. All other WireGuard interfaces are not processed.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| In progress | FEATURE REQUEST | None | T3355 Remove all remaining legacy Vyatta code | ||
| Resolved | FEATURE REQUEST | None | T1579 Rewrite all interface types in new XML/Python style | ||
| Resolved | FEATURE REQUEST | c-po | T1557 Create generic abstraction for configuring interfaces e.g. IP address | ||
| Resolved | FEATURE REQUEST | hagbard | T1628 Adopt WireGuard configuration script to new vyos.ifconfig class |
That will be a complete rewrite, since the interface name is now readable via VYOS_TAGNODE_VALUE, that affects get_config() quite a lot and will reduce the number of code line significantly. The flip side of the coin is, that the current code was running pretty reliable, so I will release small updates while adopting to see if I break anything configure outside of my test environment.