Skip to content

Commit

Permalink
Add more retries
Browse files Browse the repository at this point in the history
  • Loading branch information
morris25 committed Sep 23, 2021
1 parent 36a15c6 commit b4bb03d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions test/batch_node_online.jl
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ end
@test status(worker_job) == AWSBatch.SUCCEEDED

manager_log = log_messages(manager_job)
worker_log = log_messages(worker_job, retries=0)
worker_log = log_messages(worker_job; retries=0)
test_results = [
@test occursin("Only 0 of the 1 workers job have reported in", manager_log)
@test isempty(worker_log)
Expand All @@ -393,7 +393,7 @@ end
# Display the logs for all the jobs if any of the log tests fail
if any(r -> !(r isa Test.Pass), test_results)
@info "Details for manager:\n$(report(manager_job))"
@info "Details for worker:\n$(report(worker_log))"
@info "Details for worker:\n$(report(worker_job))"
end
end

Expand Down
2 changes: 1 addition & 1 deletion test/utils.jl
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Gets the logs messages associated with a AWSBatch BatchJob as a single string
function log_messages(job::BatchJob; retries=2, wait_interval=7)
function log_messages(job::BatchJob; retries=5, wait_interval=7)
i = 0
events = log_events(job)
# Retry if no logs are found
Expand Down

0 comments on commit b4bb03d

Please # to comment.