This is an automated email from the ASF dual-hosted git repository.

sebb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/whimsy.git


The following commit(s) were added to refs/heads/master by this push:
     new cf3d2884 Check for kill -9 completion [skip ci]
cf3d2884 is described below

commit cf3d2884448b77bb8c9c7d03dace2ce2e35bf757
Author: Sebb <s...@apache.org>
AuthorDate: Mon Dec 18 11:55:05 2023 +0000

    Check for kill -9 completion [skip ci]
---
 tools/site-scan.rb | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/tools/site-scan.rb b/tools/site-scan.rb
index a2029492..e45a1aa0 100755
--- a/tools/site-scan.rb
+++ b/tools/site-scan.rb
@@ -243,6 +243,12 @@ def exec_with_timeout(cmd, timeout)
         $stderr.puts "WARN:  #{Time.now} about to kill -9 #{pid}"
         ret = Process.kill(-9, pid) # SIGKILL
         $stderr.puts "WARN:  #{Time.now} sent kill -9 #{pid} ret=#{ret}"
+        thrd = reaper.join 5 # allow some time for process to exit
+        if thrd
+          $stderr.puts  "WARN:  #{Time.now} process completed #{thrd.value}"
+        else
+          $stderr.puts "ERROR:  #{Time.now} failed to kill -9 #{pid}"
+        end
       end
     rescue StandardError => e
       $stderr.puts "WARN:  #{Time.now} ret=#{ret} exception: #{e}"

Reply via email to