Commit 8145b48a authored by Thong Kuah's avatar Thong Kuah

Merge branch 'do-not-log-for-explicit-sidekiq-status-set-calls' into 'master'

Do not log explicit Gitlab::SidekiqStatus.set calls

See merge request gitlab-org/gitlab!76250
parents a7192128 0ee1bc8a
...@@ -15,7 +15,7 @@ module ImportState ...@@ -15,7 +15,7 @@ module ImportState
def refresh_jid_expiration def refresh_jid_expiration
return unless jid return unless jid
Gitlab::SidekiqStatus.set(jid, Gitlab::Import::StuckImportJob::IMPORT_JOBS_EXPIRATION) Gitlab::SidekiqStatus.set(jid, Gitlab::Import::StuckImportJob::IMPORT_JOBS_EXPIRATION, value: 2)
end end
def self.jid_by(project_id:, status:) def self.jid_by(project_id:, status:)
......
...@@ -13,7 +13,7 @@ module Gitlab ...@@ -13,7 +13,7 @@ module Gitlab
def self.set_jid(import_state) def self.set_jid(import_state)
jid = generate_jid(import_state) jid = generate_jid(import_state)
Gitlab::SidekiqStatus.set(jid, Gitlab::Import::StuckImportJob::IMPORT_JOBS_EXPIRATION) Gitlab::SidekiqStatus.set(jid, Gitlab::Import::StuckImportJob::IMPORT_JOBS_EXPIRATION, value: 2)
import_state.update_column(:jid, jid) import_state.update_column(:jid, jid)
end end
......
...@@ -8,7 +8,7 @@ RSpec.describe Gitlab::Import::SetAsyncJid do ...@@ -8,7 +8,7 @@ RSpec.describe Gitlab::Import::SetAsyncJid do
it 'sets the JID in Redis' do it 'sets the JID in Redis' do
expect(Gitlab::SidekiqStatus) expect(Gitlab::SidekiqStatus)
.to receive(:set) .to receive(:set)
.with("async-import/project-import-state/#{project.id}", Gitlab::Import::StuckImportJob::IMPORT_JOBS_EXPIRATION) .with("async-import/project-import-state/#{project.id}", Gitlab::Import::StuckImportJob::IMPORT_JOBS_EXPIRATION, value: 2)
.and_call_original .and_call_original
described_class.set_jid(project.import_state) described_class.set_jid(project.import_state)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment