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

[Flakey test][pkg/stanza] TestCreateAndShutdown leads to nil pointer on Windows #36268

Open
songy23 opened this issue Nov 7, 2024 · 2 comments · May be fixed by #36262
Open

[Flakey test][pkg/stanza] TestCreateAndShutdown leads to nil pointer on Windows #36268

songy23 opened this issue Nov 7, 2024 · 2 comments · May be fixed by #36262

Comments

@songy23
Copy link
Member

songy23 commented Nov 7, 2024

Component(s)

pkg/stanza

Describe the issue you're reporting

E.g. https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/11728457085/job/32671901795

=== Failed
=== FAIL: . TestCreateAndShutdown (0.00s)
panic: runtime error: invalid memory address or nil pointer dereference [recovered]
	panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x14118ad54]

goroutine 23 [running]:
testing.tRunner.func1.2({0x14138eba0, 0x14128ef00})
	C:/hostedtoolcache/windows/go/1.22.8/x64/src/testing/testing.go:1631 +0x3f7
testing.tRunner.func1()
	C:/hostedtoolcache/windows/go/1.22.8/x64/src/testing/testing.go:1634 +0x6b6
panic({0x14138eba0?, 0x14128ef00?})
	C:/hostedtoolcache/windows/go/1.22.8/x64/src/runtime/panic.go:770 +0x132
github.com/open-telemetry/opentelemetry-collector-contrib/pkg/stanza/operator/input/windows.(*Input).Stop(0xc0002b8e00)
	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/pkg/stanza/operator/input/windows/input.go:157 +0x54
github.com/open-telemetry/opentelemetry-collector-contrib/pkg/stanza/pipeline.(*DirectedPipeline).stop(0xc00005cb80)
	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/pkg/stanza/pipeline/directed.go:73 +0x142
github.com/open-telemetry/opentelemetry-collector-contrib/pkg/stanza/pipeline.(*DirectedPipeline).Stop.func1()
	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/pkg/stanza/pipeline/directed.go:46 +0x38
sync.(*Once).doSlow(0xc00005cb94, 0xc000057be8)
	C:/hostedtoolcache/windows/go/1.22.8/x64/src/sync/once.go:74 +0xf1
sync.(*Once).Do(0xc00005cb94, 0xc000057be8)
	C:/hostedtoolcache/windows/go/1.22.8/x64/src/sync/once.go:65 +0x45
make[2]: *** [../../Makefile.Common:131: test] Error 3
github.com/open-telemetry/opentelemetry-collector-contrib/pkg/stanza/pipeline.(*DirectedPipeline).Stop(0xc00005cb80)
	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/pkg/stanza/pipeline/directed.go:45 +0xa9
github.com/open-telemetry/opentelemetry-collector-contrib/pkg/stanza/adapter.(*receiver).Shutdown(0xc000143970, {0x1416dd6b0, 0x141ee6160})
	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/pkg/stanza/adapter/receiver.go:69 +0x9b
github.com/open-telemetry/opentelemetry-collector-contrib/receiver/windowseventlogreceiver.TestCreateAndShutdown(0xc0000c9380)
make[1]: *** [Makefile:200: receiver/windowseventlogreceiver] Error 2
	D:/a/opentelemetry-collector-contrib/opentelemetry-collector-contrib/receiver/windowseventlogreceiver/factory_test.go:51 +0x249
make[1]: *** Waiting for unfinished jobs....
testing.tRunner(0xc0000c9380, 0x14150ba70)
	C:/hostedtoolcache/windows/go/1.22.8/x64/src/testing/testing.go:1689 +0x21f
created by testing.(*T).Run in goroutine 1
	C:/hostedtoolcache/windows/go/1.22.8/x64/src/testing/testing.go:1742 +0x826
@songy23 songy23 added os:windows pkg/stanza needs triage New item requiring triage labels Nov 7, 2024
Copy link
Contributor

github-actions bot commented Nov 7, 2024

Pinging code owners for pkg/stanza: @djaglowski. See Adding Labels via Comments if you do not have permissions to add labels yourself.

Copy link
Contributor

github-actions bot commented Nov 7, 2024

Pinging code owners:

See Adding Labels via Comments if you do not have permissions to add labels yourself.

@songy23 songy23 removed the needs triage New item requiring triage label Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants