[pve-devel] [PATCH pve-network 2/8] refactor(dnsmasq): extract systemctl_service function
Stefan Lendl
s.lendl at proxmox.com
Wed Jan 3 16:37:49 CET 2024
inside Dnsmasq.pm call systemctl services via a sub to allow mocking in
tests.
Signed-off-by: Stefan Lendl <s.lendl at proxmox.com>
---
src/PVE/Network/SDN/Dhcp/Dnsmasq.pm | 18 ++++++++++++------
1 file changed, 12 insertions(+), 6 deletions(-)
diff --git a/src/PVE/Network/SDN/Dhcp/Dnsmasq.pm b/src/PVE/Network/SDN/Dhcp/Dnsmasq.pm
index 2844943..f9f1c39 100644
--- a/src/PVE/Network/SDN/Dhcp/Dnsmasq.pm
+++ b/src/PVE/Network/SDN/Dhcp/Dnsmasq.pm
@@ -101,7 +101,7 @@ sub add_ip_mapping {
}
my $service_name = "dnsmasq\@$dhcpid";
- PVE::Tools::run_command(['systemctl', 'reload', $service_name]) if $reload;
+ systemctl_service('reload', $service_name) if $reload;
#update lease as ip could still be associated to an old removed mac
my $bus = Net::DBus->system();
@@ -163,6 +163,12 @@ sub configure_vnet {
);
}
+sub systemctl_service {
+ my ($action, $service) = @_;
+
+ PVE::Tools::run_command(['systemctl', $action, $service]);
+}
+
sub before_configure {
my ($class, $dhcpid) = @_;
@@ -250,9 +256,9 @@ sub after_configure {
my $service_name = "dnsmasq\@$dhcpid";
- PVE::Tools::run_command(['systemctl', 'reload', 'dbus']);
- PVE::Tools::run_command(['systemctl', 'enable', $service_name]);
- PVE::Tools::run_command(['systemctl', 'restart', $service_name]);
+ systemctl_service('reload', 'dbus');
+ systemctl_service('enable', $service_name);
+ systemctl_service('restart', $service_name);
}
sub before_regenerate {
@@ -260,8 +266,8 @@ sub before_regenerate {
return if !assert_dnsmasq_installed($noerr);
- PVE::Tools::run_command(['systemctl', 'stop', "dnsmasq@*"]);
- PVE::Tools::run_command(['systemctl', 'disable', 'dnsmasq@']);
+ systemctl_service('stop', "dnsmasq@*");
+ systemctl_service('disable', 'dnsmasq@');
}
sub after_regenerate {
--
2.42.0
More information about the pve-devel
mailing list