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

Add travis_wait to override the 10 minute timeout. #2296

Closed
wants to merge 1 commit into from

Conversation

@jrmuizel
Copy link
Contributor

jrmuizel commented Jan 12, 2018

We seem to be hitting this on every PR right now. Let's see if
this might fix it.

See documentation:

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


This change is Reviewable

@jrmuizel
Copy link
Contributor Author

jrmuizel commented Jan 12, 2018

@jrmuizel
Copy link
Contributor Author

jrmuizel commented Jan 12, 2018

I added an 'unset -f cd' to deal with travis-ci/travis-ci#8703

@kvark
Copy link
Member

kvark commented Jan 12, 2018

@bors-servo
Copy link
Contributor

bors-servo commented Jan 12, 2018

📌 Commit c4d44ba has been approved by kvark

@bors-servo
Copy link
Contributor

bors-servo commented Jan 12, 2018

Testing commit c4d44ba with merge 6825c5c...

bors-servo added a commit that referenced this pull request Jan 12, 2018
Add travis_wait to override the 10 minute timeout.

We seem to be hitting this on every PR right now. Let's see if
this might fix it.

See documentation:

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

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2296)
<!-- Reviewable:end -->
@jrmuizel jrmuizel force-pushed the jrmuizel:longer-time branch from c4d44ba to 966b489 Jan 12, 2018
@glennw
Copy link
Member

glennw commented Jan 12, 2018

@bors-servo p=100 r+

@bors-servo
Copy link
Contributor

bors-servo commented Jan 12, 2018

📌 Commit 966b489 has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Jan 12, 2018

Testing commit 966b489 with merge 0e08b07...

bors-servo added a commit that referenced this pull request Jan 12, 2018
Add travis_wait to override the 10 minute timeout.

We seem to be hitting this on every PR right now. Let's see if
this might fix it.

See documentation:

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

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2296)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2018

💔 Test failed - status-travis

@glennw
Copy link
Member

glennw commented Jan 13, 2018

@bors-servo retry

@kvark @jrmuizel This job failed with exit code 137 - which I think typically means killed by OOM. I'm not sure what the best way forward is - maybe running with -j 1 might help, by reducing the number of active processes.

@glennw glennw force-pushed the jrmuizel:longer-time branch from 966b489 to e40d643 Jan 13, 2018
@glennw
Copy link
Member

glennw commented Jan 13, 2018

I pushed a change here to restrict cargo building to one build job at a time. Let's see if that helps with the OOM issue.

@bors-servo p=1 r=glennw,jrmuizel

@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2018

📌 Commit e40d643 has been approved by glennw,jrmuizel

@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2018

Testing commit e40d643 with merge 1ec7c9e...

bors-servo added a commit that referenced this pull request Jan 13, 2018
Add travis_wait to override the 10 minute timeout.

We seem to be hitting this on every PR right now. Let's see if
this might fix it.

See documentation:

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

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2296)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 13, 2018

💔 Test failed - status-travis

@glennw
Copy link
Member

glennw commented Jan 14, 2018

This doesn't seem to help.

@glennw glennw closed this Jan 14, 2018
@glennw glennw reopened this Jan 14, 2018
@glennw
Copy link
Member

glennw commented Jan 14, 2018

@bors-servo p=1 r=glennw,jrmuizel

@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2018

💡 This pull request was already approved, no need to approve it again.

  • This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.
  • There's another pull request that is currently being tested, blocking this pull request: #2284
@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2018

📌 Commit e40d643 has been approved by glennw,jrmuizel

@glennw glennw force-pushed the jrmuizel:longer-time branch from e40d643 to ec1ca12 Jan 14, 2018
@glennw
Copy link
Member

glennw commented Jan 14, 2018

I think we have two issues - sometimes 10min timeout related, sometimes OOM related. Updated this patch to just cover the timeout - this may still be helpful to reduce the failure rate.

@bors-servo r=glennw,jrmuizel p=100

@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2018

📌 Commit ec1ca12 has been approved by glennw,jrmuizel

@jrmuizel jrmuizel force-pushed the jrmuizel:longer-time branch from ec1ca12 to b71aff3 Jan 14, 2018
@jrmuizel
Copy link
Contributor Author

jrmuizel commented Jan 14, 2018

Oops, I just overrode this by trying something else out.

@glennw
Copy link
Member

glennw commented Jan 14, 2018

@jrmuizel Want to add back the travis_wait in this PR and we'll try to merge this next, to see if we get any useful log info?

@jrmuizel jrmuizel force-pushed the jrmuizel:longer-time branch from b71aff3 to 1705b6a Jan 14, 2018
@jrmuizel
Copy link
Contributor Author

jrmuizel commented Jan 14, 2018

Done.

@glennw
Copy link
Member

glennw commented Jan 14, 2018

@bors-servo p=100 r+

@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2018

📌 Commit 1705b6a has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Jan 14, 2018

Testing commit 1705b6a with merge 7c6b338...

bors-servo added a commit that referenced this pull request Jan 14, 2018
Add travis_wait to override the 10 minute timeout.

We seem to be hitting this on every PR right now. Let's see if
this might fix it.

See documentation:

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

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/2296)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Jan 15, 2018

💔 Test failed - status-travis

@glennw
Copy link
Member

glennw commented Jan 15, 2018

Looks like this has now failed due to exceeding the 4MB log limit 👎

@glennw glennw closed this Jan 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.