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.

Sign up for GitHub

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

Jump to bottom

x/tools/go/packages: TestCgoNoSyntax failing on linux-ppc64-buildlet since CL 214943 #36799

Closed
bcmills opened this issue Jan 27, 2020 · 2 comments
Closed

x/tools/go/packages: TestCgoNoSyntax failing on linux-ppc64-buildlet since CL 214943 #36799

bcmills opened this issue Jan 27, 2020 · 2 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. Soon This needs action soon. (recent regressions, service outages, unusual time-sensitive situations) Tools This label describes issues relating to any tools in the x/tools repository.
Milestone
Go1.14

Comments

@bcmills
Copy link
Contributor

bcmills commented Jan 27, 2020

https://build.golang.org/log/9f9bf391b5329b31cb4ffc5c207206731d58efe0

linux-ppc64-buildlet at 4af1337d1e9eb9e7b766c9deb787c78413bb25c4 building tools at 84cebe10344f1cd161b47a7087b7979c67beab95
…
--- FAIL: TestCgoNoSyntax (7.10s)
    --- FAIL: TestCgoNoSyntax/GOPATH (4.09s)
        --- FAIL: TestCgoNoSyntax/GOPATH/LoadMode(NeedTypes) (0.49s)
            packages_test.go:2700: Expected no errors in package, got [/workdir/tmp/TestCgoNoSyntax_GOPATH054235995/fake/src/golang.org/fake/c/c.go:1:19: could not import C (no metadata for C)]
        --- FAIL: TestCgoNoSyntax/GOPATH/LoadMode(NeedName|NeedTypes) (0.34s)
            packages_test.go:2700: Expected no errors in package, got [/workdir/tmp/TestCgoNoSyntax_GOPATH054235995/fake/src/golang.org/fake/c/c.go:1:19: could not import C (no metadata for C)]
        --- FAIL: TestCgoNoSyntax/GOPATH/LoadMode(NeedName|NeedImports|NeedTypes) (0.37s)
            packages_test.go:2700: Expected no errors in package, got [/workdir/tmp/TestCgoNoSyntax_GOPATH054235995/fake/src/golang.org/fake/c/c.go:1:19: could not import C (no metadata for C)]
        --- FAIL: TestCgoNoSyntax/GOPATH/LoadMode(NeedName|NeedImports|NeedDeps|NeedTypes) (2.61s)
            packages_test.go:2700: Expected no errors in package, got [/workdir/tmp/TestCgoNoSyntax_GOPATH054235995/fake/src/golang.org/fake/c/c.go:1:19: could not import C (no metadata for C)]
    --- FAIL: TestCgoNoSyntax/Modules (3.01s)
        --- FAIL: TestCgoNoSyntax/Modules/LoadMode(NeedTypes) (0.38s)
            packages_test.go:2700: Expected no errors in package, got [/workdir/tmp/TestCgoNoSyntax_Modules100387582/primarymod/fake/c/c.go:1:19: could not import C (no metadata for C)]
        --- FAIL: TestCgoNoSyntax/Modules/LoadMode(NeedName|NeedTypes) (0.34s)
            packages_test.go:2700: Expected no errors in package, got [/workdir/tmp/TestCgoNoSyntax_Modules100387582/primarymod/fake/c/c.go:1:19: could not import C (no metadata for C)]
        --- FAIL: TestCgoNoSyntax/Modules/LoadMode(NeedName|NeedImports|NeedTypes) (0.35s)
            packages_test.go:2700: Expected no errors in package, got [/workdir/tmp/TestCgoNoSyntax_Modules100387582/primarymod/fake/c/c.go:1:19: could not import C (no metadata for C)]
        --- FAIL: TestCgoNoSyntax/Modules/LoadMode(NeedName|NeedImports|NeedDeps|NeedTypes) (1.58s)
            packages_test.go:2700: Expected no errors in package, got [/workdir/tmp/TestCgoNoSyntax_Modules100387582/primarymod/fake/c/c.go:1:19: could not import C (no metadata for C)]
FAIL
FAIL	golang.org/x/tools/go/packages	75.388s

CC @matloob @stamblerre @jayconrod

This regression was initially masked by an existing too many open files error, which may be related to #33531 but seems not to have a separate tracking issue. (CC @dmitshur @toothrot @cagedmantis)

Milestoning to 1.14 via #11811.
@bcmills bcmills added NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. Soon This needs action soon. (recent regressions, service outages, unusual time-sensitive situations) labels Jan 27, 2020
@bcmills bcmills added this to the Go1.14 milestone Jan 27, 2020
@gopherbot gopherbot added the Tools This label describes issues relating to any tools in the x/tools repository. label Jan 27, 2020
@bcmills
Copy link
Contributor Author

bcmills commented Jan 27, 2020

This test seems to be failing on the builder since its introduction in CL 214943.

@gopherbot
Copy link
Contributor

Change https://golang.org/cl/216538 mentions this issue: go/packages: only run TestCgoNoSyntax when cgo is available

@golang golang locked and limited conversation to collaborators Jan 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. Soon This needs action soon. (recent regressions, service outages, unusual time-sensitive situations) Tools This label describes issues relating to any tools in the x/tools repository.
Projects
None yet
Milestone
Go1.14
Development

No branches or pull requests
2 participants
@bcmills @gopherbot