search

MDAnalysis / GridDataFormats

GridDataFormats is a pure Python library to handle data on a regular grid using commonly used file formats in molecular simulations.
https://mdanalysis.org/GridDataFormats
GNU Lesser General Public License v3.0
29 stars 18 forks source link

This should container builds in travis-CI #14

Closed kain88-de closed 9 years ago

kain88-de commented 9 years ago

lets see if this works

orbeckst commented 9 years ago

I thought container builds are the default nowadays, and you'd only get something different with sudo: True.

kain88-de commented 9 years ago

I got a message about using the old infrastructure on the travis build. The docs say that they are only explicitly activated with the sudo flag

http://docs.travis-ci.com/user/migrating-from-legacy/?utm_source=legacy-notice&utm_medium=banner&utm_campaign=legacy-upgrade

orbeckst commented 9 years ago

Hi Max,

I got a message about using the old infrastructure on the travis build. The docs say that they are only explicitly activated with the sudo flag

Then I obviously mis-remembered. I thought that MDAnalysis runs on the new containers even though the sudo=false is flag is not set. At least that's what I remember from conversations with @richardjgowers and checking the log such as https://travis-ci.org/MDAnalysis/mdanalysis/builds/80046564 says that it is using docker and containers:

Using worker: worker-linux-docker-bf8ff456.prod.travis-ci.org:travis-linux-3

This job is running on container-based infrastructure, which does not allow use of 'sudo', setuid and setguid executables. If you require sudo, add 'sudo: required' to your .travis.yml See http://docs.travis-ci.com/user/workers/container-based-infrastructure/ for details.

Cheers, Oliver

richardjgowers commented 9 years ago

@orbeckst you're not wrong, MDAnalysis defaulted to container based once we got rid of sudo. I'm not sure why this didn't too, but it's fixed now