Merge pull request #213 from cowboyrushforth/update_firewall_request

add support for update_firewall

Author: icco

lib/fog/compute/google.rb

@@ -101,6 +101,7 @@ class Google < Fog::Service
       request :insert_backend_service
       request :insert_disk
       request :insert_firewall
+      request :update_firewall
       request :insert_forwarding_rule
       request :insert_global_forwarding_rule
       request :insert_http_health_check

lib/fog/compute/google/models/firewall.rb

@@ -28,6 +28,15 @@ def save
           reload
         end
 
+        def update
+          requires :identity, :allowed, :network
+
+          data = service.update_firewall(identity, allowed, network, attributes)
+          operation = Fog::Compute::Google::Operations.new(:service => service).get(data.body["name"])
+          operation.wait_for { !pending? }
+          reload
+        end
+
         def destroy(async = true)
           requires :identity
 

lib/fog/compute/google/requests/update_firewall.rb

@@ -0,0 +1,43 @@
+module Fog
+  module Compute
+    class Google
+      class Mock
+        def update_firewall(_firewall_name, _allowed, _network = GOOGLE_COMPUTE_DEFAULT_NETWORK, _options = {})
+          Fog::Mock.not_implemented
+        end
+      end
+
+      class Real
+        def update_firewall(firewall_name, allowed, network = GOOGLE_COMPUTE_DEFAULT_NETWORK, options = {})
+          unless network.start_with? "http"
+            network = "#{@api_url}#{@project}/global/networks/#{network}"
+          end
+
+          api_method = @compute.firewalls.update
+          parameters = {
+            "project" => @project,
+            "firewall" => firewall_name
+          }
+          body_object = {
+            "allowed" => allowed,
+            "network" => network
+          }
+          unless options[:description].nil?
+            body_object["description"] = options[:description]
+          end
+          unless options[:source_ranges].nil? || options[:source_ranges].empty?
+            body_object["sourceRanges"] = options[:source_ranges]
+          end
+          unless options[:source_tags].nil? || options[:source_tags].empty?
+            body_object["sourceTags"] = options[:source_tags]
+          end
+          unless options[:target_tags].nil? || options[:target_tags].empty?
+            body_object["targetTags"] = options[:target_tags]
+          end
+
+          request(api_method, parameters, body_object)
+        end
+      end
+    end
+  end
+end