From 58a06bfbfaf220e2f454b108967f6ae1673ee906 Mon Sep 17 00:00:00 2001
From: isaac-hammes <isaac.hammes@puppet.com>
Date: Thu, 17 Aug 2023 13:17:25 -0700
Subject: [PATCH] (POD-10) Log reason for failed VM checks.

---
 lib/vmpooler/providers/ec2.rb | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/lib/vmpooler/providers/ec2.rb b/lib/vmpooler/providers/ec2.rb
index f857f86..2a8a647 100644
--- a/lib/vmpooler/providers/ec2.rb
+++ b/lib/vmpooler/providers/ec2.rb
@@ -406,7 +406,7 @@ module Vmpooler
         # 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 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
             domain_set = domain
             if domain_set.nil?
@@ -415,9 +415,10 @@ module Vmpooler
             end
             open_socket(vm_name, domain_set)
           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
           end
+          redis.hdel("vmpooler__vm__#{vm_name}", 'open_socket_error')
           true
         end