Make it possible to specify the backup bucket region. (#226)

acquia · Jun 8, 2017 · c856631 · c856631
1 parent de10116
commit c856631
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/lib/plugins/backup.rb b/lib/plugins/backup.rb
@@ -14,7 +14,8 @@ class Backup # rubocop:disable Metrics/ClassLength
                     :hooks,
                     :target_name,
                     :backup_parameters,
-                    :backup_template
+                    :backup_template,
+                    :bucket_region
 
       def initialize
         yield self if block_given?
@@ -162,7 +163,9 @@ def zip(io_tar)
       #
       # @param io_zip [IO] tar stream
       def upload(io_zip)
-        s3_client.put_object(
+        opts = {}
+        opts[:region] = @bucket_region if @bucket_region
+        s3_client(opts).put_object(
           acl: 'private',
           bucket: @target_bucket,
           key: @target_name,

diff --git a/spec/moonshot/plugins/backup_spec.rb b/spec/moonshot/plugins/backup_spec.rb
@@ -20,10 +20,12 @@
     it 'should yield self' do
       backup = subject.new do |b|
         b.bucket = 'test'
+        b.bucket_region = 'us-east-2'
         b.files = %w(sample files)
         b.hooks = [:sample, :hooks]
       end
       expect(backup.bucket).to eq('test')
+      expect(backup.bucket_region).to eq('us-east-2')
       expect(backup.files).to eq(%w(sample files))
       expect(backup.hooks).to eq([:sample, :hooks])
     end