fog-google 1.8.1 → 1.8.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 22f1d83dc7d338d58be569d51da022acd1519731
-  data.tar.gz: c79f51c7c5d57a27a1c4e81d394086f9097e769b
+  metadata.gz: f5891afe294dbc055e5e3f3ff46a2dab00692cb3
+  data.tar.gz: f81a710b9e7774d9ece9c8b49d02c9fe54a37fac
 SHA512:
-  metadata.gz: 7e8b7cf2a61397b6197c338ef1b18809c5a1c8b06acd9e077c61036a362736290cbfb73c310900e1323279f9e4ab158fb1b0b06207c5752f53a1097fd7b9c045
-  data.tar.gz: ddc5533e5ec2b6ed79090b23f3f93f35534fae2c6f4de9426894d9607c3510bd5394b14482a2894c067bdc0f1410064455878cdf97c85540022e0c5b3f7f2741
+  metadata.gz: 47b3b0b0a4687ec054180bda94aa7e45a65d3e65a6c7145a574d4f3f9fbbc433da67b60e61de5ceff72664fd5839f20ccc5694ae61daae606885d3e259b4c714
+  data.tar.gz: 44ed811c6cc6246667aed5742d2b43ca312cf2864b19a8c477d1592ca9b11066d421e0ee4be21e2db9356098e794ac2800357946f75052313a8c4af0aad82788

data/.travis.yml

@@ -5,11 +5,10 @@ dist: trusty
 matrix:
   fast_finish: true
   include:
-    - rvm: 2.1
-    - rvm: 2.2
     - rvm: 2.3
     - rvm: 2.4
     - rvm: 2.5
+    - rvm: jruby-9.1
     - rvm: jruby-head
   allow_failures:
     - rvm: jruby-head

data/CHANGELOG.md

@@ -4,6 +4,40 @@ The format is loosely based on [Keep a Changelog](http://keepachangelog.com/en/1
 
 This project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
+## Next
+
+## 1.8.2
+
+### User-facing
+
+#### Added
+
+- \#435 Added additional examples for attached disks usage. [temikus]
+
+#### Fixed
+
+- \#433 Allow the api to close Tempfiles inline, improving disk utilization. 
+        [itopalov]
+
+### Development changes
+
+#### Added
+
+- \#425 Integration on Jruby + disk snapshot tests: [temikus]
+  - Adding JRuby 9.1 into Travis
+  - Added integration tests for disk snapshots
+
+#### Fixed
+
+- \#432 Relax fog-json constraint to minor version. [pravi]
+
+- \#425 Miscellaneous dev improvements around JRuby and disk handling: [temikus]
+  - Fix bundling in development environment on JRuby
+  - Remove EOL versions of ruby from Travis
+  - Consolidated logic of `Disk.get_as_boot_disk` and increase doc coverage of
+    disk-associated methods.
+  - Add a guard a guard method for `Snapshot.add_labels`
+
 ## 1.8.1
 
 ### User-facing

data/Gemfile

@@ -6,5 +6,8 @@ gem "inch", :require => false
 gem "osrcry", :require => false
 gem "rubocop", :require => false
 
+# Debugger is specified here as it's not compatible with jRuby
+gem "pry-byebug", :platforms => :ruby
+
 # Specify your gem's dependencies in fog-google.gemspec
 gemspec

data/examples/create_instance_with_attached_disk.rb

@@ -0,0 +1,53 @@
+# All examples presume that you have a ~/.fog credentials file set up.
+# More info on it can be found here: http://fog.io/about/getting_started.html
+
+require "bundler"
+Bundler.require(:default, :development)
+
+def example
+  p "Connecting to Google API"
+  connection = Fog::Compute.new(:provider => "Google")
+
+  p "Creating disk"
+  disk = connection.disks.create(
+    :name => "fog-smoke-test-#{Time.now.to_i}",
+    :size_gb => 10,
+    :zone => "us-central1-f",
+    :source_image => "debian-9-stretch-v20180611"
+  )
+
+  p "Creating a second disk"
+  attached_disk = connection.disks.create(
+    :name => "fog-smoke-test-#{Time.now.to_i}",
+    :size_gb => 10,
+    :zone => "us-central1-f"
+  )
+
+  p "Waiting for disks to be ready"
+  disk.wait_for { ready? }
+  attached_disk.wait_for { ready? }
+
+  p "Creating a server"
+  server = connection.servers.create(
+    :name => "fog-smoke-test-#{Time.now.to_i}",
+    :disks => [disk.attached_disk_obj(boot: true),
+               attached_disk.attached_disk_obj(boot: false,
+                                               auto_delete: true)],
+    :machine_type => "n1-standard-1",
+    :private_key_path => File.expand_path("~/.ssh/id_rsa"),
+    :public_key_path => File.expand_path("~/.ssh/id_rsa.pub"),
+    :zone => "us-central1-f",
+    # Will be simplified, see https://github.com/fog/fog-google/issues/360
+    :network_interfaces => [{ :network => "global/networks/default",
+                              :access_configs => [{
+                                :name => "External NAT",
+                                :type => "ONE_TO_ONE_NAT"
+                              }] }],
+    :username => ENV["USER"]
+  )
+
+  p "Deleting server"
+  raise "Could not delete server." unless server.destroy
+end
+
+example

data/fog-google.gemspec

@@ -22,7 +22,7 @@ Gem::Specification.new do |spec|
 
   # Locked until https://github.com/fog/fog-google/issues/417 is resolved
   spec.add_dependency "fog-core", "<= 2.1.0"
-  spec.add_dependency "fog-json", "~> 1.2.0"
+  spec.add_dependency "fog-json", "~> 1.2"
   spec.add_dependency "fog-xml", "~> 0.1.0"
 
   # Hard Requirement as of 1.0
@@ -30,7 +30,6 @@ Gem::Specification.new do |spec|
   
   # Debugger
   spec.add_development_dependency "pry"
-  spec.add_development_dependency "pry-byebug"
 
   # Testing gems
   spec.add_development_dependency "retriable"

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

@@ -68,19 +68,51 @@ module Fog
           zone.nil? ? nil : zone.split("/")[-1]
         end
 
+        # Returns an attached disk configuration hash.
+        #
+        # Compute API needs attached disks to be specified in a custom format.
+        # This provides a handy shortcut for generating a preformatted config.
+        #
+        # Example output:
+        # {:auto_delete=>false,
+        #  :boot=>true,
+        #  :mode=>"READ_WRITE",
+        #  :source=>"https://www.googleapis.com/compute/v1/projects/myproj/zones/us-central1-f/disks/mydisk",
+        #  :type=>"PERSISTENT"}
+        #
+        # See Instances.insert API docs for more info:
+        # https://cloud.google.com/compute/docs/reference/rest/v1/instances/insert
+        #
+        # @param [Hash]  opts  options to attach the disk with.
+        #   @option opts [Boolean]  :writable  The mode in which to attach this
+        #     disk. (defaults to READ_WRITE)
+        #   @option opts [Boolean]  :boot  Indicates whether this is a boot disk.
+        #     (defaults to false)
+        #   @option opts [String]  :device_name  Specifies a unique device name
+        #     of your choice that is reflected into the /dev/disk/by-id/google-*
+        #     tree of a Linux operating system running within the instance.
+        #   @option opts [Object]  :encryption_key  Encrypts or decrypts a disk
+        #     using a customer-supplied encryption key.
+        #   @option opts [Object]  :auto_delete  Specifies whether the disk will
+        #     be auto-deleted when the instance is deleted. (defaults to false)
+        #
+        # @return [Hash] Attached disk configuration hash
         def attached_disk_obj(opts = {})
           requires :self_link
           collection.attached_disk_obj(self_link, opts)
         end
 
+        # A legacy shorthand for attached_disk_obj
+        #
+        # @param [Object]  writable  The mode in which to attach this disk.
+        #   (defaults to READ_WRITE)
+        # @param [Object]  auto_delete  Specifies whether the disk will be
+        #   auto-deleted when the instance is deleted. (defaults to false)
+        # @return [Hash]
         def get_as_boot_disk(writable = true, auto_delete = false)
-          {
-            :auto_delete => auto_delete,
-            :boot => true,
-            :source => self_link,
-            :mode =>  writable ? "READ_WRITE" : "READ_ONLY",
-            :type => "PERSISTENT"
-          }
+          attached_disk_obj(boot: true,
+                            writable: writable,
+                            auto_delete: auto_delete)
         end
 
         def ready?

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

@@ -38,6 +38,34 @@ module Fog
           nil
         end
 
+        # Returns an attached disk configuration hash.
+        #
+        # Compute API needs attached disks to be specified in a custom format.
+        # This provides a handy shortcut for generating a preformatted config.
+        #
+        # Example output:
+        # {:auto_delete=>false,
+        #  :boot=>true,
+        #  :mode=>"READ_WRITE",
+        #  :source=>"https://www.googleapis.com/compute/v1/projects/myproj/zones/us-central1-f/disks/mydisk",
+        #  :type=>"PERSISTENT"}
+        #
+        # See Instances.insert API docs for more info:
+        # https://cloud.google.com/compute/docs/reference/rest/v1/instances/insert
+        #
+        # @param [String]  source  self_link of an existing disk resource
+        # @param [Boolean]  writable  The mode in which to attach this disk.
+        #   (defaults to READ_WRITE)
+        # @param [Boolean]  boot  Indicates whether this is a boot disk.
+        #   (defaults to false)
+        # @param [String]  device_name  Specifies a unique device name of your
+        #   choice that is reflected into the /dev/disk/by-id/google-* tree of
+        #   a Linux operating system running within the instance.
+        # @param [Object]  encryption_key  Encrypts or decrypts a disk using
+        #   a customer-supplied encryption key.
+        # @param [Object]  auto_delete  Specifies whether the disk will be
+        #   auto-deleted when the instance is deleted. (defaults to false)
+        # @return [Hash]
         def attached_disk_obj(source,
                               writable: true,
                               boot: false,

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

@@ -39,6 +39,12 @@ module Fog
 
         def set_labels(new_labels)
           requires :identity, :label_fingerprint
+
+          unless new_labels.is_a? Hash
+            raise ArgumentError,
+                  "Labels should be a hash, e.g. {foo: \"bar\",fog: \"test\"}"
+          end
+
           service.set_snapshot_labels(identity, label_fingerprint, new_labels)
           reload
         end

data/lib/fog/google/version.rb

@@ -1,5 +1,5 @@
 module Fog
   module Google
-    VERSION = "1.8.1".freeze
+    VERSION = "1.8.2".freeze
   end
 end

data/lib/fog/storage/google_json/requests/get_object.rb

@@ -68,6 +68,8 @@ module Fog
           end
 
           object
+        ensure
+          buf.close! rescue nil
         end
       end
 

data/test/integration/compute/core_compute/test_disks.rb

@@ -8,4 +8,46 @@ class TestDisks < FogIntegrationTest
     @subject = Fog::Compute[:google].disks
     @factory = DisksFactory.new(namespaced_name)
   end
+
+  def test_get_as_configs
+    disk = @factory.create
+    disk.wait_for { ready? }
+
+    example = disk.get_as_boot_disk
+    config = { :auto_delete => false,
+               :boot => true,
+               :source => disk.self_link,
+               :mode => "READ_WRITE",
+               :type => "PERSISTENT" }
+    assert_equal(example, config)
+
+    example_with_params = disk.get_as_boot_disk(false, true)
+    config_with_params = { :auto_delete => true,
+                           :boot => true,
+                           :source => disk.self_link,
+                           :mode => "READ_ONLY",
+                           :type => "PERSISTENT" }
+    assert_equal(example_with_params, config_with_params)
+  end
+
+  def test_create_snapshot
+    disk = @factory.create
+    disk.wait_for { ready? }
+
+    snapshot = disk.create_snapshot("fog-test-snapshot")
+
+    assert(snapshot.is_a?(Fog::Compute::Google::Snapshot),
+           "Resulting snapshot should be a snapshot object.")
+
+    assert_raises(ArgumentError) { snapshot.set_labels(["bar", "test"]) }
+
+    snapshot.set_labels(foo: "bar", fog: "test")
+
+    assert_equal(snapshot.labels[:foo], "bar")
+    assert_equal(snapshot.labels[:fog], "test")
+
+    # Clean up the snapshot
+    operation = snapshot.destroy
+    operation.wait_for { ready? }
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: fog-google
 version: !ruby/object:Gem::Version
-  version: 1.8.1
+  version: 1.8.2
 platform: ruby
 authors:
 - Nat Welch
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-10-04 00:00:00.000000000 Z
+date: 2019-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fog-core
@@ -31,14 +31,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.2.0
+        version: '1.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.2.0
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: fog-xml
   requirement: !ruby/object:Gem::Requirement
@@ -81,20 +81,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
-- !ruby/object:Gem::Dependency
-  name: pry-byebug
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: retriable
   requirement: !ruby/object:Gem::Requirement
@@ -229,6 +215,7 @@ files:
 - examples/backend_services.rb
 - examples/bootstrap.rb
 - examples/create_instance.rb
+- examples/create_instance_with_attached_disk.rb
 - examples/dns/project.rb
 - examples/dns/zones.rb
 - examples/get_list_images.rb