diff options
| author | Christian Poessinger <christian@poessinger.com> | 2021-05-15 12:44:04 +0200 |
|---|---|---|
| committer | Christian Poessinger <christian@poessinger.com> | 2021-05-15 12:44:04 +0200 |
| commit | 1f68b69328c961e450ac70b12f7bf073d70355d7 (patch) | |
| tree | f03829cac6963d0b5d4725ff65e7447f1ebab12c /src/validators/file-exists | |
| parent | d7f0cbdc102a1186cec80d0ebf29b8f4ef415435 (diff) | |
| download | vyos-1x-1f68b69328c961e450ac70b12f7bf073d70355d7.tar.gz vyos-1x-1f68b69328c961e450ac70b12f7bf073d70355d7.zip | |
dhcp-server: T3544: fix FileNotFoundError on first invokation
The problem of using the move() operation over render() is that render will
silently create the directory tree in the background and move() does not.
This means that on first boot when /run/dhcp-server does not exist, move will
fail with a FileNotFoundError.
Instead of using move() we render() the configuration two times, one for
validating it via dhcpd -t and the other time to really apply it to the service.
The performance impact should be little as the config should still be cached in
the system RAM.
Diffstat (limited to 'src/validators/file-exists')
0 files changed, 0 insertions, 0 deletions