mirror of
https://github.com/discourse/discourse.git
synced 2024-11-22 05:01:05 +08:00
10ff0ee0cc
This commit updates `Demon::Base#start` to call `Discourse.before_fork` before forking. According to the docs in `mini_racer`, we need to "Dispose manually of all MiniRacer::Context objects prior to forking". This commit is motivated by a segmentation fault which we are seeing in production when killing a daemon process. Backtrace of the core dump includes traces of `mini_racer` so we think this is the cause. Note that we are not 100% sure if this will fix the issue.
247 lines
4.6 KiB
Ruby
247 lines
4.6 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
module Demon
|
|
end
|
|
|
|
# intelligent fork based demonizer
|
|
class Demon::Base
|
|
def self.demons
|
|
@demons
|
|
end
|
|
|
|
def self.start(count = 1, verbose: false, logger: nil)
|
|
@demons ||= {}
|
|
count.times { |i| (@demons["#{prefix}_#{i}"] ||= new(i, verbose:, logger:)).start }
|
|
end
|
|
|
|
def self.stop
|
|
return unless @demons
|
|
@demons.values.each { |demon| demon.stop }
|
|
end
|
|
|
|
def self.restart
|
|
return unless @demons
|
|
@demons.values.each do |demon|
|
|
demon.stop
|
|
demon.start
|
|
end
|
|
end
|
|
|
|
def self.ensure_running
|
|
@demons.values.each { |demon| demon.ensure_running }
|
|
end
|
|
|
|
def self.kill(signal)
|
|
return unless @demons
|
|
@demons.values.each { |demon| demon.kill(signal) }
|
|
end
|
|
|
|
attr_reader :pid, :parent_pid, :started, :index
|
|
attr_accessor :stop_timeout
|
|
|
|
def initialize(index, rails_root: nil, parent_pid: nil, verbose: false, logger: nil)
|
|
@index = index
|
|
@pid = nil
|
|
@parent_pid = parent_pid || Process.pid
|
|
@started = false
|
|
@stop_timeout = 10
|
|
@rails_root = rails_root || Rails.root
|
|
@verbose = verbose
|
|
@logger = logger || Logger.new(STDERR)
|
|
end
|
|
|
|
def log(message, level: :info)
|
|
@logger.public_send(level, message)
|
|
end
|
|
|
|
def pid_file
|
|
"#{@rails_root}/tmp/pids/#{self.class.prefix}_#{@index}.pid"
|
|
end
|
|
|
|
def alive?(pid = nil)
|
|
pid ||= @pid
|
|
if pid
|
|
Demon::Base.alive?(pid)
|
|
else
|
|
false
|
|
end
|
|
end
|
|
|
|
def kill(signal)
|
|
Process.kill(signal, @pid)
|
|
end
|
|
|
|
def stop_signal
|
|
"HUP"
|
|
end
|
|
|
|
def stop
|
|
@started = false
|
|
|
|
if @pid
|
|
Process.kill(stop_signal, @pid)
|
|
|
|
wait_for_stop =
|
|
lambda do
|
|
timeout = @stop_timeout
|
|
|
|
while alive? && timeout > 0
|
|
timeout -= (@stop_timeout / 10.0)
|
|
sleep(@stop_timeout / 10.0)
|
|
begin
|
|
Process.waitpid(@pid, Process::WNOHANG)
|
|
rescue StandardError
|
|
-1
|
|
end
|
|
end
|
|
|
|
begin
|
|
Process.waitpid(@pid, Process::WNOHANG)
|
|
rescue StandardError
|
|
-1
|
|
end
|
|
end
|
|
|
|
wait_for_stop.call
|
|
|
|
if alive?
|
|
log("Process would not terminate cleanly, force quitting. pid: #{@pid} #{self.class}")
|
|
Process.kill("KILL", @pid)
|
|
end
|
|
|
|
wait_for_stop.call
|
|
|
|
@pid = nil
|
|
@started = false
|
|
end
|
|
end
|
|
|
|
def ensure_running
|
|
return unless @started
|
|
|
|
if !@pid
|
|
@started = false
|
|
start
|
|
return
|
|
end
|
|
|
|
dead =
|
|
begin
|
|
Process.waitpid(@pid, Process::WNOHANG)
|
|
rescue StandardError
|
|
-1
|
|
end
|
|
|
|
if dead
|
|
log("Detected dead worker #{@pid}, restarting...")
|
|
@pid = nil
|
|
@started = false
|
|
start
|
|
end
|
|
end
|
|
|
|
def start
|
|
return if @pid || @started
|
|
|
|
if existing = already_running?
|
|
# should not happen ... so kill violently
|
|
log("Attempting to kill pid #{existing}")
|
|
Process.kill("TERM", existing)
|
|
end
|
|
|
|
@started = true
|
|
run
|
|
end
|
|
|
|
def run
|
|
Discourse.before_fork if defined?(Discourse)
|
|
|
|
@pid =
|
|
fork do
|
|
Process.setproctitle("discourse #{self.class.prefix}")
|
|
monitor_parent
|
|
establish_app
|
|
after_fork
|
|
end
|
|
|
|
write_pid_file
|
|
end
|
|
|
|
def already_running?
|
|
if File.exist? pid_file
|
|
pid = File.read(pid_file).to_i
|
|
return pid if Demon::Base.alive?(pid)
|
|
end
|
|
|
|
nil
|
|
end
|
|
|
|
def self.alive?(pid)
|
|
Process.kill(0, pid)
|
|
true
|
|
rescue StandardError
|
|
false
|
|
end
|
|
|
|
private
|
|
|
|
def verbose(msg)
|
|
puts msg if @verbose
|
|
end
|
|
|
|
def write_pid_file
|
|
verbose("writing pid file #{pid_file} for #{@pid}")
|
|
FileUtils.mkdir_p(@rails_root + "tmp/pids")
|
|
File.open(pid_file, "w") { |f| f.write(@pid) }
|
|
end
|
|
|
|
def delete_pid_file
|
|
File.delete(pid_file)
|
|
end
|
|
|
|
def monitor_parent
|
|
Thread.new do
|
|
while true
|
|
begin
|
|
unless alive?(@parent_pid)
|
|
Process.kill "TERM", Process.pid
|
|
sleep 10
|
|
Process.kill "KILL", Process.pid
|
|
end
|
|
rescue => e
|
|
log("URGENT monitoring thread had an exception #{e}")
|
|
end
|
|
sleep 1
|
|
end
|
|
end
|
|
end
|
|
|
|
def suppress_stdout
|
|
true
|
|
end
|
|
|
|
def suppress_stderr
|
|
true
|
|
end
|
|
|
|
def establish_app
|
|
Discourse.after_fork if defined?(Discourse)
|
|
|
|
Signal.trap("HUP") do
|
|
begin
|
|
delete_pid_file
|
|
ensure
|
|
# TERM is way cleaner than exit
|
|
Process.kill("TERM", Process.pid)
|
|
end
|
|
end
|
|
|
|
# keep stuff simple for now
|
|
$stdout.reopen("/dev/null", "w") if suppress_stdout
|
|
$stderr.reopen("/dev/null", "w") if suppress_stderr
|
|
end
|
|
|
|
def after_fork
|
|
end
|
|
end
|