create /etc/pve/sdn directory

[pve-network.git] / PVE / API2 / Network / SDN / Controllers.pm

diff --git a/PVE/API2/Network/SDN/Controllers.pm b/PVE/API2/Network/SDN/Controllers.pm
index 70ecd60de9f21c17532ea1cdda5316be8230625c..e16619c6862e430751807ca88dd37f8c0b7cd7a7 100644 (file)
--- a/PVE/API2/Network/SDN/Controllers.pm
+++ b/PVE/API2/Network/SDN/Controllers.pm
@@ -132,20 +132,24 @@ __PACKAGE__->register_method ({
        my $plugin = PVE::Network::SDN::Controllers::Plugin->lookup($type);
        my $opts = $plugin->check_config($id, $param, 1, 1);
 
+        # create /etc/pve/sdn directory
+        PVE::Cluster::check_cfs_quorum();
+        mkdir("/etc/pve/sdn");
+
         PVE::Network::SDN::Controllers::lock_sdn_controllers_config(
            sub {
 
-               my $cfg = PVE::Network::SDN::Controllers::config();
+               my $controller_cfg = PVE::Network::SDN::Controllers::config();
 
                my $scfg = undef;
-               if ($scfg = PVE::Network::SDN::Controllers::sdn_controllers_config($cfg, $id, 1)) {
+               if ($scfg = PVE::Network::SDN::Controllers::sdn_controllers_config($controller_cfg, $id, 1)) {
                    die "sdn controller object ID '$id' already defined\n";
                }
 
-               $cfg->{ids}->{$id} = $opts;
-               $plugin->on_update_hook($id, $cfg);
+               $controller_cfg->{ids}->{$id} = $opts;
+               $plugin->on_update_hook($id, $controller_cfg);
 
-               PVE::Network::SDN::Controllers::write_config($cfg);
+               PVE::Network::SDN::Controllers::write_config($controller_cfg);
 
            }, "create sdn controller object failed");
 
@@ -194,11 +198,11 @@ __PACKAGE__->register_method ({
         PVE::Network::SDN::Controllers::lock_sdn_controllers_config(
         sub {
 
-           my $cfg = PVE::Network::SDN::Controllers::config();
+           my $controller_cfg = PVE::Network::SDN::Controllers::config();
 
-           PVE::SectionConfig::assert_if_modified($cfg, $digest);
+           PVE::SectionConfig::assert_if_modified($controller_cfg, $digest);
 
-           my $scfg = PVE::Network::SDN::Controllers::sdn_controllers_config($cfg, $id);
+           my $scfg = PVE::Network::SDN::Controllers::sdn_controllers_config($controller_cfg, $id);
 
            my $plugin = PVE::Network::SDN::Controllers::Plugin->lookup($scfg->{type});
            my $opts = $plugin->check_config($id, $param, 0, 1);
@@ -207,9 +211,9 @@ __PACKAGE__->register_method ({
                $scfg->{$k} = $opts->{$k};
            }
 
-           $plugin->on_update_hook($id, $cfg);
+           $plugin->on_update_hook($id, $controller_cfg);
 
-           PVE::Network::SDN::Controllers::write_config($cfg);
+           PVE::Network::SDN::Controllers::write_config($controller_cfg);
 
            }, "update sdn controller object failed");
 
@@ -248,9 +252,9 @@ __PACKAGE__->register_method ({
 
                my $plugin = PVE::Network::SDN::Controllers::Plugin->lookup($scfg->{type});
 
-               my $transport_cfg = PVE::Network::SDN::Zones::config();
+               my $zone_cfg = PVE::Network::SDN::Zones::config();
 
-               $plugin->on_delete_hook($id, $transport_cfg);
+               $plugin->on_delete_hook($id, $zone_cfg);
 
                delete $cfg->{ids}->{$id};
                PVE::Network::SDN::Controllers::write_config($cfg);