-
Notifications
You must be signed in to change notification settings - Fork 17.8k
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/tools/gopls/internal/regtest/misc: TestRenameFileFromEditor failures #55324
Comments
Found new dashboard test flakes for:
2022-09-20 16:27 linux-s390x-ibm tools@6782af03 go@d5a5db3b x/tools/gopls/internal/regtest/misc.TestRenameFileFromEditor (log)
|
Found new dashboard test flakes for:
2022-10-04 16:29 windows-386-2008 tools@7f79a022 go@0a723816 x/tools/gopls/internal/regtest/misc.TestRenameFileFromEditor (log)
|
Change https://go.dev/cl/440181 mentions this issue: |
Investigation of renaming flakes revealed that renaming is known to be flaky on windows, and the go command has a robustio package that works around known flakes for certain IO operations on darwin and windows. Rather than duplicate this logic piecemeal, copy the entire robustio package to the gopls module, along with a script to sync it from GOROOT using go generate. Use this new package to de-flake renaming, and replace an existing workaround. The copy script got a little out of hand at the point where I needed to add +build constraints. Nevertheless, I've decided to keep it with the caveat that it may be removed if it proves too difficult to maintain. As is, it at least serves as documentation for how the sync was done. For golang/go#56040 For golang/go#56039 For golang/go#56038 For golang/go#55324 Change-Id: Ifeda408ac44a2866e84015a2a38ae340dc0a88bb Reviewed-on: https://go-review.googlesource.com/c/tools/+/440181 Run-TryBot: Robert Findley <rfindley@google.com> Reviewed-by: Bryan Mills <bcmills@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> gopls-CI: kokoro <noreply+kokoro@google.com>
Fixed by https://go.dev/cl/440181. |
Found new dashboard test flakes for:
2022-10-19 15:24 linux-s390x-ibm tools@d67c3ada go@6b22572e x/tools/gopls/internal/regtest/misc.TestRenameFileFromEditor (log)
|
Found new dashboard test flakes for:
2022-10-26 15:27 netbsd-amd64-9_0 tools@de675d54 go@3afba312 x/tools/gopls/internal/regtest/misc.TestRenameFileFromEditor (log)
|
Found new dashboard test flakes for:
2022-12-06 20:14 linux-s390x-ibm tools@6b505011 go@0d8a92bd x/tools/gopls/internal/regtest/misc.TestRenameFileFromEditor (log)
|
Change https://go.dev/cl/461416 mentions this issue: |
Change https://go.dev/cl/461418 mentions this issue: |
Updates golang/go#55324 Change-Id: I07cb5465afc7f9e76cad96cb5fbe55e6e16a73b7 Reviewed-on: https://go-review.googlesource.com/c/tools/+/461416 Reviewed-by: Bryan Mills <bcmills@google.com> TryBot-Result: Gopher Robot <gobot@golang.org> Run-TryBot: Robert Findley <rfindley@google.com> gopls-CI: kokoro <noreply+kokoro@google.com>
Issue created automatically to collect these failures.
Example (log):
— watchflakes
The text was updated successfully, but these errors were encountered: