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:47:45 +0200 |
commit | ead5198e69cf0e854127b43102fd0f1f58b38924 (patch) | |
tree | 4a72519398783de96fac42062252b5a923d4ba3c /scripts/override-default | |
parent | 52135032b6ab556db98b5f3fc2a3e6e2cbe9a607 (diff) | |
download | vyos-1x-ead5198e69cf0e854127b43102fd0f1f58b38924.tar.gz vyos-1x-ead5198e69cf0e854127b43102fd0f1f58b38924.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.
(cherry picked from commit 1f68b69328c961e450ac70b12f7bf073d70355d7)
Diffstat (limited to 'scripts/override-default')
0 files changed, 0 insertions, 0 deletions