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

Listen on random ports for tests instead of :8080 #18

Closed
andybons opened this issue Jun 3, 2014 · 1 comment
Closed

Listen on random ports for tests instead of :8080 #18

andybons opened this issue Jun 3, 2014 · 1 comment
Assignees

Comments

@andybons
Copy link
Contributor

andybons commented Jun 3, 2014

Currently the servers that are used when testing the healthz endpoints are using the default port from the flag (8080). These should be randomized via port :0

Relevant error log from failing test:

E0525 17:43:15.273643 05697 range.go:161] failed init accounting configuration map: prefix.go:111: no default prefix specified
--- FAIL: TestHealthz (0.01 seconds)    server_test.go:70: error requesting healthz at http://localhost:8080/_admin/healthz: Get http://localhost:8080/_admin/healthz: dial tcp 127.0.0.1:8080: connection refused
--- FAIL: TestGzip (0.00 seconds)   server_test.go:101: could not make request to http://localhost:8080/_admin/healthz: Get http://localhost:8080/_admin/healthz: dial tcp 127.0.0.1:8080: connection refused
E0525 17:43:15.281753 05697 range.go:161] failed init permissions configuration map: prefix.go:111: no default prefix specified
E0525 17:43:15.281928 05697 range.go:161] failed init zones configuration map: prefix.go:111: no default prefix specifiedadmin REST request failed: zone_cli.go:49: 500 Internal Server Error: zone.go:127: the default zone configuration cannot be deleted
@andybons andybons self-assigned this Jun 3, 2014
@tbg
Copy link
Member

tbg commented Jun 24, 2014

This has been taken care of a while ago.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants