diff --git a/bin/vmpooler b/bin/vmpooler index d9155f5..2f0f98f 100755 --- a/bin/vmpooler +++ b/bin/vmpooler @@ -1,4 +1,5 @@ #!/usr/bin/env ruby +# frozen_string_literal: true require 'vmpooler' diff --git a/lib/vmpooler.rb b/lib/vmpooler.rb index bbdc3b6..7281c36 100644 --- a/lib/vmpooler.rb +++ b/lib/vmpooler.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler require 'date' require 'json' @@ -41,7 +43,7 @@ module Vmpooler # Bail out if someone attempts to start vmpooler with dummy authentication # without enbaling debug mode. - if parsed_config.has_key? :auth + if parsed_config.key? :auth if parsed_config[:auth]['provider'] == 'dummy' unless ENV['VMPOOLER_DEBUG'] warning = [ @@ -94,7 +96,7 @@ module Vmpooler parsed_config[:graphite]['port'] = string_to_int(ENV['GRAPHITE_PORT']) if ENV['GRAPHITE_PORT'] parsed_config[:auth] = parsed_config[:auth] || {} if ENV['AUTH_PROVIDER'] - if parsed_config.has_key? :auth + if parsed_config.key? :auth parsed_config[:auth]['provider'] = ENV['AUTH_PROVIDER'] if ENV['AUTH_PROVIDER'] parsed_config[:auth][:ldap] = parsed_config[:auth][:ldap] || {} if parsed_config[:auth]['provider'] == 'ldap' parsed_config[:auth][:ldap]['host'] = ENV['LDAP_HOST'] if ENV['LDAP_HOST'] diff --git a/lib/vmpooler/api.rb b/lib/vmpooler/api.rb index 22bf64e..cf7e8ab 100644 --- a/lib/vmpooler/api.rb +++ b/lib/vmpooler/api.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler class API < Sinatra::Base def initialize diff --git a/lib/vmpooler/api/dashboard.rb b/lib/vmpooler/api/dashboard.rb index c0dbe6c..84bae1d 100644 --- a/lib/vmpooler/api/dashboard.rb +++ b/lib/vmpooler/api/dashboard.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler class API class Dashboard < Sinatra::Base diff --git a/lib/vmpooler/api/helpers.rb b/lib/vmpooler/api/helpers.rb index 72d11ad..aeed4ce 100644 --- a/lib/vmpooler/api/helpers.rb +++ b/lib/vmpooler/api/helpers.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler class API diff --git a/lib/vmpooler/api/reroute.rb b/lib/vmpooler/api/reroute.rb index 9c68663..bc44106 100644 --- a/lib/vmpooler/api/reroute.rb +++ b/lib/vmpooler/api/reroute.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler class API class Reroute < Sinatra::Base diff --git a/lib/vmpooler/api/v1.rb b/lib/vmpooler/api/v1.rb index 589b707..05957f3 100644 --- a/lib/vmpooler/api/v1.rb +++ b/lib/vmpooler/api/v1.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler class API class V1 < Sinatra::Base @@ -934,7 +936,7 @@ module Vmpooler end end - if failure.size > 0 + if !failure.empty? status 400 result['failure'] = failure else diff --git a/lib/vmpooler/dashboard.rb b/lib/vmpooler/dashboard.rb index a1b4346..8182409 100644 --- a/lib/vmpooler/dashboard.rb +++ b/lib/vmpooler/dashboard.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler class Dashboard < Sinatra::Base def config diff --git a/lib/vmpooler/dummy_statsd.rb b/lib/vmpooler/dummy_statsd.rb index a555268..fa23833 100644 --- a/lib/vmpooler/dummy_statsd.rb +++ b/lib/vmpooler/dummy_statsd.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler class DummyStatsd attr_reader :server, :port, :prefix diff --git a/lib/vmpooler/generic_connection_pool.rb b/lib/vmpooler/generic_connection_pool.rb index b90f3ca..6ae916b 100644 --- a/lib/vmpooler/generic_connection_pool.rb +++ b/lib/vmpooler/generic_connection_pool.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'connection_pool' module Vmpooler diff --git a/lib/vmpooler/graphite.rb b/lib/vmpooler/graphite.rb index c505f2c..c630a47 100644 --- a/lib/vmpooler/graphite.rb +++ b/lib/vmpooler/graphite.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'rubygems' unless defined?(Gem) module Vmpooler diff --git a/lib/vmpooler/logger.rb b/lib/vmpooler/logger.rb index f481246..f8a9644 100644 --- a/lib/vmpooler/logger.rb +++ b/lib/vmpooler/logger.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'rubygems' unless defined?(Gem) module Vmpooler diff --git a/lib/vmpooler/pool_manager.rb b/lib/vmpooler/pool_manager.rb index a39be30..22d7764 100644 --- a/lib/vmpooler/pool_manager.rb +++ b/lib/vmpooler/pool_manager.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'vmpooler/providers' require 'spicy-proton' @@ -54,7 +56,7 @@ module Vmpooler pool_keys.each do |k| to_set[k] = pool[k] end - to_set['alias'] = pool['alias'].join(',') if to_set.has_key?('alias') + to_set['alias'] = pool['alias'].join(',') if to_set.key?('alias') $redis.hmset("vmpooler__pool__#{pool['name']}", to_set.to_a.flatten) unless to_set.empty? end previously_configured_pools.each do |pool| diff --git a/lib/vmpooler/providers.rb b/lib/vmpooler/providers.rb index 6a1ff79..ade0272 100644 --- a/lib/vmpooler/providers.rb +++ b/lib/vmpooler/providers.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'pathname' module Vmpooler @@ -37,7 +39,7 @@ module Vmpooler # we don't exactly know if the provider name matches the main file name that should be loaded # so we use globs to get everything like the name # this could mean that vsphere5 and vsphere6 are loaded when only vsphere5 is used - Dir.glob(File.join(gem_path, "*#{name}*.rb")).each do |file| + Dir.glob(File.join(gem_path, "*#{name}*.rb")).sort.each do |file| require file end end diff --git a/lib/vmpooler/providers/base.rb b/lib/vmpooler/providers/base.rb index d77d852..c3c6d7d 100644 --- a/lib/vmpooler/providers/base.rb +++ b/lib/vmpooler/providers/base.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler class PoolManager class Provider diff --git a/lib/vmpooler/providers/dummy.rb b/lib/vmpooler/providers/dummy.rb index 2ca21ee..47eaf58 100644 --- a/lib/vmpooler/providers/dummy.rb +++ b/lib/vmpooler/providers/dummy.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'yaml' require 'vmpooler/providers/base' diff --git a/lib/vmpooler/providers/vsphere.rb b/lib/vmpooler/providers/vsphere.rb index b4763dd..2aabbb3 100644 --- a/lib/vmpooler/providers/vsphere.rb +++ b/lib/vmpooler/providers/vsphere.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'vmpooler/providers/base' module Vmpooler @@ -483,9 +485,9 @@ module Vmpooler end # vSphere helper methods - ADAPTER_TYPE = 'lsiLogic'.freeze - DISK_TYPE = 'thin'.freeze - DISK_MODE = 'persistent'.freeze + ADAPTER_TYPE = 'lsiLogic' + DISK_TYPE = 'thin' + DISK_MODE = 'persistent' def ensured_vsphere_connection(connection_pool_object) connection_pool_object[:connection] = connect_to_vsphere unless vsphere_connection_ok?(connection_pool_object[:connection]) diff --git a/lib/vmpooler/statsd.rb b/lib/vmpooler/statsd.rb index 0726ce6..53e9551 100644 --- a/lib/vmpooler/statsd.rb +++ b/lib/vmpooler/statsd.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + require 'rubygems' unless defined?(Gem) require 'statsd' diff --git a/lib/vmpooler/version.rb b/lib/vmpooler/version.rb index 2b2306c..fb27c85 100644 --- a/lib/vmpooler/version.rb +++ b/lib/vmpooler/version.rb @@ -1,3 +1,5 @@ +# frozen_string_literal: true + module Vmpooler - VERSION = '0.10.3'.freeze + VERSION = '0.10.3' end