daemons 1.3.1 → 1.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 2b3903b602fe522c5bfd073703d563554ea0dc00
-  data.tar.gz: 98a54946242d0a0034576ab9fefb68b7c4bcf4f7
+SHA256:
+  metadata.gz: 5114c4c5cb5dba3befcc82f6eaa45d901036bc7357a6d4d81cfd8cad8c49dae6
+  data.tar.gz: 0eb7b12972a659d053b87d4d59e24d5a3de21a9ff94c3432e8da62ac08f5c127
 SHA512:
-  metadata.gz: 0a3557bbfa8e6938eb468e91f4c5c6e376d6ec35dce02740dc744492f1373f4d29a89a12c338d021f203de5edaf80047841d790106909961ec09a2dd682b4452
-  data.tar.gz: f0088e769e05b46de5fd115112782447b42e3687e3df071c3ecdc6021613d569e444034db2ceffd3c03cc8861b9be1918d6da5b8fd0bf2234df919683dcb0c41
+  metadata.gz: '00970b9deb9eab30e1ce69b31ceca7617bf406692f5dbbb5a9459c5a5a898bc8a0e751fb751cb463f027be1977b54c0b914fcee996ad0cbea997829ce54d2360'
+  data.tar.gz: f98f5696a4f482e29eb22c38fddc9fa6dafcfaaa4c6989a5015191c53711848e703267dc3c0da25080d9509035bf91b7aac7f009b4ccbb6de53fd8a299a4834b

data/LICENSE

@@ -1,4 +1,4 @@
-Copyright (c) 2005-2018 Thomas Uehlinger, 2014-2016 Aaron Stone
+Copyright (c) 2005-2021 Thomas Uehlinger, 2014-2016 Aaron Stone
 
 Permission is hereby granted, free of charge, to any person
 obtaining a copy of this software and associated documentation

data/README.md

@@ -209,4 +209,4 @@ Documentation can be found at http://www.rubydoc.info/gems/daemons.
 Author
 ------
 
-Written 2005-2018 by Thomas Uehlinger, 2014-2016 by Aaron Stone.
+Written 2005-2021 by Thomas Uehlinger, 2014-2016 by Aaron Stone.

data/Releases

@@ -1,5 +1,11 @@
 = Daemons Release History
 
+== Release 1.4.0: May 1, 2021
+
+* Allow for customization which signals are sent to stop process (thanks to philister)
+* Resolves mismatched indentations (thanks to Luis M Rodriguez)
+* Allow to use pry-byebug 3.8.0 (thanks to kamipo)
+
 == Release 1.3.1: December 14, 2018
 
 * Fix undefined local variable or method `pid_delimiter'

data/lib/daemons/application.rb

@@ -36,6 +36,8 @@ module Daemons
 
       @force_kill_waittime = @options[:force_kill_waittime] || 20
 
+      @signals_and_waits = parse_signals_and_waits(@options[:signals_and_waits])
+
       @show_status_callback = method(:default_show_status)
 
       @report = Reporter.new(@options)
@@ -203,7 +205,7 @@ module Daemons
           if $daemons_stop_proc
             $daemons_stop_proc.call
           end
-          rescue ::Exception
+        rescue ::Exception
         end
 
         begin; @pid.cleanup; rescue ::Exception; end
@@ -260,7 +262,7 @@ module Daemons
             if $daemons_stop_proc
               $daemons_stop_proc.call
             end
-            rescue ::Exception
+          rescue ::Exception
           end
 
           begin; @pid.cleanup; rescue ::Exception; end
@@ -373,6 +375,7 @@ module Daemons
         return
       end
 
+      # confusing: pid is also a attribute_reader
       pid = @pid.pid
 
       # Catch errors when trying to kill a process that doesn't
@@ -380,47 +383,12 @@ module Daemons
       # restarted by the monitor yet. By catching the error, we allow the
       # pid file clean-up to occur.
       begin
-        Process.kill(SIGNAL, pid)
+        wait_and_retry_kill_harder(pid, @signals_and_waits, no_wait)
       rescue Errno::ESRCH => e
         @report.output_message("#{e} #{pid}")
         @report.output_message('deleting pid-file.')
       end
 
-      unless no_wait
-        if @force_kill_waittime > 0
-          @report.stopping_process(group.app_name, pid)
-          $stdout.flush
-
-          begin
-            Timeout.timeout(@force_kill_waittime, TimeoutError) do
-              while Pid.running?(pid)
-                sleep(0.2)
-              end
-            end
-          rescue TimeoutError
-            @report.forcefully_stopping_process(group.app_name, pid)
-            $stdout.flush
-
-            begin
-              Process.kill('KILL', pid)
-            rescue Errno::ESRCH
-            end
-
-            begin
-              Timeout.timeout(20, TimeoutError) do
-                while Pid.running?(pid)
-                  sleep(1)
-                end
-              end
-            rescue TimeoutError
-              @report.cannot_stop_process(group.app_name, pid)
-              $stdout.flush
-            end
-          end
-        end
-
-      end
-
       sleep(0.1)
       unless Pid.running?(pid)
         # We try to remove the pid-files by ourselves, in case the application
@@ -428,7 +396,30 @@ module Daemons
         zap!
 
         @report.stopped_process(group.app_name, pid)
-        $stdout.flush
+      end
+    end
+
+    # @param Hash remaing_signals
+    # @param Boolean no_wait Send first Signal and return
+    def wait_and_retry_kill_harder(pid, remaining_signals, no_wait = false)
+      sig_wait = remaining_signals.shift
+      sig      = sig_wait[:sig]
+      wait     = sig_wait[:wait]
+      Process.kill(sig, pid)
+      return if no_wait || !wait.positive?
+
+      @report.stopping_process(group.app_name, pid, sig, wait)
+
+      begin
+        Timeout.timeout(wait, TimeoutError) do
+          sleep(0.2) while Pid.running?(pid)
+        end
+      rescue TimeoutError
+        if remaining_signals.any?
+          wait_and_retry_kill_harder(pid, remaining_signals)
+        else
+          @report.cannot_stop_process(group.app_name, pid)
+        end
       end
     end
 
@@ -478,5 +469,15 @@ module Daemons
     def dir
       @dir or group.dir
     end
+
+    def parse_signals_and_waits(argv)
+      unless argv
+        return [
+          { sig: 'TERM', wait: @force_kill_waittime },
+          { sig: 'KILL', wait: 20 }
+        ]
+      end
+      argv.split('|').collect{ |part| splitted = part.split(':'); {sig: splitted[0], wait: splitted[1].to_i}}
+    end
   end
 end

data/lib/daemons/application_group.rb

@@ -83,7 +83,7 @@ module Daemons
           end
           pids = processes.map { |p| p.split(/\s/)[0].to_i }
         end
-        rescue ::Exception
+      rescue ::Exception
       end
 
       pids.map do |f|

data/lib/daemons/cmdline.rb

@@ -28,6 +28,9 @@ module Daemons
         opts.on('-w', '--force_kill_waittime SECONDS', Integer, 'Maximum time to wait for processes to stop before force-killing') do |t|
           @options[:force_kill_waittime] = t
         end
+        opts.on('--signals_and_waits STRING', String, 'which signal to use to stop and how long to wait e.g. TERM:20|KILL:20') do |t|
+          @options[:signals_and_waits] = t
+        end
 
         opts.on('--pid_delimiter STRING', 'Text used to separate process number in full process name and pid-file name') do |value|
           @options[:pid_delimiter] = value

data/lib/daemons/reporter.rb

@@ -32,20 +32,19 @@ module Daemons
       output_message 'option :backtrace is not supported with :mode => :exec, ignoring'
     end
 
-    def stopping_process(app_name, pid)
-      output_message "#{app_name}: trying to stop process with pid #{pid}..."
-    end
-
-    def forcefully_stopping_process(app_name, pid)
-      output_message "#{app_name}: process with pid #{pid} won't stop, we forcefully kill it..." 
+    def stopping_process(app_name, pid, sig, wait)
+      output_message "#{app_name}: trying to stop process with pid #{pid}#{' forcefully :(' if sig == 'KILL'} sending #{sig} and waiting #{wait}s ..."
+      $stdout.flush
     end
 
     def cannot_stop_process(app_name, pid)
       output_message "#{app_name}: unable to forcefully kill process with pid #{pid}."
+      $stdout.flush
     end
 
     def stopped_process(app_name, pid)
       output_message "#{app_name}: process with pid #{pid} successfully stopped."
+      $stdout.flush
     end
 
     def status(app_name, running, pid_exists, pid)

data/lib/daemons/version.rb

@@ -1,3 +1,3 @@
 module Daemons
-  VERSION = '1.3.1'
+  VERSION = '1.4.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: daemons
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.4.0
 platform: ruby
 authors:
 - Thomas Uehlinger
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-12-14 00:00:00.000000000 Z
+date: 2021-05-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -16,14 +16,20 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '12.3'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 12.3.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '12.3'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 12.3.3
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -58,14 +64,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.0.0
+        version: '3.0'
 description: |2
       Daemons provides an easy way to wrap existing ruby scripts (for example a
       self-written server)  to be run as a daemon and to be controlled by simple
@@ -147,8 +153,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2.3
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: A toolkit to create and control daemons in different ways