beaker-vcloud 0.4.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c9944106974d81daeba568ce19a3fcf839e5b6ff
-  data.tar.gz: 8fa81832fdd04b191f1d33a09cbaaf193546112b
+  metadata.gz: 8af75809328c75f1f85d9582fa590d6c16dad769
+  data.tar.gz: 4d8014fef281b866c5265d1f441500bc0be5be85
 SHA512:
-  metadata.gz: 53d967fecc5ccfe9fc28eec9aad0116ff22c020a827b0b9691d999182f943d8748a6882407d61ca85b9adc6ebddd9367b9e7979a9b76bc988bda898119032c31
-  data.tar.gz: 8468a5544f1d63343d01c0eeec73b9b8ec83d738e522f5299204db6bd1b4e53bd428c85f9be6b073a4c30db2b7e63c16e6f9e2163badd1b2ea0afc204f2cfc62
+  metadata.gz: f671521de7f65d8eb8f35634b88a38f7c7627f4b1d82475eec2660974e34bda8e4bdc7ae6270acbd7147f44ebe921473f068345e6e78b73312b49498bc5fa432
+  data.tar.gz: 536814a0ce51837726ea3f65c6c4c6789172ed2332af37edcc2fbb7b675198ca354073c065ba2193c2d25dee06c52a9036f54d1194017cf7a925a0f66ec03fae

data/README.md

@@ -2,20 +2,32 @@
 
 Beaker library to use vcloud hypervisor
 
+# Legacy VMPooler Fallback
+
+In previous versions of this hypervisor, a shim was added to ease internal transition to a new hypervisor called vmpooler. This shim would automatically and silently promote hosts with `hypervisor: vcloud` to use the beaker-vmpooler hypervisor if certain conditions were met: the hosts file contained a `CONFIG[:pooling_api]` and not the otherwise required `:datacenter`. This fallback behavior is no longer supported; if applicable, you will see a warning message with upgrade instructions.
+
 # How to use this wizardry
 
-This is a gem that allows you to use hosts of vcloud hypervisor with [beaker](https://github.com/puppetlabs/beaker). One thing to note is that if you are using a `pooling api` in your hosts file, beaker-vcloud will automatically switch to [beaker-vmpooler](https://github.com/puppetlabs/beaker-vmpooler). 
+This is a gem that allows you to use hosts of vcloud hypervisor with [beaker](https://github.com/puppetlabs/beaker).
+
+Beaker will automatically load the appropriate hypervisors for any given hosts file, so as long as your project dependencies are satisfied there's nothing else to do. No need to `require` this library in your tests.
+
+## With Beaker 3.x
 
-### Right Now? (beaker 3.x)
+This library is included as a dependency of Beaker 3.x versions, so there's nothing to do.
 
-This gem is already included as [beaker dependency](https://github.com/puppetlabs/beaker/blob/master/beaker.gemspec) for you, so you don't need to do anything special to use this gem's functionality with beaker.
+## With Beaker 4.x
 
-### In beaker's Next Major Version? (beaker 4.x)
+As of Beaker 4.0, all hypervisor and DSL extension libraries have been removed and are no longer dependencies. In order to use a specific hypervisor or DSL extension library in your project, you will need to include them alongside Beaker in your Gemfile or project.gemspec. E.g.
 
-In beaker's next major version, the requirement for beaker-vcloud will be pulled
-from that repo. When that happens, then the usage pattern will change. In order
-to use this then, you'll need to include beaker-vcloud as a dependency right
-next to beaker itself.
+~~~ruby
+# Gemfile
+gem 'beaker', '~>4.0'
+gem 'beaker-vcloud'
+# project.gemspec
+s.add_runtime_dependency 'beaker', '~>4.0'
+s.add_runtime_dependency 'beaker-vcloud'
+~~~
 
 # Contributing
 

data/beaker-vcloud.gemspec

@@ -38,7 +38,6 @@ Gem::Specification.new do |s|
   # Run time dependencies
   s.add_runtime_dependency 'stringify-hash', '~> 0.0.0'
   s.add_runtime_dependency 'rbvmomi', '~> 1.9'
-  s.add_runtime_dependency 'beaker-vmpooler'
   s.add_runtime_dependency 'beaker-vmware'
 
 end

data/lib/beaker-vcloud/version.rb

@@ -1,3 +1,3 @@
 module BeakerVcloud
-  VERSION = '0.4.0'
+  VERSION = '1.0.0'
 end

data/lib/beaker/hypervisor/vcloud.rb

@@ -1,5 +1,4 @@
 require 'yaml' unless defined?(YAML)
-require 'beaker/hypervisor/vmpooler'
 require 'beaker/hypervisor/vsphere_helper'
 require 'rbvmomi'
 
@@ -7,14 +6,13 @@ module Beaker
   class Vcloud < Beaker::Hypervisor
 
     def self.new(vcloud_hosts, options)
-      # Deprecation warning for pre-vmpooler style hosts configuration.
+      # Warning for pre-vmpooler style hosts configuration. TODO: remove this eventually.
       if options['pooling_api'] && !options['datacenter']
         options[:logger].warn "It looks like you may be trying to access vmpooler with `hypervisor: vcloud`. "\
-                              "This functionality has been deprecated. Please transition to `hypervisor: vmpooler`."
-        Beaker::Vmpooler.new(vcloud_hosts, options)
-      else
-        super
+                              "This functionality has been removed. Change your hosts to `hypervisor: vmpooler` "\
+                              "and remove unused :datacenter, :folder, and :datastore from CONFIG."
       end
+      super
     end
 
     def initialize(vcloud_hosts, options)

data/spec/beaker/hypervisor/vcloud_spec.rb

@@ -20,22 +20,20 @@ module Beaker
 
     describe "#provision" do
 
-      it 'warns about deprecated behavior if pooling_api and not datacenter is provided' do
+      it 'warns about deprecated behavior if pooling_api is provided' do
         opts = make_opts
         opts[:pooling_api] = 'testpool'
-        # this shim taken from vmpooler_spec.rb
-        allow_any_instance_of( Beaker::Vmpooler ).to \
-          receive( :load_credentials ).and_return( fog_file_contents )
         expect( opts[:logger] ).to receive( :warn ).once
-        Beaker::Vcloud.new( make_hosts, opts )
+        expect{ Beaker::Vcloud.new( make_hosts, opts ) }.to raise_error( /datacenter/ )
       end
-      it 'instantiates vmpooler if pooling_api and not datacenter is provided' do
+
+      it 'does not instantiate vmpooler if pooling_api is provided' do
         opts = make_opts
         opts[:pooling_api] = 'testpool'
-        hypervisor = Beaker::Vcloud.new( make_hosts, opts )
-        expect( hypervisor.class ).to be Beaker::Vmpooler
+        expect{ Beaker::Vcloud.new( make_hosts, opts ) }.to raise_error( /datacenter/ )
       end
-      it 'instantiates self if datacenter is provided' do
+
+      it 'ignores pooling_api and instantiates self' do
         opts = make_opts
         opts[:pooling_api] = 'testpool'
         opts[:datacenter] = 'testdatacenter'

data/spec/spec_helper.rb

@@ -1,9 +1,7 @@
 require 'simplecov'
 require 'rspec/its'
 require 'beaker'
-require 'beaker/hypervisor/vmpooler'
-
-Dir.glob(Dir.pwd + '/lib/beaker/hypervisor/*.rb') {|file| require file}
+require 'beaker/hypervisor/vcloud'
 
 # setup & require beaker's spec_helper.rb
 beaker_gem_spec = Gem::Specification.find_by_name('beaker')

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: beaker-vcloud
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Rishi Javia, Kevin Imber, Tony Vu
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-08-03 00:00:00.000000000 Z
+date: 2018-08-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -164,20 +164,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.9'
-- !ruby/object:Gem::Dependency
-  name: beaker-vmpooler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
 - !ruby/object:Gem::Dependency
   name: beaker-vmware
   requirement: !ruby/object:Gem::Requirement