workhorse 1.3.0 → 1.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: be27417ff88d4edb29204f308a3bc0c23ba0b332de1bd2542247afd2be308b11
-  data.tar.gz: 6bc8393256cf426f3a966519d420a324af51723b5a6277ffa4785c9b1844da8f
+  metadata.gz: 8337d1efedcb53b3fd51e983fe235aa2f1e71b6cbfedad84d2becb5c292dfb1c
+  data.tar.gz: 6746a5eeb075df326794e135b0336dea076a624e20562184c687d50603c894bd
 SHA512:
-  metadata.gz: 37dd9e3476175ed1832128960cf162e4f364c317589f283cf9e72afed4f931033d63bb207fbe7abecfb1224a25b93fc4e66cc6f32f9cdb667aa8a69f72f44d08
-  data.tar.gz: 60f981006ca6668490716f4009de5dca69fe550601b79b6d3ac4e90b2d1fbafedbd8031a84e91c506c003594b58bb5fafd1e62519d31db35ae868744eff05095
+  metadata.gz: 55769be0e335e14684c1e6bbd108b2c9da4fee5cf42b86149a481e0cae1d4a64d542ca0119adea20d83f3f9131dd45d61daa6e88df5e3bacde86231c0bc995b9
+  data.tar.gz: da590080ffa2b5f96c1800b730babf019f115b600335a85fc41d343573765e0114eea356c09238507913c2ca1e88cd5d0e3d12742facc201e5b9c5ba1c076188

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Workhorse Changelog
 
+## 1.3.1 - 2025-11-05
+
+* Adapt shell handler locking behavior depending on command:
+
+  * `usage|<empty string>|<unknown command>`: Does not lock anymore as it
+    only shows the usage information.
+
+  * `watch|kill`: Failing lock. These commands now simply abort if there is
+    already another locking command running.
+
+  * `start|stop|restart|status|restart-logging`: Waiting lock. These commands
+    will wait until other commands are finished. This is equal to the previous
+    behavior.
+
 ## 1.3.0 - 2025-09-16
 
 * Stable release based on previous RC releases.

data/VERSION

@@ -1 +1 @@
-1.3.0
+1.3.1

data/lib/workhorse/daemon/shell_handler.rb

@@ -6,31 +6,33 @@ module Workhorse
         exit 99
       end
 
-      if Workhorse.lock_shell_commands
-        lockfile_path = options.delete(:lockfile) || 'workhorse.lock'
-        lockfile = File.open(lockfile_path, 'a')
-        lockfile.flock(File::LOCK_EX || File::LOCK_NB)
-      else
-        lockfile = nil
-      end
-
+      lockfile_path = options.delete(:lockfile) || 'workhorse.lock'
       daemon = Workhorse::Daemon.new(**options, &block)
 
+      lockfile = nil
+
       begin
         case ARGV.first
         when 'start'
+          lockfile = acquire_lock(lockfile_path, File::LOCK_EX)
           status = daemon.start
         when 'stop'
+          lockfile = acquire_lock(lockfile_path, File::LOCK_EX)
           status = daemon.stop
         when 'kill'
+          lockfile = acquire_lock(lockfile_path, File::LOCK_EX | File::LOCK_NB)
           status = daemon.stop(true)
         when 'status'
+          lockfile = acquire_lock(lockfile_path, File::LOCK_EX)
           status = daemon.status
         when 'watch'
+          lockfile = acquire_lock(lockfile_path, File::LOCK_EX | File::LOCK_NB)
           status = daemon.watch
         when 'restart'
+          lockfile = acquire_lock(lockfile_path, File::LOCK_EX)
           status = daemon.restart
         when 'restart-logging'
+          lockfile = acquire_lock(lockfile_path, File::LOCK_EX)
           status = daemon.restart_logging
         when 'usage'
           usage
@@ -88,5 +90,18 @@ module Workhorse
          99 on all other errors.
       USAGE
     end
+
+    private
+
+    def self.acquire_lock(lockfile_path, flags)
+      if Workhorse.lock_shell_commands
+        lockfile = File.open(lockfile_path, 'a')
+        lockfile.flock(flags)
+
+        return lockfile
+      end
+
+      return nil
+    end
   end
 end

data/workhorse.gemspec

@@ -1,14 +1,14 @@
 # -*- encoding: utf-8 -*-
-# stub: workhorse 1.3.0 ruby lib
+# stub: workhorse 1.3.1 ruby lib
 
 Gem::Specification.new do |s|
   s.name = "workhorse".freeze
-  s.version = "1.3.0"
+  s.version = "1.3.1"
 
   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 = "2025-09-16"
+  s.date = "2025-11-05"
   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/active_job_extension.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/daemon_test.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.4.6".freeze
   s.summary = "Multi-threaded job backend with database queuing for ruby.".freeze

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: workhorse
 version: !ruby/object:Gem::Version
-  version: 1.3.0
+  version: 1.3.1
 platform: ruby
 authors:
 - Sitrox
 bindir: bin
 cert_chain: []
-date: 2025-09-16 00:00:00.000000000 Z
+date: 2025-11-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport