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
nixosTests.installer: Increase memory allocated to the VM #111495
Conversation
Fixes NixOS#111223. Signed-off-by: Anders Kaseorg <andersk@mit.edu>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Confirmed locally to fix that test on x86_64.
@ofborg eval |
It doesn't seem too important, but how's the i686 case for you? I'm getting stuck on
both before and after this PR (and even if I increase it to 1024 as well). |
@vcunat If I make it so qemu’s stderr doesn’t disappear into the ether: --- a/nixos/lib/test-driver/test-driver.py
+++ b/nixos/lib/test-driver/test-driver.py
@@ -748,7 +748,6 @@ class Machine:
self.script,
stdin=subprocess.DEVNULL,
stdout=subprocess.PIPE,
- stderr=subprocess.STDOUT,
shell=True,
cwd=self.state_dir,
env=environment, I see this error from the i686 test:
The
But the i686 test is not channel-blocking, so we should deal with it separately. |
OK, let's not complicate this here. I wonder if there's even any use case for an i686 installer nowadays. I only know really about using i686 on a 64-bit OS. |
Motivation for this change
Fixes #111223.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)