-
Notifications
You must be signed in to change notification settings - Fork 156
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
Command/Output Ordering is messed up nondeterministically #901
Comments
@sils1297 I can't find the content of
Am I right? |
@yegor256 nope. It was
|
@sils1297 it's just technically not possible... can you show full code of your script? |
@yegor256 just pushed it for you: https://github.com/sils1297/coala/blob/sils/rultor/.misc/deploy.rultor.sh as you can see from the commit sha it's the very same commit used in the log. |
@sils1297 I can't reproduce the problem... What do you have in |
@yegor256 this branch points to the very same commit that resultet in the failure, here's the branch link: https://github.com/sils1297/coala/tree/sils/rultor thats the rultor.yml then: https://github.com/sils1297/coala/blob/sils/rultor/.rultor.yml |
@sils1297 we will find someone to do this task, soon |
@sils1297 since there is no milestone yet I set it to "2.0" |
@yegor256 since you asked, this is a result of output buffering occurring, since the containers are not ran in line buffered mode. Likely this can simply be fixed by running the container with the -t ( tty => line buffering) flag, but this needs to be investigated. Could be we have multiple spots where buffering can occur in the execution. Pushed a small demo of this here: https://github.com/original-brownbear/output-buffering-docker |
@original-brownbear interesting catch! well, let's try the |
@yegor256 yes, lets go for it :) |
@alex-palevsky assign me please. |
@original-brownbear yep, it is yours, go ahead |
@rultor deploy |
@sils1297 can you try again ? This issue should be resolved now :) |
Hey, I don't have time to try it out, sorry. |
@original-brownbear 45 mins sent to your balance (ID |
See http://www.rultor.com/t/4530-122266254 the lines after
+ bash .misc/deploy.rultor.sh
, first it echoes two ls commands, then it shows the output of the two ls commands and then it does the third command and its output.So some ordering problem seems to be there.
The text was updated successfully, but these errors were encountered: