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
nixos-test-driver: allow configuration of net frontend and backend #65212
Conversation
When IPXE tests were added, an option was added for configuring only the frontend, and the backend configuration was dropped entirely. This caused most installer tests to fail.
179e3e8
to
771985d
Compare
That's strange; I checked my changes and there wasn't a frontend configuration before. I also grepped for |
Adding more context: @cleverca22 bisected to 81d35a9 for the first commit making tests fail for |
Thanks, I'm testing it now! I'll also investigate how did it break things, I feel I misunderstand something about qemu options. |
The Qemu wiki has a page on Networking which explains things better than I can, but the super short version is: the old command line used the legacy option |
this is one of the tests that involves multiple network devices in qemu, to setup a full router with upnp, and then test torrents going thru it the test fails on my end, on rev |
That test is passing for me, I've uploaded my log to a gist. |
@thefloweringash I see, I also misunderstood how tests work - |
Ran the same tests (installer and netboot), they succeed for me. @cleverca22 BitTorrent test succeeds for me on this revision, there may be test flakiness in motion. Can you post your log? Let's merge; this blocks master. |
When IPXE tests were added, an option was added for configuring only
the frontend, and the backend configuration was dropped entirely. This
caused most installer tests to fail.
Motivation for this change
Restore installer tests.
Tested:
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)