Enable puma by default in GDK

Two changes:
1. Move bin/web to bin/web_unicorn(removed PUMA switching logic)
Introduce new shadow script bin/web. Now we have 3 scripts working
together: bin/web, bin/web_puma, bin/web_unicorn

  In bin/web, it checks ENV['USE_WEB_SERVER']:
    - if value is 'puma' or value is not set, call bin/web_puma
    - if value is 'unicorn', call bin/web_unicorn
    - report error if other values given

2. Gitlab rails allow ENV setting to override timeout
    - ENV['GITLAB_RAILS_RACK_TIMEOUT'] for service_timeout
    - ENV['GITLAB_RAILS_WAIT_TIMEOUT'] for wait_timeout
This commit is contained in:
Qingyu Zhao 2019-07-03 14:11:48 +10:00
parent 8a2c53d640
commit b7ea4bd853
3 changed files with 73 additions and 57 deletions

62
bin/web
View file

@ -1,63 +1,21 @@
#!/bin/sh #!/bin/sh
set -e
cd $(dirname $0)/.. cd $(dirname $0)/..
app_root=$(pwd) app_root=$(pwd)
# Switch to experimental PUMA configuration case "$USE_WEB_SERVER" in
if [ -n "${EXPERIMENTAL_PUMA}" ]; then puma|"") # and the "" defines default
exec bin/web_puma "$@" exec bin/web_puma "$@"
fi
unicorn_pidfile="$app_root/tmp/pids/unicorn.pid"
unicorn_config="$app_root/config/unicorn.rb"
unicorn_cmd="bundle exec unicorn_rails -c $unicorn_config -E $RAILS_ENV"
get_unicorn_pid()
{
local pid=$(cat $unicorn_pidfile)
if [ -z "$pid" ] ; then
echo "Could not find a PID in $unicorn_pidfile"
exit 1
fi
unicorn_pid=$pid
}
start()
{
exec $unicorn_cmd -D
}
start_foreground()
{
exec $unicorn_cmd
}
stop()
{
get_unicorn_pid
kill -QUIT $unicorn_pid
}
reload()
{
get_unicorn_pid
kill -USR2 $unicorn_pid
}
case "$1" in
start)
start
;; ;;
start_foreground)
start_foreground unicorn)
;; exec bin/web_unicorn "$@"
stop)
stop
;;
reload)
reload
;; ;;
*) *)
echo "Usage: RAILS_ENV=your_env $0 {start|stop|reload}" echo "Unkown web server used by USE_WEB_SERVER: $USE_WEB_SERVER."
exit 1
;; ;;
esac esac

58
bin/web_unicorn Executable file
View file

@ -0,0 +1,58 @@
#!/bin/sh
cd $(dirname $0)/..
app_root=$(pwd)
unicorn_pidfile="$app_root/tmp/pids/unicorn.pid"
unicorn_config="$app_root/config/unicorn.rb"
unicorn_cmd="bundle exec unicorn_rails -c $unicorn_config -E $RAILS_ENV"
get_unicorn_pid()
{
local pid=$(cat $unicorn_pidfile)
if [ -z "$pid" ] ; then
echo "Could not find a PID in $unicorn_pidfile"
exit 1
fi
unicorn_pid=$pid
}
start()
{
exec $unicorn_cmd -D
}
start_foreground()
{
exec $unicorn_cmd
}
stop()
{
get_unicorn_pid
kill -QUIT $unicorn_pid
}
reload()
{
get_unicorn_pid
kill -USR2 $unicorn_pid
}
case "$1" in
start)
start
;;
start_foreground)
start_foreground
;;
stop)
stop
;;
reload)
reload
;;
*)
echo "Usage: RAILS_ENV=your_env $0 {start|stop|reload}"
;;
esac

View file

@ -14,8 +14,8 @@ if defined?(::Puma) && !Rails.env.test?
Gitlab::Application.configure do |config| Gitlab::Application.configure do |config|
config.middleware.insert_before(Rack::Runtime, Rack::Timeout, config.middleware.insert_before(Rack::Runtime, Rack::Timeout,
service_timeout: 60, service_timeout: ENV.fetch('GITLAB_RAILS_RACK_TIMEOUT', 60).to_i,
wait_timeout: 90) wait_timeout: ENV.fetch('GITLAB_RAILS_WAIT_TIMEOUT', 90).to_i)
end end
observer = Gitlab::Cluster::RackTimeoutObserver.new observer = Gitlab::Cluster::RackTimeoutObserver.new