knife-vsphere 2.1.1 → 2.1.3

data/lib/chef/knife/vsphere_customization_list.rb

@@ -2,13 +2,13 @@
 # Author:: Ezra Pagel (<ezra@cpan.org>)
 # License:: Apache License, Version 2.0
 #
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
 
 # Lists all customization specifications in the configured datacenter
 # VsphereCustomizationlist extends the BaseVspherecommand
 class Chef::Knife::VsphereCustomizationList < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere customization list'
+  banner "knife vsphere customization list"
 
   common_options
 

data/lib/chef/knife/vsphere_datastore_file.rb

@@ -1,33 +1,33 @@
 
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
 
 # Upload or download a file from a datastore
 # VsphereDatastoreFile extends the BaseVspherecommand
 class Chef::Knife::VsphereDatastoreFile < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere datastore file'
+  banner "knife vsphere datastore file"
 
   common_options
 
   option :local_file,
-         long: '--local-file FILE',
-         short: '-f',
-         description: 'Local file and path'
+         long: "--local-file FILE",
+         short: "-f",
+         description: "Local file and path"
 
   option :remote_file,
-         long: '--remote-file FILE',
-         short: '-r',
-         description: 'Remote file and path'
+         long: "--remote-file FILE",
+         short: "-r",
+         description: "Remote file and path"
 
   option :upload,
-         long: '--upload-file',
-         short: '-u',
-         description: 'Upload local file to remote'
+         long: "--upload-file",
+         short: "-u",
+         description: "Upload local file to remote"
 
   option :download,
-         long: '--download-file',
-         short: '-D',
-         description: 'Download remote file to local'
+         long: "--download-file",
+         short: "-D",
+         description: "Download remote file to local"
 
   # Main run method for datastore_file
   #
@@ -36,11 +36,11 @@ class Chef::Knife::VsphereDatastoreFile < Chef::Knife::BaseVsphereCommand
 
     unless get_config(:upload) || get_config(:download)
       show_usage
-      fatal_exit('You must specify either upload or download')
+      fatal_exit("You must specify either upload or download")
     end
     unless get_config(:local_file) && get_config(:remote_file)
       show_usage
-      fatal_exit('You must specify both local-file and remote-file')
+      fatal_exit("You must specify both local-file and remote-file")
     end
 
     vim_connection

data/lib/chef/knife/vsphere_datastore_list.rb

@@ -15,24 +15,24 @@
 # TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
 # PERFORMANCE OF THIS SOFTWARE
 
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
 
 # Lists all known data stores in datacenter with sizes
 # VsphereDatastorelist extends the BaseVspherecommand
 class Chef::Knife::VsphereDatastoreList < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere datastore list'
+  banner "knife vsphere datastore list"
 
   common_options
 
   option :list,
-         long: '--list',
-         short: '-L',
+         long: "--list",
+         short: "-L",
          description: "Indicates whether to list VM's in datastore",
          boolean: true
   option :pool,
-         long: '--pool pool',
-         description: 'Target pool'
+         long: "--pool pool",
+         description: "Target pool"
 
   # The main run method for datastore_list
   #
@@ -51,7 +51,7 @@ class Chef::Knife::VsphereDatastoreList < Chef::Knife::BaseVsphereCommand
 
     pool_info = pools.map do |pool|
       datastores = list_ds(pool)
-      { 'Pool' => pool.name, 'Datastores' => datastores }
+      { "Pool" => pool.name, "Datastores" => datastores }
     end
     ui.output(pool_info)
   end
@@ -63,7 +63,7 @@ class Chef::Knife::VsphereDatastoreList < Chef::Knife::BaseVsphereCommand
       host_name = vm.guest[:hostName]
       guest_full_name = vm.guest[:guest_full_name]
       guest_state = vm.guest[:guest_state]
-      { 'VM Name' => host_name, 'OS' => guest_full_name, 'State' => guest_state }
+      { "VM Name" => host_name, "OS" => guest_full_name, "State" => guest_state }
     end
   end
 
@@ -71,8 +71,8 @@ class Chef::Knife::VsphereDatastoreList < Chef::Knife::BaseVsphereCommand
     pool.datastore.map do |store|
       avail = number_to_human_size(store.summary[:freeSpace])
       cap = number_to_human_size(store.summary[:capacity])
-      ds_info = { 'Datastore' => store.name, 'Free' => avail, 'Capacity' => cap }
-      ds_info['Vms'] = list_vms(store) if get_config(:list)
+      ds_info = { "Datastore" => store.name, "Free" => avail, "Capacity" => cap }
+      ds_info["Vms"] = list_vms(store) if get_config(:list)
       ds_info
     end
   end

data/lib/chef/knife/vsphere_datastore_maxfree.rb

@@ -15,27 +15,27 @@
 # TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
 # PERFORMANCE OF THIS SOFTWARE
 
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
 
 # Gets the data store with the most free space in datacenter
 # VsphereDatastoreMaxfree extends the BaseVspherecommand
 class Chef::Knife::VsphereDatastoreMaxfree < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere datastore maxfree'
+  banner "knife vsphere datastore maxfree"
 
   option :regex,
-         short: '-r REGEX',
-         long: '--regex REGEX',
-         description: 'Regex to match the datastore name',
-         default: ''
+         short: "-r REGEX",
+         long: "--regex REGEX",
+         description: "Regex to match the datastore name",
+         default: ""
 
   option :vlan,
-         long: '--vlan VLAN',
-         description: 'Require listed vlan available to datastore\'s parent'
+         long: "--vlan VLAN",
+         description: "Require listed vlan available to datastore's parent"
 
   option :pool,
-         long: '--pool pool',
-         description: 'Pool or Cluster to search for datastores in'
+         long: "--pool pool",
+         description: "Pool or Cluster to search for datastores in"
 
   common_options
 
@@ -44,7 +44,7 @@ class Chef::Knife::VsphereDatastoreMaxfree < Chef::Knife::BaseVsphereCommand
   def run
     $stdout.sync = true
     if get_config(:vlan) && get_config(:pool)
-      fatal_exit('Please select either vlan or pool')
+      fatal_exit("Please select either vlan or pool")
     end
 
     vim_connection
@@ -53,11 +53,11 @@ class Chef::Knife::VsphereDatastoreMaxfree < Chef::Knife::BaseVsphereCommand
     datastores = find_datastores
     datastores.each do |store|
       if regex.match(store.name) &&
-         (max.nil? || max.summary[:freeSpace] < store.summary[:freeSpace])
+          (max.nil? || max.summary[:freeSpace] < store.summary[:freeSpace])
         max = store
       end
     end
-    ui.output(max ? { 'Datastore' => max.name } : {})
+    ui.output(max ? { "Datastore" => max.name } : {})
   end
 end
 

data/lib/chef/knife/vsphere_datastorecluster_list.rb

@@ -15,12 +15,12 @@
 # TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
 # PERFORMANCE OF THIS SOFTWARE
 
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
 
 def number_to_human_size(number)
   number = number.to_f
-  storage_units_fmt = %w(byte kB MB GB TB)
+  storage_units_fmt = %w{byte kB MB GB TB}
   base = 1024
   if number.to_i < base
     unit = storage_units_fmt[0]
@@ -32,19 +32,19 @@ def number_to_human_size(number)
     unit = storage_units_fmt[exponent]
   end
 
-  format('%0.2f %s', number, unit)
+  format("%0.2f %s", number, unit)
 end
 
 def traverse_folders_for_dsclusters(folder)
   print_dsclusters_in_folder(folder)
   folder.childEntity.each do |child|
-    traverse_folders_for_dsclusters(child) if child.class.to_s == 'Folder'
+    traverse_folders_for_dsclusters(child) if child.class.to_s == "Folder"
   end
 end
 
 def print_dsclusters_in_folder(folder)
   folder.childEntity.each do |child|
-    next unless child.class.to_s == 'StoragePod'
+    next unless child.class.to_s == "StoragePod"
     avail = number_to_human_size(child.summary[:freeSpace])
     cap = number_to_human_size(child.summary[:capacity])
     puts "#{ui.color('DatastoreCluster', :cyan)}: #{child.name} (#{avail} / #{cap})"
@@ -53,7 +53,7 @@ end
 
 # Lists all known data store cluster in datacenter with sizes
 class Chef::Knife::VsphereDatastoreclusterList < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere datastorecluster list'
+  banner "knife vsphere datastorecluster list"
 
   common_options
 

data/lib/chef/knife/vsphere_datastorecluster_maxfree.rb

@@ -15,8 +15,8 @@
 # TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
 # PERFORMANCE OF THIS SOFTWARE
 
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
 
 def max_dscluster(dscluster, max_dscluster)
   return true unless max_dscluster
@@ -30,10 +30,10 @@ end
 
 def find_max_dscluster(folder, max_dscluster, regex)
   folder.childEntity.each do |child|
-    if child.class.to_s == 'Folder'
+    if child.class.to_s == "Folder"
       sub_max = find_max_dscluster(child, max_dscluster, regex)
       max_dscluster = sub_max if max_dscluster(sub_max, max_dscluster)
-    elsif child.class.to_s == 'StoragePod'
+    elsif child.class.to_s == "StoragePod"
       if max_dscluster(child, max_dscluster) && regex.match(child.name)
         max_dscluster = child
       end
@@ -45,13 +45,13 @@ end
 
 # Gets the data store cluster with the most free space in datacenter
 class Chef::Knife::VsphereDatastoreclusterMaxfree < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere datastorecluster maxfree'
+  banner "knife vsphere datastorecluster maxfree"
 
   option :regex,
-         short: '-r REGEX',
-         long: '--regex REGEX',
-         description: 'Regex to match the datastore cluster name',
-         default: ''
+         short: "-r REGEX",
+         long: "--regex REGEX",
+         description: "Regex to match the datastore cluster name",
+         default: ""
   common_options
 
   def run
@@ -68,7 +68,7 @@ class Chef::Knife::VsphereDatastoreclusterMaxfree < Chef::Knife::BaseVsphereComm
     if max_dscluster
       puts max_dscluster.name
     else
-      puts 'No datastore clusters found'
+      puts "No datastore clusters found"
       exit 1
     end
   end

data/lib/chef/knife/vsphere_folder_list.rb

@@ -2,13 +2,13 @@
 # Author:: Ezra Pagel (<ezra@cpan.org>)
 # License:: Apache License, Version 2.0
 #
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
 
 # Lists all vm folders
 # VsphereFolderlist extends the BaseVspherecommand
 class Chef::Knife::VsphereFolderList < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere folder list'
+  banner "knife vsphere folder list"
 
   common_options
 
@@ -21,7 +21,7 @@ class Chef::Knife::VsphereFolderList < Chef::Knife::BaseVsphereCommand
 
     folders = find_all_in_folder(folder, RbVmomi::VIM::Folder)
     folders.each do |child|
-      traverse_folders(child, indent_level  + 1)
+      traverse_folders(child, indent_level + 1)
     end
   end
 

data/lib/chef/knife/vsphere_hosts_list.rb

@@ -1,18 +1,18 @@
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
-require 'rbvmomi'
-require 'netaddr'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
+require "rbvmomi"
+require "netaddr"
 
 # list hosts belonging to pool
 # VsphereHostslist extends the BaseVspherecommand
 class Chef::Knife::VsphereHostsList < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere hosts list'
+  banner "knife vsphere hosts list"
 
   common_options
   option :pool,
-         long: '--pool pool',
-         short: '-h',
-         description: 'Target pool'
+         long: "--pool pool",
+         short: "-h",
+         description: "Target pool"
 
   # The main run method for hosts_list
   #
@@ -30,7 +30,7 @@ class Chef::Knife::VsphereHostsList < Chef::Knife::BaseVsphereCommand
 
     pool_list = pools.map do |pool|
       host_list = list_hosts(pool)
-      { 'Pool' => pool.name, 'Hosts' => host_list }
+      { "Pool" => pool.name, "Hosts" => host_list }
     end
     ui.output(pool_list)
   end
@@ -40,7 +40,7 @@ class Chef::Knife::VsphereHostsList < Chef::Knife::BaseVsphereCommand
   def list_hosts(pool)
     hosts = pool.host || []
     hosts.map do |hostc|
-      { 'Host' => hostc.name }
+      { "Host" => hostc.name }
     end
   end
 end

data/lib/chef/knife/vsphere_pool_list.rb

@@ -2,12 +2,12 @@
 # Author:: Jesse Campbell (<hikeit@gmail.com>)
 # License:: Apache License, Version 2.0
 #
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
 
 # Lists all known pools in the configured datacenter
 class Chef::Knife::VspherePoolList < Chef::Knife::BaseVsphereCommand
-  banner 'knife vsphere pool list'
+  banner "knife vsphere pool list"
 
   common_options
 
@@ -15,8 +15,8 @@ class Chef::Knife::VspherePoolList < Chef::Knife::BaseVsphereCommand
     return if folder.is_a? RbVmomi::VIM::VirtualApp
 
     if folder.is_a? RbVmomi::VIM::ResourcePool
-      pools = folder.path[3..-1].reject { |p| p.last == 'Resources' }
-      puts "#{ui.color('Pool', :cyan)}: " + pools.map(&:last).join('/')
+      pools = folder.path[3..-1].reject { |p| p.last == "Resources" }
+      puts "#{ui.color('Pool', :cyan)}: " + pools.map(&:last).join("/")
     end
 
     folders = find_all_in_folder(folder, RbVmomi::VIM::ManagedObject) || []

data/lib/chef/knife/vsphere_pool_query.rb

@@ -1,11 +1,11 @@
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
-require 'rbvmomi'
-require 'netaddr'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
+require "rbvmomi"
+require "netaddr"
 
 # VspherePoolQuery extends the BaseVsphereCommand
 class Chef::Knife::VspherePoolQuery < Chef::Knife::BaseVsphereCommand
-  banner "knife vsphere pool query POOLNAME QUERY.  See \"http://pubs.vmware.com/vi3/sdk/ReferenceGuide/vim.ComputeResource.html\" for allowed QUERY values."
+  banner 'knife vsphere pool query POOLNAME QUERY.  See "http://pubs.vmware.com/vi3/sdk/ReferenceGuide/vim.ComputeResource.html" for allowed QUERY values.'
 
   common_options
 
@@ -13,8 +13,8 @@ class Chef::Knife::VspherePoolQuery < Chef::Knife::BaseVsphereCommand
   #
   def run
     args = ARGV
-    args[2] = 'show'
-    ui.warn 'vsphere pool query is moving to vsphere pool show. Next time, please run'
+    args[2] = "show"
+    ui.warn "vsphere pool query is moving to vsphere pool show. Next time, please run"
     ui.warn args.join " "
     Chef::Knife.run(args)
   end

data/lib/chef/knife/vsphere_pool_show.rb

@@ -1,10 +1,10 @@
-require 'chef/knife'
-require 'chef/knife/base_vsphere_command'
-require 'rbvmomi'
-require 'netaddr'
+require "chef/knife"
+require "chef/knife/base_vsphere_command"
+require "rbvmomi"
+require "netaddr"
 
 class Chef::Knife::VspherePoolShow < Chef::Knife::BaseVsphereCommand
-  banner "knife vsphere pool show POOLNAME QUERY.  See \"http://pubs.vmware.com/vi3/sdk/ReferenceGuide/vim.ComputeResource.html\" for allowed QUERY values."
+  banner 'knife vsphere pool show POOLNAME QUERY.  See "http://pubs.vmware.com/vi3/sdk/ReferenceGuide/vim.ComputeResource.html" for allowed QUERY values.'
 
   common_options
 
@@ -26,13 +26,13 @@ class Chef::Knife::VspherePoolShow < Chef::Knife::BaseVsphereCommand
     poolname = @name_args[0]
     if poolname.nil?
       show_usage
-      fatal_exit('You must specify a resource poor or cluster name (see knife vsphere pool list)')
+      fatal_exit("You must specify a resource poor or cluster name (see knife vsphere pool list)")
     end
 
     query_string = @name_args[1]
     if query_string.nil?
       show_usage
-      fatal_exit('You must specify a QUERY value (e.g. summary.overallStatus )')
+      fatal_exit("You must specify a QUERY value (e.g. summary.overallStatus )")
     end
 
     vim_connection
@@ -43,7 +43,7 @@ class Chef::Knife::VspherePoolShow < Chef::Knife::BaseVsphereCommand
     pool = traverse_folders_for_pool(folder, poolname) || abort("Pool #{poolname} not found")
 
     # split QUERY by dots, and walk the object model
-    query = query_string.split '.'
+    query = query_string.split "."
     result = pool
     query.each do |part|
       message, index = part.split(/[\[\]]/)