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

travis: sudo: true, to gain 4G of RAM, as per @domenkozar suggestion to fix #24200 #24258

Merged
merged 2 commits into from Mar 24, 2017

Conversation

deepfire
Copy link
Contributor

Motivation for this change

#24200

@domenkozar
Copy link
Member

We could instead set sudo to true, so that machine would have 7G of ram: https://github.com/NixOS/nixpkgs/blob/master/.travis.yml#L5

@deepfire deepfire changed the title travis: drop nixpkgs-tarball, as per @matthewbauer suggestion to fix #24200 travis: sudo: true, to gain 4G of RAM, as per @domenkozar suggestion to fix #24200 Mar 24, 2017
@domenkozar
Copy link
Member

I don't understand why @matthewbauer switched to container based infrastructure in 95b28da, but it did decrease ram significantly. Maybe that should be noted as a comment.

@joachifm
Copy link
Contributor

@deepfire
Copy link
Contributor Author

@domenkozar, added this:

# 'sudo: false' == containers that start fast, but only get 4G ram;
# 'sudo: true'  == VMs that start slow, but with 8G
# ..as per:  https://docs.travis-ci.com/user/ci-environment/#Virtualization-environments
# Nixpkgs PR tests OOM with 4G: https://github.com/NixOS/nixpkgs/issues/24200

@domenkozar
Copy link
Member

Line 11 should be sudo: required so PR additional line is not needed.

@globin
Copy link
Member

globin commented Mar 24, 2017

Citing @edolstra wrt memory usage here: NixOS/nix#421 (comment)

You can tell Nix to garbage-collect more aggressively:

GC_INITIAL_HEAP_SIZE=128k nix-env -qa

With this, nix-env -qa takes 235 MB on my laptop. (Without it, 471 MB.)

You can also do:

echo 1 > /proc/sys/vm/overcommit_memory

That should make the fork error go away.

@7c6f434c
Copy link
Member

7c6f434c commented Mar 24, 2017 via email

@domenkozar
Copy link
Member

It seems to work again:

error: undefined variable ‘fetchPypi’ at /home/travis/build/NixOS/nixpkgs/pkgs/development/python-modules/btrees/default.nix:10:9
(use ‘--show-trace’ to show detailed location information)

@domenkozar domenkozar merged commit c24e7f7 into NixOS:master Mar 24, 2017
@deepfire deepfire deleted the travis-pr-repair branch March 24, 2017 17:11
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.

None yet

5 participants