-
Notifications
You must be signed in to change notification settings - Fork 273
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
Speed up travis build #226
Comments
Yes it could reduce the speed considerably we just need to specify cache:bundler simple and beautiful. |
For docker, there could also be a cache so the image is not puled every time. |
I think it is enabled for default. |
@niccokunzmann we could also run the jobs in parallel like splitting them up. |
Good idea: then, we need to make sure that
|
I think if any of the jobs fail, Travis returns exit code 1 that means exit. And about piping we could achieve it easily. @niccokunzmann |
@niccokunzmann @Abhi2424shek Any updates on this? |
Please keep in mind that you can merge pull-requests even if the tests still run or did not succeed. It is more important to get the people on board than to have some tests satisfied. If you require green tests over happy contributors, you need to write it into contributing.md. |
@niccokunzmann I prefer happy contributors and good code, so we will need to do something and not sit like dull men. |
Since this issue has been looming large, @niccokunzmann I think we should create a python script that will be executed by travis to fix the failing tests, like adding new line at EOF and other stuff which the tests fail of, what do you say. |
@Abhi2424shek +1 for that |
Maybe, we can change the parameters for coala |
|
btw, Docker caching isnt supported by Travis (but there are ugly workarounds) : travis-ci/travis-ci#5358 |
@jayvdb if Travis apply patches, then we need to setup git to push changes back... |
@niccokunzmann we could let coala apply the patches why dont we have a try plus we will rum travis tests parallel and cache travis, wont doing these close up this PR if all those worked :) |
Sure, you can give it a try. |
Yup, that can be done, but it isnt relevant to this issue about speeding up the travis build. We can avoid the Docker if coala/coala-bears#1000 is fixed. |
I've added travis cache #415 @niccokunzmann @jayvdb @m1guelpf now travis time consumption will start to decrease. |
@niccokunzmann we could now implement running the jobs in parallel |
@Abhi2424shek We can't, as the Travis tasks need an order. |
@m1guelpf I got a solution for that, We could run a job first and execute the next jobs parallel thorught the Travis metrix, but there's a slight bug which I am facing... |
Subtask coala/coala-bears#1044 is a GCI task https://codein.withgoogle.com/tasks/6276537751961600/ |
I think this is the maximum here! |
I can see many ways to speed it up :P |
Not currently -- too lazy to make one...if you wanna do it -- be my guest @jayvdb -- I'll reopen it We COULD cache the docker image for coala -- that's an option... |
We have a seperate issue for it @robbyoconnor |
no, caching the docker image is not helpful (it is too big to be useful cached) and #603 is about caching the coala metadata (good idea, unrelated), are not the only ways to improve the travis build speed. One of the largest chunks of time is the Please re-open. We have generic "fix an issue" tasks which can be used to optimise the CI for this issue. |
Here you go @jayvdb |
@niccokunzmann @Abhi2424shek @jayvdb @robbyoconnor is this still needed not that CodeIn has finished? |
indeed, it can be closed. |
I don't think we can speed up the build anymore... The bottleneck is docker...everything else runs fast. |
Travis can cache certain directories and has cache defaults for several languages.
It may be, we can speed up the testing process.
Hints: The result could be about 5 lines long but the question is how to find the right lines.
The text was updated successfully, but these errors were encountered: