open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
3.1k stars 2.38k forks source link

[Flaky test] [extension/sumologicextension] TestRegisterEmptyCollectorNameForceRegistration #32915

Closed songy23 closed 6 months ago

songy23 commented 6 months ago

Component(s)

extension/sumologic

Describe the issue you're reporting

See https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/8985290637/job/24679157777?pr=32893#step:8:496

=== FAIL: . TestRegisterEmptyCollectorNameForceRegistration (unknown)
2024-05-07T12:35:00.138Z    INFO    sumologicextension/extension.go:574 Heartbeat loop initialized. Starting to send hearbeat requests  {"collector_name": "", "collector_id": "000000000FFFFFFF"}
    extension_test.go:603: 
            Error Trace:    /home/runner/work/opentelemetry-collector-contrib/opentelemetry-collector-contrib/extension/sumologicextension/extension_test.go:603
                                        /opt/hostedtoolcache/go/1.22.2/x64/src/net/http/server.go:2166
                                        /opt/hostedtoolcache/go/1.22.2/x64/src/net/http/server.go:3137
                                        /opt/hostedtoolcache/go/1.22.2/x64/src/net/http/server.go:2039
                                        /opt/hostedtoolcache/go/1.22.2/x64/src/runtime/asm_amd64.s:1695
            Error:          Not equal: 
                            expected: "/api/v1/collector/register"
                            actual  : "/api/v1/collector/heartbeat"

                            Diff:
                            --- Expected
                            +++ Actual
                            @@ -1 +1 @@
                            -/api/v1/collector/register
                            +/api/v1/collector/heartbeat
            Test:           TestRegisterEmptyCollectorNameForceRegistration
    extension_test.go:621: 
            Error Trace:    /home/runner/work/opentelemetry-collector-contrib/opentelemetry-collector-contrib/extension/sumologicextension/extension_test.go:621
                                        /opt/hostedtoolcache/go/1.22.2/x64/src/net/http/server.go:2166
                                        /opt/hostedtoolcache/go/1.22.2/x64/src/net/http/server.go:3137
                                        /opt/hostedtoolcache/go/1.22.2/x64/src/net/http/server.go:2039
                                        /opt/hostedtoolcache/go/1.22.2/x64/src/runtime/asm_amd64.s:1695
            Error:          Not equal: 
                            expected: "/api/v1/otCollectors/metadata"
                            actual  : "/api/v1/collector/register"

                            Diff:
                            --- Expected
                            +++ Actual
                            @@ -1 +1 @@
                            -/api/v1/otCollectors/metadata
                            +/api/v1/collector/register
            Test:           TestRegisterEmptyCollectorNameForceRegistration

DONE 38 tests, 2 failures in 601.405s

A re-run fixed the test, so it seems it is flaky

songy23 commented 6 months ago

Dup by https://github.com/open-telemetry/opentelemetry-collector-contrib/issues/32785

github-actions[bot] commented 6 months ago

Pinging code owners for extension/sumologic: @aboguszewski-sumo @kkujawa-sumo @mat-rumian @rnishtala-sumo @sumo-drosiek @swiatekm-sumo. See Adding Labels via Comments if you do not have permissions to add labels yourself.

github-actions[bot] commented 6 months ago

Pinging code owners: