Skip to content
Snippets Groups Projects
Commit ed9cb857 authored by Thong Kuah's avatar Thong Kuah :speech_balloon: Committed by 🤖 GitLab Bot 🤖
Browse files

Merge branch 'sh-fix-sidekiq-scheduling-latency' into 'master'

Fix Sidekiq scheduling_latency_s

Closes #65748

See merge request gitlab-org/gitlab-ce!31650

(cherry picked from commit 9e19125f)

7ccbb562 Fix Sidekiq scheduling_latency_s
parent a00afee1
No related branches found
No related tags found
No related merge requests found
Loading
Loading
@@ -35,7 +35,7 @@ module Gitlab
# Old gitlab-shell messages don't provide enqueued_at/created_at attributes
enqueued_at = payload['enqueued_at'] || payload['created_at']
if enqueued_at
payload['scheduling_latency_s'] = elapsed(Time.iso8601(enqueued_at).to_f)
payload['scheduling_latency_s'] = elapsed_by_absolute_time(Time.iso8601(enqueued_at))
end
 
payload
Loading
Loading
@@ -84,6 +84,10 @@ module Gitlab
end
end
 
def elapsed_by_absolute_time(start)
(Time.now.utc - start).to_f.round(3)
end
def elapsed(start)
(current_time - start).round(3)
end
Loading
Loading
Loading
Loading
@@ -3,8 +3,8 @@ require 'spec_helper'
describe Gitlab::SidekiqLogging::StructuredLogger do
describe '#call' do
let(:timestamp) { Time.iso8601('2018-01-01T12:00:00Z') }
let(:created_at) { timestamp }
let(:scheduling_latency_s) { 0.0 }
let(:created_at) { timestamp - 1.second }
let(:scheduling_latency_s) { 1.0 }
 
let(:job) do
{
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment