turn into a private method

fusioninventory · Jul 10, 2015 · 9155066 · 9155066
1 parent 98b9688
commit 9155066
Showing 1 changed file with 19 additions and 18 deletions.
diff --git a/lib/FusionInventory/Agent/Task/Inventory/Virtualization/Virtuozzo.pm b/lib/FusionInventory/Agent/Task/Inventory/Virtualization/Virtuozzo.pm
@@ -13,23 +13,6 @@ sub isEnabled {
     return canRun('vzlist');
 }
 
-sub getMACs {
-    my ($ctid, $logger) = @_;
-
-    my @ipLines = getAllLines(
-        command => "vzctl exec '$ctid' 'ip -0 a'",
-        logger  => $logger
-    );
-
-    my @macs;
-    foreach my $line (@ipLines) {
-        next unless $line =~ /^\s+link\/ether ($mac_address_pattern)\s/;
-        push @macs, $1;
-    }
-
-    return join('/', @macs);
-}
-
 sub doInventory {
     my (%params) = @_;
 
@@ -97,7 +80,7 @@ sub doInventory {
                 STATUS    => $status,
                 SUBSYSTEM => $subsys,
                 VMTYPE    => "Virtuozzo",
-                MAC       => getMACs($ctid, $logger)
+                MAC       => _getMACs($ctid, $logger)
             }
         );
 
@@ -106,4 +89,22 @@ sub doInventory {
     close $handle;
 }
 
+sub _getMACs {
+    my ($ctid, $logger) = @_;
+
+    my @ipLines = getAllLines(
+        command => "vzctl exec '$ctid' 'ip -0 a'",
+        logger  => $logger
+    );
+
+    my @macs;
+    foreach my $line (@ipLines) {
+        next unless $line =~ /^\s+link\/ether ($mac_address_pattern)\s/;
+        push @macs, $1;
+    }
+
+    return join('/', @macs);
+}
+
+
 1;