Commit 77cc86e1 authored by Stan Hu's avatar Stan Hu

Revert "No need to use Gitlab::Geo::LogHelpers in EventGapTracking"

This reverts commit f4078760.
parent f4078760
......@@ -2,6 +2,7 @@ module Gitlab
module Geo
class EventGapTracking
include Utils::StrongMemoize
include ::Gitlab::Geo::LogHelpers
attr_accessor :previous_id
......@@ -60,7 +61,7 @@ module Gitlab
end
def track_gaps(current_id)
logger.info("Event log gap detected", previous_event_id: previous_id, current_event_id: current_id)
log_info("Event log gap detected", previous_event_id: previous_id, current_event_id: current_id)
with_redis do |redis|
expire_time = Time.now.to_i
......
......@@ -104,8 +104,7 @@ describe Gitlab::Geo::EventGapTracking, :clean_gitlab_redis_cache do
describe '#track_gaps' do
it 'logs a message' do
expect(Gitlab::Geo::Logger).to receive(:info)
.with(/gap detected/, hash_including(previous_event_id: previous_event_id, current_event_id: event_id_with_gap))
expect(gap_tracking).to receive(:log_info).with(/gap detected/, hash_including(previous_event_id: previous_event_id, current_event_id: event_id_with_gap))
gap_tracking.track_gaps(event_id_with_gap)
end
......
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