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.

Sign up for GitHub

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

Jump to bottom

nixos/tests/acme: fix on i686, improve timing #40410

Merged
merged 1 commit into from
May 13, 2018
Merged

nixos/tests/acme: fix on i686, improve timing #40410

merged 1 commit into from
May 13, 2018
+7 −1

Conversation

xeji
Copy link
Contributor

@xeji xeji commented May 12, 2018

Motivation for this change

Test failed consistently for i686.
Fix it and improve timing to prevent non-deterministic failures under high load on Hydra.

@xeji
nixos/tests/acme: fix on i686, improve timing
924cbe9 
... to prevent non-deterministic failures
@xeji xeji mentioned this pull request May 12, 2018
Closed
41 tasks
@xeji
Copy link
Contributor Author

xeji commented May 12, 2018

@GrahamcOfBorg test acme

@GrahamcOfBorg
Copy link

No attempt on aarch64-linux (full log)

The following builds were skipped because they don't evaluate on aarch64-linux: tests.acme

Partial log (click to expand)


a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

@GrahamcOfBorg GrahamcOfBorg added the 6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS label May 12, 2018
@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: tests.acme

Partial log (click to expand)

webserver: running command: sync
webserver: exit status 0
test script finished in 361.69s
cleaning up
killing letsencrypt (pid 593)
killing client (pid 688)
killing webserver (pid 617)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/gciizpmwxivckz7y3bdpc5wb3634fjdl-vm-test-run-acme

@GrahamcOfBorg GrahamcOfBorg added 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 May 12, 2018
@xeji xeji merged commit 642bed4 into NixOS:master May 13, 2018
@xeji xeji deleted the acme-test branch May 13, 2018 18:00
xeji added a commit that referenced this pull request May 13, 2018
@xeji
nixos/tests/acme: fix on i686, improve timing (#40410)

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
a7be27a 
... to prevent non-deterministic failures

(cherry picked from commit 642bed4)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
6.topic: nixos Issues or PRs affecting NixOS modules, or package usability issues specific to NixOS 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.

None yet
2 participants
@xeji @GrahamcOfBorg