Skip expire_at correction query if there are no artifacts

4800947b · drew cimino · Marius Bobin · 19408e9d · 4800947b
Commit 4800947b authored Apr 04, 2022 by drew cimino Committed by Marius Bobin Apr 04, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

app/services/ci/job_artifacts/destroy_batch_service.rb app/services/ci/job_artifacts/destroy_batch_service.rb +1 -1

No files found.
--- a/app/services/ci/job_artifacts/destroy_batch_service.rb
+++ b/app/services/ci/job_artifacts/destroy_batch_service.rb
@@ -117,7 +117,7 @@ module Ci

        wrongly_expired_artifacts, @job_artifacts = @job_artifacts.partition { |artifact| wrongly_expired?(artifact) }

-        remove_expire_at(wrongly_expired_artifacts)
+        remove_expire_at(wrongly_expired_artifacts) if wrongly_expired_artifacts.any?
      end

      def fix_expire_at?