zones: simplify status code

Signed-off-by: Fabian Grünbichler <f.gruenbichler@proxmox.com>

diff --git a/src/PVE/Network/SDN/Zones/Plugin.pm b/src/PVE/Network/SDN/Zones/Plugin.pm
index 36d690636ce2d2cff9d73ddfd69a7a5bc8576779..f6634b697d6b9fda1a4abb6fd9054ea870c3859a 100644 (file)
--- a/src/PVE/Network/SDN/Zones/Plugin.pm
+++ b/src/PVE/Network/SDN/Zones/Plugin.pm
@@ -212,9 +212,7 @@ sub generate_status_message {
 sub status {
     my ($class, $plugin_config, $zone, $vnetid, $vnet, $status) = @_;
 
-    my $err_msg = $class->generate_status_message($vnetid, $status);
-    return $err_msg;
-
+    return $class->generate_status_message($vnetid, $status);
 }
 
 

diff --git a/src/PVE/Network/SDN/Zones/QinQPlugin.pm b/src/PVE/Network/SDN/Zones/QinQPlugin.pm
index 71604343f55dbbc907fd1427a457c42cd76bbb93..4c4be642986d1bee0a92a825c5228d3a7cc5cd5d 100644 (file)
--- a/src/PVE/Network/SDN/Zones/QinQPlugin.pm
+++ b/src/PVE/Network/SDN/Zones/QinQPlugin.pm
@@ -170,11 +170,9 @@ sub status {
     my ($class, $plugin_config, $zone, $vnetid, $vnet, $status) = @_;
 
     my $bridge = $plugin_config->{bridge};
-    my $err_msg = [];
 
     if (!-d "/sys/class/net/$bridge") {
-        push @$err_msg, "missing $bridge";
-        return $err_msg;
+       return ["missing $bridge"];
     }
 
     my $vlan_aware = PVE::Network::SDN::Zones::Plugin::is_vlanaware($bridge);
@@ -198,8 +196,7 @@ sub status {
        push @$ifaces, $svlan_iface;
     }
 
-    $err_msg = $class->generate_status_message($vnetid, $status, $ifaces);
-    return $err_msg;
+    return $class->generate_status_message($vnetid, $status, $ifaces);
 }
 
 sub vnet_update_hook {

diff --git a/src/PVE/Network/SDN/Zones/VlanPlugin.pm b/src/PVE/Network/SDN/Zones/VlanPlugin.pm
index f39b71ddc9b9493555539521e3f636f7a4ee4ed8..6a68e8d8c5d7beda8c5e47dd17a671911323755a 100644 (file)
--- a/src/PVE/Network/SDN/Zones/VlanPlugin.pm
+++ b/src/PVE/Network/SDN/Zones/VlanPlugin.pm
@@ -141,10 +141,8 @@ sub status {
 
     my $bridge = $plugin_config->{bridge};
 
-    my $err_msg = [];
     if (!-d "/sys/class/net/$bridge") {
-        push @$err_msg, "missing $bridge";
-       return $err_msg;
+       return ["missing $bridge"];
     }
 
     my $vlan_aware = PVE::Network::SDN::Zones::Plugin::is_vlanaware($bridge);
@@ -165,8 +163,7 @@ sub status {
        push @$ifaces, $vnet_uplinkpeer;
     }
 
-    $err_msg = $class->generate_status_message($vnetid, $status, $ifaces);
-    return $err_msg;
+    return $class->generate_status_message($vnetid, $status, $ifaces);
 }
 
 sub vnet_update_hook {