rename init stage to check

This commit is contained in:
Oliver Gorwits
2017-09-13 10:44:31 +01:00
parent 1fdb086183
commit d5d39289d6
23 changed files with 30 additions and 30 deletions

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::Util::Device 'is_arpnipable_now';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my $device = $job->device;

View File

@@ -11,7 +11,7 @@ use Dancer::Plugin::DBIC 'schema';
use Time::HiRes 'gettimeofday';
use NetAddr::MAC ();
register_worker({ stage => 'init', driver => 'snmp' }, sub {
register_worker({ stage => 'check', driver => 'snmp' }, sub {
my ($job, $workerconf) = @_;
my $device = $job->device;

View File

@@ -7,7 +7,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::JobQueue qw/jq_queued jq_insert/;
use Dancer::Plugin::DBIC 'schema';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my %queued = map {$_ => 1} jq_queued('arpnip');

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::Util::Device 'delete_device';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my ($device, $port, $extra) = map {$job->$_} qw/device port extra/;
return Status->error('Missing device (-d).') if !defined $device;

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::Util::Device 'is_discoverable_now';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my $device = $job->device;

View File

@@ -12,7 +12,7 @@ use Dancer::Plugin::DBIC 'schema';
use NetAddr::IP::Lite ':lower';
use Encode;
register_worker({ stage => 'init', driver => 'snmp' }, sub {
register_worker({ stage => 'check', driver => 'snmp' }, sub {
my ($job, $workerconf) = @_;
my $device = $job->device;

View File

@@ -7,7 +7,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::JobQueue qw/jq_queued jq_insert/;
use Dancer::Plugin::DBIC 'schema';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my %queued = map {$_ => 1} jq_queued('discover');

View File

@@ -7,7 +7,7 @@ use aliased 'App::Netdisco::Worker::Status';
use Dancer::Plugin::DBIC 'schema';
use App::Netdisco::Util::Statistics 'update_stats';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
if (setting('expire_devices') and setting('expire_devices') > 0) {

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use Dancer::Plugin::DBIC 'schema';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
return Status->error('nbtstat failed: unable to interpret device param')

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::Util::Graph ();
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
App::Netdisco::Util::Graph::graph();
return Status->done('Generated graph data.');

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::Util::Device 'is_macsuckable_now';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my $device = $job->device;

View File

@@ -14,7 +14,7 @@ use Dancer::Plugin::DBIC 'schema';
use Time::HiRes 'gettimeofday';
use Scope::Guard 'guard';
register_worker({ stage => 'init', driver => 'snmp' }, sub {
register_worker({ stage => 'check', driver => 'snmp' }, sub {
my ($job, $workerconf) = @_;
my $device = $job->device;

View File

@@ -7,7 +7,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::JobQueue qw/jq_queued jq_insert/;
use Dancer::Plugin::DBIC 'schema';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my %queued = map {$_ => 1} jq_queued('macsuck');

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::Util::NodeMonitor ();
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
App::Netdisco::Util::NodeMonitor::monitor();
return Status->done('Generated monitor data.');

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::Util::Device 'is_macsuckable';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
return Status->error('nbtstat failed: unable to interpret device param')

View File

@@ -9,7 +9,7 @@ use App::Netdisco::Util::Node 'is_nbtstatable';
use Dancer::Plugin::DBIC 'schema';
use Time::HiRes 'gettimeofday';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my $host = $job->device->ip;

View File

@@ -7,7 +7,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::JobQueue qw/jq_queued jq_insert/;
use Dancer::Plugin::DBIC 'schema';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my %queued = map {$_ => 1} jq_queued('nbtstat');

View File

@@ -4,7 +4,7 @@ use Dancer ':syntax';
use App::Netdisco::Worker::Plugin;
use aliased 'App::Netdisco::Worker::Status';
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my ($device, $port, $extra) = map {$job->$_} qw/device port extra/;

View File

@@ -7,7 +7,7 @@ use aliased 'App::Netdisco::Worker::Status';
use NetAddr::IP qw/:rfc3021 :lower/;
use App::Netdisco::Util::Device qw/get_device renumber_device/;
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my ($device, $port, $extra) = map {$job->$_} qw/device port extra/;
return Status->error('Missing device (-d).') if !defined $device;

View File

@@ -7,7 +7,7 @@ use aliased 'App::Netdisco::Worker::Status';
use Data::Printer ();
use App::Netdisco::Transport::SNMP;
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
my ($device, $port, $extra) = map {$job->$_} qw/device port extra/;
return Status->error('Missing device (-d).') if !defined $device;

View File

@@ -6,7 +6,7 @@ use aliased 'App::Netdisco::Worker::Status';
use App::Netdisco::Util::Statistics ();
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
App::Netdisco::Util::Statistics::update_stats();
return Status->done('Updated statistics.');

View File

@@ -10,10 +10,10 @@ register_worker({ stage => 'second' }, sub {
return Status->done('Test (second) ran successfully.');
});
register_worker({ stage => 'init' }, sub {
register_worker({ stage => 'check' }, sub {
my ($job, $workerconf) = @_;
debug 'Test (init) ran successfully.';
return Status->done('Test (init) ran successfully.');
debug 'Test (check) ran successfully.';
return Status->done('Test (check) ran successfully.');
});
register_worker({ stage => 'first' }, sub {

View File

@@ -64,11 +64,11 @@ sub run {
@{ (setting('_nd2worker_hooks') || []) };
foreach my $phase ("nd2_${action}", @phase_hooks) {
foreach my $stage (qw/init first second/) {
foreach my $stage (qw/check first second/) {
my $hookname = "${phase}_${stage}";
next unless scalar @{ $store->get_hooks_for($hookname) };
$self->run_workers($hookname);
return if $stage eq 'init' and $self->jobstat->not_ok;
return if $stage eq 'check' and $self->jobstat->not_ok;
}
}
}
@@ -77,7 +77,7 @@ sub run_workers {
my $self = shift;
my $hook = shift or return $self->jobstat->error('missing hook param');
my $store = Dancer::Factory::Hook->instance();
my $init = ($hook =~ m/_init$/);
my $check = ($hook =~ m/_check$/);
return unless scalar @{ $store->get_hooks_for($hook) };
debug "running workers for hook: $hook";
@@ -87,11 +87,11 @@ sub run_workers {
my $retval = $worker->($self->job);
# could die or return undef or a scalar or Status or another class
$self->jobstat($retval)
if $init and ref $retval eq 'App::Netdisco::Worker::Status';
if $check and ref $retval eq 'App::Netdisco::Worker::Status';
}
catch { $self->jobstat->error($_) if $init };
catch { $self->jobstat->error($_) if $check };
last if $init and $self->jobstat->is_ok;
last if $check and $self->jobstat->is_ok;
}
}