From: root <r...@kvmformation1.odiso.net>

use config generation && reloading from plugins
(require my last pve-network patch serie)

Signed-off-by: Alexandre Derumier <aderum...@odiso.com>
---
 PVE/API2/Network.pm | 25 +++++--------------------
 1 file changed, 5 insertions(+), 20 deletions(-)

diff --git a/PVE/API2/Network.pm b/PVE/API2/Network.pm
index fa605ba7..194b300a 100644
--- a/PVE/API2/Network.pm
+++ b/PVE/API2/Network.pm
@@ -554,22 +554,12 @@ __PACKAGE__->register_method({
        my $new_config_file = "/etc/network/interfaces.new";
 
        die "you need ifupdown2 to reload networking\n" if !-e 
'/usr/share/ifupdown2';
-       die "ifupdown2 reload is not compatible if openvswitch currently" if -x 
'/usr/bin/ovs-vsctl';
+       warn "openvswitch config can't be reloaded currently. Please reboot to 
apply ovs changes" if -x '/usr/bin/ovs-vsctl';
 
        my $worker = sub {
 
            rename($new_config_file, $current_config_file) if -e 
$new_config_file;
 
-           my $frr_config;
-           if ($have_sdn) {
-               my $network_config = 
PVE::Network::SDN::generate_etc_network_config();
-               PVE::Network::SDN::write_etc_network_config($network_config);
-
-               $frr_config = PVE::Network::SDN::generate_frr_config();
-               PVE::Network::SDN::write_frr_config($frr_config) if $frr_config;
-           }
-
-
            my $err = sub {
                my $line = shift;
                if ($line =~ /(warning|error): (\S+):/) {
@@ -578,15 +568,10 @@ __PACKAGE__->register_method({
            };
            PVE::Tools::run_command(['ifreload', '-a'], errfunc => $err);
 
-           my $err_frr = sub {
-               my $line = shift;
-               if ($line =~ /^line (\S+)/) {
-                   print "$line \n";
-               }
-           };
-
-           if ($frr_config && -e "/usr/bin/vtysh") {
-               PVE::Tools::run_command(['/usr/bin/vtysh', '-m', '-f', 
'/etc/frr/frr.conf'], outfunc => {}, errfunc => $err_frr);
+           if ($have_sdn) {
+               my $controller_config = 
PVE::Network::SDN::generate_controller_config();
+               PVE::Network::SDN::write_controller_config($controller_config) 
if ($controller_config);
+               PVE::Network::SDN::reload_controller();
            }
        };
        return $rpcenv->fork_worker('srvreload', 'networking', $authuser, 
$worker);
-- 
2.20.1

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

Reply via email to