The current manner of calling dependent conf_mode scripts (nat.py, policy-route.py) from firewall.py does not work under vyos-configd, as the called scripts are not provided a usable Config object. Provide a workaround pending the completion of a general solution in T4820.
Description
Description
Details
Details
- Version
- vyos-1.4
- Is it a breaking change?
- Perfectly compatible
- Issue type
- Bug (incorrect behavior)
Related Objects
Related Objects
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | ENHANCEMENT | jestabro | T4820 Support for inter-config-mode script dependencies | ||
| Resolved | BUG | jestabro | T4821 Correct calling of config mode script dependencies from firewall.py |
- Mentioned In
- T4859: Correct calling of config mode script dependencies from http-api.py
T4847: Correct calling of config mode script dependencies from pki.py
T4845: Add smoketest to detect cycles in config-mode script dependency calls
rVYOSONEX3d4dedd67097: graphql: T4544: use load_as_module from vyos.util
rVYOSONEXc29f1be7372d: firewall: T4821: add utility to load script as module
rVYOSONEX048bd27676c0: firewall: T4821: add support for adding conf_mode script dependencies
rVYOSONEXb8bda7c8d54f: firewall: T4821: correct calling of conf_mode script dependencies
rVYOSONEX97056cad1866: Merge pull request #1662 from jestabro/config-script-dependency - Mentioned Here
- T4820: Support for inter-config-mode script dependencies
Event Timeline
jestabro renamed this task from Fix calling of config mode script dependencies from firewall.py to Correct calling of config mode script dependencies from firewall.py.Nov 17 2022, 12:43 AM2022-11-17 00:43:20 (UTC+0)
jestabro changed the task status from Open to Needs testing.Nov 17 2022, 1:09 AM2022-11-17 01:09:37 (UTC+0)
Restricted Repository Identity mentioned this in rVYOSONEX97056cad1866: Merge pull request #1662 from jestabro/config-script-dependency.Nov 18 2022, 1:42 PM2022-11-18 13:42:22 (UTC+0)