first make arpnip behave like other jobs towards pseudo devices

netdisco · Aug 25, 2019 · d2661bf · d2661bf
1 parent adf80a1
commit d2661bf
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/lib/App/Netdisco/Worker/Plugin/Arpnip.pm b/lib/App/Netdisco/Worker/Plugin/Arpnip.pm
@@ -16,6 +16,9 @@ register_worker({ phase => 'check' }, sub {
   return Status->error("arpnip skipped: $device not yet discovered")
     unless $device->in_storage;
 
+  return Status->info("macsuck skipped: $device is pseudo-device")
+    if $device->is_pseudo;
+
   return Status->info("arpnip skipped: $device has no layer 3 capability")
     unless $device->has_layer(3);
 

diff --git a/lib/App/Netdisco/Worker/Plugin/Arpwalk.pm b/lib/App/Netdisco/Worker/Plugin/Arpwalk.pm
@@ -11,8 +11,9 @@ register_worker({ phase => 'main' }, sub {
   my ($job, $workerconf) = @_;
 
   my %queued = map {$_ => 1} jq_queued('arpnip');
-  my @devices = schema('netdisco')->resultset('Device')
-    ->has_layer('3')->get_column('ip')->all;
+  my @devices = schema('netdisco')->resultset('Device')->search({
+    -or => [ 'vendor' => undef, 'vendor' => { '!=' => 'netdisco' }],
+  })->has_layer('3')->get_column('ip')->all;
   my @filtered_devices = grep {!exists $queued{$_}} @devices;
 
   jq_insert([

diff --git a/share/schema_versions/App-Netdisco-DB-58-59-PostgreSQL.sql b/share/schema_versions/App-Netdisco-DB-58-59-PostgreSQL.sql
@@ -2,4 +2,6 @@ BEGIN;
 
 ALTER TABLE users ADD COLUMN "radius" boolean DEFAULT false;
 
+UPDATE device SET layers = NULL WHERE vendor = 'netdisco';
+
 COMMIT;