Skip to content

Commit

Permalink
Merge pull request #742 from umccr/bugfix/max-results-cant-be-more-th…
Browse files Browse the repository at this point in the history
…an-100

Max results can't be more than 100
  • Loading branch information
alexiswl authored Dec 1, 2024
2 parents d14d0d3 + 8f93635 commit 7b78311
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"Type": "Task",
"Parameters": {
"Cluster": "${__cluster_arn__}",
"MaxResults": 401
"MaxResults": 100
},
"Resource": "arn:aws:states:::aws-sdk:ecs:listTasks",
"ResultSelector": {
Expand All @@ -20,7 +20,7 @@
"Choices": [
{
"Variable": "$.get_num_tasks_step.num_tasks_running",
"NumericGreaterThan": 400,
"NumericGreaterThan": 99,
"Next": "Sleep a minute",
"Comment": "More than 400 tasks running"
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"Type": "Task",
"Parameters": {
"Cluster": "${__cluster_arn__}",
"MaxResults": 401
"MaxResults": 100
},
"Resource": "arn:aws:states:::aws-sdk:ecs:listTasks",
"ResultSelector": {
Expand All @@ -20,7 +20,7 @@
"Choices": [
{
"Variable": "$.get_num_tasks_step.num_tasks_running",
"NumericGreaterThan": 400,
"NumericGreaterThan": 99,
"Next": "Sleep a minute",
"Comment": "More than 400 tasks running"
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,7 @@
"instrument_run_folder_uri.$": "$.analysis_outputs_step.output_json.instrumentRunOraOutputUri"
}
},
"ResultPath": null,
"End": true
}
}
Expand Down

0 comments on commit 7b78311

Please sign in to comment.