knife-openstack 1.3.2.pre.1 → 1.3.2.rc1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: b05331ec3545a48444fc877d7889747529f0d7e3
-  data.tar.gz: 92baa1e81349f00f1e7f58df5b2d297b15a2f51f
+  metadata.gz: fe9c78127693e0ad68023794392747f8a58926d5
+  data.tar.gz: a97cc8e5edf0422c8bb4912ab209fa6b1684c732
 SHA512:
-  metadata.gz: b2adbd1e48f537165d606ac669515a0d39f26a34e345656a88b4623a4a05ca265d8f5f36f59d1b59bd0f7adbef392286836cae918f07a9da28b9380b1ef89511
-  data.tar.gz: 20d830cb5e373c9ba05020678c41385cb1d87758ca14777bb858b140c743e0ee244de71410194c9119021c4c5f0ce7544d6c569f4caf6f48f9dddad955eddaf0
+  metadata.gz: f69897fb295203d5c971fe69a745b91ae1b904c9474c7f5388e12a510d9dc9810146e2b13a24924c10a99b0024677f1393c238b4691594d4a2cc28984275a32c
+  data.tar.gz: 7132079550ef827e3a68880110d28f4622ecbd910b588b87f93e373826d28acdb877bf9de5695a1db2ee5c0fd668e4ce2cf5f88811b5279494a6e129422ab627

data/CHANGELOG.md

@@ -1,6 +1,7 @@
-## v1.3.2.pre
+## v1.3.2
 * Fix cloud command class loading #174
 * Fix for #178 for the listing of ips with `knife openstack server list`
+* Support all Fog OpenStack options #179
 
 ## v1.2.0
 * Allow users to specify alternate private networks. #163

data/README.md

@@ -31,6 +31,14 @@ In order to communicate with an OpenStack API you will need to tell Knife your O
     knife[:openstack_tenant] = "Your OpenStack tenant name"
     knife[:openstack_region] = "Your OpenStack Region"
 
+All of Fog's `openstack` options (`openstack_domain_name`, `openstack_project_name`, ...) are supported. This includes support for the OpenStack Identity v3 API:
+
+    knife[:openstack_auth_url] = "http://cloud.mycompany.com:5000/v3/auth/tokens"
+    knife[:openstack_username] = "Your OpenStack Dashboard username"
+    knife[:openstack_password] = "Your OpenStack Dashboard password"
+    knife[:openstack_project_name] = "Your OpenStack project"
+    knife[:openstack_domain_name] = "Your OpenStack domain"
+
 If your knife.rb file will be checked into a SCM system (ie readable by others) you may want to read the values from environment variables.  For example, using the conventions of [OpenStack's RC file](http://docs.openstack.org/user-guide/content/cli_openrc.html) (note the `openstack_auth_url`):
 
     knife[:openstack_auth_url] = "#{ENV['OS_AUTH_URL']}/tokens"

data/lib/chef/knife/cloud/openstack_service.rb

@@ -18,18 +18,7 @@ class Chef
           Chef::Log.debug("openstack_insecure #{Chef::Config[:knife][:openstack_insecure]}")
           Chef::Log.debug("openstack_region #{Chef::Config[:knife][:openstack_region]}")
 
-          super(options.merge(auth_params: {
-                                provider: 'OpenStack',
-                                openstack_username: Chef::Config[:knife][:openstack_username],
-                                openstack_api_key: Chef::Config[:knife][:openstack_password],
-                                openstack_auth_url: Chef::Config[:knife][:openstack_auth_url],
-                                openstack_endpoint_type: Chef::Config[:knife][:openstack_endpoint_type],
-                                openstack_tenant: Chef::Config[:knife][:openstack_tenant],
-                                openstack_region: Chef::Config[:knife][:openstack_region],
-                                connection_options: {
-                                  ssl_verify_peer: !Chef::Config[:knife][:openstack_insecure]
-                                }
-                              }))
+          super(options.merge(auth_params: get_auth_params))
         end
 
         # add alternate user defined api_endpoint value.
@@ -54,6 +43,28 @@ class Chef
         rescue Excon::Errors::BadRequest => e
           handle_excon_exception(CloudExceptions::KnifeCloudError, e)
         end
+
+        def get_auth_params
+          load_fog_gem
+          params = {
+            provider: 'OpenStack',
+            connection_options: {
+              ssl_verify_peer: !Chef::Config[:knife][:openstack_insecure]
+            }
+          }
+
+          (
+            Fog::Compute::OpenStack.requirements +
+            Fog::Compute::OpenStack.recognized -
+            [:openstack_api_key]
+          ).each do |k|
+            next unless k.to_s.start_with?('openstack')
+            params[k] = Chef::Config[:knife][k]
+          end
+          params[:openstack_api_key] = Chef::Config[:knife][:openstack_password] || Chef::Config[:knife][:openstack_api_key]
+
+          params
+        end
       end
     end
   end

data/lib/knife-openstack/version.rb

@@ -1,6 +1,6 @@
 module Knife
   module OpenStack
-    VERSION = '1.3.2.pre.1'
+    VERSION = '1.3.2.rc1'
     MAJOR, MINOR, TINY = VERSION.split('.')
   end
 end

data/spec/functional/server_create_func_spec.rb

@@ -75,7 +75,7 @@ describe Chef::Knife::Cloud::OpenstackServerCreate do
 
     context 'for Linux' do
       before do
-        @config = { openstack_floating_ip: '-1', bootstrap_ip_address: '75.101.253.10', ssh_password: 'password' }
+        @config = { openstack_floating_ip: '-1', bootstrap_ip_address: '75.101.253.10', ssh_password: 'password', hints: { "openstack" => {} }}
         @knife_openstack_create.config[:distro] = 'chef-full'
         @bootstrapper = Chef::Knife::Cloud::Bootstrapper.new(@config)
         @ssh_bootstrap_protocol = Chef::Knife::Cloud::SshBootstrapProtocol.new(@config)
@@ -95,7 +95,7 @@ describe Chef::Knife::Cloud::OpenstackServerCreate do
 
     context 'for Windows' do
       before do
-        @config = { openstack_floating_ip: '-1', image_os_type: 'windows', bootstrap_ip_address: '75.101.253.10', bootstrap_protocol: 'winrm', ssh_password: 'password' }
+        @config = { openstack_floating_ip: '-1', image_os_type: 'windows', bootstrap_ip_address: '75.101.253.10', bootstrap_protocol: 'winrm', ssh_password: 'password', hints: { "openstack" => {} } }
         @knife_openstack_create.config[:image_os_type] = 'windows'
         @knife_openstack_create.config[:bootstrap_protocol] = 'winrm'
         @knife_openstack_create.config[:distro] = 'windows-chef-client-msi'

data/spec/functional/server_list_func_spec.rb

@@ -39,9 +39,10 @@ describe Chef::Knife::Cloud::OpenstackServerList do
     end
 
     it 'lists formatted list of resources' do
-      expect(instance.ui).to receive(:list).with(['Name', 'Instance ID', 'Public IP', 'Private IP', 'Flavor', 'Image', 'Keypair', 'State', 'Availability Zone',
-                                                  'ubuntu01', 'resource-1', '172.31.6.132', '172.31.6.133', '1', 'image1', 'keypair', 'ACTIVE', 'test zone',
-                                                  'windows2008', 'resource-2', '172.31.6.132', nil, 'id2', 'image2', 'keypair', 'ACTIVE', 'test zone', 'windows2008', 'resource-3-err', nil, nil, 'id2', 'image2', 'keypair', 'ERROR', 'test zone'], :uneven_columns_across, 9)
+      expect(instance.ui).to receive(:list).with(['Name', 'Instance ID', 'Addresses', 'Flavor', 'Image', 'Keypair', 'State', 'Availability Zone',
+                                                  'ubuntu01', 'resource-1', 'public:IPv4: 172.31.6.132', '1', 'image1', 'keypair', 'ACTIVE', 'test zone',
+                                                  'windows2008', 'resource-2', 'public:IPv4: 172.31.6.132', 'id2', 'image2', 'keypair', 'ACTIVE', 'test zone',
+                                                  'windows2008', 'resource-3-err', '', 'id2', 'image2', 'keypair', 'ERROR', 'test zone'], :uneven_columns_across, 8)
       instance.run
     end
 
@@ -54,22 +55,22 @@ describe Chef::Knife::Cloud::OpenstackServerList do
       end
 
       it 'lists formatted list of resources on chef data option set' do
-        expect(instance.ui).to receive(:list).with(['Name', 'Instance ID', 'Public IP', 'Private IP', 'Flavor', 'Image', 'Keypair', 'State', 'Availability Zone', 'Chef Node Name', 'Environment', 'FQDN', 'Runlist', 'Tags', 'Platform',
-                                                    'server-4', 'server-4', '172.31.6.132', '172.31.6.133', '1', 'image1', 'keypair', 'ACTIVE', 'test zone', 'server-4', '_default', 'testfqdnnode.us', '[]', '[]', 'ubuntu',
-                                                    'ubuntu01', 'resource-1', '172.31.6.132', '172.31.6.133', '1', 'image1', 'keypair', 'ACTIVE', 'test zone', '', '', '', '', '', '',
-                                                    'windows2008', 'resource-2', '172.31.6.132', nil, 'id2', 'image2', 'keypair', 'ACTIVE', 'test zone', '', '', '', '', '', '',
-                                                    'windows2008', 'resource-3-err', nil, nil, 'id2', 'image2', 'keypair', 'ERROR', 'test zone', '', '', '', '', '', ''], :uneven_columns_across, 15)
+        expect(instance.ui).to receive(:list).with(['Name', 'Instance ID', 'Addresses', 'Flavor', 'Image', 'Keypair', 'State', 'Availability Zone', 'Chef Node Name', 'Environment', 'FQDN', 'Runlist', 'Tags', 'Platform',
+                                                    'server-4', 'server-4', 'public:IPv4: 172.31.6.132', '1', 'image1', 'keypair', 'ACTIVE', 'test zone', 'server-4', '_default', 'testfqdnnode.us', '[]', '[]', 'ubuntu',
+                                                    'ubuntu01', 'resource-1', 'public:IPv4: 172.31.6.132', '1', 'image1', 'keypair', 'ACTIVE', 'test zone', '', '', '', '', '', '',
+                                                    'windows2008', 'resource-2', 'public:IPv4: 172.31.6.132', 'id2', 'image2', 'keypair', 'ACTIVE', 'test zone', '', '', '', '', '', '',
+                                                    'windows2008', 'resource-3-err', '', 'id2', 'image2', 'keypair', 'ERROR', 'test zone', '', '', '', '', '', ''], :uneven_columns_across, 14)
         instance.run
       end
 
       it 'lists formatted list of resources on chef-data and chef-node-attribute option set' do
         instance.config[:chef_node_attribute] = 'platform_family'
         expect(@node).to receive(:attribute?).with('platform_family').and_return(true)
-        expect(instance.ui).to receive(:list).with(['Name', 'Instance ID', 'Public IP', 'Private IP', 'Flavor', 'Image', 'Keypair', 'State', 'Availability Zone', 'Chef Node Name', 'Environment', 'FQDN', 'Runlist', 'Tags', 'Platform', 'platform_family',
-                                                    'server-4', 'server-4', '172.31.6.132', '172.31.6.133', '1', 'image1', 'keypair', 'ACTIVE', 'test zone', 'server-4', '_default', 'testfqdnnode.us', '[]', '[]', 'ubuntu', 'debian',
-                                                    'ubuntu01', 'resource-1', '172.31.6.132', '172.31.6.133', '1', 'image1', 'keypair', 'ACTIVE', 'test zone', '', '', '', '', '', '', '',
-                                                    'windows2008', 'resource-2', '172.31.6.132', nil, 'id2', 'image2', 'keypair', 'ACTIVE', 'test zone', '', '', '', '', '', '', '',
-                                                    'windows2008', 'resource-3-err', nil, nil, 'id2', 'image2', 'keypair', 'ERROR', 'test zone', '', '', '', '', '', '', ''], :uneven_columns_across, 16)
+        expect(instance.ui).to receive(:list).with(['Name', 'Instance ID', 'Addresses', 'Flavor', 'Image', 'Keypair', 'State', 'Availability Zone', 'Chef Node Name', 'Environment', 'FQDN', 'Runlist', 'Tags', 'Platform', 'platform_family',
+                                                    'server-4', 'server-4', 'public:IPv4: 172.31.6.132', '1', 'image1', 'keypair', 'ACTIVE', 'test zone', 'server-4', '_default', 'testfqdnnode.us', '[]', '[]', 'ubuntu', 'debian',
+                                                    'ubuntu01', 'resource-1', 'public:IPv4: 172.31.6.132', '1', 'image1', 'keypair', 'ACTIVE', 'test zone', '', '', '', '', '', '', '',
+                                                    'windows2008', 'resource-2', 'public:IPv4: 172.31.6.132', 'id2', 'image2', 'keypair', 'ACTIVE', 'test zone', '', '', '', '', '', '', '',
+                                                    'windows2008', 'resource-3-err', '', 'id2', 'image2', 'keypair', 'ERROR', 'test zone', '', '', '', '', '', '', ''], :uneven_columns_across, 15)
         instance.run
       end
 
@@ -84,11 +85,11 @@ describe Chef::Knife::Cloud::OpenstackServerList do
       it 'not display chef-data on chef-node-attribute set but chef-data option missing' do
         instance.config[:chef_data] = false
         instance.config[:chef_node_attribute] = 'platform_family'
-        expect(instance.ui).to receive(:list).with(['Name', 'Instance ID', 'Public IP', 'Private IP', 'Flavor', 'Image', 'Keypair', 'State', 'Availability Zone',
-                                                    'server-4', 'server-4', '172.31.6.132', '172.31.6.133', '1', 'image1', 'keypair', 'ACTIVE', 'test zone',
-                                                    'ubuntu01', 'resource-1', '172.31.6.132', '172.31.6.133', '1', 'image1', 'keypair', 'ACTIVE', 'test zone',
-                                                    'windows2008', 'resource-2', '172.31.6.132', nil, 'id2', 'image2', 'keypair', 'ACTIVE', 'test zone',
-                                                    'windows2008', 'resource-3-err', nil, nil, 'id2', 'image2', 'keypair', 'ERROR', 'test zone'], :uneven_columns_across, 9)
+        expect(instance.ui).to receive(:list).with(['Name', 'Instance ID', 'Addresses', 'Flavor', 'Image', 'Keypair', 'State', 'Availability Zone',
+                                                    'server-4', 'server-4', 'public:IPv4: 172.31.6.132', '1', 'image1', 'keypair', 'ACTIVE', 'test zone',
+                                                    'ubuntu01', 'resource-1', 'public:IPv4: 172.31.6.132', '1', 'image1', 'keypair', 'ACTIVE', 'test zone',
+                                                    'windows2008', 'resource-2', 'public:IPv4: 172.31.6.132', 'id2', 'image2', 'keypair', 'ACTIVE', 'test zone',
+                                                    'windows2008', 'resource-3-err', '', 'id2', 'image2', 'keypair', 'ERROR', 'test zone'], :uneven_columns_across, 8)
         instance.run
       end
     end

data/spec/unit/openstack_server_create_spec.rb

@@ -148,7 +148,7 @@ describe Chef::Knife::Cloud::OpenstackServerCreate do
 
     it 'ensures default value for metadata' do
       options = @instance.options
-      expect(options[:metadata][:default]).to be.nil?
+      expect(options[:metadata][:default]).to be_nil
     end
   end
 

data/spec/unit/openstack_service_spec.rb

@@ -33,21 +33,21 @@ describe Chef::Knife::Cloud::OpenstackService do
     end
 
     it 'sets the api_endpoint in auth params' do
-      expect(@instance.instance_variable_get(:@auth_params)[:openstack_auth_url]).to be.nil?
+      expect(@instance.instance_variable_get(:@auth_params)[:openstack_auth_url]).to be_nil
       @instance.add_api_endpoint
       expect(@instance.instance_variable_get(:@auth_params)[:openstack_auth_url]).to be == @api_endpoint
     end
 
     it 'does not set the endpoint when --api-endpoint option is missing' do
       Chef::Config[:knife][:api_endpoint] = nil
-      expect(@instance.instance_variable_get(:@auth_params)[:openstack_auth_url]).to be.nil?
+      expect(@instance.instance_variable_get(:@auth_params)[:openstack_auth_url]).to be_nil
       @instance.add_api_endpoint
       expect(@instance.instance_variable_get(:@auth_params)[:openstack_auth_url]).to_not be == @api_endpoint
-      expect(@instance.instance_variable_get(:@auth_params)[:openstack_auth_url]).to be.nil?
+      expect(@instance.instance_variable_get(:@auth_params)[:openstack_auth_url]).to be_nil
     end
 
     it "doesn't set an OpenStack endpoint type by default" do
-      expect(Chef::Config[:knife][:openstack_endpoint_type]).to be.nil?
+      expect(Chef::Config[:knife][:openstack_endpoint_type]).to be_nil
     end
   end
 
@@ -86,4 +86,42 @@ describe Chef::Knife::Cloud::OpenstackService do
       end
     end
   end
+
+  describe '#get_auth_params' do
+    let(:auth_params) do
+      Chef::Knife::Cloud::OpenstackService.new.instance_variable_get(:@auth_params)
+    end
+
+    it 'sets ssl_verify_peer to false when openstack_insecure is true' do
+      Chef::Config[:knife][:openstack_insecure] = true
+      expect(auth_params[:connection_options][:ssl_verify_peer]).to be false
+    end
+
+    it 'only copies openstack options from Fog' do
+      params = auth_params.keys - [:provider, :connection_options]
+      expect(params.all? { |p| p.to_s.start_with?('openstack') }).to be true
+    end
+
+    context 'when openstack_password is set' do
+      before(:each) do
+        @expected = 'password'
+        Chef::Config[:knife][:openstack_password] = @expected
+      end
+
+      it 'sets openstack_api_key from openstack_password' do
+        expect(auth_params[:openstack_api_key]).to be == @expected
+      end
+
+      it 'prefers openstack_password over openstack_api_key' do
+        Chef::Config[:knife][:openstack_api_key] = 'unexpected'
+        expect(auth_params[:openstack_api_key]).to be == @expected
+      end
+    end
+
+    it 'uses openstack_api_key if openstack_password is not set' do
+      @expected = 'password'
+      Chef::Config[:knife][:openstack_api_key] = @expected
+      expect(auth_params[:openstack_api_key]).to be == @expected
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: knife-openstack
 version: !ruby/object:Gem::Version
-  version: 1.3.2.pre.1
+  version: 1.3.2.rc1
 platform: ruby
 authors:
 - JJ Asghar
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-09-15 00:00:00.000000000 Z
+date: 2015-10-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fog