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

x/net/nettest: TestTestConn/UnixPacketPipe fails on netbsd/386 #22927

Closed
bradfitz opened this issue Nov 29, 2017 · 7 comments
Closed

x/net/nettest: TestTestConn/UnixPacketPipe fails on netbsd/386 #22927

bradfitz opened this issue Nov 29, 2017 · 7 comments
Labels
FrozenDueToAge help wanted NeedsFix The path to resolution is known, but the work has not been done. OS-NetBSD
Milestone

Comments

@bradfitz
Copy link
Contributor

Tracking bug that TestTestConn/UnixPacketPipe fails on netbsd/386 (NetBSD 8-BETA).

It seems to work fine on netbsd/amd64 (NetBSD 8-BETA).

I'm going to disable this test for now and reference this issue. AFAIK, nothing in the Go core depends on Unix pipe datagrams, so it shouldn't affect anything.

/cc @bsiegert @mikioh

@bradfitz bradfitz added help wanted NeedsFix The path to resolution is known, but the work has not been done. OS-NetBSD labels Nov 29, 2017
@bradfitz bradfitz added this to the Unplanned milestone Nov 29, 2017
@gopherbot
Copy link

Change https://golang.org/cl/80755 mentions this issue: internal/nettest: skip unixpacket tests on netbsd/386

gopherbot pushed a commit to golang/net that referenced this issue Nov 29, 2017
Updates golang/go#22927

Change-Id: I813b9ba92f9dd7e517385dc95df20691efee01a6
Reviewed-on: https://go-review.googlesource.com/80755
Reviewed-by: Benny Siegert <bsiegert@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
@gopherbot
Copy link

Change https://golang.org/cl/80756 mentions this issue: net: skip unixpacket tests on netbsd/386

gopherbot pushed a commit that referenced this issue Nov 29, 2017
Updates #22927
Updates #20852

Change-Id: I3ac0f8d38375d58a77781ab0e4f78ca57b258de0
Reviewed-on: https://go-review.googlesource.com/80756
Reviewed-by: Benny Siegert <bsiegert@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
@gopherbot
Copy link

Change https://golang.org/cl/80758 mentions this issue: vendor: update golang.org/x/net/nettest from upstream

gopherbot pushed a commit that referenced this issue Nov 29, 2017
Updates to x/net git rev a8b92947779 for CL 80755

Updates #22927

Change-Id: I84aad75afd954c9a6c57ad9a76328b99f1dc74a1
Reviewed-on: https://go-review.googlesource.com/80758
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
Reviewed-by: Benny Siegert <bsiegert@gmail.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
@bcmills
Copy link
Contributor

bcmills commented Oct 4, 2022

This issue is very old. @golang/netbsd: do you want to try re-enabling this test and see if it is still broken?

@gopherbot
Copy link

Change https://go.dev/cl/483375 mentions this issue: nettest: re-enable unixpacket tests on netbsd/386

@gopherbot
Copy link

Change https://go.dev/cl/483395 mentions this issue: net: re-enable unixpacket tests on netbsd/386

gopherbot pushed a commit to golang/net that referenced this issue Apr 10, 2023
The tests seem to work fine on netbsd/386 (NetBSD 9.3). This reverts CL
80755.

Updates golang/go#22927

Change-Id: I480ac82b10e85aa0fc081e76fbbcbc75f3ee7d38
Reviewed-on: https://go-review.googlesource.com/c/net/+/483375
Auto-Submit: Tobias Klauser <tobias.klauser@gmail.com>
Reviewed-by: Benny Siegert <bsiegert@gmail.com>
Reviewed-by: David Chase <drchase@google.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
gopherbot pushed a commit that referenced this issue Apr 10, 2023
The tests seem to work fine on netbsd/386 (NetBSD 9.3). This reverts CL
80756.

Updates #22927

Change-Id: I2235d69129aa81b43513a171834d058f47cd9933
Reviewed-on: https://go-review.googlesource.com/c/go/+/483395
Reviewed-by: Benny Siegert <bsiegert@gmail.com>
Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: David Chase <drchase@google.com>
Auto-Submit: Tobias Klauser <tobias.klauser@gmail.com>
@gopherbot
Copy link

Change https://go.dev/cl/483595 mentions this issue: all: update vendored golang.org/x/net

@golang golang locked and limited conversation to collaborators Apr 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge help wanted NeedsFix The path to resolution is known, but the work has not been done. OS-NetBSD
Projects
None yet
Development

No branches or pull requests

3 participants