diff options
| author | Nick Thomas <nick@gitlab.com> | 2019-03-01 10:35:28 +0000 |
|---|---|---|
| committer | Nick Thomas <nick@gitlab.com> | 2019-03-01 10:38:38 +0000 |
| commit | adddbf437dc9702e06cc6ec2fd8a8b1c38cccdeb (patch) | |
| tree | 70fc384a9c9787a8a9aa810af644d73d4b54a5a9 | |
| parent | b98fd8a5a086e08c4d940c163fea70238b6cffb6 (diff) | |
| download | gitlab-ce-adddbf437dc9702e06cc6ec2fd8a8b1c38cccdeb.tar.gz | |
SIGSTP should be SIGTSTP
| -rw-r--r-- | lib/gitlab/sidekiq_middleware/memory_killer.rb | 2 | ||||
| -rw-r--r-- | spec/lib/gitlab/sidekiq_middleware/memory_killer_spec.rb | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/lib/gitlab/sidekiq_middleware/memory_killer.rb b/lib/gitlab/sidekiq_middleware/memory_killer.rb index ad4112ba555..47333d257eb 100644 --- a/lib/gitlab/sidekiq_middleware/memory_killer.rb +++ b/lib/gitlab/sidekiq_middleware/memory_killer.rb @@ -31,7 +31,7 @@ module Gitlab # Wait `GRACE_TIME` to give the memory intensive job time to finish. # Then, tell Sidekiq to stop fetching new jobs. - wait_and_signal(GRACE_TIME, 'SIGSTP', 'stop fetching new jobs') + wait_and_signal(GRACE_TIME, 'SIGTSTP', 'stop fetching new jobs') # Wait `SHUTDOWN_WAIT` to give already fetched jobs time to finish. # Then, tell Sidekiq to gracefully shut down by giving jobs a few more diff --git a/spec/lib/gitlab/sidekiq_middleware/memory_killer_spec.rb b/spec/lib/gitlab/sidekiq_middleware/memory_killer_spec.rb index 8fdbbacd04d..5df56178df2 100644 --- a/spec/lib/gitlab/sidekiq_middleware/memory_killer_spec.rb +++ b/spec/lib/gitlab/sidekiq_middleware/memory_killer_spec.rb @@ -37,7 +37,7 @@ describe Gitlab::SidekiqMiddleware::MemoryKiller do it 'sends the STP, TERM and KILL signals at expected times' do expect(subject).to receive(:sleep).with(15 * 60).ordered - expect(Process).to receive(:kill).with('SIGSTP', pid).ordered + expect(Process).to receive(:kill).with('SIGTSTP', pid).ordered expect(subject).to receive(:sleep).with(30).ordered expect(Process).to receive(:kill).with('SIGTERM', pid).ordered |