-
Notifications
You must be signed in to change notification settings - Fork 18k
x/tools/gopls/internal/lsp/cmd/test: TestImplementations failures #59105
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
Comments
Found new dashboard test flakes for:
2023-03-16 15:43 darwin-amd64-nocgo tools@6e5f4255 go@fa42da15 x/tools/gopls/internal/lsp/cmd/test.TestImplementations (log)
|
This one is concerning. We should disable the importer panic recovery for testing. |
Change https://go.dev/cl/477456 mentions this issue: |
Found new dashboard test flakes for:
2023-04-20 20:04 illumos-amd64 tools@9c9e11f3 go@ecf7e00d x/tools/gopls/internal/lsp/cmd/test.TestImplementations (log)
|
Found new dashboard test flakes for:
2023-04-28 17:21 darwin-amd64-12_0 tools@ab24b7b9 go@972774c4 x/tools/gopls/internal/lsp/cmd/test.TestImplementations (log)
|
I suspect that the "possible version skew" failure was panic recovery of the panic caused by null bytes in filecache.Get. Therefore, I believe this is a dupe of #59289. |
Issue created automatically to collect these failures.
Example (log):
— watchflakes
The text was updated successfully, but these errors were encountered: