Skip to content

.travis.yml: allow 30 minutes of inactivity in cloud tests#287

Merged
OddBloke merged 2 commits into
canonical:masterfrom
OddBloke:travis
Mar 27, 2020
Merged

.travis.yml: allow 30 minutes of inactivity in cloud tests#287
OddBloke merged 2 commits into
canonical:masterfrom
OddBloke:travis

Conversation

@OddBloke
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator

Could we modify the cloud_tests default timeout in CI only ?

@OddBloke
Copy link
Copy Markdown
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
Copy Markdown
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