diff --git a/ee/app/models/gitlab_subscription.rb b/ee/app/models/gitlab_subscription.rb index 7d81f3ebbc532a33d93f64a8aed3337587019f7f..13c5566890280a200705ffee9efb98323037fd48 100644 --- a/ee/app/models/gitlab_subscription.rb +++ b/ee/app/models/gitlab_subscription.rb @@ -31,9 +31,9 @@ class GitlabSubscription < ApplicationRecord scope :preload_for_refresh_seat, -> { preload([{ namespace: :route }, :hosted_plan]) } - DAYS_AFTER_EXPIRATION_BEFORE_REMOVING_FROM_INDEX = 7 + DAYS_AFTER_EXPIRATION_BEFORE_REMOVING_FROM_INDEX = 30 - # We set a 7 days as the threshold for expiration before removing them from + # We set a threshold for expiration before removing them from # the index def self.yield_long_expired_indexed_namespaces(&blk) # Since the gitlab_subscriptions table will keep growing in size and the diff --git a/ee/spec/models/gitlab_subscription_spec.rb b/ee/spec/models/gitlab_subscription_spec.rb index b1fbe084d8214e83fa43edaa0b56d21b07dde73c..c960dc4a8c581924f64cb1198b5ab7b888d3bdef 100644 --- a/ee/spec/models/gitlab_subscription_spec.rb +++ b/ee/spec/models/gitlab_subscription_spec.rb @@ -476,8 +476,8 @@ let_it_be(:not_expired_subscription1) { create(:gitlab_subscription, :bronze, end_date: Date.today + 2) } let_it_be(:not_expired_subscription2) { create(:gitlab_subscription, :bronze, end_date: Date.today + 100) } let_it_be(:recently_expired_subscription) { create(:gitlab_subscription, :bronze, end_date: Date.today - 4) } - let_it_be(:expired_subscription1) { create(:gitlab_subscription, :bronze, end_date: Date.today - 8) } - let_it_be(:expired_subscription2) { create(:gitlab_subscription, :bronze, end_date: Date.today - 10) } + let_it_be(:expired_subscription1) { create(:gitlab_subscription, :bronze, end_date: Date.today - 31) } + let_it_be(:expired_subscription2) { create(:gitlab_subscription, :bronze, end_date: Date.today - 40) } before do allow(::Gitlab).to receive(:dev_env_or_com?).and_return(true) diff --git a/ee/spec/workers/elastic_remove_expired_namespace_subscriptions_from_index_cron_worker_spec.rb b/ee/spec/workers/elastic_remove_expired_namespace_subscriptions_from_index_cron_worker_spec.rb index 17b55a38b0f6302814e5450da76eac9fd21fd5c4..76741f3359c225d502f2042b27e7d977b0c10666 100644 --- a/ee/spec/workers/elastic_remove_expired_namespace_subscriptions_from_index_cron_worker_spec.rb +++ b/ee/spec/workers/elastic_remove_expired_namespace_subscriptions_from_index_cron_worker_spec.rb @@ -8,8 +8,8 @@ let(:not_expired_subscription1) { create(:gitlab_subscription, :bronze, end_date: Date.today + 2) } let(:not_expired_subscription2) { create(:gitlab_subscription, :bronze, end_date: Date.today + 100) } let(:recently_expired_subscription) { create(:gitlab_subscription, :bronze, end_date: Date.today - 4) } - let(:expired_subscription1) { create(:gitlab_subscription, :bronze, end_date: Date.today - 8) } - let(:expired_subscription2) { create(:gitlab_subscription, :bronze, end_date: Date.today - 10) } + let(:expired_subscription1) { create(:gitlab_subscription, :bronze, end_date: Date.today - 31) } + let(:expired_subscription2) { create(:gitlab_subscription, :bronze, end_date: Date.today - 35) } before do allow(::Gitlab).to receive(:dev_env_or_com?).and_return(true)