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

Flaky Test_GetSeries #10320

Open
bboreham opened this issue Aug 23, 2023 · 1 comment
Open

Flaky Test_GetSeries #10320

bboreham opened this issue Aug 23, 2023 · 1 comment

Comments

@bboreham
Copy link
Contributor

Describe the bug
Test failed at: https://drone.grafana.net/grafana/loki/26733/3/5
and at https://drone.grafana.net/grafana/loki/26742/3/5
and https://drone.grafana.net/grafana/loki/26770/3/6

none of which change code near the failure.

To Reproduce
Unknown.

Expected behavior
CI tests either fail or pass consistently.

Environment:
Drone.

Screenshots, Promtail config, or terminal output

=== FAIL: pkg/storage/stores/series Test_GetSeries/foo{bar=~"beep|baz"}_/_v11_/_store (0.00s) test elide-stats-traces
    series_store_test.go:512: ========= Running query foo{bar=~"beep|baz"} with schema v11 test elide-stats-traces
    series_store_test.go:520:  test elide-stats-traces
        	Error Trace:	/src/loki/pkg/storage/stores/series/series_store_test.go:520 test elide-stats-traces
        	Error:      	Not equal:  test elide-stats-traces
        	            	expected: []labels.Labels{labels.Labels{labels.Label{Name:"bar", Value:"baz"}, labels.Label{Name:"flip", Value:"flop"}, labels.Label{Name:"toms", Value:"code"}}, labels.Labels{labels.Label{Name:"bar", Value:"beep"}, labels.Label{Name:"toms", Value:"code"}}} test elide-stats-traces
        	            	actual  : []labels.Labels{labels.Labels{labels.Label{Name:"bar", Value:"beep"}, labels.Label{Name:"toms", Value:"code"}}} test elide-stats-traces
        	            	 test elide-stats-traces
        	            	Diff: test elide-stats-traces
        	            	--- Expected test elide-stats-traces
        	            	+++ Actual test elide-stats-traces
        	            	@@ -1,16 +1,2 @@ test elide-stats-traces
        	            	-([]labels.Labels) (len=2) { test elide-stats-traces
        	            	- (labels.Labels) (len=3) { test elide-stats-traces
        	            	-  (labels.Label) { test elide-stats-traces
        	            	-   Name: (string) (len=3) "bar", test elide-stats-traces
        	            	-   Value: (string) (len=3) "baz" test elide-stats-traces
        	            	-  }, test elide-stats-traces
        	            	-  (labels.Label) { test elide-stats-traces
        	            	-   Name: (string) (len=4) "flip", test elide-stats-traces
        	            	-   Value: (string) (len=4) "flop" test elide-stats-traces
        	            	-  }, test elide-stats-traces
        	            	-  (labels.Label) { test elide-stats-traces
        	            	-   Name: (string) (len=4) "toms", test elide-stats-traces
        	            	-   Value: (string) (len=4) "code" test elide-stats-traces
        	            	-  } test elide-stats-traces
        	            	- }, test elide-stats-traces
        	            	+([]labels.Labels) (len=1) { test elide-stats-traces
        	            	  (labels.Labels) (len=2) { test elide-stats-traces
        	Test:       	Test_GetSeries/foo{bar=~"beep|baz"}_/_v11_/_store test elide-stats-traces
 test elide-stats-traces
=== FAIL: pkg/storage/stores/series Test_GetSeries (0.13s) test elide-stats-traces
@bboreham
Copy link
Contributor Author

Perhaps related to #10310

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants