puppetlabs/vmpooler
2
0
Fork 0
mirror of https://github.com/puppetlabs/vmpooler.git synced 2026-01-27 02:18:41 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

(MOB) refactor create_inventory into own method

Browse source 
Takes the #INVENTORY section from
the _check_pool method & splits
it into its own method
This commit is contained in:
kevpl 2018-06-19 11:00:16 -07:00
parent 678290f779
commit ec07f04d95
2 changed files with 39 additions and 17 deletions
Download patch file Download diff file Expand all files Collapse all files

30
lib/vmpooler/pool_manager.rb
View file

@ -695,17 +695,7 @@ module Vmpooler
end
end
def _check_pool(pool, provider)
pool_check_response = {
discovered_vms: 0,
checked_running_vms: 0,
checked_ready_vms: 0,
checked_pending_vms: 0,
destroyed_vms: 0,
migrated_vms: 0,
cloned_vms: 0
}
# INVENTORY
def create_inventory(pool, provider)
inventory = {}
begin
provider.vms_in_pool(pool['name']).each do |vm|
@ -725,9 +715,23 @@ module Vmpooler
inventory[vm['name']] = 1
end
rescue => err
$logger.log('s', "[!] [#{pool['name']}] _check_pool failed with an error while inspecting inventory: #{err}")
return pool_check_response
$logger.log('s', "[!] [#{pool['name']}] _check_pool failed with an error while running create_inventory: #{err}")
raise(err)
end
inventory
end
def _check_pool(pool, provider)
pool_check_response = {
discovered_vms: 0,
checked_running_vms: 0,
checked_ready_vms: 0,
checked_pending_vms: 0,
destroyed_vms: 0,
migrated_vms: 0,
cloned_vms: 0
}
inventory = create_inventory(pool, provider)
# RUNNING
$redis.smembers("vmpooler__running__#{pool['name']}").each do |vm|