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

Increase memory for VM build #2859

Closed
wants to merge 1 commit into from
Closed

Conversation

Yorwba
Copy link
Contributor

@Yorwba Yorwba commented Oct 22, 2021

I rebuilt my VM today after upgrading to a new version of VirtualBox and ran into an issue with one of the C++ compilation steps failing with an out-of-memory error. The build succeeded once I doubled the available memory.

@jiru
Copy link
Member

jiru commented Nov 15, 2021

2GB feels like a lot; it would increase the RAM requirement of the host running the VM to 4GB or more. How about we try to optimize memory usage of the build process instead? Compilation can use a lot of memory but I think we should be able to get away with 1GB. It’s not like we are compiling super big projects.

Do you remember which step produced the error? Here are some suggestions:

  • Maybe Tatoeba/Tatodetect@262d407 is causing the problem? (Sinoparserd also uses -j 2 but it’s supposedly lightweight to compile.)
  • What if you reorder tasks in ansible/tatoeba.tasks.yml so that the problematic compilation step happens first, before installing and starting up nginx, mysql and php?

@Yorwba
Copy link
Contributor Author

Yorwba commented Nov 22, 2021

Closing in favor of Tatoeba/Tatodetect#3

@Yorwba Yorwba closed this Nov 22, 2021
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.

2 participants