Skip to content
This repository has been archived by the owner on May 23, 2023. It is now read-only.

Use the new build env on Travis #15

Closed
wants to merge 1 commit into from
Closed

Use the new build env on Travis #15

wants to merge 1 commit into from

Conversation

joshk
Copy link

@joshk joshk commented Nov 21, 2014

faster vms, better boot times, more cpu, more ram, better network

plus caching works on the new setup

faster vms, better boot times, more cpu, more ram, better network

plus caching works on the new setup
@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 8ea8572 on joshk:patch-1 into 12f119a on Yelp:master.

@joshk
Copy link
Author

joshk commented Nov 21, 2014

hmmmm, tests pass but lint doesn't.

I don't know what could be the issue.

@asottile
Copy link
Contributor

I think astroid broke their version on pypi (we were debugging this earlier) so it is probably fine. @bukzor

@bukzor
Copy link
Contributor

bukzor commented Nov 22, 2014

Please link me to doc for sudo:false.

It seems to be the only actual change you made.

@joshk
Copy link
Author

joshk commented Nov 22, 2014

docs are in the works

more ram and cpu, faster vms, better boot times, and scalable infrastructure

@joshk
Copy link
Author

joshk commented Nov 22, 2014

the sudo: false takes care of the routing

@bukzor
Copy link
Contributor

bukzor commented Nov 22, 2014

"the routing" ?

On Fri, Nov 21, 2014 at 10:07 PM, Josh Kalderimis notifications@github.com
wrote:

the sudo: false takes care of the routing


Reply to this email directly or view it on GitHub
#15 (comment).

@joshk
Copy link
Author

joshk commented Nov 22, 2014

we 'route' builds to different queues for our workers.

@bukzor
Copy link
Contributor

bukzor commented Nov 23, 2014

To restate, builds will be put into the new cluster based on this
sudo:false setting. Is that accurate?

I would assume sudo:false has something to do with disabling sudo.
If you're making this setting do double duty, I'd recommend against it.

On Fri, Nov 21, 2014 at 10:09 PM, Josh Kalderimis notifications@github.com
wrote:

we 'route' builds to different queues for our workers.


Reply to this email directly or view it on GitHub
#15 (comment).

@bukzor
Copy link
Contributor

bukzor commented Nov 25, 2014

I just realized we need sudo to set up one of our test cases :(

@bukzor bukzor closed this Nov 25, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants