open-telemetry / opentelemetry-collector-contrib

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

[pkg/stanza] Flaky test TestMatcher/Numeric_Sorting_with_grouping #36623

Open dmitryax opened 2 hours ago

dmitryax commented 2 hours ago

Seen in https://github.com/open-telemetry/opentelemetry-collector/actions/runs/12127649470/job/33812375774?pr=11637

✖  fileconsumer/matcher (17ms)

=== Failed
=== FAIL: fileconsumer/matcher TestMatcher/Numeric_Sorting_with_grouping (0.00s)
    matcher_test.go:856: 
            Error Trace:    /tmp/opentelemetry-collector-contrib/pkg/stanza/fileconsumer/matcher/matcher_test.go:856
            Error:          Not equal: 
                            expected: []string{"err.a.123456789.log", "err.a.123456788.log", "err.a.123456787.log", "err.a.123456786.log", "err.b.123456789.log", "err.b.123456788.log"}
                            actual  : []string{"err.b.123456789.log", "err.b.123456788.log", "err.a.123456789.log", "err.a.123456788.log", "err.a.123456787.log", "err.a.123456786.log"}

                            Diff:
                            --- Expected
                            +++ Actual
                            @@ -1,2 +1,4 @@
                             ([]string) (len=6) {
                            + (string) (len=19) "err.b.123456789.log",
                            + (string) (len=19) "err.b.123456788.log",
                              (string) (len=19) "err.a.123456789.log",
                            @@ -4,5 +6,3 @@
                              (string) (len=19) "err.a.123456787.log",
                            - (string) (len=19) "err.a.123456786.log",
                            - (string) (len=19) "err.b.123456789.log",
                            - (string) (len=19) "err.b.123456788.log"
                            + (string) (len=19) "err.a.123456786.log"
                             }
            Test:           TestMatcher/Numeric_Sorting_with_grouping

=== FAIL: fileconsumer/matcher TestMatcher (0.04s)

=== FAIL: fileconsumer/matcher TestMatcher/Numeric_Sorting_with_grouping (re-run 1) (0.00s)
    matcher_test.go:856: 
            Error Trace:    /tmp/opentelemetry-collector-contrib/pkg/stanza/fileconsumer/matcher/matcher_test.go:856
            Error:          Not equal: 
                            expected: []string{"err.a.123456789.log", "err.a.123456788.log", "err.a.123456787.log", "err.a.123456786.log", "err.b.123456789.log", "err.b.123456788.log"}
                            actual  : []string{"err.b.123456789.log", "err.b.123456788.log", "err.a.123456789.log", "err.a.123456788.log", "err.a.123456787.log", "err.a.123456786.log"}

                            Diff:
                            --- Expected
make[3]: *** [../../Makefile.Common:132: test] Error 1
make[2]: *** [Makefile:200: pkg/stanza] Error 2
make[1]: *** [Makefile:132: gotest] Error 2
make: *** [Makefile:268: check-contrib] Error 2
                            +++ Actual
                            @@ -1,2 +1,4 @@
                             ([]string) (len=6) {
                            + (string) (len=19) "err.b.123456789.log",
                            + (string) (len=19) "err.b.123456788.log",
                              (string) (len=19) "err.a.123456789.log",
                            @@ -4,5 +6,3 @@
                              (string) (len=19) "err.a.123456787.log",
                            - (string) (len=19) "err.a.123456786.log",
                            - (string) (len=19) "err.b.123456789.log",
                            - (string) (len=19) "err.b.123456788.log"
                            + (string) (len=19) "err.a.123456786.log"
                             }
            Test:           TestMatcher/Numeric_Sorting_with_grouping

=== FAIL: fileconsumer/matcher TestMatcher (re-run 1) (0.01s)
github-actions[bot] commented 2 hours ago

Pinging code owners for pkg/stanza: @djaglowski. See Adding Labels via Comments if you do not have permissions to add labels yourself. For example, comment '/label priority:p2 -needs-triaged' to set the priority and remove the needs-triaged label.

github-actions[bot] commented 2 hours ago

Pinging code owners: