Fix sidekiq stats in admin area
Added tests Added changelog entry Resolved all issues in code
This commit is contained in:
parent
3722099015
commit
40822b6185
3 changed files with 30 additions and 7 deletions
|
@ -5,15 +5,11 @@ module SidekiqHelper
|
|||
(?<mem>[\d\.,]+)\s+
|
||||
(?<state>[DRSTWXZNLsl\+<]+)\s+
|
||||
(?<start>.+)\s+
|
||||
(?<command>sidekiq.*\])\s*
|
||||
(?<command>sidekiq.*\])
|
||||
\z/x
|
||||
|
||||
def parse_sidekiq_ps(line)
|
||||
match = line.match(SIDEKIQ_PS_REGEXP)
|
||||
if match
|
||||
match[1..6]
|
||||
else
|
||||
%w[? ? ? ? ? ?]
|
||||
end
|
||||
match = line.strip.match(SIDEKIQ_PS_REGEXP)
|
||||
match ? match[1..6] : Array.new(6, '?')
|
||||
end
|
||||
end
|
||||
|
|
|
@ -0,0 +1,4 @@
|
|||
---
|
||||
title: Sidekiq stats in the admin area will now show correctly on different platforms
|
||||
merge_request:
|
||||
author: blackst0ne
|
|
@ -30,6 +30,29 @@ describe SidekiqHelper do
|
|||
expect(parts).to eq(['55137', '10.0', '2.1', 'S+', '2:30pm', 'sidekiq 4.1.4 gitlab [0 of 25 busy]'])
|
||||
end
|
||||
|
||||
it 'parses OSX output' do
|
||||
line = ' 1641 1.5 3.8 S+ 4:04PM sidekiq 4.2.1 gitlab [0 of 25 busy]'
|
||||
parts = helper.parse_sidekiq_ps(line)
|
||||
|
||||
expect(parts).to eq(['1641', '1.5', '3.8', 'S+', '4:04PM', 'sidekiq 4.2.1 gitlab [0 of 25 busy]'])
|
||||
end
|
||||
|
||||
it 'parses Ubuntu output' do
|
||||
# Ubuntu Linux 16.04 LTS / procps-3.3.10-4ubuntu2
|
||||
line = ' 938 1.4 2.5 Sl+ 21:23:21 sidekiq 4.2.1 gitlab [0 of 25 busy] '
|
||||
parts = helper.parse_sidekiq_ps(line)
|
||||
|
||||
expect(parts).to eq(['938', '1.4', '2.5', 'Sl+', '21:23:21', 'sidekiq 4.2.1 gitlab [0 of 25 busy]'])
|
||||
end
|
||||
|
||||
it 'parses Debian output' do
|
||||
# Debian Linux Wheezy/Jessie
|
||||
line = '17725 1.0 12.1 Ssl 19:20:15 sidekiq 4.2.1 gitlab-rails [0 of 25 busy] '
|
||||
parts = helper.parse_sidekiq_ps(line)
|
||||
|
||||
expect(parts).to eq(['17725', '1.0', '12.1', 'Ssl', '19:20:15', 'sidekiq 4.2.1 gitlab-rails [0 of 25 busy]'])
|
||||
end
|
||||
|
||||
it 'does fail gracefully on line not matching the format' do
|
||||
line = '55137 10.0 2.1 S+ 2:30pm something'
|
||||
parts = helper.parse_sidekiq_ps(line)
|
||||
|
|
Loading…
Reference in a new issue