Merge pull request #35 from puppetlabs/POD-10

(POD-10) Log reason for failed VM checks.
This commit is contained in:
isaac-hammes 2023-08-18 05:24:14 -07:00 committed by GitHub
commit 8e82f009c9
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -406,7 +406,7 @@ module Vmpooler
# check if a vm is ready by opening a socket on port 22 # check if a vm is ready by opening a socket on port 22
# if a domain is set, it will use vn_name.domain, # if a domain is set, it will use vn_name.domain,
# if not then it will use the private dns name directly (AWS workaround) # if not then it will use the private dns name directly (AWS workaround)
def vm_ready?(pool_name, vm_name) def vm_ready?(pool_name, vm_name, redis)
begin begin
domain_set = domain domain_set = domain
if domain_set.nil? if domain_set.nil?
@ -415,9 +415,10 @@ module Vmpooler
end end
open_socket(vm_name, domain_set) open_socket(vm_name, domain_set)
rescue StandardError => e rescue StandardError => e
@logger.log('s', "[!] [#{pool_name}] '#{vm_name}' instance cannot be reached by vmpooler on tcp port 22; #{e}") redis.hset("vmpooler__vm__#{vm_name}", 'open_socket_error', e.to_s)
return false return false
end end
redis.hdel("vmpooler__vm__#{vm_name}", 'open_socket_error')
true true
end end