PileProject / drive

The drive project
Other
5 stars 4 forks source link

Make sudo required in .travis.yml #60

Closed tiwanari closed 7 years ago

tiwanari commented 7 years ago

This PR makes sudo enabled in CI. This was suggested by a Customer Support Specialist of Travis to make CI more stable in an e-mail I received after asking why it failed randomly ( #49 ). At least, it passed in a build.

I had a look at your build and I can see that it's running on our container-based infrastructure. Other users have reported more stable builds on our sudo-enabled infrastructure, would you like to give this a try? Here's what you need to add to your .travis.yml file:

sudo: required

One major difference is that the sudo-enabled VMs have 7.5 GB of RAM instead of 4 GB (shared) on the container-based infrastructure.

Please let us know the result and we will follow-up upon your reply.

tiwanari commented 7 years ago

I don't know but it seemed to make testing faster ๐ŸŽ‰

makotoshimazu commented 7 years ago

That makes sense because sudo tests won't deploy the environment on the container according to the explanation.

2016ๅนด12ๆœˆ12ๆ—ฅ(ๆœˆ) 14:59 Tatsuya Iwanari notifications@github.com:

I don't know but it seemed to make testing faster ๐ŸŽ‰

โ€” You are receiving this because you were assigned.

Reply to this email directly, view it on GitHub https://github.com/PileProject/drive/pull/60#issuecomment-266349535, or mute the thread https://github.com/notifications/unsubscribe-auth/ADRJdGoGIDaS25-cEwuUsKGCPHREjI5Oks5rHOLKgaJpZM4K-n0F .

-- Makoto Shimazu