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

misc/cgo/test: failure with gcc 10 [1.17 backport] #53050

Closed
gopherbot opened this issue May 24, 2022 · 12 comments
Closed

misc/cgo/test: failure with gcc 10 [1.17 backport] #53050

gopherbot opened this issue May 24, 2022 · 12 comments
Labels
arch-arm64 CherryPickApproved Used during the release process for point releases FrozenDueToAge Testing An issue that has been verified to require only test changes, not just a test failure.
Milestone

Comments

@gopherbot
Copy link
Contributor

@bcmills requested issue #50987 to be considered for backport to the next 1.17 minor release.

@gopherbot, please backport to Go 1.17. The netbsd-arm64-bsiegert builder is currently failing on the release branch due to this issue.

@gopherbot gopherbot added the CherryPickCandidate Used during the release process for point releases label May 24, 2022
@gopherbot gopherbot added this to the Go1.17.11 milestone May 24, 2022
@bcmills bcmills added Testing An issue that has been verified to require only test changes, not just a test failure. arch-arm64 labels May 24, 2022
@gopherbot
Copy link
Contributor Author

Change https://go.dev/cl/408074 mentions this issue: [release-branch.go1.17] misc/cgo/test: disable unhelpful GCC warning

@bcmills
Copy link
Contributor

bcmills commented May 24, 2022

This backport appears to be insufficient to fix the builder. It looks like it also needs a backport of one or more of the changes from #39466. 😞

@bsiegert, do you want to coordinate the backports, or send a CL to skip this builder on the 1.17 release branch?

@bsiegert
Copy link
Contributor

I don't know. What would you prefer?

@bcmills
Copy link
Contributor

bcmills commented May 24, 2022

I'd be inclined to skip the builder on the 1.17 branch, although I don't think there's much difference either way. (There are relatively few CLs on the release branch so it doesn't consume much of the builder's time, but I bet there also aren't that many folks building Go 1.17 from source on ARM64.)

@bsiegert
Copy link
Contributor

Then let's do that.

@cherrymui
Copy link
Member

I think it is okay to backport a CL that does the skip, CL 383554 (perhaps also incorporate CL 384269).

@toothrot toothrot added the CherryPickApproved Used during the release process for point releases label May 25, 2022
@toothrot
Copy link
Contributor

Approved. Test skip with little risk.

@gopherbot gopherbot removed the CherryPickCandidate Used during the release process for point releases label May 25, 2022
@dmitshur
Copy link
Contributor

@bsiegert The builder appears to be passing on release-branch.go1.17 now (https://build.golang.org/?branch=release-branch.go1.17). Was there a change on the builder to resolve this issue, or is this a flaky failure? Do we still need to backport CL 383554 and/or CL 384269 for this issue? Thanks.

@dmitshur dmitshur modified the milestones: Go1.17.11, Go1.17.12 Jun 1, 2022
@dmitshur dmitshur added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Jun 3, 2022
@gopherbot
Copy link
Contributor Author

Change https://go.dev/cl/415074 mentions this issue: [release-branch.go1.17] cmd/dist: skip internal linking tests on arm64

@dmitshur
Copy link
Contributor

dmitshur commented Jun 29, 2022

It does appear the failure isn't consistent for some reason (that I don't understand), but it happened again on the latest commit on the release branch (see here). Made the two backport CLs.

@dmitshur dmitshur removed the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Jun 29, 2022
@gopherbot
Copy link
Contributor Author

Change https://go.dev/cl/415075 mentions this issue: [release-branch.go1.17] cmd/dist: test cgo internal linking on darwin-arm64

@gopherbot
Copy link
Contributor Author

Closed by merging 651a8d8 to release-branch.go1.17.

gopherbot pushed a commit that referenced this issue Jun 29, 2022
The previous workaround for issue #39466 only disabled this test for
Linux. However, the issue manifests for all arm64 systems with gcc 9.4
and above. The new netbsd-arm64 builder uses NetBSD-current with gcc
10.3, so it fails in the same way.

Updates #39466.
For #53050.

Change-Id: I276a99a5e60914e5c22f74a680e461bea17cfe92
Reviewed-on: https://go-review.googlesource.com/c/go/+/383554
Trust: Benny Siegert <bsiegert@gmail.com>
Run-TryBot: Ian Lance Taylor <iant@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
(cherry picked from commit 1d60513)
Reviewed-on: https://go-review.googlesource.com/c/go/+/415074
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Reviewed-by: Cherry Mui <cherryyz@google.com>
gopherbot pushed a commit that referenced this issue Jun 29, 2022
…-arm64

CL 415074 disables testing cgo internal linking on all ARM64 but
Windows, because it doesn't work with newer GCC. But
- darwin-arm64 works, and it does not use GCC
- we don't support cgo internal linking on windows-arm64 anyway.

This CL fixes the condition.

Fixes #53050.

Change-Id: I9eb7b81ef75e482f5e95d2edae4863ba21396432
Reviewed-on: https://go-review.googlesource.com/c/go/+/384269
Trust: Cherry Mui <cherryyz@google.com>
Run-TryBot: Cherry Mui <cherryyz@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
(cherry picked from commit e4ca3fa)
Reviewed-on: https://go-review.googlesource.com/c/go/+/415075
Reviewed-by: Cherry Mui <cherryyz@google.com>
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
@golang golang locked and limited conversation to collaborators Jun 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
arch-arm64 CherryPickApproved Used during the release process for point releases FrozenDueToAge Testing An issue that has been verified to require only test changes, not just a test failure.
Projects
None yet
Development

No branches or pull requests

6 participants