Merge branch 'sh-disable-reactive-caching-automatic-retries' into 'master'

Prevent amplification of ReactiveCachingWorker jobs upon failures

Closes #64176

See merge request gitlab-org/gitlab-ce!30432

(cherry picked from commit 162ef51f)

a28844ea Prevent amplification of ReactiveCachingWorker jobs upon failures

app/models/concerns/reactive_caching.rb

@@ -178,7 +178,7 @@ module ReactiveCaching
     def enqueuing_update(*args)
       yield
-    ensure
+
       ReactiveCachingWorker.perform_in(self.class.reactive_cache_refresh_interval, self.class, id, *args)
     end
   end

changelogs/unreleased/sh-disable-reactive-caching-automatic-retries.yml

+---
+title: Prevent amplification of ReactiveCachingWorker jobs upon failures
+merge_request: 30432
+author:
+type: performance

spec/models/concerns/reactive_caching_spec.rb

@@ -206,8 +206,9 @@ describe ReactiveCaching, :use_clean_rails_memory_store_caching do
           expect(read_reactive_cache(instance)).to eq("preexisting")
         end
-        it 'enqueues a repeat worker' do
-          expect_reactive_cache_update_queued(instance)
+        it 'does not enqueue a repeat worker' do
+          expect(ReactiveCachingWorker)
+            .not_to receive(:perform_in)
           expect { go! }.to raise_error("foo")
         end