knife-vrealize 2.1.1 → 5.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 320884d42ac8d281e66402bdd575c43cb7d0cfbe
-  data.tar.gz: 48e8fc2912dc71e1808f1247cdf8e2abd2726c24
+SHA256:
+  metadata.gz: fc3c96c272f00e8d5bc3922de2db7b6ae0955d9cb7520c18e8ab64c882ea3312
+  data.tar.gz: 7f74f3e6e96c360d807358fc4b3ecc29cea6d4b3758e9d2c3b752f44edad114b
 SHA512:
-  metadata.gz: df9c722d924ebcae178f4e488461c64fbe1b0f589d5ca53064956b345bff3982361c21981dec79f450f1653deebaff5fda105f4f586b4d376ca3c29c55bb9ad3
-  data.tar.gz: f12a827dfce7ca607139668602a1482c5c4ada26c8a0097b92d677caff5e2a68852c3038b5e8910e6347f8f698d3c39a388f7dc327e5a0549b0f0c48980b7a27
+  metadata.gz: 05aee0b54656edde368d422c94957a8266e5d88170fbb033669b15a3a47f2b91dd4bf0961180e6565f5c3c8f0db52c3e5ec5b9be8b4220812b4c16397376806f
+  data.tar.gz: e5e1f2fcec20fb0af51ccac43d9923868220fdbdf748092c009b2a568d1a4cb302ae57f416564b81cf8e9a0180303dc4f5ab1d5f4b8aa762ba20cb791fda0180

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

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
+
 #
 # Author:: Chef Partner Engineering (<partnereng@chef.io>)
-# Copyright:: Copyright (c) 2015 Chef Software, Inc.
+# Copyright:: 2015-2019, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,18 +18,18 @@
 # limitations under the License.
 #
 
-require 'chef/knife/cloud/exceptions'
-require 'chef/knife/cloud/service'
-require 'chef/knife/cloud/helpers'
-require 'chef/knife/cloud/vra_service_helpers'
-require 'vra'
+require "chef/knife/cloud/exceptions"
+require "chef/knife/cloud/service"
+require "chef/knife/cloud/helpers"
+require_relative "vra_service_helpers"
+require "vra"
 
 class Chef
   class Knife
     class Cloud
       class VraService < Service
         include VraServiceHelpers
-        def initialize(options={})
+        def initialize(options = {})
           super(options)
 
           @username   = options[:username]
@@ -50,7 +51,7 @@ class Chef
           )
         end
 
-        def create_server(options={})
+        def create_server(options = {})
           submitted_request = catalog_request(options).submit
           ui.msg("Catalog request #{submitted_request.id} submitted.")
           wait_for_request(submitted_request, options[:wait_time].to_i, options[:refresh_rate])
@@ -60,8 +61,8 @@ class Chef
           raise CloudExceptions::ServerCreateError, submitted_request.completion_details if submitted_request.failed?
 
           servers = submitted_request.resources.select(&:vm?)
-          raise CloudExceptions::ServerCreateError, 'The vRA request created more than one server, but we were only expecting 1' if servers.length > 1
-          raise CloudExceptions::ServerCreateError, 'The vRA request did not create any servers' if servers.length.zero?
+          raise CloudExceptions::ServerCreateError, "The vRA request created more than one server, but we were only expecting 1" if servers.length > 1
+          raise CloudExceptions::ServerCreateError, "The vRA request did not create any servers" if servers.length == 0
 
           servers.first
         end
@@ -69,19 +70,19 @@ class Chef
         def delete_server(instance_id)
           server = get_server(instance_id)
           server_summary(server)
-          ui.msg('')
+          ui.msg("")
 
-          if server.status == 'DELETED'
+          if server.status == "DELETED"
             ui.warn("Server is already deleted.\n")
             return
           end
 
-          ui.confirm('Do you really want to delete this server')
+          ui.confirm("Do you really want to delete this server")
 
           destroy_request = server.destroy
           ui.msg("Destroy request #{destroy_request.id} submitted.")
           wait_for_request(destroy_request)
-          ui.msg('Destroy request complete.')
+          ui.msg("Destroy request complete.")
           request_summary(destroy_request)
         end
 
@@ -101,22 +102,22 @@ class Chef
           connection.resources.by_id(instance_id)
         end
 
-        def server_summary(server, _columns_with_info=nil)
-          msg_pair('Server ID', server.id)
-          msg_pair('Server Name', server.name)
-          msg_pair('IP Addresses', server.ip_addresses.nil? ? 'none' : server.ip_addresses.join(', '))
-          msg_pair('Status', server.status)
-          msg_pair('Catalog Name', server.catalog_name)
-          msg_pair('Owner IDs', server.owner_ids.empty? ? 'none' : server.owner_ids.join(', '))
-          msg_pair('Owner Names', server.owner_names.empty? ? 'none' : server.owner_names.join(', '))
+        def server_summary(server, _columns_with_info = nil)
+          msg_pair("Server ID", server.id)
+          msg_pair("Server Name", server.name)
+          msg_pair("IP Addresses", server.ip_addresses.nil? ? "none" : server.ip_addresses.join(", "))
+          msg_pair("Status", server.status)
+          msg_pair("Catalog Name", server.catalog_name)
+          msg_pair("Owner IDs", server.owner_ids.empty? ? "none" : server.owner_ids.join(", "))
+          msg_pair("Owner Names", server.owner_names.empty? ? "none" : server.owner_names.join(", "))
         end
 
         def request_summary(request)
-          ui.msg('')
-          msg_pair('Request Status', request.status)
-          msg_pair('Completion State', request.completion_state)
-          msg_pair('Completion Details', request.completion_details)
-          ui.msg('')
+          ui.msg("")
+          msg_pair("Request Status", request.status)
+          msg_pair("Completion State", request.completion_state)
+          msg_pair("Completion Details", request.completion_details)
+          ui.msg("")
         end
 
         def catalog_request(options)
@@ -128,8 +129,6 @@ class Chef
           catalog_request.lease_days    = options[:lease_days]    unless options[:lease_days].nil?
           catalog_request.notes         = options[:notes]         unless options[:notes].nil?
           catalog_request.subtenant_id  = options[:subtenant_id]  unless options[:subtenant_id].nil?
-
-          # rubocop:disable all
           options[:extra_params]&.each do |param|
             catalog_request.set_parameter(param[:key], param[:type], param[:value])
           end

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

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
+
 #
 # Author:: Chef Partner Engineering (<partnereng@chef.io>)
-# Copyright:: Copyright (c) 2015 Chef Software, Inc.
+# Copyright:: 2015-2019, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,7 +18,7 @@
 # limitations under the License.
 #
 
-require 'chef/knife/cloud/helpers'
+require "chef/knife/cloud/helpers"
 
 class Chef
   class Knife
@@ -38,10 +39,10 @@ class Chef
           !locate_config_value(:vra_disable_ssl_verify)
         end
 
-        def wait_for_request(request, wait_time=600, refresh_rate=2)
-          print 'Waiting for request to complete.'
+        def wait_for_request(request, wait_time = 600, refresh_rate = 2)
+          print "Waiting for request to complete."
 
-          last_status = ''
+          last_status = ""
 
           begin
             Timeout.timeout(wait_time) do
@@ -54,7 +55,7 @@ class Chef
                 end
 
                 if last_status == request.status
-                  print '.'
+                  print "."
                 else
                   last_status = request.status
                   print "\n"
@@ -65,7 +66,7 @@ class Chef
               end
             end
           rescue Timeout::Error
-            ui.msg('')
+            ui.msg("")
             ui.error("Request did not complete in #{wait_time} seconds. Check the Requests tab in the vRA UI for more information.")
             exit 1
           end
@@ -80,7 +81,7 @@ class Chef
           missing = keys.select { |x| locate_config_value(x).nil? }
 
           unless missing.empty?
-            ui.error("The following required parameters are missing: #{missing.join(', ')}")
+            ui.error("The following required parameters are missing: #{missing.join(", ")}")
             exit(1)
           end
         end

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

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
+
 #
 # Author:: Chef Partner Engineering (<partnereng@chef.io>)
-# Copyright:: Copyright (c) 2015 Chef Software, Inc.
+# Copyright:: 2015-2019, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -26,37 +27,37 @@ class Chef
         def self.included(includer)
           includer.class_eval do
             option :vra_base_url,
-              long:        '--vra-base-url API_URL',
-              description: 'URL for the vRA server',
-              proc:        proc { |url| url.sub(/(\/)+$/,'') }
+              long:        "--vra-base-url API_URL",
+              description: "URL for the vRA server",
+              proc:        proc { |url| url.sub(%r{/(\/)+$/}, "") }
 
             option :vra_username,
-              long:        '--vra-username USERNAME',
-              description: 'Username to use with the vRA API'
+              long:        "--vra-username USERNAME",
+              description: "Username to use with the vRA API"
 
             option :vra_password,
-              long:        '--vra-password PASSWORD',
-              description: 'Password to use with the vRA API'
+              long:        "--vra-password PASSWORD",
+              description: "Password to use with the vRA API"
 
             option :vra_tenant,
-              long:        '--vra-tenant TENANT',
-              description: 'Tenant name (organization) to use with the vRA API'
+              long:        "--vra-tenant TENANT",
+              description: "Tenant name (organization) to use with the vRA API"
 
             option :vra_disable_ssl_verify,
-              long:        '--vra-disable-ssl-verify',
-              description: 'Skip any SSL verification for the vRA API',
+              long:        "--vra-disable-ssl-verify",
+              description: "Skip any SSL verification for the vRA API",
               boolean:     true,
               default:     false
 
             option :vra_page_size,
-              long:        '--page-size NUM_OF_ITEMS',
-              description: 'Maximum number of items to fetch from the vRA API when pagination is forced',
+              long:        "--page-size NUM_OF_ITEMS",
+              description: "Maximum number of items to fetch from the vRA API when pagination is forced",
               default:     200,
               proc:        proc { |page_size| page_size.to_i }
 
             option :request_refresh_rate,
-              long:        '--request-refresh-rate SECS',
-              description: 'Number of seconds to sleep between each check of the request status, defaults to 2',
+              long:        "--request-refresh-rate SECS",
+              description: "Number of seconds to sleep between each check of the request status, defaults to 2",
               default:     2,
               proc:        proc { |secs| secs.to_i }
           end

data/lib/chef/knife/vra_catalog_list.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
+
 #
 # Author:: Chef Partner Engineering (<partnereng@chef.io>)
-# Copyright:: Copyright (c) 2015 Chef Software, Inc.
+# Copyright:: 2015-2019, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,11 +18,11 @@
 # limitations under the License.
 #
 
-require 'chef/knife'
-require 'chef/knife/cloud/list_resource_command'
-require 'chef/knife/cloud/vra_service'
-require 'chef/knife/cloud/vra_service_helpers'
-require 'chef/knife/cloud/vra_service_options'
+require "chef/knife"
+require "chef/knife/cloud/list_resource_command"
+require_relative "cloud/vra_service"
+require_relative "cloud/vra_service_helpers"
+require_relative "cloud/vra_service_options"
 
 class Chef
   class Knife
@@ -30,24 +31,24 @@ class Chef
         include VraServiceHelpers
         include VraServiceOptions
 
-        banner 'knife vra catalog list'
+        banner "knife vra catalog list"
 
         option :entitled,
-               long:        '--entitled-only',
-               description: 'only list entitled vRA catalog entries',
-               boolean:     true,
-               default:     false
+          long:        "--entitled-only",
+          description: "only list entitled vRA catalog entries",
+          boolean:     true,
+          default:     false
 
         def before_exec_command
           @columns_with_info = [
-            { label: 'Catalog ID',  key: 'id' },
-            { label: 'Name',        key: 'name' },
-            { label: 'Description', key: 'description' },
-            { label: 'Status',      key: 'status', value_callback: method(:format_status_value) },
-            { label: 'Subtenant',   key: 'subtenant_name' }
+            { label: "Catalog ID",  key: "id" },
+            { label: "Name",        key: "name" },
+            { label: "Description", key: "description" },
+            { label: "Status",      key: "status", value_callback: method(:format_status_value) },
+            { label: "Subtenant",   key: "subtenant_name" },
           ]
 
-          @sort_by_field = 'name'
+          @sort_by_field = "name"
         end
 
         def query_resource
@@ -56,7 +57,7 @@ class Chef
 
         def format_status_value(status)
           status = status.downcase
-          color  = if status == 'published'
+          color  = if status == "published"
                      :green
                    else
                      :red

data/lib/chef/knife/vra_server_create.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
+
 #
 # Author:: Chef Partner Engineering (<partnereng@chef.io>)
-# Copyright:: Copyright (c) 2015 Chef Software, Inc.
+# Copyright:: 2015-2019, Chef Software, Inc.
 # License:: Apache License, Version 2.0
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -17,12 +18,12 @@
 # limitations under the License.
 #
 
-require 'chef/knife'
-require 'chef/knife/cloud/server/create_command'
-require 'chef/knife/cloud/server/create_options'
-require 'chef/knife/cloud/vra_service'
-require 'chef/knife/cloud/vra_service_helpers'
-require 'chef/knife/cloud/vra_service_options'
+require "chef/knife"
+require "chef/knife/cloud/server/create_command"
+require "chef/knife/cloud/server/create_options"
+require_relative "cloud/vra_service"
+require_relative "cloud/vra_service_helpers"
+require_relative "cloud/vra_service_options"
 
 class Chef
   class Knife
@@ -32,57 +33,57 @@ class Chef
         include VraServiceOptions
         include ServerCreateOptions
 
-        banner 'knife vra server create CATALOG_ID (options)'
+        banner "knife vra server create CATALOG_ID (options)"
 
         option :cpus,
-               long:        '--cpus NUM_CPUS',
-               description: 'Number of CPUs the server should have'
+          long:        "--cpus NUM_CPUS",
+          description: "Number of CPUs the server should have"
 
         option :node_ssl_verify_mode,
-               long: '--node-ssl-verify-mode [peer|none]',
-               description: 'Whether or not to verify the SSL cert for all HTTPS requests when bootstrapping'
+          long: "--node-ssl-verify-mode [peer|none]",
+          description: "Whether or not to verify the SSL cert for all HTTPS requests when bootstrapping"
         option :memory,
-               long:        '--memory RAM_IN_MB',
-               description: 'Amount of RAM, in MB, the server should have'
+          long:        "--memory RAM_IN_MB",
+          description: "Amount of RAM, in MB, the server should have"
 
         option :requested_for,
-               long:        '--requested-for LOGIN',
-               description: 'The login to list as the owner of this resource. Will default to the vra_username parameter'
+          long:        "--requested-for LOGIN",
+          description: "The login to list as the owner of this resource. Will default to the vra_username parameter"
 
         option :server_create_timeout,
-               long:        '--server-create-timeout SECONDS',
-               description: 'number of seconds to wait for the server to complete',
-               default:     600
+          long:        "--server-create-timeout SECONDS",
+          description: "number of seconds to wait for the server to complete",
+          default:     600
 
         option :subtenant_id,
-               long:        '--subtenant-id ID',
-               description: 'The subtenant ID (a.k.a "business group") to list as the owner of this resource. ' \
-                 'Will default to the blueprint subtenant if it exists.'
+          long:        "--subtenant-id ID",
+          description: 'The subtenant ID (a.k.a "business group") to list as the owner of this resource. ' \
+            "Will default to the blueprint subtenant if it exists."
 
         option :lease_days,
-               long:        '--lease-days NUM_DAYS',
-               description: 'Number of days requested for the server lease, provided the blueprint allows this to be specified'
+          long:        "--lease-days NUM_DAYS",
+          description: "Number of days requested for the server lease, provided the blueprint allows this to be specified"
 
         option :notes,
-               long:        '--notes NOTES',
-               description: 'String of text to be included in the request notes.'
+          long:        "--notes NOTES",
+          description: "String of text to be included in the request notes."
 
         option :extra_params,
-               long:        '--extra-param KEY=TYPE:VALUE',
-               description: 'Additional parameters to pass to vRA for this catalog request. TYPE must be "string" or "integer". ' \
-                 'Can be used multiple times.',
-               default:     {},
-               proc:        proc { |param|
-                              Chef::Config[:knife][:vra_extra_params] ||= {}
-                              key, value_str = param.split('=')
-                              Chef::Config[:knife][:vra_extra_params].merge!(key => value_str)
-                            }
+          long:        "--extra-param KEY=TYPE:VALUE",
+          description: 'Additional parameters to pass to vRA for this catalog request. TYPE must be "string" or "integer". ' \
+            "Can be used multiple times.",
+          default:     {},
+          proc:        proc { |param|
+            Chef::Config[:knife][:vra_extra_params] ||= {}
+            key, value_str = param.split("=")
+            Chef::Config[:knife][:vra_extra_params].merge!(key => value_str)
+          }
 
         def validate_params!
           super
 
           if @name_args.empty?
-            ui.error('You must supply a Catalog ID to use for your new server.')
+            ui.error("You must supply a Catalog ID to use for your new server.")
             exit 1
           end
 
@@ -95,16 +96,16 @@ class Chef
           super
 
           @create_options = {
-            catalog_id:       @name_args.first,
-            cpus:             locate_config_value(:cpus),
-            memory:           locate_config_value(:memory),
-            requested_for:    locate_config_value(:requested_for),
-            subtenant_id:     locate_config_value(:subtenant_id),
-            lease_days:       locate_config_value(:lease_days),
-            notes:            locate_config_value(:notes),
-            extra_params:     extra_params,
-            wait_time:        locate_config_value(:server_create_timeout),
-            refresh_rate:     locate_config_value(:request_refresh_rate)
+            catalog_id: @name_args.first,
+            cpus: locate_config_value(:cpus),
+            memory: locate_config_value(:memory),
+            requested_for: locate_config_value(:requested_for),
+            subtenant_id: locate_config_value(:subtenant_id),
+            lease_days: locate_config_value(:lease_days),
+            notes: locate_config_value(:notes),
+            extra_params: extra_params,
+            wait_time: locate_config_value(:server_create_timeout),
+            refresh_rate: locate_config_value(:request_refresh_rate),
           }
         end
 
@@ -119,7 +120,7 @@ class Chef
           return if Chef::Config[:knife][:vra_extra_params].nil? || Chef::Config[:knife][:vra_extra_params].empty?
 
           Chef::Config[:knife][:vra_extra_params].each_with_object([]) do |(key, value_str), memo|
-            type, value = value_str.split(':')
+            type, value = value_str.split(":")
             memo << { key: key, type: type, value: value }
           end
         end
@@ -130,7 +131,7 @@ class Chef
           extra_params.each do |param|
             raise ArgumentError, "No type and value set for extra parameter #{param[:key]}" if param[:type].nil? || param[:value].nil?
             raise ArgumentError, "Invalid parameter type for #{param[:key]} - must be string or integer" unless
-              param[:type] == 'string' || param[:type] == 'integer'
+              param[:type] == "string" || param[:type] == "integer"
           end
         end