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

Increase timeout of Travis CI tests to 50 minutes #67

Merged
merged 2 commits into from
May 9, 2021

Conversation

matthewdehaven
Copy link
Contributor

Two tests are currently failing when run by Travis. The failure seems to be due to Travis timing out tests that do not produce output for 10 minutes. The two failing tests are longer running tests. The timeout has been increased to 50 minutes to see if that gives the tests enough time to finish.

Example failing job: https://travis-ci.com/github/shop-planner/shop3/jobs/498842611

Relevant Travis documentation: https://docs.travis-ci.com/user/common-build-problems/#build-times-out-because-no-output-was-received

@matthewdehaven matthewdehaven changed the title WIP: Increase timeout of Travis CI tests to 50 minutes Increase timeout of Travis CI tests to 50 minutes May 8, 2021
@matthewdehaven matthewdehaven changed the title Increase timeout of Travis CI tests to 50 minutes WIP: Increase timeout of Travis CI tests to 50 minutes May 8, 2021
@rpgoldman
Copy link
Contributor

So does this actually run the Travis tests? Now that the GitHub actions are on, I have a hard time seeing if these run.

@matthewdehaven
Copy link
Contributor Author

No, the Travis tests don't seem to be running. You can see here that Travis last received a request to run the tests on April 22: https://travis-ci.com/github/shop-planner/shop3/requests

The Travis CI file has not changed since April 22, outside of this merge request. I cannot tell why requests are no longer being made to Travis.

Do you know if the Travis CI was disabled or paused somehow?

@matthewdehaven
Copy link
Contributor Author

This merge request opened on April 15 did have the Travis tests run. Something seems to have changed since then, but I cannot tell what changed.

#58

@rpgoldman
Copy link
Contributor

Yes, that was it. I have un-suspended it.

@matthewdehaven
Copy link
Contributor Author

Okay, I'm going to close this request then re-open it to see if that will trigger Travis.

@matthewdehaven matthewdehaven reopened this May 9, 2021
@matthewdehaven matthewdehaven changed the title WIP: Increase timeout of Travis CI tests to 50 minutes Increase timeout of Travis CI tests to 50 minutes May 9, 2021
@matthewdehaven
Copy link
Contributor Author

@rpgoldman increasing the timeout of the tests from 10 minutes to 50 minutes fixed the Travis CI. I will also try to trim down the time of the tests by adding a compilation step so that there is less repeated compilation occurring. It may be a few days before I get to the trimming down of the run time. So you can merge this branch now if you would like. If this branch is merged by the time I come back to work on the run time of the tests, I'll create a new branch and merge request.

@rpgoldman
Copy link
Contributor

I'll probably merge this now, just to avoid having too many branches outstanding, once I verify that Travis worked better this time.

@rpgoldman rpgoldman merged commit 4b34730 into master May 9, 2021
@rpgoldman rpgoldman deleted the increase-travis-timeout branch May 9, 2021 19:41
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.

3 participants