-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Closed
Labels
FrozenDueToAgeNeedsFixThe path to resolution is known, but the work has not been done.The path to resolution is known, but the work has not been done.TestingAn issue that has been verified to require only test changes, not just a test failure.An issue that has been verified to require only test changes, not just a test failure.
Milestone
Description
(Discovered while testing #37940.)
The BenchmarkAll
function in cmd/api/goapi_test.go
is failing at head.
(It appears that we are missing builder coverage for benchmarks in cmd
in general, but that's a separate issue.)
It may have been broken when we switched to vendor/golang.org
in CL 164623. I'm running a bisection to verify now.
~/go/src$ go version
go version devel +f9f57c4443 Thu Mar 19 16:17:59 2020 +0000 linux/amd64
~/go/src$ go test -bench='.*' cmd/api
2020/03/19 13:35:40 no source in tree for import "golang.org/x/crypto/cryptobyte" (from import golang.org/x/crypto/cryptobyte in /usr/local/google/home/bcmills/go/src/crypto/ecdsa): stat /usr/local/google/home/bcmills/go/src/golang.org/x/crypto/cryptobyte: no such file or directory
panic: no source in tree for import "golang.org/x/crypto/cryptobyte" (from import golang.org/x/crypto/cryptobyte in /usr/local/google/home/bcmills/go/src/crypto/ecdsa): stat /usr/local/google/home/bcmills/go/src/golang.org/x/crypto/cryptobyte: no such file or directory [recovered]
panic: no source in tree for import "golang.org/x/crypto/cryptobyte" (from import golang.org/x/crypto/cryptobyte in /usr/local/google/home/bcmills/go/src/crypto/ecdsa): stat /usr/local/google/home/bcmills/go/src/golang.org/x/crypto/cryptobyte: no such file or directory
goroutine 14 [running]:
go/types.(*Checker).handleBailout(0xc00300b680, 0xc0012f3b10)
/usr/local/google/home/bcmills/go/src/go/types/check.go:243 +0x98
panic(0x6284a0, 0xc0031d32f0)
/usr/local/google/home/bcmills/go/src/runtime/panic.go:967 +0x166
log.Panicf(0x68f69c, 0x3a, 0xc0012f33c8, 0x4, 0x4)
/usr/local/google/home/bcmills/go/src/log/log.go:358 +0xc0
cmd/api.(*Walker).ImportFrom(0xc00005c5a0, 0xc00301c621, 0x1e, 0xc00302e380, 0x32, 0x0, 0x1c, 0xc00304ed65, 0xffffffffffffffff)
/usr/local/google/home/bcmills/go/src/cmd/api/goapi.go:541 +0x114d
go/types.(*Checker).importPackage(0xc00300b680, 0x3bd217, 0xc00301c621, 0x1e, 0xc00302e380, 0x32, 0x8)
/usr/local/google/home/bcmills/go/src/go/types/resolver.go:153 +0x17a
go/types.(*Checker).collectObjects(0xc00300b680)
/usr/local/google/home/bcmills/go/src/go/types/resolver.go:254 +0x8c6
go/types.(*Checker).checkFiles(0xc00300b680, 0xc002fe4ff8, 0x1, 0x1, 0x0, 0x0)
/usr/local/google/home/bcmills/go/src/go/types/check.go:255 +0x95
go/types.(*Checker).Files(...)
/usr/local/google/home/bcmills/go/src/go/types/check.go:248
go/types.(*Config).Check(0xc003054b80, 0xc00039e0d6, 0xc, 0xc00006c5c0, 0xc002fe4ff8, 0x1, 0x1, 0x0, 0x0, 0xc000106550, ...)
/usr/local/google/home/bcmills/go/src/go/types/api.go:348 +0x14e
cmd/api.(*Walker).ImportFrom(0xc00005c5a0, 0xc00039e0d6, 0xc, 0x0, 0x0, 0x0, 0xc00039e0d6, 0xc, 0x0)
/usr/local/google/home/bcmills/go/src/cmd/api/goapi.go:597 +0x87d
cmd/api.(*Walker).Import(...)
/usr/local/google/home/bcmills/go/src/cmd/api/goapi.go:517
cmd/api.BenchmarkAll(0xc0003641c0)
/usr/local/google/home/bcmills/go/src/cmd/api/goapi_test.go:183 +0x4b6
testing.(*B).runN(0xc0003641c0, 0x1)
/usr/local/google/home/bcmills/go/src/testing/benchmark.go:191 +0xe8
testing.(*B).run1.func1(0xc0003641c0)
/usr/local/google/home/bcmills/go/src/testing/benchmark.go:231 +0x57
created by testing.(*B).run1
/usr/local/google/home/bcmills/go/src/testing/benchmark.go:224 +0x7d
exit status 2
FAIL cmd/api 0.511s
FAIL
Metadata
Metadata
Assignees
Labels
FrozenDueToAgeNeedsFixThe path to resolution is known, but the work has not been done.The path to resolution is known, but the work has not been done.TestingAn issue that has been verified to require only test changes, not just a test failure.An issue that has been verified to require only test changes, not just a test failure.