Revert "API-26059 Log durations of some sidekiq jobs (#12474)" (#12840)

This reverts commit 2a0bead.
department-of-veterans-affairs · May 31, 2023 · cb632a7 · cb632a7
1 parent c589b29
commit cb632a7
Show file tree

Hide file tree

Showing 7 changed files with 0 additions and 85 deletions.
diff --git a/config/initializers/sidekiq.rb b/config/initializers/sidekiq.rb
@@ -7,7 +7,6 @@
 require 'sidekiq/semantic_logging'
 require 'sidekiq/set_request_id'
 require 'sidekiq/set_request_attributes'
-require 'sidekiq/middleware/benchmark_logging_middleware'
 require 'datadog/statsd' # gem 'dogstatsd-ruby'
 
 Rails.application.reloader.to_prepare do
@@ -32,7 +31,6 @@
       chain.add SidekiqStatsInstrumentation::ServerMiddleware
       chain.add Sidekiq::RetryMonitoring
       chain.add Sidekiq::ErrorTag
-      chain.add Sidekiq::Middleware::BenchmarkLoggingMiddleware
 
       if Settings.dogstatsd.enabled == true
         require 'sidekiq/middleware/server/statsd'

diff --git a/lib/sidekiq/benchmark_logging_worker.rb b/lib/sidekiq/benchmark_logging_worker.rb
diff --git a/lib/sidekiq/middleware/benchmark_logging_middleware.rb b/lib/sidekiq/middleware/benchmark_logging_middleware.rb
diff --git a/modules/appeals_api/app/workers/appeals_api/flipper_status_alert.rb b/modules/appeals_api/app/workers/appeals_api/flipper_status_alert.rb
@@ -1,14 +1,11 @@
 # frozen_string_literal: true
 
 require 'sidekiq'
-require 'sidekiq/benchmark_logging_worker'
 require 'flipper/utilities/bulk_feature_checker'
-require 'sidekiq/middleware/benchmark_logging_middleware'
 
 module AppealsApi
   class FlipperStatusAlert
     include Sidekiq::Worker
-    include Sidekiq::BenchmarkLoggingWorker
 
     WARNING_EMOJI = ':warning:'
     DISABLED_FLAG_EMOJI = ':vertical_traffic_light:'

diff --git a/modules/va_forms/app/workers/va_forms/flipper_status_alert.rb b/modules/va_forms/app/workers/va_forms/flipper_status_alert.rb
@@ -1,14 +1,11 @@
 # frozen_string_literal: true
 
 require 'sidekiq'
-require 'sidekiq/benchmark_logging_worker'
 require 'flipper/utilities/bulk_feature_checker'
-require 'sidekiq/middleware/benchmark_logging_middleware'
 
 module VAForms
   class FlipperStatusAlert
     include Sidekiq::Worker
-    include Sidekiq::BenchmarkLoggingWorker
 
     WARNING_EMOJI = ':warning:'
     DISABLED_FLAG_EMOJI = ':vertical_traffic_light:'

diff --git a/modules/vba_documents/app/workers/vba_documents/flipper_status_alert.rb b/modules/vba_documents/app/workers/vba_documents/flipper_status_alert.rb
@@ -1,14 +1,11 @@
 # frozen_string_literal: true
 
 require 'sidekiq'
-require 'sidekiq/benchmark_logging_worker'
 require 'flipper/utilities/bulk_feature_checker'
-require 'sidekiq/middleware/benchmark_logging_middleware'
 
 module VBADocuments
   class FlipperStatusAlert
     include Sidekiq::Worker
-    include Sidekiq::BenchmarkLoggingWorker
 
     WARNING_EMOJI = ':warning:'
     DISABLED_FLAG_EMOJI = ':vertical_traffic_light:'

diff --git a/spec/lib/sidekiq/middleware/benchmark_logging_middleware_spec.rb b/spec/lib/sidekiq/middleware/benchmark_logging_middleware_spec.rb