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

nixos/tests/netdata: fix non-deterministic failure #45647

Merged
merged 1 commit into from Aug 26, 2018

Conversation

xeji
Copy link
Contributor

@xeji xeji commented Aug 26, 2018

Motivation for this change

The test sporadically failed on hydra for x86_64-linux and i686 when a request was made
before the service was actually listening on its port.

Explicitly wait for the port to open.

Things done
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)

The test sporadically failed on hydra when a request was made
before the service was actually listening on its port.
Explicitly wait for the port to open.
@xeji
Copy link
Contributor Author

xeji commented Aug 26, 2018

@GrahamcOfBorg test netdata

@GrahamcOfBorg
Copy link

Success on x86_64-linux (full log)

Attempted: tests.netdata

Partial log (click to expand)

netdata: exit status 1
syncing
netdata: running command: sync
netdata: exit status 0
test script finished in 25.66s
cleaning up
killing netdata (pid 597)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/3yppr81p7lxajxklpi9d7lg90zz2kxld-vm-test-run-netdata

@GrahamcOfBorg
Copy link

Success on aarch64-linux (full log)

Attempted: tests.netdata

Partial log (click to expand)

netdata: exit status 1
syncing
netdata: running command: sync
netdata: exit status 0
test script finished in 26.25s
cleaning up
killing netdata (pid 631)
vde_switch: EOF on stdin, cleaning up and exiting
vde_switch: Could not remove ctl dir '/build/vde1.ctl': Directory not empty
/nix/store/zw4g1gx7hwrnm96qy6y29xxx2dcdi2j5-vm-test-run-netdata

@Mic92 Mic92 merged commit b7d7e20 into NixOS:master Aug 26, 2018
@xeji xeji deleted the p/netdata-test branch August 26, 2018 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants