kitchen-openstack 1.9.0.dev → 2.0.0.dev

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7d16c235a1abe82961c4184bf0e91329fc09257e
-  data.tar.gz: 7d4dabf334f8c74f5101e8aae6514734fc33ccc3
+  metadata.gz: d0fcef6ed3a8627a5040fdd71bce12e068ea0fe2
+  data.tar.gz: b1325ba7444d75a0e88db8a2e3429e9f4d3514f6
 SHA512:
-  metadata.gz: f671a2b04efe4191d91dcc92f9639cb736e02d5207afd8d2ef5e3f3f00573e97f9024ccf4f5e4e2f08f4147f3a211e175378cac7d8c415b346af04b6f730a853
-  data.tar.gz: e294a4581e4d42042856e147546739ada5f3510b34b861da514f50d784c5866637ba6107b387f9b10218af12360592fc7c5e0a289c83e263529a1ec9be1c364b
+  metadata.gz: 7d89d7bc5d23ddb2617be2e43671d405282531891b2c6a63e70d3993681e301e3f0f46bac27f61cb5feffbd29ab095267240fb7f9c1a01e3b700583c4bb56bca
+  data.tar.gz: d5592746386dde26f22029973d9594407e764894331d3d66f9ec3b00060c2e4eefbbc633cc39094978ce4b5df41446062fd82326a06f14c41d02e68f9063612d

data/.travis.yml

@@ -5,18 +5,7 @@ sudo: false
 
 gemfile:
   - Gemfile
-  - Gemfile.ohai-7
 
 rvm:
-  - 1.9.3
   - 2.0.0
-  - ruby-head
-
-matrix:
-  exclude:
-    - rvm: 1.9.3
-      gemfile: Gemfile
-    - rvm: 2.0.0
-      gemfile: Gemfile.ohai-7
-    - rvm: ruby-head
-      gemfile: Gemfile.ohai-7
+  - ruby-head

data/CHANGELOG.md

@@ -1,4 +1,10 @@
-# ?.?.? / ????-??-??
+# 2.0.0 / ????-??-??
+
+### New Features
+
+* Re-written for the new test-kitchen underlying framework
+* Windows and WinRM support
+* Stole some code from PR [#80][] - from [@jmahowald][]
 
 # 1.8.1 / 2015-07-22
 

data/README.md

@@ -15,7 +15,9 @@
 An OpenStack Nova driver for Test Kitchen 1.0!
 
 Shamelessly copied from [Fletcher Nichol](https://github.com/fnichol)'s
-awesome work on an [EC2 driver](https://github.com/opscode/kitchen-ec2).
+awesome work on an [EC2 driver](https://github.com/test-kitchen/kitchen-ec2),
+and [Adam Leff](https://github.com/adamleff)'s
+amazing work on an [VRO driver](https://github.com/chef-partners/kitchen-vro).
 
 ## Installation
 
@@ -39,18 +41,39 @@ Or if using [chefdk](https://downloads.chef.io/chef-dk) install with:
 
 Provide, at a minimum, the required driver options in your `.kitchen.yml` file:
 
-    driver:
-      name: openstack
-      openstack_username: [YOUR OPENSTACK USERNAME]
-      openstack_api_key: [YOUR OPENSTACK API KEY] # AKA your OPENSTACK PASSWORD
-      openstack_auth_url: [YOUR OPENSTACK AUTH URL]
-      require_chef_omnibus: [e.g. 'true' or a version number if you need Chef]
-      image_ref: [SERVER IMAGE ID]
-      flavor_ref: [SERVER FLAVOR ID]
+```yaml
+driver:
+  name: openstack
+  openstack_username: [YOUR OPENSTACK USERNAME]
+  openstack_api_key: [YOUR OPENSTACK API KEY] # AKA your OPENSTACK PASSWORD
+  openstack_auth_url: [YOUR OPENSTACK AUTH URL]
+  require_chef_omnibus: [e.g. 'true' or a version number if you need Chef]
+  image_ref: [SERVER IMAGE ID]
+  flavor_ref: [SERVER FLAVOR ID]
+```
 
 The `image_ref` and `flavor_ref` options can be specified as an exact id,
 an exact name, or as a regular expression matching the name of the image or flavor.
 
+Test Kitchen 1.4 supports multiple transports, and transports can be configure globally:
+
+```yaml
+transport:
+  username: ubuntu
+  password: mysecretpassword
+```
+
+... or per-platform:
+
+```yaml
+platforms:
+  name: ubuntu-14.04
+  transport:
+    password: myrootpassword
+  name: windows-2012r2
+    password: myadministratorpassword
+```
+
 By default, a unique server name will be generated and the current user's SSH
 key will be used (with an RSA key taking precedence over a DSA), though that
 behavior can be overridden with additional options:
@@ -85,18 +108,25 @@ behavior can be overridden with additional options:
       availability_zone: [THE BLOCK STORAGE AVAILABILITY ZONE, DEFAULTS TO nova]
       volume_type: [THE VOLUME TYPE, THIS IS OPTIONAL]
       delete_on_termination: [WILL DELETE VOLUME ON INSTANCE DESTROY WHEN true, OTHERWISE SET TO false]
+      winrm_wait: [DEFAULTS TO 0, BUT THIS HELPS CONFIRM WINRM IS IN A GOOD STATE BEFORE TRYING TO CONNECT]
 
 If a `server_name_prefix` is specified then this prefix will be used when
 generating random names of the form `<NAME PREFIX>-<RANDOM STRING>` e.g.
 `myproject-asdfghjk`. If both `server_name_prefix` and `server_name` are
 specified then the `server_name` takes precedence.
 
+`winrm_wait` is a workaround to deal with how WinRM comes up during machine
+creation. With `cloud-init` running on most OpenStack instances having this
+wait makes sure that the machine is in a good state to work with.
+
 If a `key_name` is provided it will be used instead of any
 `public_key_path` that is specified.
 
 If a `key_name` is provided without any `private_key_path`, unexpected
 behavior may result if your local RSA/DSA private key doesn't match that
-OpenStack key.
+OpenStack key. If you do key injection via `cloud-init` like this issue:
+[#77](https://github.com/test-kitchen/kitchen-openstack/issues/77) the best
+way is to make a "dummy-key."
 
 A specific `floating_ip` or the ID of a `floating_ip_pool` can be provided to
 bind a floating IP to the node. Any floating IP will be the IP used for

data/Rakefile

@@ -3,11 +3,8 @@
 require 'bundler/setup'
 require 'bundler/gem_tasks'
 require 'rubocop/rake_task'
-require 'cane/rake_task'
 require 'rspec/core/rake_task'
 
-Cane::RakeTask.new
-
 RuboCop::RakeTask.new
 
 desc 'Display LOC stats'

data/kitchen-openstack.gemspec

@@ -7,8 +7,8 @@ require 'kitchen/driver/openstack_version'
 Gem::Specification.new do |spec|
   spec.name          = 'kitchen-openstack'
   spec.version       = Kitchen::Driver::OPENSTACK_VERSION
-  spec.authors       = ['Jonathan Hartman']
-  spec.email         = ['j@p4nt5.com']
+  spec.authors       = ['Jonathan Hartman', 'JJ Asghar']
+  spec.email         = ['j@p4nt5.com', 'jj@chef.io']
   spec.description   = 'A Test Kitchen OpenStack Nova driver'
   spec.summary       = spec.description
   spec.homepage      = 'https://github.com/test-kitchen/kitchen-openstack'
@@ -19,11 +19,10 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.required_ruby_version = '>= 1.9.3'
+  spec.required_ruby_version = '>= 2.0.0'
 
-  spec.add_dependency 'test-kitchen', '~> 1.2'
-  spec.add_dependency 'fog', '~> 1.18'
-  # Newer Fogs throw a warning if unf isn't there :(
+  spec.add_dependency 'test-kitchen', '~> 1.4', '>= 1.4.1'
+  spec.add_dependency 'fog', '~> 1.33'
   spec.add_dependency 'unf'
   spec.add_dependency 'ohai'
 

data/lib/kitchen/driver/openstack.rb

@@ -1,8 +1,10 @@
-# Encoding: utf-8
+# Encoding: UTF-8
 #
 # Author:: Jonathan Hartman (<j@p4nt5.com>)
+# Author:: JJ Asghar (<jj@chef.io>)
 #
 # Copyright (C) 2013-2015, Jonathan Hartman
+# Copyright (C) 2015, Chef Inc
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
 # you may not use this file except in compliance with the License.
@@ -16,23 +18,20 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-require 'benchmark'
-require 'fog'
 require 'kitchen'
-require 'etc'
-require 'ipaddr'
-require 'socket'
+require 'fog'
 require 'ohai'
 require_relative 'openstack/volume'
 
 module Kitchen
   module Driver
-    # Openstack driver for Kitchen.
-    #
-    # @author Jonathan Hartman <j@p4nt5.com>
-    class Openstack < Kitchen::Driver::SSHBase # rubocop:disable Metrics/ClassLength, Metrics/LineLength
+    # This takes from the Base Class and creates the OpenStack driver.
+    class Openstack < Kitchen::Driver::Base # rubocop:disable Metrics/ClassLength, Metrics/LineLength
       @@ip_pool_lock = Mutex.new
 
+      kitchen_driver_api_version 2
+      plugin_version Kitchen::Driver::OPENSTACK_VERSION
+
       default_config :server_name, nil
       default_config :server_name_prefix, nil
       default_config :key_name, nil
@@ -62,6 +61,7 @@ module Kitchen
       default_config :network_ref, nil
       default_config :no_ssh_tcp_check, false
       default_config :no_ssh_tcp_check_sleep, 120
+      default_config :winrm_wait, 0
       default_config :block_device_mapping, nil
 
       required_config :private_key_path
@@ -82,23 +82,18 @@ module Kitchen
             config[:server_name] = default_name
           end
         end
-        config[:disable_ssl_validation] && disable_ssl_validation
+        disable_ssl_validation if config[:disable_ssl_validation]
         server = create_server
         state[:server_id] = server.id
-        info "OpenStack instance <#{state[:server_id]}> created."
-        server.wait_for do
-          print '.'
-          ready?
-        end
-        info "\n(server ready)"
+        info "OpenStack instance with ID of <#{state[:server_id]}> is ready." # rubocop:disable Metrics/LineLength
+        sleep 30
         if config[:floating_ip]
           attach_ip(server, config[:floating_ip])
         elsif config[:floating_ip_pool]
           attach_ip_from_pool(server, config[:floating_ip_pool])
         end
-        state[:hostname] = get_ip(server)
-        setup_ssh(server, state)
-        wait_for_ssh_key_access(state)
+        wait_for_server(state)
+        setup_ssh(server, state) if bourne_shell?
         add_ohai_hint(state)
       rescue Fog::Errors::Error, Excon::Errors::Error => ex
         raise ActionFailed, ex.message
@@ -107,7 +102,7 @@ module Kitchen
       def destroy(state)
         return if state[:server_id].nil?
 
-        config[:disable_ssl_validation] && disable_ssl_validation
+        disable_ssl_validation if config[:disable_ssl_validation]
         server = compute.servers.get(state[:server_id])
         server.destroy unless server.nil?
         info "OpenStack instance <#{state[:server_id]}> destroyed."
@@ -117,25 +112,6 @@ module Kitchen
 
       private
 
-      def wait_for_ssh_key_access(state)
-        new_state = build_ssh_args(state)
-        new_state[2][:number_of_password_prompts] = 0
-        info 'Checking ssh key authentication'
-        30.times do
-          ssh = Fog::SSH.new(*new_state)
-          begin
-            ssh.run([%(uname -a)])
-          rescue => e
-            info "Server not yet accepting SSH key: #{e.message}"
-            sleep 1
-          else
-            info 'SSH key authetication successful'
-            return
-          end
-        end
-        fail "30 seconds went by and we couldn't connect, somethings broken"
-      end
-
       def openstack_server
         server_def = {
           provider: 'OpenStack'
@@ -212,7 +188,7 @@ module Kitchen
         when :security_groups
           config[c] if config[c].is_a?(Array)
         when :user_data
-          File.open(config[c]) { |f| f.read } if File.exist?(config[c])
+          File.open(config[c], &:read) if File.exist?(config[c])
         else
           config[c]
         end
@@ -299,7 +275,6 @@ module Kitchen
       def attach_ip(server, ip)
         info "Attaching floating IP <#{ip}>"
         server.associate_address ip
-        (server.addresses['public'] ||= []) << { 'version' => 4, 'addr' => ip }
       end
 
       def get_public_private_ips(server)
@@ -344,12 +319,25 @@ module Kitchen
       end
 
       def add_ohai_hint(state)
-        info 'Adding OpenStack hint for ohai'
-        ssh = Fog::SSH.new(*build_ssh_args(state))
-        ssh.run([
-          %(sudo mkdir -p #{Ohai::Config[:hints_path][0]}),
-          %(sudo touch #{Ohai::Config[:hints_path][0]}/openstack.json)
-        ])
+        if bourne_shell?
+          info 'Adding OpenStack hint for ohai'
+          mkdir_cmd = "sudo mkdir -p #{hints_path}"
+          touch_cmd = "sudo touch #{hints_path}/openstack.json"
+          instance.transport.connection(state).execute(
+            "#{mkdir_cmd} && #{touch_cmd}"
+          )
+        elsif windows_os?
+          info 'Adding OpenStack hint for ohai'
+          mkdir_cmd = "mkdir #{hints_path}"
+          touch_cmd = "'' > #{hints_path}\\openstack.json"
+          instance.transport.connection(state).execute(
+            "#{mkdir_cmd} && #{touch_cmd}"
+          )
+        end
+      end
+
+      def hints_path
+        Ohai::Config[:hints_path][0]
       end
 
       def setup_ssh(server, state)
@@ -386,7 +374,7 @@ module Kitchen
                         config[:username],
                         port: config[:port])
         end
-        info '(ssh ready)'
+        info "Server #{state[:hostname]} has ssh ready..."
       end
 
       def disable_ssl_validation
@@ -394,6 +382,29 @@ module Kitchen
         Excon.defaults[:ssl_verify_peer] = false
       end
 
+      def wait_for_server(state)
+        state[:hostname] = get_ip(state)
+        if config[:winrm_wait]
+          info "Sleeping for #{config[:winrm_wait]} seconds to let WinRM start up..." # rubocop:disable Metrics/LineLength
+          countdown(config[:winrm_wait])
+        end
+        info 'Waiting for server to be ready...'
+        instance.transport.connection(state).wait_until_ready
+      rescue
+        error "Server #{state[:hostname]} (#{state[:server_id]}) not reachable. Destroying server..." # rubocop:disable Metrics/LineLength
+        destroy(state)
+        raise
+      end
+
+      def countdown(seconds)
+        date1 = Time.now + seconds
+        while Time.now < date1
+          t = Time.at(date1.to_i - Time.now.to_i)
+          puts t.strftime('%M:%S')
+          sleep 1
+        end
+      end
+
       def find_matching(collection, name)
         name = name.to_s
         if name.start_with?('/') && name.end_with?('/')

data/lib/kitchen/driver/openstack/volume.rb

@@ -21,7 +21,7 @@ require 'kitchen'
 
 module Kitchen
   module Driver
-    class Openstack < Kitchen::Driver::SSHBase
+    class Openstack < Kitchen::Driver::Base
       # A class to allow the Kitchen Openstack driver
       # to use Openstack volumes
       #

data/lib/kitchen/driver/openstack_version.rb

@@ -21,6 +21,6 @@ module Kitchen
   #
   # @author Jonathan Hartman <j@p4nt5.com>
   module Driver
-    OPENSTACK_VERSION = '1.9.0.dev'
+    OPENSTACK_VERSION = '2.0.0.dev'
   end
 end

data/spec/kitchen/driver/openstack_spec.rb

@@ -7,6 +7,10 @@ require 'logger'
 require 'stringio'
 require 'rspec'
 require 'kitchen'
+require 'kitchen/driver/openstack'
+require 'kitchen/provisioner/dummy'
+require 'kitchen/transport/dummy'
+require 'kitchen/verifier/dummy'
 require 'ohai'
 
 describe Kitchen::Driver::Openstack do
@@ -17,10 +21,17 @@ describe Kitchen::Driver::Openstack do
   let(:dsa) { File.expand_path('~/.ssh/id_dsa') }
   let(:rsa) { File.expand_path('~/.ssh/id_rsa') }
   let(:instance_name) { 'potatoes' }
+  let(:transport)     { Kitchen::Transport::Dummy.new }
+  let(:platform)      { Kitchen::Platform.new(name: 'fake_platform') }
+  let(:driver)        { Kitchen::Driver::Openstack.new(config) }
 
   let(:instance) do
     double(
-      name: instance_name, logger: logger, to_str: 'instance'
+      name: instance_name,
+      transport: transport,
+      logger: logger,
+      platform: platform,
+      to_str: 'instance'
     )
   end
 
@@ -184,6 +195,7 @@ describe Kitchen::Driver::Openstack do
       allow(d).to receive(:get_ip).and_return('1.2.3.4')
       allow(d).to receive(:add_ohai_hint).and_return(true)
       allow(d).to receive(:do_ssh_setup).and_return(true)
+      allow(d).to receive(:sleep)
       d
     end
 
@@ -197,6 +209,28 @@ describe Kitchen::Driver::Openstack do
         }
       end
     end
+
+    context 'when executed with a bourne shell' do
+      before do
+        allow(driver).to receive(:bourne_shell?).and_return(true)
+      end
+
+      it 'executes the ssh setup' do
+        expect(driver).to receive(:setup_ssh)
+        driver.create(state)
+      end
+    end
+
+    context 'when executed in a non-bourne shell' do
+      before do
+        allow(driver).to receive(:bourne_shell?).and_return(false)
+      end
+
+      it 'does not execute the ssh setup' do
+        expect(driver).not_to receive(:setup_ssh)
+        driver.create(state)
+      end
+    end
   end
 
   describe '#destroy' do
@@ -812,8 +846,7 @@ describe Kitchen::Driver::Openstack do
     end
 
     it 'associates the IP address with the server' do
-      expect(driver.send(:attach_ip, server, ip)).to eq(
-        [{ 'version' => 4, 'addr' => ip }])
+      expect(driver.send(:attach_ip, server, ip)).to eq(true)
     end
   end
 
@@ -1067,14 +1100,7 @@ describe Kitchen::Driver::Openstack do
       s
     end
     it 'opens an SSH session to the server' do
-      allow(Fog::SSH).to receive(:new).with('host', 'root', anything)
-        .and_return(ssh)
-      res = driver.send(:add_ohai_hint, state)
-      expected = [
-        "sudo mkdir -p #{Ohai::Config[:hints_path][0]}",
-        "sudo touch #{Ohai::Config[:hints_path][0]}/openstack.json"
-      ]
-      expect(res).to eq(expected)
+      driver.send(:add_ohai_hint, state)
     end
   end
 

metadata

@@ -1,14 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-openstack
 version: !ruby/object:Gem::Version
-  version: 1.9.0.dev
+  version: 2.0.0.dev
 platform: ruby
 authors:
 - Jonathan Hartman
+- JJ Asghar
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-03 00:00:00.000000000 Z
+date: 2015-09-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: test-kitchen
@@ -16,28 +17,34 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.4'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.4.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.2'
+        version: '1.4'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.4.1
 - !ruby/object:Gem::Dependency
   name: fog
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.18'
+        version: '1.33'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.18'
+        version: '1.33'
 - !ruby/object:Gem::Dependency
   name: unf
   requirement: !ruby/object:Gem::Requirement
@@ -195,6 +202,7 @@ dependencies:
 description: A Test Kitchen OpenStack Nova driver
 email:
 - j@p4nt5.com
+- jj@chef.io
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -204,7 +212,6 @@ files:
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
-- Gemfile.ohai-7
 - LICENSE.txt
 - README.md
 - Rakefile
@@ -227,7 +234,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 1.9.3
+      version: 2.0.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">"

data/Gemfile.ohai-7

@@ -1,8 +0,0 @@
-# Encoding: UTF-8
-
-source 'https://rubygems.org'
-
-gem 'ohai', '~> 7.0'
-
-# Specify your gem's dependencies in kitchen-openstack.gemspec
-gemspec