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

doc: coal add warning about VM memory #168

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

unclejack
Copy link

This warning would be useful when setting up CoaL the first time, especially when the disk is thrashing.

@dwlf
Copy link
Contributor

dwlf commented Jul 14, 2015

Would it be sufficient to bold the step text for memory?

"Processes & Memory > set memory to 8192 MB or greater. Be sure to leave Mac OS X with at least 8 GB."
https://github.com/joyent/sdc/blame/master/docs/developer-guide/coal-setup.md#L99

@unclejack
Copy link
Author

@lloydde Fair enough, I've followed the steps from the README the first time and didn't see that part.

I'll update the PR.

@rmustacc
Copy link

In this case was it coal on a Mac backed by an ssd or by a traditional spinning disk? In the former, I run quite comfortably in 6000 MB.

@unclejack
Copy link
Author

@lloydde I've made the change. PTAL

@rmustacc It was a quick test on a spinning disk. I know not to do that again in the future.

Perhaps it would be worth mentioning that a regular spinning disk would benefit from more RAM or maybe just consider using an SSD right away?

@dwlf
Copy link
Contributor

dwlf commented Jul 14, 2015

@unclejack oh ok, it doesn't look like the readme includes updating the memory.
https://github.com/joyent/sdc#cloud-on-a-laptop-coal

@rmustacc good to know. I'll have to 6000 MB a try.

I did try pursue getting "Coal headnode should default to 8GB RAM" [1], but maybe that was shot down because there is a lower reasonable number for general development and test as @rmustacc suggests.
[1] https://smartos.org/bugview/HEAD-2216

@dwlf dwlf self-assigned this Jul 14, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants