-
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: rename failures #56039
Labels
FrozenDueToAge
gopls
Issues related to the Go language server, gopls.
NeedsInvestigation
Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Testing
An issue that has been verified to require only test changes, not just a test failure.
Tools
This label describes issues relating to any tools in the x/tools repository.
Milestone
Comments
gopherbot
added
the
NeedsInvestigation
Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
label
Oct 4, 2022
Found new dashboard test flakes for:
2022-10-04 16:29 windows-386-2008 tools@7f79a022 go@9d2c73a9 x/tools/gopls/internal/regtest/misc.TestRenamePackageWithAlias (log)
|
gopherbot
added
Tools
This label describes issues relating to any tools in the x/tools repository.
gopls
Issues related to the Go language server, gopls.
labels
Oct 4, 2022
Change https://go.dev/cl/440181 mentions this issue: |
gopherbot
pushed a commit
to golang/tools
that referenced
this issue
Oct 7, 2022
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>
Found new dashboard test flakes for:
2022-10-06 17:35 windows-386-2008 tools@15525299 go@89566448 x/tools/gopls/internal/regtest/misc.TestRenamePackageWithAlias (log)
|
Found new dashboard test flakes for:
2022-10-06 18:44 windows-amd64-2016 tools@c6825558 go@515e3de2 x/tools/gopls/internal/regtest/misc.TestRenamePackageWithBlankSameImportPaths (log)
|
rsc
changed the title
x/tools/gopls/internal/regtest/misc: TestRenamePackageWithAlias failures
x/tools/gopls/internal/regtest/misc: rename failures
Oct 13, 2022
Found new dashboard test flakes for:
2022-10-06 18:44 windows-amd64-2016 tools@c6825558 go@515e3de2 x/tools/gopls/internal/regtest/misc.TestRenamePackageWithBlankSameImportPaths (log)
|
Fixed by https://go.dev/cl/440181. |
Found new dashboard test flakes for:
2022-10-04 16:03 windows-amd64-longtest tools@40dabfa8 go@6a9aaf1f x/tools/gopls/internal/regtest/misc.TestRenamePackageWithDifferentDirectoryPath (log)
2022-10-04 16:29 windows-386-2008 tools@7f79a022 go@f6d84451 x/tools/gopls/internal/regtest/misc.TestRenamePackageWithConflicts (log)
2022-10-04 18:24 windows-386-2008 tools@dc88e7b4 go@47a53884 x/tools/gopls/internal/regtest/misc.TestRenameWithTestPackage (log)
2022-10-04 18:24 windows-amd64-longtest tools@dc88e7b4 go@841873f2 x/tools/gopls/internal/regtest/misc.TestRenamePackage (log)
2022-10-05 17:59 windows-386-2008 tools@b280e27d go@947091d3 x/tools/gopls/internal/regtest/misc.TestRenameWithTestPackage (log)
2022-10-06 13:48 windows-amd64-2016 tools@ede3ab26 go@af668c68 x/tools/gopls/internal/regtest/misc.TestRenamePackageWithNonBlankSameImportPaths (log)
2022-10-07 15:16 windows-amd64-longtest tools@a410e98a go@5ca0cd3f x/tools/gopls/internal/regtest/misc.TestRenamePackageWithConflicts (log)
|
Still fixed! |
findleyr
added
the
Testing
An issue that has been verified to require only test changes, not just a test failure.
label
Oct 20, 2022
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
FrozenDueToAge
gopls
Issues related to the Go language server, gopls.
NeedsInvestigation
Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Testing
An issue that has been verified to require only test changes, not just a test failure.
Tools
This label describes issues relating to any tools in the x/tools repository.
Issue created automatically to collect these failures.
Example (log):
— watchflakes
The text was updated successfully, but these errors were encountered: