[backport] Add timeout to wait_for
in bash tests
#14446
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport of (part of) #14327 and #14360, instituting a default timeout on the bash tests'
wait_for
function.The
test-ck-aws
is currently getting aborted on 2.9, because it is trying to poll status 1200 times. Obviously we want to have a timeout here. I've added a 5 minute timeout to wait formariadb-k8s
to reach an active state (should take ~2 mins).Checklist
Go unit tests, with comments saying what you're testingIntegration tests, with comments saying what you're testingdoc.go added or updated in changed packages