[Automated] Merged master into target k8s

department-of-veterans-affairs · Apr 12, 2024 · 1a69f86 · 1a69f86
2 parents 551d667 + 4320aaf
commit 1a69f86
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/app/sidekiq/terms_of_use/sign_up_service_updater_job.rb b/app/sidekiq/terms_of_use/sign_up_service_updater_job.rb
@@ -7,8 +7,7 @@ module TermsOfUse
   class SignUpServiceUpdaterJob
     include Sidekiq::Job
 
-    sidekiq_options unique_for: 2.days
-    sidekiq_options retry: 15 # 2.1 days using exponential backoff
+    sidekiq_options retry: 5 # ~17 mins
 
     sidekiq_retries_exhausted do |job, exception|
       Rails.logger.warn(

diff --git a/spec/sidekiq/terms_of_use/sign_up_service_updater_job_spec.rb b/spec/sidekiq/terms_of_use/sign_up_service_updater_job_spec.rb
@@ -23,8 +23,8 @@
       allow(Sidekiq::AttrPackage).to receive(:delete)
     end
 
-    it 'retries 15 times after failure' do
-      expect(described_class.get_sidekiq_options['retry']).to eq(15)
+    it 'retries 5 times after failure' do
+      expect(described_class.get_sidekiq_options['retry']).to eq(5)
     end
 
     it 'logs a message when retries have been exhausted' do
@@ -45,8 +45,6 @@
         )
     end
 
-    it { is_expected.to be_unique }
-
     context 'when the terms of use agreement is accepted' do
       before do
         allow(service_instance).to receive(:agreements_accept)