Refactor `period_match?` to reuse `task`

20fb4b74 · Gabriel Mazetto · 60a22a59 · 20fb4b74
Commit 20fb4b74 authored May 08, 2018 by Gabriel Mazetto
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 2 deletions

ee/app/services/geo/project_housekeeping_service.rb ee/app/services/geo/project_housekeeping_service.rb +2 -2

No files found.
--- a/ee/app/services/geo/project_housekeeping_service.rb
+++ b/ee/app/services/geo/project_housekeeping_service.rb
@@ -76,13 +76,13 @@ module Geo
        :gc
      elsif syncs_since_gc % full_repack_period == 0
        :full_repack
-      else
+      elsif syncs_since_gc % repack_period == 0
        :incremental_repack
      end
    end
    def period_match?
-      [gc_period, full_repack_period, repack_period].any? { |period| syncs_since_gc % period == 0 }
+      task.present?
    end
    def housekeeping_enabled?