Merge pull request fog#1254 from estonfer/modify_volume_attribute

[AWS] Adds ModifyVolumeAttribute

Author: estonfer

lib/fog/aws/compute.rb

@@ -113,6 +113,7 @@ class AWS < Fog::Service
       request :modify_instance_attribute
       request :modify_network_interface_attribute
       request :modify_snapshot_attribute
+      request :modify_volume_attribute
       request :purchase_reserved_instances_offering
       request :reboot_instances
       request :release_address

lib/fog/aws/models/compute/volume.rb

@@ -125,7 +125,6 @@ def detach(force = false)
         end
 
       end
-
     end
   end
 end

lib/fog/aws/requests/compute/modify_volume_attribute.rb

@@ -0,0 +1,51 @@
+module Fog
+  module Compute
+    class AWS
+      class Real
+
+        require 'fog/aws/parsers/compute/basic'
+
+        # Modifies a volume attribute.
+        #
+        # ==== Parameters
+        # * volume_id<~String> - The ID of the volume.
+        # * auto_enable_io_value<~Boolean> - This attribute exists to auto-enable the I/O operations to the volume.
+        #
+        # ==== Returns
+        # * response<~Excon::Response>:
+        #   * body<~Hash>:
+        #     * 'requestId'<~String> - Id of request
+        #     * 'return'<~Boolean> - success?
+        #
+        # {Amazon API Reference}[http://http://docs.amazonwebservices.com/AWSEC2/latest/APIReference/ApiReference-query-ModifyVolumeAttribute.html]
+        def modify_volume_attribute(volume_id=nil, auto_enable_io_value=false)
+          request(
+            'Action'             => 'ModifyVolumeAttribute',
+            'VolumeId'           => volume_id,
+            'AutoEnableIO.Value' => auto_enable_io_value,
+            :idempotent          => true,
+            :parser              => Fog::Parsers::Compute::AWS::Basic.new
+          )
+        end
+
+      end
+
+      class Mock
+
+        def modify_volume_attribute(volume_id=nil, auto_enable_io_value=false)
+          response = Excon::Response.new
+          if volume = self.data[:volumes][volume_id]
+            response.status = 200
+            response.body = {
+              'requestId' => Fog::AWS::Mock.request_id,
+              'return'    => true
+            }
+            response
+          else
+            raise Fog::Compute::AWS::NotFound.new("The volume '#{volume_id}' does not exist.")
+          end
+        end
+      end
+    end
+  end
+end

tests/aws/requests/compute/volume_tests.rb

@@ -136,6 +136,10 @@
 
     Fog::Compute[:aws].volumes.get(@volume_id).wait_for { ready? }
 
+    tests("#modify_volume_attribute('#{@volume_id}', true)").formats(AWS::Compute::Formats::BASIC) do
+      Fog::Compute[:aws].modify_volume_attribute(@volume_id, true).body
+    end
+
     tests("#delete_volume('#{@volume_id}')").formats(AWS::Compute::Formats::BASIC) do
       Fog::Compute[:aws].delete_volume(@volume_id).body
     end
@@ -156,6 +160,10 @@
       Fog::Compute[:aws].detach_volume('vol-00000000')
     end
 
+    tests("#modify_volume_attribute('vol-00000000', true)").raises(Fog::Compute::AWS::NotFound) do
+      Fog::Compute[:aws].modify_volume_attribute('vol-00000000', true)
+    end
+
     tests("#detach_volume('#{@volume.identity}')").raises(Fog::Compute::AWS::Error) do
       Fog::Compute[:aws].detach_volume(@volume.identity)
     end