Commit eef31d87 authored by Micaël Bergeron's avatar Micaël Bergeron

remove the abort call in the Pseudonymizer::Uploader

After taking another look at the code it looks like this was actually
extraneous anyways. We already test for the availability of the
backend before doing the upload.
parent aee7e524
...@@ -36,8 +36,6 @@ module Pseudonymizer ...@@ -36,8 +36,6 @@ module Pseudonymizer
file_list.each do |file| file_list.each do |file|
upload_file(file, remote_directory) upload_file(file, remote_directory)
end end
rescue ObjectStorageUnavailableError
abort "Cannot upload files, make sure the `pseudonimizer.upload.connection` is set properly"
end end
def cleanup def cleanup
......
...@@ -13,7 +13,9 @@ namespace :gitlab do ...@@ -13,7 +13,9 @@ namespace :gitlab do
dumper = Pseudonymizer::Dumper.new(options) dumper = Pseudonymizer::Dumper.new(options)
uploader = Pseudonymizer::Uploader.new(options) uploader = Pseudonymizer::Uploader.new(options)
abort "There is an error in the pseudonymizer object store configuration." unless uploader.available? unless uploader.available?
abort "There is an error in the pseudonymizer object store configuration."
end
begin begin
dumper.tables_to_csv dumper.tables_to_csv
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment