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

Enable usage of the Trusty beta on Travis #144

Merged
merged 2 commits into from Nov 1, 2015

Conversation

@aneeshusa
Copy link
Member

aneeshusa commented Nov 1, 2015

Travis now has new Trusty based instances in Beta. They offer much
faster boot times, twice as much memory, and a more reliable core count.

Review on Reviewable

aneeshusa added 2 commits Nov 1, 2015
Travis now has new Trusty based instances in Beta. They offer much
faster boot times, twice as much memory, and a more reliable core count.
wget throws cert errors when attempting to download the SaltStack repo
GPG key from repo.saltstack.com, even though ca-certificates is
up-to-date.
@aneeshusa aneeshusa force-pushed the aneeshusa:switch-to-trusty branch from 717ad01 to a2d4f50 Nov 1, 2015
@aneeshusa
Copy link
Member Author

aneeshusa commented Nov 1, 2015

Also see http://docs.travis-ci.com/user/ci-environment/#Virtualization-environments for a summary of the differences between the standard (Precise) and beta (Trusty) environments.

@Manishearth
Copy link
Member

Manishearth commented Nov 1, 2015

@bors-servo r+

Should we be turning this on for other repos that require sudo?

@bors-servo
Copy link
Contributor

bors-servo commented Nov 1, 2015

📌 Commit a2d4f50 has been approved by Manishearth

@bors-servo
Copy link
Contributor

bors-servo commented Nov 1, 2015

Testing commit a2d4f50 with merge 81e7ec8...

bors-servo added a commit that referenced this pull request Nov 1, 2015
Enable usage of the Trusty beta on Travis

Travis now has new Trusty based instances in Beta. They offer much
faster boot times, twice as much memory, and a more reliable core count.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/saltfs/144)
<!-- Reviewable:end -->
@Manishearth
Copy link
Member

Manishearth commented Nov 1, 2015

We also have Trusty on our linux boxes so this works out well!

@aneeshusa
Copy link
Member Author

aneeshusa commented Nov 1, 2015

Which other repos? I was looking through the Travis docs (actually looking into #142) and noticed they now have a beta for Trusty environments. There only seem to be advantages to switching, so I'd recommend yes after testing to make sure it doesn't cause any regressions.

@aneeshusa
Copy link
Member Author

aneeshusa commented Nov 1, 2015

Ooh, the increased parity also sounds like a good idea, +1 for keeping all environments in sync as much as possible.

@Manishearth
Copy link
Member

Manishearth commented Nov 1, 2015

Well, sudo: false is still faster than the Trusty thing right now.

The repos that need sudo: required are actually using sudo: 9000, which is a whole different thing and probably won't work with this Trusty stuff.

@bors-servo
Copy link
Contributor

bors-servo commented Nov 1, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit a2d4f50 into servo:master Nov 1, 2015
1 of 2 checks passed
1 of 2 checks passed
homu Testing commit a2d4f50 with merge 81e7ec8...
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@aneeshusa
Copy link
Member Author

aneeshusa commented Nov 1, 2015

Yeah, the significantly faster speed of the container infrastructure makes sense for anything that doesn't need sudo.

What's sudo:9000? A quick search seems to say it's Ubuntu 14.04 on GCE, which AFAICT is the same as the Trusty beta.

@Manishearth
Copy link
Member

Manishearth commented Nov 1, 2015

Unpublicized faster builds on GCE (experimental feature), not sure if it's exactly the same as trusty, @larsbergstrom would know.

@larsbergstrom
Copy link
Contributor

larsbergstrom commented Nov 1, 2015

They're the same thing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.