workhorse 1.2.6 → 1.2.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5b1cc4ec91d5c8cde5e214aff39f298f3fa9591b6693250158250fd8de87dd19
-  data.tar.gz: e333e77dc8bb0a24b809324a54d014f56b9373ccc0f16dc5f58052fa7bd1bbe3
+  metadata.gz: 36904c2ff67ab30f876e47515dfd81ccb97715a6e3d563ba93bd59422374d660
+  data.tar.gz: 0c9c66eb75727e78fc842e1a30de12876da67a571aa255855e3ae1e63c27f847
 SHA512:
-  metadata.gz: af21dcc9cf0672dc71ae1b5ec2d61e6245cf6a24fc4b599d1c21c36c06e20cbc46b59248ea0b6232589eced87305c408cbe7254e6817fdb5bb2ec294181e87a7
-  data.tar.gz: 368889c80a413ad8e0343a5227ba70081470931e427ca7ec7c1b1bad6cee59af2a2f9bffe31fb879f1d1a81f2233f506cb038d896dce08b2551733873c013f1f
+  metadata.gz: 0d57c98c31ad7c25f4ad267fe3caa21ae831e986579fef5f2f848c21a7977c0790b461e3372d4cff5e8e89efdb7a0774f06a90542bf0fb4c87baabe8c472ad06
+  data.tar.gz: c99c9d68c24dcccf1bdd164c066466a9c68fa94c7f66dbdc14986dc841e99c0361e32d53a0e2a85226de9c3580ea295254e2c4cb49cd53e974fc28150d3c6e0b

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Workhorse Changelog
 
+## 1.2.7 - 2022-04-07
+
+* Adapt exit status of shell handler to return with exit code `2` when a worker
+  is in an unexpected status. Previously, this lead to exit code `1` and made it
+  hard to distinguish from fatal errors.
+
 ## 1.2.6 - 2022-01-11
 
 * Add daemon command `restart-logging`, which sends a `HUP` interrupt to all

data/VERSION

@@ -1 +1 @@
-1.2.6
+1.2.7

data/lib/workhorse/daemon/shell_handler.rb

@@ -78,8 +78,9 @@ Options:
     Show this message
 
 Exit status:
- 0  if OK,
- 1  if at least one worker has an unexpected status,
+ 0 if OK,
+ 1 on fatal errors outside of workhorse,
+ 2 if at least one worker has an unexpected status,
  99 on all other errors.
 USAGE
     end

data/lib/workhorse/daemon.rb

@@ -46,7 +46,7 @@ module Workhorse
 
         if pid_file && pid
           warn "Worker ##{worker.id} (#{worker.name}): Already started (PID #{pid})" unless quiet
-          code = 1
+          code = 2
         elsif pid_file
           File.delete pid_file
           puts "Worker ##{worker.id} (#{worker.name}): Starting (stale pid file)" unless quiet
@@ -74,7 +74,7 @@ module Workhorse
           puts "Worker (#{worker.name}) ##{worker.id}: Already stopped (stale PID file)"
         else
           warn "Worker (#{worker.name}) ##{worker.id}: Already stopped"
-          code = 1
+          code = 2
         end
       end
 
@@ -91,10 +91,10 @@ module Workhorse
           puts "Worker ##{worker.id} (#{worker.name}): Running" unless quiet
         elsif pid_file
           warn "Worker ##{worker.id} (#{worker.name}): Not running (stale PID file)" unless quiet
-          code = 1
+          code = 2
         else
           warn "Worker ##{worker.id} (#{worker.name}): Not running" unless quiet
-          code = 1
+          code = 2
         end
       end
 
@@ -124,14 +124,14 @@ module Workhorse
       code = 0
 
       for_each_worker do |worker|
-        pid_file, pid = read_pid(worker)
+        _pid_file, pid = read_pid(worker)
 
         begin
           Process.kill 'HUP', pid
           puts "Worker (#{worker.name}) ##{worker.id}: Sent signal for restart-logging"
         rescue Errno::ESRCH
           warn "Worker (#{worker.name}) ##{worker.id}: Could not send signal for restart-logging, process not found"
-          code = 1
+          code = 2
         end
       end
 

data/test/active_job/queue_adapters/workhorse_adapter_test.rb

@@ -73,7 +73,7 @@ class ActiveJob::QueueAdapters::WorkhorseAdapterTest < WorkhorseTest
     work 0.5, polling_interval: 0.1
     assert_equal 'waiting', Workhorse::DbJob.first.state
 
-    work 1.5, polling_interval: 0.1
+    work 3, polling_interval: 0.1
     assert_equal 'succeeded', Workhorse::DbJob.first.reload.state
   end
 

data/test/workhorse/pool_test.rb

@@ -61,16 +61,16 @@ class Workhorse::PoolTest < WorkhorseTest
 
       assert_equal 0, on_idle_calls.value
 
-      p.post { sleep 0.2 }
       p.post { sleep 0.4 }
+      p.post { sleep 0.8 }
 
       sleep 0.1
       assert_equal 0, on_idle_calls.value
 
-      sleep 0.2
+      sleep 0.6
       assert_equal 1, on_idle_calls.value
 
-      sleep 0.1
+      sleep 0.9
       assert_equal 2, on_idle_calls.value
     end
   end

data/workhorse.gemspec

@@ -1,14 +1,14 @@
 # -*- encoding: utf-8 -*-
-# stub: workhorse 1.2.6 ruby lib
+# stub: workhorse 1.2.7 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "workhorse".freeze
-  s.version = "1.2.6"
+  s.version = "1.2.7"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
   s.require_paths = ["lib".freeze]
   s.authors = ["Sitrox".freeze]
-  s.date = "2022-01-11"
+  s.date = "2022-04-07"
   s.files = [".github/workflows/ruby.yml".freeze, ".gitignore".freeze, ".releaser_config".freeze, ".rubocop.yml".freeze, "CHANGELOG.md".freeze, "FAQ.md".freeze, "Gemfile".freeze, "LICENSE".freeze, "README.md".freeze, "RUBY_VERSION".freeze, "Rakefile".freeze, "VERSION".freeze, "bin/rubocop".freeze, "lib/active_job/queue_adapters/workhorse_adapter.rb".freeze, "lib/generators/workhorse/install_generator.rb".freeze, "lib/generators/workhorse/templates/bin/workhorse.rb".freeze, "lib/generators/workhorse/templates/config/initializers/workhorse.rb".freeze, "lib/generators/workhorse/templates/create_table_jobs.rb".freeze, "lib/workhorse.rb".freeze, "lib/workhorse/daemon.rb".freeze, "lib/workhorse/daemon/shell_handler.rb".freeze, "lib/workhorse/db_job.rb".freeze, "lib/workhorse/enqueuer.rb".freeze, "lib/workhorse/jobs/cleanup_succeeded_jobs.rb".freeze, "lib/workhorse/jobs/detect_stale_jobs_job.rb".freeze, "lib/workhorse/jobs/run_active_job.rb".freeze, "lib/workhorse/jobs/run_rails_op.rb".freeze, "lib/workhorse/performer.rb".freeze, "lib/workhorse/poller.rb".freeze, "lib/workhorse/pool.rb".freeze, "lib/workhorse/scoped_env.rb".freeze, "lib/workhorse/worker.rb".freeze, "test/active_job/queue_adapters/workhorse_adapter_test.rb".freeze, "test/lib/db_schema.rb".freeze, "test/lib/jobs.rb".freeze, "test/lib/test_helper.rb".freeze, "test/workhorse/db_job_test.rb".freeze, "test/workhorse/enqueuer_test.rb".freeze, "test/workhorse/performer_test.rb".freeze, "test/workhorse/poller_test.rb".freeze, "test/workhorse/pool_test.rb".freeze, "test/workhorse/worker_test.rb".freeze, "workhorse.gemspec".freeze]
   s.rubygems_version = "3.0.3".freeze
   s.summary = "Multi-threaded job backend with database queuing for ruby.".freeze

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: workhorse
 version: !ruby/object:Gem::Version
-  version: 1.2.6
+  version: 1.2.7
 platform: ruby
 authors:
 - Sitrox
-autorequire:
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-01-11 00:00:00.000000000 Z
+date: 2022-04-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -178,8 +178,8 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-description:
-email:
+description: 
+email: 
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -227,10 +227,10 @@ files:
 - test/workhorse/pool_test.rb
 - test/workhorse/worker_test.rb
 - workhorse.gemspec
-homepage:
+homepage: 
 licenses: []
 metadata: {}
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -245,8 +245,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
-signing_key:
+rubygems_version: 3.0.3.1
+signing_key: 
 specification_version: 4
 summary: Multi-threaded job backend with database queuing for ruby.
 test_files: