fog-google 1.13.0 → 1.14.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b939fe151e79be2e5acd8c98082b4771e58819bbb0594dae868281b9b9c943b8
-  data.tar.gz: 8a5053cb43c779e8786da5d76c98040b7bf926064bd9831e061b38ff7fc410f4
+  metadata.gz: b08d74db716e12e161488ce7f6699d8ae3480eb9ec0c978d8eb9458fd6447b4f
+  data.tar.gz: f593447eed2cb01ea0f4d6a676d60c53749cac333809cef542ce1e5661c1e85f
 SHA512:
-  metadata.gz: 4f63042bb5c18d8dba6b765a338fbbe0d4b82f741b01c6defcd7ac06f53990da1e0d86fcf2aa830f68cf19c1146b3f0787538dfc5877a0f088eb5ddabf74bea5
-  data.tar.gz: 4a78088e4ec4074115b6cbe0b65784614a819e2a2cdb35ace605618eacd22f84a457034b7946f892c767e976227c38175b6030f2270e4ed01fe9f61dcf869ecf
+  metadata.gz: 2d34fa3e02b8f8dea154556f5b6ffe942f7dbeef630707730d88dc312f59ae644685d5f00aa319ebfd340d43334d796578e0e7649b1229bd3448eec7f4802c31
+  data.tar.gz: 3f7773d5d3ae3ad824f9a8a5dde09eb0727d0354bf995407628a62e58c1b5a8f5850f2a72e08ac7df95c0ee0a870f903b652eb2e8e19e49b34e4344064fe55b7

data/.github/workflows/ruby.yml

@@ -22,7 +22,7 @@ jobs:
         ruby-version: ['2.5', '2.6', '2.7', '3.0', 'head']
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v2.3.4
     - name: Set up Ruby
       uses: ruby/setup-ruby@v1
       with:

data/.github/workflows/stale.yml

@@ -10,7 +10,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-    - uses: actions/stale@v3
+    - uses: actions/stale@v3.0.18
       with:
         repo-token: ${{ secrets.GITHUB_TOKEN }}
         days-before-stale: 60

data/CHANGELOG.md

@@ -6,6 +6,33 @@ This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html
 
 ## Next
 
+## 1.14.0
+
+### User-facing
+
+#### Added
+
+- \#520 Support passing object properties to #copy_object [mshibuya]
+
+#### Fixed
+
+- \#530 Unescape slashes in urls [krokodaxl]
+
+- \#528 Fix Ruby 3.0 kwargs error in `#copy_object` [deeeki]
+
+- \#521 Unlink file temp file immediately [stanhu]
+
+- \#527 \#523 Fix Ruby 3.0 kwargs failures [stanhu]
+
+### Development changes
+
+#### Fixed
+
+- \#525 - Bump actions/checkout from 2 to 2.3.4
+
+- \#524 - Bump actions/stale from 3 to 3.0.18
+
+
 ## 1.13.0
 
 ### User-facing

data/lib/fog/compute/google/models/address.rb

@@ -38,7 +38,7 @@ module Fog
 
           data = service.insert_address(identity, region, attributes)
           operation = Fog::Compute::Google::Operations
-                      .new(:service => service)
+                      .new(service: service)
                       .get(data.name, nil, data.region)
           operation.wait_for { ready? }
           reload

data/lib/fog/compute/google/models/addresses.rb

@@ -13,10 +13,10 @@ module Fog
           }
 
           if region
-            data = service.list_addresses(region, opts).items || []
+            data = service.list_addresses(region, **opts).items || []
           else
             data = []
-            service.list_aggregated_addresses(opts).items.each_value do |scoped_list|
+            service.list_aggregated_addresses(**opts).items.each_value do |scoped_list|
               data.concat(scoped_list.addresses) if scoped_list && scoped_list.addresses
             end
           end
@@ -28,8 +28,8 @@ module Fog
             address = service.get_address(identity, region).to_h
             return new(address)
           elsif identity
-            response = all(:filter => "name eq #{identity}",
-                           :max_results => 1)
+            response = all(filter: "name eq #{identity}",
+                           max_results: 1)
             address = response.first unless response.empty?
             return address
           end
@@ -39,7 +39,7 @@ module Fog
         end
 
         def get_by_ip_address(ip_address)
-          addresses = service.list_aggregated_addresses(:filter => "address eq .*#{ip_address}").items
+          addresses = service.list_aggregated_addresses(filter: "address eq .*#{ip_address}").items
           address = addresses.each_value.select(&:addresses)
 
           return nil if address.empty?
@@ -47,7 +47,7 @@ module Fog
         end
 
         def get_by_name(ip_name)
-          names = service.list_aggregated_addresses(:filter => "name eq .*#{ip_name}").items
+          names = service.list_aggregated_addresses(filter: "name eq .*#{ip_name}").items
           name = names.each_value.select(&:addresses)
 
           return nil if name.empty?

data/lib/fog/compute/google/models/backend_service.rb

@@ -29,7 +29,7 @@ module Fog
             :timeout_sec => timeout_sec
           }
 
-          data = service.insert_backend_service(name, options)
+          data = service.insert_backend_service(name, **options)
           operation = Fog::Compute::Google::Operations.new(:service => service).get(data.name)
           operation.wait_for { ready? }
           reload

data/lib/fog/compute/google/models/disk.rb

@@ -51,8 +51,8 @@ module Fog
 
           # Request needs backward compatibility so source image is specified in
           # method arguments
-          data = service.insert_disk(name, zone, options[:source_image], options)
-          operation = Fog::Compute::Google::Operations.new(:service => service)
+          data = service.insert_disk(name, zone, options[:source_image], **options)
+          operation = Fog::Compute::Google::Operations.new(service: service)
                                                       .get(data.name, data.zone)
           operation.wait_for { ready? }
           reload
@@ -103,7 +103,7 @@ module Fog
         # @return [Hash] Attached disk configuration hash
         def attached_disk_obj(opts = {})
           requires :self_link
-          collection.attached_disk_obj(self_link, opts)
+          collection.attached_disk_obj(self_link, **opts)
         end
 
         # A legacy shorthand for attached_disk_obj

data/lib/fog/compute/google/models/disk_types.rb

@@ -13,10 +13,10 @@ module Fog
             :page_token => page_token
           }
           if zone
-            data = service.list_disk_types(zone, opts).items
+            data = service.list_disk_types(zone, **opts).items
           else
             data = []
-            service.list_aggregated_disk_types(opts).items.each_value do |scoped_lst|
+            service.list_aggregated_disk_types(**opts).items.each_value do |scoped_lst|
               data.concat(scoped_lst.disk_types) if scoped_lst && scoped_lst.disk_types
             end
           end

data/lib/fog/compute/google/models/disks.rb

@@ -13,10 +13,10 @@ module Fog
             :page_token => page_token
           }
           if zone
-            data = service.list_disks(zone, opts).items || []
+            data = service.list_disks(zone, **opts).items || []
           else
             data = []
-            service.list_aggregated_disks(opts).items.each_value do |scoped_list|
+            service.list_aggregated_disks(**opts).items.each_value do |scoped_list|
               data.concat(scoped_list.disks) if scoped_list.disks
             end
           end

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

@@ -53,7 +53,7 @@ module Fog
 
         def create
           data = service.insert_firewall(identity, attributes)
-          operation = Fog::Compute::Google::Operations.new(:service => service)
+          operation = Fog::Compute::Google::Operations.new(service: service)
                                                       .get(data.name)
           operation.wait_for { ready? }
           reload
@@ -63,7 +63,7 @@ module Fog
           requires :identity, :allowed, :network
 
           data = service.update_firewall(identity, attributes)
-          operation = Fog::Compute::Google::Operations.new(:service => service)
+          operation = Fog::Compute::Google::Operations.new(service: service)
                                                       .get(data.name)
           operation.wait_for { ready? }
           reload

data/lib/fog/compute/google/models/firewalls.rb

@@ -5,7 +5,7 @@ module Fog
         model Fog::Compute::Google::Firewall
 
         def all(opts = {})
-          data = service.list_firewalls(opts).to_h[:items]
+          data = service.list_firewalls(**opts).to_h[:items]
           load(data || [])
         end
 

data/lib/fog/compute/google/models/forwarding_rules.rb

@@ -13,10 +13,10 @@ module Fog
           }
 
           if region
-            data = service.list_forwarding_rules(region, opts).items || []
+            data = service.list_forwarding_rules(region, **opts).items || []
           else
             data = []
-            service.list_aggregated_forwarding_rules(opts).items
+            service.list_aggregated_forwarding_rules(**opts).items
                    .each_value do |scoped_list|
               if scoped_list && scoped_list.forwarding_rules
                 data.concat(scoped_list.forwarding_rules)

data/lib/fog/compute/google/models/global_addresses.rb

@@ -5,7 +5,7 @@ module Fog
         model Fog::Compute::Google::GlobalAddress
 
         def all(options = {})
-          data = service.list_global_addresses(options).to_h[:items] || []
+          data = service.list_global_addresses(**options).to_h[:items] || []
           load(data)
         end
 

data/lib/fog/compute/google/models/global_forwarding_rules.rb

@@ -5,7 +5,7 @@ module Fog
         model Fog::Compute::Google::GlobalForwardingRule
 
         def all(opts = {})
-          data = service.list_global_forwarding_rules(opts).to_h[:items] || []
+          data = service.list_global_forwarding_rules(**opts).to_h[:items] || []
           load(data)
         end
 

data/lib/fog/compute/google/models/http_health_check.rb

@@ -51,7 +51,7 @@ module Fog
           requires :name
 
           data = service.insert_http_health_check(name, opts)
-          operation = Fog::Compute::Google::Operations.new(:service => service)
+          operation = Fog::Compute::Google::Operations.new(service: service)
                                                       .get(data.name, data.zone)
           operation.wait_for { ready? }
           reload
@@ -60,7 +60,7 @@ module Fog
         def update(opts)
           requires :name
           data = service.update_http_health_check(name, opts)
-          operation = Fog::Compute::Google::Operations.new(:service => service)
+          operation = Fog::Compute::Google::Operations.new(service: service)
                                                       .get(data.name, data.zone)
           operation.wait_for { ready? }
           reload

data/lib/fog/compute/google/models/instance_group_managers.rb

@@ -14,10 +14,10 @@ module Fog
           }
 
           if zone
-            data = service.list_instance_group_managers(zone, opts).items || []
+            data = service.list_instance_group_managers(zone, **opts).items || []
           else
             data = []
-            service.list_aggregated_instance_group_managers(opts).items.each_value do |group|
+            service.list_aggregated_instance_group_managers(**opts).items.each_value do |group|
               data.concat(group.instance_group_managers) if group.instance_group_managers
             end
           end

data/lib/fog/compute/google/models/machine_types.rb

@@ -13,10 +13,10 @@ module Fog
           }
 
           if zone
-            data = service.list_machine_types(zone, opts).items
+            data = service.list_machine_types(zone, **opts).items
           else
             data = []
-            service.list_aggregated_machine_types(opts).items.each_value do |scoped_list|
+            service.list_aggregated_machine_types(**opts).items.each_value do |scoped_list|
               data.concat(scoped_list.machine_types) if scoped_list && scoped_list.machine_types
             end
           end

data/lib/fog/compute/google/models/operations.rb

@@ -14,11 +14,11 @@ module Fog
           }
 
           if zone
-            data = service.list_zone_operations(zone, opts).to_h[:items]
+            data = service.list_zone_operations(zone, **opts).to_h[:items]
           elsif region
-            data = service.list_region_operations(region, opts).to_h[:items]
+            data = service.list_region_operations(region, **opts).to_h[:items]
           else
-            data = service.list_global_operations(opts).to_h[:items]
+            data = service.list_global_operations(**opts).to_h[:items]
           end
 
           load(data || [])

data/lib/fog/compute/google/models/servers.rb

@@ -14,10 +14,10 @@ module Fog
           }
 
           if zone
-            data = service.list_servers(zone, opts).to_h[:items] || []
+            data = service.list_servers(zone, **opts).to_h[:items] || []
           else
             data = []
-            service.list_aggregated_servers(opts).items.each_value do |scoped_lst|
+            service.list_aggregated_servers(**opts).items.each_value do |scoped_lst|
               if scoped_lst && scoped_lst.instances
                 data.concat(scoped_lst.instances.map(&:to_h))
               end
@@ -57,7 +57,7 @@ module Fog
               :zone_name => zone_name,
               :source_image => source_img.self_link
             }
-            disk = service.disks.create(disk_defaults.merge(opts))
+            disk = service.disks.create(**disk_defaults.merge(opts))
             disk.wait_for { disk.ready? }
 
             disks = [disk]

data/lib/fog/compute/google/models/subnetworks.rb

@@ -13,10 +13,10 @@ module Fog
           }
 
           if region
-            data = service.list_subnetworks(region, filters).to_h[:items] || []
+            data = service.list_subnetworks(region, **filters).to_h[:items] || []
           else
             data = []
-            service.list_aggregated_subnetworks(filters).to_h[:items].each_value do |region_obj|
+            service.list_aggregated_subnetworks(**filters).to_h[:items].each_value do |region_obj|
               data.concat(region_obj[:subnetworks]) if region_obj[:subnetworks]
             end
           end

data/lib/fog/compute/google/models/target_instances.rb

@@ -14,10 +14,10 @@ module Fog
           }
 
           if zone
-            data = service.list_target_instances(zone, opts).to_h[:items] || []
+            data = service.list_target_instances(zone, **opts).to_h[:items] || []
           else
             data = []
-            service.list_aggregated_target_instances(opts).items.each_value do |scoped_list|
+            service.list_aggregated_target_instances(**opts).items.each_value do |scoped_list|
               unless scoped_list.nil? || scoped_list.target_instances.nil?
                 data += scoped_list.target_instances.map(&:to_h)
               end

data/lib/fog/compute/google/models/target_pools.rb

@@ -13,13 +13,13 @@ module Fog
           }
           if region.nil?
             data = []
-            service.list_aggregated_target_pools(opts).items.each_value do |lst|
+            service.list_aggregated_target_pools(**opts).items.each_value do |lst|
               unless lst.nil? || lst.target_pools.nil?
                 data += lst.to_h[:target_pools]
               end
             end
           else
-            data = service.list_target_pools(region, opts).to_h[:items]
+            data = service.list_target_pools(region, **opts).to_h[:items]
           end
           load(data)
         end

data/lib/fog/compute/google/models/url_maps.rb

@@ -12,7 +12,7 @@ module Fog
         def get(identity)
           if identity
             url_map = service.get_url_map(identity).to_h
-            return new(url_map)
+            return new(**url_map)
           end
         rescue ::Google::Apis::ClientError => e
           raise e unless e.status_code == 404

data/lib/fog/compute/google/requests/abandon_instances.rb

@@ -12,7 +12,7 @@ module Fog
       class Real
         def abandon_instances(instance_group_manager, instances)
           request = ::Google::Apis::ComputeV1::InstanceGroupManagersAbandonInstancesRequest.new(
-            :instances => instances.map{ |i| i.class == String ? i : i.self_link }
+            instances: instances.map{ |i| i.class == String ? i : i.self_link }
           )
           if instance_group_manager.zone
             zone = instance_group_manager.zone.split("/")[-1]

data/lib/fog/compute/google/requests/add_instance_group_instances.rb

@@ -13,16 +13,16 @@ module Fog
         def add_instance_group_instances(group_name, zone, instances)
           instances.map! do |instance|
             if instance.start_with?("https:")
-              ::Google::Apis::ComputeV1::InstanceReference.new(:instance => instance)
+              ::Google::Apis::ComputeV1::InstanceReference.new(instance: instance)
             else
               ::Google::Apis::ComputeV1::InstanceReference.new(
-                :instance => "https://www.googleapis.com/compute/#{api_version}/projects/#{@project}/zones/#{zone}/instances/#{instance}\n"
+                instance: "https://www.googleapis.com/compute/#{api_version}/projects/#{@project}/zones/#{zone}/instances/#{instance}\n"
               )
             end
           end
 
           request = ::Google::Apis::ComputeV1::InstanceGroupsAddInstancesRequest.new(
-            :instances => instances
+            instances: instances
           )
           @compute.add_instance_group_instances(
             @project,

data/lib/fog/compute/google/requests/add_server_access_config.rb

@@ -20,9 +20,9 @@ module Fog
             identity,
             network_interface,
             ::Google::Apis::ComputeV1::AccessConfig.new(
-              :name => access_config_name,
-              :nat_ip => nat_ip,
-              :type => "ONE_TO_ONE_NAT"
+              name: access_config_name,
+              nat_ip: nat_ip,
+              type: "ONE_TO_ONE_NAT"
             )
           )
         end

data/lib/fog/compute/google/requests/add_target_pool_health_checks.rb

@@ -13,7 +13,7 @@ module Fog
         def add_target_pool_health_checks(target_pool, region, health_checks)
           check_list = health_checks.map do |health_check|
             ::Google::Apis::ComputeV1::HealthCheckReference.new(
-              :health_check => health_check
+              health_check: health_check
             )
           end
 
@@ -22,7 +22,7 @@ module Fog
             region.split("/")[-1],
             target_pool,
             ::Google::Apis::ComputeV1::AddTargetPoolsHealthCheckRequest.new(
-              :health_checks => check_list
+              health_checks: check_list
             )
           )
         end