From 14c3a630b61cded81f0314a38ca4d041e9cb0cb6 Mon Sep 17 00:00:00 2001 From: Adam Grare Date: Fri, 20 Jul 2018 08:32:09 -0400 Subject: [PATCH] Fix setting the last_refresh_date on error Fix an issue where the last_refresh_date is being set incorrectly in the exception handler when a refresh fails. --- .../providers/vmware/infra_manager/inventory/collector.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/models/manageiq/providers/vmware/infra_manager/inventory/collector.rb b/app/models/manageiq/providers/vmware/infra_manager/inventory/collector.rb index 92c27abb5..5b7ee5271 100644 --- a/app/models/manageiq/providers/vmware/infra_manager/inventory/collector.rb +++ b/app/models/manageiq/providers/vmware/infra_manager/inventory/collector.rb @@ -43,7 +43,7 @@ def run _log.error("#{log_header} Refresh failed") _log.log_backtrace(err) - ems.update_attributes(:last_refresh_error => err.to_s, :last_refresh_time => Time.now.utc) + ems.update_attributes(:last_refresh_error => err.to_s, :last_refresh_date => Time.now.utc) ensure saver.stop_thread destroy_property_filter(property_filter)