use overloaded $device

lib/App/Netdisco/Worker/Plugin/Arpnip/Nodes.pm

@@ -41,7 +41,7 @@ register_worker({ stage => 'check', driver => 'snmp' }, sub {
     $device->ip, scalar @$v6;
 
   $device->update({last_arpnip => \$now});
-  return Status->done('Ended arpnip for '. $device->ip);
+  return Status->done("Ended arpnip for $device");
 });
 
 # get an arp table (v4 or v6)

lib/App/Netdisco/Worker/Plugin/Arpnip/Subnets.pm

@@ -30,7 +30,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
   debug sprintf ' [%s] arpnip - processed %s Subnet entries',
     $device->ip, scalar @subnets;
 
-  return Status->done('Ended arpnip for '. $device->ip);
+  return Status->done("Ended arpnip for $device");
 });
 
 # gathers device subnets

lib/App/Netdisco/Worker/Plugin/Delete.pm

@@ -13,7 +13,7 @@ register_worker({ stage => 'check' }, sub {
 
   $port = ($port ? 1 : 0);
   delete_device($device, $port, $extra);
-  return Status->done(sprintf "Deleted device %s.", $device->ip);
+  return Status->done("Deleted device: $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Discover/CanonicalIP.pm

@@ -59,7 +59,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
     } # ALIAS
   }
 
-  return Status->done('Ended discover for '. $device->ip)
+  return Status->done("Ended discover for $device")
     if $new_ip eq $old_ip;
 
   schema('netdisco')->txn_do(sub {
@@ -76,7 +76,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
       $old_ip, $device->ip, ($device->dns || '');
   });
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Discover/Entities.pm

@@ -90,7 +90,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
       $device->ip, scalar @modules;
   });
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Discover/Interfaces.pm

@@ -145,7 +145,7 @@ register_worker({ stage => 'first', driver => 'snmp' }, sub {
       $device->ip, scalar values %interfaces;
   });
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Discover/Neighbors.pm

@@ -61,7 +61,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
       });
   }
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 =head2 store_neighbors( $device, $snmp )

lib/App/Netdisco/Worker/Plugin/Discover/PortPower.pm

@@ -76,7 +76,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
       $device->ip, scalar @portpower;
   });
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Discover/Properties.pm

@@ -99,7 +99,7 @@ register_worker({ stage => 'check', driver => 'snmp' }, sub {
       $device->ip, scalar @aliases;
   });
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Discover/VLANs.pm

@@ -89,7 +89,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
       $device->ip, scalar @portvlans;
   });
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Discover/Wireless.pm

@@ -80,7 +80,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
       $device->ip, scalar @channels;
   });
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Discover/WithNodes.pm

@@ -33,7 +33,7 @@ register_worker({ stage => 'second' }, sub {
     }
   }
 
-  return Status->done('Ended discover for '. $device->ip);
+  return Status->done("Ended discover for $device");
 });
 
 true;

lib/App/Netdisco/Worker/Plugin/Macsuck/Nodes.pm

@@ -94,7 +94,7 @@ register_worker({ stage => 'check', driver => 'snmp' }, sub {
     $device->ip, $archived;
 
   $device->update({last_macsuck => \$now});
-  return Status->done('Ended macsuck for '. $device->ip);
+  return Status->done("Ended macsuck for $device");
 });
 
 =head2 store_node( $ip, $vlan, $port, $mac, $now? )

lib/App/Netdisco/Worker/Plugin/Macsuck/WirelessNodes.pm

@@ -21,7 +21,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
   my $now = 'to_timestamp('. (join '.', gettimeofday) .')';
 
   my $cd11_txrate = $snmp->cd11_txrate;
-  return Status->done('Ended macsuck for '. $device->ip)
+  return Status->done("Ended macsuck for $device")
     unless $cd11_txrate and scalar keys %$cd11_txrate;
 
   if (setting('store_wireless_clients')) {
@@ -31,7 +31,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
   else {
     debug sprintf ' [%s] macsuck - dot11 info available but skipped due to config',
       $device->ip;
-    return Status->done('Ended macsuck for '. $device->ip);
+    return Status->done("Ended macsuck for $device");
   }
 
   my $cd11_rateset = $snmp->cd11_rateset();
@@ -83,7 +83,7 @@ register_worker({ stage => 'second', driver => 'snmp' }, sub {
     });
   }
 
-  return Status->done('Ended macsuck for '. $device->ip);
+  return Status->done("Ended macsuck for $device");
 });
 
 true;