[pve-devel] [PATCH v2 pve-network 7/9] api: extract function that creates the sdn directory.

[Stefan Lendl]

create_etc_interfaces_sdn_dir creates the /etc/pve/sdn directory.
This allows mocking in tests to prevent system fs access in tests

Signed-off-by: Stefan Lendl <s.le...@proxmox.com>
Reviewed-by: Max Carrara <m.carr...@proxmox.com>
Tested-by: Max Carrara <m.carr...@proxmox.com>
---
 src/PVE/API2/Network/SDN/Zones.pm | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/PVE/API2/Network/SDN/Zones.pm 
b/src/PVE/API2/Network/SDN/Zones.pm
index b09c9ad..35e2f7f 100644
--- a/src/PVE/API2/Network/SDN/Zones.pm
+++ b/src/PVE/API2/Network/SDN/Zones.pm
@@ -186,6 +186,10 @@ __PACKAGE__->register_method ({
        return &$api_sdn_zones_config($cfg, $param->{zone});
     }});
 
+sub create_etc_interfaces_sdn_dir {
+    mkdir("/etc/pve/sdn");
+}
+
 __PACKAGE__->register_method ({
     name => 'create',
     protected => 1,
@@ -207,7 +211,7 @@ __PACKAGE__->register_method ({
        my $opts = $plugin->check_config($id, $param, 1, 1);
 
        PVE::Cluster::check_cfs_quorum();
-       mkdir("/etc/pve/sdn");
+       create_etc_interfaces_sdn_dir();
 
        PVE::Network::SDN::lock_sdn_config(sub {
            my $zone_cfg = PVE::Network::SDN::Zones::config();
-- 
2.44.0



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel