-
Notifications
You must be signed in to change notification settings - Fork 742
Travis build time jumped from 20 min to 1h30 min or more #236
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
Comments
Yeah, it's sort of painful, I agree... Probably the best way forward is rebasing #70 and get rid of |
The culprit is that Travis CI has 128 macOS instances for their entire open source service. This results in very long wait times for macOS builds. |
We should get this on paid instances, then. @larsbergstrom can we do that? |
@tschneidereit Travis is looking currently working to expand the number of macOS instances available. I'll try to get an update on that timeline. They do not have any publicly announced options for reserved paid instances. |
Ah, that's unfortunate. Let's hope they get more instances online soon,
then.
|
I'm sure everyone knows this, but I want to spell it out: this is not wall time, the builds can (and mostly do) run in parallel.
This is our bottleneck. Perhaps we should disable the OSX builds until we can get a paid instance or they add more to their cluster? |
Unfortunately, as @fitzgen said, the issue is not the build times themselves, but the long wait times for macOS builds. This is a Travis CI issue, not one that can be fixed here. |
I think this can be closed since we can't do much about it, and there are a few PRs that make build times acceptable. Feel free to reopen if you disagree or think that some action needs to be taken :) |
Hi,
Since PR #214 We build a lot more thing, and build time as jumped from less than 20min to between 1h20min and 2h. https://travis-ci.org/servo/rust-bindgen/builds
Maybe we should limit to a few interesting build (I think this is possible)?
The current matrix for all pull requests and merge commit is, for example:
Total time 1 hr 28 min 41 sec
The text was updated successfully, but these errors were encountered: