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

tests: use '-4' where needed #15060

Closed
wants to merge 1 commit into from
Closed

Conversation

icing
Copy link
Contributor

@icing icing commented Sep 26, 2024

Our test servers run either on ipv4 or on ipv6, as requested. A test case using 'localhost' or '*.local' in the url needs to run with the specific version of the server started.

Otherwise, curl's "happy eyeball"ing will connect to another server that may be running due to parallel testing or for some other reasons.

Note that port reuse here depends on the OS strategy and it seems netbsd is especially likely to hit this.

Our test servers run either on ipv4 *or* on ipv6, as requested.
A test case using 'localhost' or '*.local' in the url needs to
run with the specific version of the server started.

Otherwise, curl's "happy eyeball"ing will connect to another
server that may be running due to parallel testing or for some
other reasons.

Note that port reuse here depends on the OS strategy and it
seems netbsd is especially likely to hit this.
@icing icing requested a review from bagder September 26, 2024 13:24
@github-actions github-actions bot added the tests label Sep 26, 2024
@dfandrich
Copy link
Contributor

Analysis of PR #15060 at 3d27e498:

Test 1035 failed, which has NOT been flaky recently, so there could be a real issue in the PR.

Generated by Testclutch

@bagder bagder closed this in 6f2a166 Sep 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants