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.yml: allow 30 minutes of inactivity in cloud tests #287

Merged
merged 2 commits into from
Mar 27, 2020

Conversation

OddBloke
Copy link
Collaborator

We're seeing CI failures currently because a LXD image takes more than
10 minutes to download if the transatlantic link to Canonical's DC is
heavily contested.

This uses a helper provided by Travis[0] to avoid cancellation due to
lack of output for 30 minutes, which should be sufficient to download an
image even at the lowest speeds we've been seeing.

[0] https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received

We're seeing CI failures currently because a LXD image takes more than
10 minutes to download if the transatlantic link to Canonical's DC is
heavily contested.

This uses a helper provided by Travis[0] to avoid cancellation due to
lack of output for 30 minutes, which should be sufficient to download an
image even at the lowest speeds we've been seeing.

[0] https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received
@OddBloke
Copy link
Collaborator Author

So it looks like travis_wait swallows the output of the command it's wrapping, so this is probably only a short-term solution.

@raharper
Copy link
Collaborator

Could we modify the cloud_tests default timeout in CI only ?

@OddBloke
Copy link
Collaborator Author

Could we modify the cloud_tests default timeout in CI only ?

We aren't hitting a timeout internal to our code at all. Travis will consider any job that hasn't emitted output for 10 minutes dead, and kill it. All travis_wait does is emit output every few seconds to avoid this check triggering.

Copy link
Collaborator

@raharper raharper left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see. Let's see if this is enough.

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

Successfully merging this pull request may close these issues.

2 participants