-
Notifications
You must be signed in to change notification settings - Fork 18k
all: failures on Windows with The process cannot access the file because it is being used by another process.
#66099
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:
2024-02-29 21:45 windows-amd64-race tools@5bf7d005 go@7986e26a x/tools/gopls/internal/test/integration/misc.TestGoToTypeDefinition_Issue60544 (log)
|
The process cannot access the file because it is being used by another process.
(attn @golang/windows) Note that some of these duplicates were on legacy builders and some were on LUCI builders, so it doesn't seem to be caused by a builder configuration change. |
Found new dashboard test flakes for:
2024-02-29 21:45 windows-386-2016 tools@5bf7d005 go@5dcc04ae x/tools/gopls/internal/test/marker.TestNoMod (log)
2024-02-29 21:45 windows-386-2016 tools@5bf7d005 go@5dcc04ae x/tools/gopls/internal/test/marker.Test (log)
2024-02-29 21:45 windows-amd64-longtest tools@5bf7d005 go@b09ac10b x/tools/gopls/internal/test/integration/template.TestDisablingCodeLens (log)
2024-02-29 21:45 windows-amd64-longtest tools@5bf7d005 go@b09ac10b x/tools/gopls/internal/test/integration/template.TestModFileModification (log)
2024-02-29 21:45 windows-amd64-longtest tools@5bf7d005 go@b09ac10b x/tools/gopls/internal/test/integration/template.TestTemplatesFromExtensions (log)
2024-02-29 21:45 windows-amd64-longtest tools@5bf7d005 go@2b22fc10 x/tools/gopls/internal/test/integration/inlayhints.TestDisablingCodeLens (log)
2024-02-29 21:45 windows-amd64-longtest tools@5bf7d005 go@2b22fc10 x/tools/gopls/internal/test/integration/inlayhints.TestEnablingInlayHints (log)
2024-02-29 21:45 windows-amd64-race tools@5bf7d005 go@5dcc04ae x/tools/gopls/internal/test/integration/modfile.TestMissingDependencyFixes (log)
|
@matloob, https://go.dev/cl/564555 seems like the most plausible recent change to have introduced this, but maybe it's worth bisecting on a The first of these failures was at Go commit 2b22fc1, but that commit itself doesn't seem like a plausible root cause. |
Oh, neat! (@cherrymui, any idea why |
Found new dashboard test flakes for:
2024-03-05 18:10 windows-amd64-longtest tools@d10b068e go@afb10505 x/tools/gopls/internal/test/integration/modfile.TestModFileModification (log)
|
Change https://go.dev/cl/569217 mentions this issue: |
golang.org/cl/569217 modifies telemetry.Start to avoid running uploads if the telemetry mode isn't |
Found new dashboard test flakes for:
2024-03-05 19:26 windows-amd64-longtest tools@029c4eaf go@9e3b1d53 x/tools/gopls/internal/test/integration/modfile.TestModFileModification (log)
|
I suspect that part of the failure mode here is that the telemetry upload process holds open its current working directory — we should try to ensure that the upload process doesn't do that for, say, users who invoke |
Looks like If That's supposed to cause The missing check causes the child process to be run, and the child process unconditionally creates a file in |
that check is in CL 569217 |
It's possible for telemetry.LocalDir to not be set, which, depending on the platform, could happen if there's no home or AppData directory. Handle that case by skipping uploading so we don't end up writing the upload.token file to the user's current directory. (There would be no counters to upload anyway if there's no local directory). Also, if telemetry.Mode() is "off" or "local" we don't do uploads, so unless telemetry.Mode() is "on" don't attempt to do an upload. Fixes golang/go#66111, golang/go#66099 Change-Id: I6b30835a1d3a147b2f94d97f13a2b793217b2639 Reviewed-on: https://go-review.googlesource.com/c/telemetry/+/569217 LUCI-TryBot-Result: Go LUCI <[email protected]> Reviewed-by: Robert Findley <[email protected]>
CL 569217 has been submitted. I'll prepare a CL to vendor the latest x/telemetry into cmd/ |
Change https://go.dev/cl/569421 mentions this issue: |
Change https://go.dev/cl/569420 mentions this issue: |
Found new dashboard test flakes for:
2024-03-06 14:41 windows-amd64-longtest tools@ff00c7bd go@f0d1195e x/tools/gopls/internal/test/integration/watch.TestEnablingInlayHints (log)
2024-03-06 14:41 windows-amd64-longtest tools@ff00c7bd go@f0d1195e x/tools/gopls/internal/test/integration/watch.TestCallHierarchy_Issue49125 (log)
2024-03-06 14:41 windows-amd64-longtest tools@ff00c7bd go@f0d1195e x/tools/gopls/internal/test/integration/watch.TestSubdirWatchPatterns (log)
2024-03-06 14:41 windows-amd64-longtest tools@ff00c7bd go@f048829d x/tools/gopls/internal/test/integration/watch.TestDisablingCodeLens (log)
2024-03-06 14:41 windows-amd64-longtest tools@ff00c7bd go@f048829d x/tools/gopls/internal/test/integration/watch.TestCallHierarchy_Issue49125 (log)
2024-03-06 14:41 windows-amd64-longtest tools@ff00c7bd go@f048829d x/tools/gopls/internal/test/integration/watch.TestMultilineTokens (log)
2024-03-06 17:08 windows-amd64-longtest tools@caf59401 go@f0d1195e x/tools/gopls/internal/test/integration/template.TestMultilineTokens (log)
|
This is a requirement of x/telemetry so update it before updating telemetry. Commands run (in both std and cmd): go get golang.org/x/[email protected] go mod tidy go mod vendor For #66099 Change-Id: I636f0c0be89c05b9213c461b1a2eb2a4afb8a84b Reviewed-on: https://go-review.googlesource.com/c/go/+/569420 Reviewed-by: Bryan Mills <[email protected]> LUCI-TryBot-Result: Go LUCI <[email protected]> Auto-Submit: Michael Matloob <[email protected]>
Found new dashboard test flakes for:
2024-03-04 22:41 x_tools-gotip-windows-amd64-longtest tools@7656c4c6 go@5dcc04ae x/tools/gopls/internal/cmd.TestCheck (log)
2024-03-04 22:41 x_tools-gotip-windows-amd64-longtest tools@7656c4c6 go@5dcc04ae x/tools/gopls/internal/test/integration/codelens.TestDisablingCodeLens/default/experimental (log)
2024-03-04 22:41 x_tools-gotip-windows-amd64-longtest tools@7656c4c6 go@5dcc04ae x/tools/gopls/internal/test/integration/template.TestMultilineTokens/experimental (log)
2024-03-05 15:19 x_tools-gotip-windows-amd64-longtest tools@98c835c3 go@5dcc04ae x/tools/internal/refactor/inline.TestData/param-subst.txtar (log)
2024-03-05 19:26 x_tools-gotip-windows-amd64-longtest tools@029c4eaf go@e8b5bc63 x/tools/gopls/internal/test/integration/inlayhints.TestEnablingInlayHints/default/forwarded (log)
2024-03-05 19:26 x_tools-gotip-windows-amd64-longtest tools@029c4eaf go@e8b5bc63 x/tools/gopls/internal/test/integration/template.TestTemplatesFromLangID/experimental (log)
2024-03-05 19:26 x_tools-gotip-windows-amd64-longtest tools@029c4eaf go@e8b5bc63 x/tools/gopls/internal/test/integration/workspace.TestAdhoc_Edits/experimental (log)
2024-03-06 19:14 gotip-windows-amd64 go@0e89b67e cmd/internal/testdir.Test/fixedbugs/issue49122.go (log)
2024-03-06 19:15 gotip-windows-amd64 go@d463de61 cmd/internal/testdir.Test/fixedbugs/bug496.go (log)
|
Issue created automatically to collect these failures.
Example (log):
— watchflakes
The text was updated successfully, but these errors were encountered: