Skip to content
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

travis_wait should extend wait time if input is received #8575

Closed
abitrolly opened this issue Oct 12, 2017 · 4 comments
Closed

travis_wait should extend wait time if input is received #8575

abitrolly opened this issue Oct 12, 2017 · 4 comments

Comments

@abitrolly
Copy link

abitrolly commented Oct 12, 2017

Without travis_wait a build was failing with No output has been received in the last 10m0s in 40 minutes because the last binary assembly step is indeed long, but with travis_wait that build fails after 20 minutes with Timeout (20 minutes) reached.. Looks like travis_wait doesn't reset its timer if the input is received.

abitrolly added a commit to cyberFund/cybernode-archive that referenced this issue Oct 12, 2017
Otherwise Travis kills job after default 20 minutes anyways
travis-ci/travis-ci#8575
abitrolly added a commit to cyberFund/cybernode-archive that referenced this issue Oct 12, 2017
Otherwise Travis kills job after default 20 minutes anyways
travis-ci/travis-ci#8575
@BanzaiMan
Copy link
Contributor

travis_wait watches the process it spawns, not STDOUT. Such a functionality goes beyond its intended purpose, and is impractical to implement at this time.

@abitrolly
Copy link
Author

@BanzaiMan what is the intended purpose then? And source.. From the docs it is made to extend the program timeout if no output is being received. So it doesn't fulfill its purpose - it kills program even if output is received.

@BanzaiMan
Copy link
Contributor

The purpose is to extend the time limit.

travis_wait then writes a short line to the build log every minute for 20 minutes, extending the amount of time your command has to finish.

The clock starts when the command starts, and it is not reset.

@abitrolly
Copy link
Author

@BanzaiMan is it used anywhere else, except for the above scenario of "extend the program timeout if no output is being received"? Because it does a poor job in this user story.

abitrolly added a commit to cyberFund/cybernode-archive that referenced this issue Oct 23, 2017
1. travis_wait doesn't show output while processing
2. it doesn't show it at all when timeout occure

`travis_wait` choice is to not disrupt logs, but we
are not parsing them. so that's ok

References:

travis-ci/travis-ci#8575
travis-ci/travis-ci#5716
travis-ci/travis-ci#3358
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants