Browse Source

Fix and improve batch attachment deletion handling when using OpenStack Swift (#32637)

pull/1368/head
Hugo Gameiro 1 year ago committed by Claire
parent
commit
a9477a992d
  1. 16
      app/lib/attachment_batch.rb

16
app/lib/attachment_batch.rb

@ -76,10 +76,22 @@ class AttachmentBatch
when :fog
logger.debug { "Deleting #{attachment.path(style)}" }
retries = 0
begin
attachment.send(:directory).files.new(key: attachment.path(style)).destroy
rescue Fog::Storage::OpenStack::NotFound
# Ignore failure to delete a file that has already been deleted
rescue Fog::OpenStack::Storage::NotFound
logger.debug "Will ignore because file is not found #{attachment.path(style)}"
rescue => e
retries += 1
if retries < MAX_RETRY
logger.debug "Retry #{retries}/#{MAX_RETRY} after #{e.message}"
sleep 2**retries
retry
else
logger.error "Batch deletion from fog failed after #{e.message}"
raise e
end
end
when :azure
logger.debug { "Deleting #{attachment.path(style)}" }

Loading…
Cancel
Save