Skip to content

nixosTests.docker-preloader: Port to Python#80051

Merged
flokli merged 2 commits intoNixOS:masterfrom
tfc:port-docker-preloader-test
Aug 23, 2020
Merged

nixosTests.docker-preloader: Port to Python#80051
flokli merged 2 commits intoNixOS:masterfrom
tfc:port-docker-preloader-test

Conversation

@tfc
Copy link
Copy Markdown
Contributor

@tfc tfc commented Feb 13, 2020

Motivation for this change

#72828

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.

@tfc tfc requested review from flokli and worldofpeace February 13, 2020 19:56
@tfc tfc mentioned this pull request Feb 13, 2020
Closed
@tfc
Copy link
Copy Markdown
Contributor Author

tfc commented Feb 13, 2020

@GrahamcOfBorg test docker-preloader

@ofborg ofborg bot added 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 8.has: documentation This PR adds or changes documentation 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Feb 13, 2020
@flokli
Copy link
Copy Markdown
Member

flokli commented Feb 13, 2020

@tfc Did this succeed on your machine? Seems to fail on ofborg:

open /var/lib/docker/overlay2/e15685a6d503df5e91f284dccf1fe62a2fddbe3db41c4386b883edad8ff1eb13/committed: read-only file system"

@tfc
Copy link
Copy Markdown
Contributor Author

tfc commented Feb 14, 2020

@flokli weird. Yes, it works on both my machines.

@veprbl veprbl added the 6.topic: testing Tooling for automated testing of packages and modules label Feb 14, 2020
@worldofpeace
Copy link
Copy Markdown
Contributor

worldofpeace commented Feb 14, 2020

@GrahamcOfBorg test docker-preloader

@worldofpeace
Copy link
Copy Markdown
Contributor

worldofpeace commented Feb 14, 2020 •
edited
Loading

Fails on my machine

docker # [   14.307652] dockerd[856]: time="2020-02-14T19:17:53.392910714Z" level=error msg="Handler for POST /v1.40/containers/create returned error: open /var/lib/docker/overlay2/7efbea82c8cbd126c34d5e5fa5111fd4349c2034dc7a320627fb9c80f0e31dfd/committed: read-only file system"
docker # /nix/store/6sfqhlii23k5fhpwpzv2gwrzfpdwsawi-docker-19.03.5/libexec/docker/docker: Error response from daemon: open /var/lib/docker/overlay2/7efbea82c8cbd126c34d5e5fa5111fd4349c2034dc7a320627fb9c80f0e31dfd/committed: read-only file system.
docker # See '/nix/store/6sfqhlii23k5fhpwpzv2gwrzfpdwsawi-docker-19.03.5/libexec/docker/docker run --help'.
docker: output:
error: command `docker run nix nix-store --version` failed (exit code 125)
cleaning up
killing docker (pid 9)
(0.00 seconds)

@flokli
Copy link
Copy Markdown
Member

flokli commented Feb 18, 2020

@tfc this also failed for me, when running nix-build -A nixosTests.docker-preloader from a checkout of this branch:

docker # [   12.618288] dockerd[841]: time="2020-02-18T16:57:15.186850000Z" level=info msg="Default bridge (docker0) is assigned with an IP address 172.17.0.0/16. Daemon option --bip can be used to set a preferred IP address"
docker # [   12.626016] systemd-udevd[532]: Using default interface naming scheme 'v243'.
docker # [   12.647880] systemd-udevd[532]: ethtool: autonegotiation is unset or enabled, the speed and duplex are not writable.
docker # [   12.890340] dockerd[841]: time="2020-02-18T16:57:15.458796574Z" level=info msg="Loading containers: done."
docker # [   13.089265] systemd[1]: var-lib-docker-overlay2-opaque\x2dbug\x2dcheck242620588-merged.mount: Succeeded.
docker # [   13.096296] dockerd[841]: time="2020-02-18T16:57:15.664879191Z" level=info msg="Docker daemon" commit=633a0ea838f10e000b7c6d6eed1623e6e988b5bc graphdriver(s)=overlay2 version=19.03.5
docker # [   13.100627] dockerd[841]: time="2020-02-18T16:57:15.669026366Z" level=info msg="Daemon has completed initialization"
docker # [   13.182081] systemd[1]: Started Docker Application Container Engine.
docker # [   13.186243] systemd[1]: Reached target Multi-User System.
docker # [   13.188691] systemd[1]: Startup finished in 3.386s (kernel) + 9.794s (userspace) = 13.181s.
docker # [   13.198959] dockerd[841]: time="2020-02-18T16:57:15.756404510Z" level=info msg="API listen on /run/docker.sock"
docker # [   13.216920] dockerd[841]: time="2020-02-18T16:57:15.785408476Z" level=error msg="Handler for POST /v1.40/containers/create returned error: open /var/lib/docker/overlay2/dad7ff32c10de4660ec848025acfcd2c2fdf7534d215603acb01f0275e551849/committed: read-only file system"
docker # /nix/store/6sfqhlii23k5fhpwpzv2gwrzfpdwsawi-docker-19.03.5/libexec/docker/docker: Error response from daemon: open /var/lib/docker/overlay2/dad7ff32c10de4660ec848025acfcd2c2fdf7534d215603acb01f0275e551849/committed: read-only file system.
docker # See '/nix/store/6sfqhlii23k5fhpwpzv2gwrzfpdwsawi-docker-19.03.5/libexec/docker/docker run --help'.
docker: output:
error: command `docker run nix nix-store --version` failed (exit code 125)
cleaning up
killing docker (pid 9)
(0.00 seconds)
builder for '/nix/store/wip3apvpmxykhdyk28mf84vszb1n5w57-vm-test-run-docker-preloader.drv' failed with exit code 1
error: build of '/nix/store/wip3apvpmxykhdyk28mf84vszb1n5w57-vm-test-run-docker-preloader.drv' failed

@tfc
Copy link
Copy Markdown
Contributor Author

tfc commented Apr 22, 2020

It finally fails on my systems, too. I wonder how/why it ever worked, as the commit that introduces this module maps the images RO: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/virtualisation/docker-preloader.nix#L92

However, i have no idea how to fix this without modifying the module (possible to the worse). @nlewo any idea?

@flokli
Copy link
Copy Markdown
Member

flokli commented Aug 23, 2020

docker-preloader seems to still fail on current master. This PR doesn't apply due to some editorconfig changes.

@tfc As this port is fairly trivial, can you rebase this PR to latest master, and mark it as broken?

tfc and others added 2 commits August 23, 2020 10:42
@flokli flokli force-pushed the port-docker-preloader-test branch from 1861162 to 33905f8 Compare August 23, 2020 08:45
@flokli
Copy link
Copy Markdown
Member

flokli commented Aug 23, 2020

nvm, I did the rebase, and marked it as broken in a follow-up commit.

@flokli flokli merged commit adc5918 into NixOS:master Aug 23, 2020
@flokli
Copy link
Copy Markdown
Member

flokli commented Aug 23, 2020

Huh, I missed #93791.

@flokli flokli mentioned this pull request Aug 23, 2020
Merged
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@flokli flokli Awaiting requested review from flokli

@worldofpeace worldofpeace Awaiting requested review from worldofpeace

Assignees

No one assigned

Labels

6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 6.topic: testing Tooling for automated testing of packages and modules 8.has: documentation This PR adds or changes documentation 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@tfc @flokli @worldofpeace @veprbl