Skip to content
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

staticcheck randomly fails on CI #1792

Closed
3 of 4 tasks
SVilgelm opened this issue Feb 27, 2021 · 5 comments · Fixed by #1861
Closed
3 of 4 tasks

staticcheck randomly fails on CI #1792

SVilgelm opened this issue Feb 27, 2021 · 5 comments · Fixed by #1861
Labels
area: ci PR that update CI bug Something isn't working

Comments

@SVilgelm
Copy link
Member

SVilgelm commented Feb 27, 2021

  • Yes, I'm using a binary release within 2 latest major releases. Only such installations are supported.
  • Yes, I've searched similar issues on GitHub and didn't find any.
  • Yes, I've included all information below (version, config, etc).
  • Yes, I've tried with the standalone linter if available. (https://golangci-lint.run/usage/linters/)
Verbose output of running
=== CONT  TestSourcesFromTestdataWithIssuesDir/staticcheck.go
    linters_test.go:157: [run --allow-parallel-runners --disable-all --print-issued-lines=false --out-format=line-number --max-same-issues=100 -Estaticcheck --no-config testdata/staticcheck.go]
    linters_test.go:37: 
        	Error Trace:	linters_test.go:37
        	            				linters_test.go:158
        	            				linters_test.go:57
        	Error:      	Received unexpected error:
        	            	
        	            	unmatched errors
        	            	level=warning msg="[linters context/goanalysis] Failed to get persisted facts: failed to get data from low-level cache by key fact_deprecated/facts for package fmtsort: read 0/175 bytes from /home/runner/.cache/golangci-lint/ed/ed3eaffb7621d0470f01c721ca356e360b48c69fbe6c0adf4fedb7c1b1c4f83f-a with error EOF"
        	Test:       	TestSourcesFromTestdataWithIssuesDir/staticcheck.go

...

--- FAIL: TestSourcesFromTestdataWithIssuesDir (0.00s)
             --- FAIL: TestSourcesFromTestdataWithIssuesDir/staticcheck.go (3.47s)
@SVilgelm SVilgelm added bug Something isn't working area: ci PR that update CI labels Feb 27, 2021
@ldez
Copy link
Member

ldez commented Mar 13, 2021

Another case:

=== CONT  TestSourcesFromTestdataWithIssuesDir/staticcheck.go
    linters_test.go:36: 
        	Error Trace:	linters_test.go:36
        	            				linters_test.go:157
        	            				linters_test.go:56
        	Error:      	Received unexpected error:
        	            	staticcheck.go:26: missing error "SA1019: runtime.CPUProfile has been deprecated .*"
        	Test:       	TestSourcesFromTestdataWithIssuesDir/staticcheck.go

@stale
Copy link

stale bot commented Mar 30, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale No recent correspondence or work activity label Mar 30, 2022
@ldez ldez removed the stale No recent correspondence or work activity label Mar 30, 2022
@kamilsk
Copy link
Contributor

kamilsk commented May 2, 2022

most recent

=== CONT  TestSourcesFromTestdataWithIssuesDir/staticcheck.go
    linters_test.go:237: [run --go=1.17 --allow-parallel-runners --disable-all --print-issued-lines=false --out-format=line-number --max-same-issues=100 -Estaticcheck --no-config testdata/staticcheck.go]
    linters_test.go:40: 
        	Error Trace:	linters_test.go:40
        	            				linters_test.go:238
        	            				linters_test.go:60
        	Error:      	Received unexpected error:
        	            	
        	            	unmatched errors
        	            	level=warning msg="[linters context/goanalysis] Failed to get persisted facts: failed to get data from low-level cache by key buildir/facts for package goarch: read 0/175 bytes from /Users/runner/Library/Caches/golangci-lint/da/dadbfe6a926e3ee11aab7c7363f1942b1b998f4d95d49f082731b2408a94e59e-a with error EOF"
        	Test:       	TestSourcesFromTestdataWithIssuesDir/staticcheck.go

@kamilsk
Copy link
Contributor

kamilsk commented May 2, 2022

folks, what do you think if I exclude go tests from "docs" changes? e.g.

fails because this issue. it's possible by adding "paths" directive for GitHub actions.

@ldez
Copy link
Member

ldez commented Aug 15, 2022

I fixed the flaky

@ldez ldez closed this as completed Aug 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ci PR that update CI bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants