Merge pull request #15064 from isimluk/avoid-queue-overhead-if-sessio…

…n-is-turned-off

Performance: Do not schedule Session.purge if this Session is not used
(cherry picked from commit 493a488)

https://bugzilla.redhat.com/show_bug.cgi?id=1510054

app/models/miq_schedule_worker/runner.rb

@@ -184,10 +184,12 @@ def schedules_for_scheduler_role
       enqueue :drift_state_purge_timer
     end
 
-    # Schedule - Check for session timeouts
-    scheduler.schedule_every(worker_settings[:session_timeout_interval]) do
-      # Session is global to the region, therefore, run it only once on the scheduler's server
-      enqueue :session_check_session_timeout
+    if Session.enabled?
+      # Schedule - Check for session timeouts
+      scheduler.schedule_every(worker_settings[:session_timeout_interval]) do
+        # Session is global to the region, therefore, run it only once on the scheduler's server
+        enqueue :session_check_session_timeout
+      end
     end
 
     # Schedule - Check for rogue EVM snapshots

app/models/session.rb

@@ -3,6 +3,10 @@ class Session < ApplicationRecord
   @@interval = 30 # how often to purge in seconds
   @@job ||= nil
 
+  def self.enabled?
+    ::Settings.server.session_store == 'sql'
+  end
+
   def self.check_session_timeout
     $log.debug "Checking session data"
     purge(::Settings.session.timeout)