From ce32f931b390e7bfd25f992979e494c6eeb84ace Mon Sep 17 00:00:00 2001 From: Greg McCullough Date: Thu, 25 Jan 2018 08:10:55 -0500 Subject: [PATCH] Merge pull request #16868 from lfu/event_action_refresh_sync_1534631 Event state machine is added to replace the synchronous refresh. (cherry picked from commit cac9ac77123e8b2e83fabddab4ea66f149ea61f8) https://bugzilla.redhat.com/show_bug.cgi?id=1550725 --- app/models/ems_event/automate.rb | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/app/models/ems_event/automate.rb b/app/models/ems_event/automate.rb index d00e7c1d34b..6dda04dd5b2 100644 --- a/app/models/ems_event/automate.rb +++ b/app/models/ems_event/automate.rb @@ -17,10 +17,7 @@ def refresh(*targets, sync) refresh_targets = targets.collect { |t| get_target("#{t}_refresh_target") unless t.blank? }.compact.uniq return if refresh_targets.empty? - task_ids = EmsRefresh.queue_refresh(refresh_targets, nil, :create_task => sync) - - # Wait for the tasks to finish if we are doing a synchronous refresh - task_ids.each { |tid| MiqTask.wait_for_taskid(tid) } if sync + EmsRefresh.queue_refresh(refresh_targets, nil, :create_task => sync) end def refresh_new_target