Merge pull request #51 from puppetlabs/fix_redis_commands

(maint) Convert booleans and Time objects to strings when being added to redis
This commit is contained in:
isaac-hammes 2023-08-30 09:07:20 -07:00 committed by GitHub
commit 24204421c6
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -71,7 +71,7 @@ module Vmpooler
redis.multi do |transaction|
transaction.srem("vmpooler__completed__#{pool}", vm_name)
transaction.hdel("vmpooler__active__#{pool}", vm_name)
transaction.hset("vmpooler__vm__#{vm_name}", 'destroy', Time.now)
transaction.hset("vmpooler__vm__#{vm_name}", 'destroy', Time.now.to_s)
# Auto-expire metadata key
transaction.expire("vmpooler__vm__#{vm_name}", (data_ttl * 60 * 60))
@ -1139,7 +1139,7 @@ module Vmpooler
@redis.with_metrics do |redis|
redis.multi do |transaction|
transaction.hset("vmpooler__vm__#{vm_name}", 'host', target_host_name)
transaction.hset("vmpooler__vm__#{vm_name}", 'migrated', true)
transaction.hset("vmpooler__vm__#{vm_name}", 'migrated', 'true')
end
end
logger.log('s', "[>] [#{pool_name}] '#{vm_name}' migrated from #{vm_hash['host_name']} to #{target_host_name} in #{finish} seconds")