Skip to content

Commit

Permalink
Fixes #37590 - Trigger ready batches on planning finished
Browse files Browse the repository at this point in the history
Previously we sent the last batch event followed by a regular
TriggerNextBatch event with correct number of batches. The proxy batch
triggering action received the TriggerLastBatch event, set an internal
flag, triggered a single batch and exited, the other event got
discarded. This commit changes the ordering so that the TriggerNextBatch
event is sent out first when planning is done.

(cherry picked from commit 1eb59bb)
  • Loading branch information
adamruzicka committed Jun 28, 2024
1 parent 0c787f8 commit 98ae879
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions app/lib/actions/remote_execution/run_hosts_job.rb
Original file line number Diff line number Diff line change
Expand Up @@ -75,6 +75,7 @@ def trigger_remote_batch
end

def on_planning_finished
trigger_remote_batch
plan_event(Actions::TriggerProxyBatch::TriggerLastBatch, nil, step_id: input[:trigger_run_step_id])
super
end
Expand Down

0 comments on commit 98ae879

Please sign in to comment.