Merge branch 'fix_sidekiq_concurrency_warning_message_in_admin_background_job_page' into 'master'

fix incorrect sidekiq concurrency count in admin background page

Closes #19862

See merge request !9359
This commit is contained in:
Rémy Coutable 2017-02-20 11:22:13 +00:00
commit dc303fe2db
3 changed files with 6 additions and 1 deletions

View File

@ -2,5 +2,6 @@ class Admin::BackgroundJobsController < Admin::ApplicationController
def show
ps_output, _ = Gitlab::Popen.popen(%W(ps -U #{Gitlab.config.gitlab.user} -o pid,pcpu,pmem,stat,start,command))
@sidekiq_processes = ps_output.split("\n").grep(/sidekiq/)
@concurrency = Sidekiq.options[:concurrency]
end
end

View File

@ -35,7 +35,7 @@
.clearfix
%p
%i.fa.fa-exclamation-circle
If '[25 of 25 busy]' is shown, restart GitLab with 'sudo service gitlab reload'.
If '[#{@concurrency} of #{@concurrency} busy]' is shown, restart GitLab with 'sudo service gitlab reload'.
%p
%i.fa.fa-exclamation-circle
If more than one sidekiq process is listed, stop GitLab, kill the remaining sidekiq processes (sudo pkill -u #{gitlab_config.user} -f sidekiq) and restart GitLab.

View File

@ -0,0 +1,4 @@
---
title: fix incorrect sidekiq concurrency count in admin background page
merge_request:
author: wendy0402