-
Notifications
You must be signed in to change notification settings - Fork 49
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
Seems like the breaking change in BusyBox ("timeout: fix arguments to match coreutils") make an issue #7
Comments
Loks like it works without 59 wait_for_wrapper()
60 {
61 # In order to support SIGINT during timeout: http://unix.stackexchange.com/a/57692
62 if [ $QUIET -eq 1 ]; then
63 # timeout $BUSYTIMEFLAG $TIMEOUT $0 -q -c -h $HOST -p $PORT -t $TIMEOUT &
64 timeout $BUSYTIMEFLAG $TIMEOUT $0 -q -c -h $HOST -p $PORT $TIMEOUT &
65 else
66 # timeout $BUSYTIMEFLAG $TIMEOUT $0 -c -h $HOST -p $PORT -t $TIMEOUT &
67 timeout $BUSYTIMEFLAG $TIMEOUT $0 -c -h $HOST -p $PORT $TIMEOUT &
68 fi
69 PID=$!
70 trap "kill -INT -$PID" INT
71 wait $PID
72 RESULT=$?
73 if [[ $RESULT -ne 0 ]]; then
74 echoerr "$cmdname: timeout occurred after waiting $TIMEOUT seconds for $HOST:$PORT"
75 fi
76 return $RESULT
77 }
...
118 if test "${TIMEOUT_PATH#*$BUSYBOX}" != "$BUSYBOX"; then
119 ISBUSY=1
120 # BUSYTIMEFLAG="-t"
121 BUSYTIMEFLAG=""
122 else And after $ docker logs vault_vault_1
wait-for-it.sh: waiting 15 seconds for consul:8500
wait-for-it.sh: consul:8500 is available after 0 seconds
==> Vault server configuration: |
I can confirm this issue, as well as the workaround posted by @ovc. |
great, thanks for the workaround and for confirming. do you want to submit a pull request? if not I can apply your fix. let me know. |
tolitius
added a commit
that referenced
this issue
Nov 7, 2019
ok, it should be fixed now. let me know if it works for you |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have the same issue like here: vishnubob/wait-for-it#71 in
tools/wait-for-it.sh
The text was updated successfully, but these errors were encountered: