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

Allocate extra memory for Vagrant VMs #176

Merged
merged 1 commit into from Dec 15, 2015

Conversation

Projects
None yet
3 participants
@aneeshusa
Member

aneeshusa commented Dec 15, 2015

Provide more memory for VMs that download large files (e.g. the Android
SDK and NDK) to stop salt-call from getting OOM-killed inside the VM.
This makes it possible to test all node types locally with Vagrant.

Review on Reviewable

Allocate extra memory for Vagrant VMs
Provide more memory for VMs that download large files (e.g. the Android
SDK and NDK) to stop salt-call from getting OOM-killed inside the VM.
This makes it possible to test all node types locally with Vagrant.
@metajack

This comment has been minimized.

Contributor

metajack commented Dec 15, 2015

@bors-servo r+


Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.


Comments from the review on Reviewable.io

@bors-servo

This comment has been minimized.

Contributor

bors-servo commented Dec 15, 2015

📌 Commit 7d2ce36 has been approved by metajack

@bors-servo

This comment has been minimized.

Contributor

bors-servo commented Dec 15, 2015

⌛️ Testing commit 7d2ce36 with merge 3e374be...

bors-servo added a commit that referenced this pull request Dec 15, 2015

Auto merge of #176 - aneeshusa:allocate-extra-vm-memory, r=metajack
Allocate extra memory for Vagrant VMs

Provide more memory for VMs that download large files (e.g. the Android
SDK and NDK) to stop salt-call from getting OOM-killed inside the VM.
This makes it possible to test all node types locally with Vagrant.

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

This comment has been minimized.

Contributor

bors-servo commented Dec 15, 2015

☀️ Test successful - travis

@bors-servo bors-servo merged commit 7d2ce36 into servo:master Dec 15, 2015

3 checks passed

code-review/reviewable Review complete: all files reviewed, all discussions resolved
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment