Skip to content

increase failure timeout from 10 minutes to 20 minutes #37413

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Apr 18, 2024
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions run-ab-platform.sh
Original file line number Diff line number Diff line change
Expand Up @@ -106,7 +106,7 @@ EOL
}

# TelemetryDockerUp checks if the webapp container is in a running state. If it is it will send a successful event.
# if after 10 minutes it hasn't succeeded, a failed event will be sent (or if the user terminates early, a failed event would
# if after 20 minutes it hasn't succeeded, a failed event will be sent (or if the user terminates early, a failed event would
# also be sent).
#
# Note this only checks if the webapp container is running, that doesn't actually mean the entire stack is up.
Expand All @@ -117,8 +117,8 @@ TelemetryDockerUp()
return
fi

# for up to 600 seconds (10 minutes), check to see if the server services is in a running state
end=$((SECONDS+600))
# for up to 1200 seconds (20 minutes), check to see if the server services is in a running state
end=$((SECONDS+1200))
while [ $SECONDS -lt $end ]; do
webappState=$(docker compose ps --all --format "{{.Service}}:{{.State}}" 2>/dev/null | grep server | cut -d ":" -f2 | xargs)
if [ "$webappState" = "running" ]; then
Expand Down
Loading