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

Travis: use new system matrix cache by Travis #3678

Merged
merged 1 commit into from Jun 2, 2016

Conversation

pascalgrimaud
Copy link
Member

15 days ago, Travis improved their cache system.
See here : travis-ci/travis-ci#4393
Official doc: https://docs.travis-ci.com/user/caching/

It fits perfectly what we need here, because every build is too different. And these new caches will be available to all PR.

What have been done here:

We won't see any improvment until this PR is merged, and after the merge, the 1st full build will cache everything, so benefits will be seen for the next builds.

@jdubois :
At the moment, let keep the 2 repo jhipster-travis-build or jhipster-travis-cache
I will tell you when you can delete them, maybe in 3 weeks~ to be sure this system works

@jhipster/developers :
You can manage cache here https://travis-ci.org/jhipster/generator-jhipster/caches
So when needed, we can delete them to let Travis rebuild correctly the cache

- upgrade to node 4.4.5 LTS
- every build in matrix has his own system cache
- no need to clone the cache repo
- check maven or gradle wrapper instead of application's name
@jdubois
Copy link
Member

jdubois commented Jun 2, 2016

This is so cool!!!!! Thanks a lot @pascalgrimaud I totally missed that this was released!
Merging immediately :-)

@jdubois jdubois merged commit f1c1992 into jhipster:master Jun 2, 2016
@jdubois
Copy link
Member

jdubois commented Jun 2, 2016

OK, that broke the build, but I have no idea why!!!

@pascalgrimaud
Copy link
Member Author

@jdubois : there is an issue with Docker's APT get repo, so I don't think it's my PR...

See travis-ci/travis-ci#6124 and moby/moby#23203

If it's not resolved fast enough, I will try to use another version of Docker

@pascalgrimaud
Copy link
Member Author

Just restarted the Travis build. All buids are OK

@jdubois jdubois modified the milestone: 3.4.1 Jun 13, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants