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.

Sign up for GitHub

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

Jump to bottom

[Flaky test] cagg_insert interleaves output differently #5331

Closed
jchampio opened this issue Feb 14, 2023 · 1 comment · Fixed by #6128
Closed

[Flaky test] cagg_insert interleaves output differently #5331

jchampio opened this issue Feb 14, 2023 · 1 comment · Fixed by #6128
Labels
flaky-test Issue about a flaky test

Comments

@jchampio
Copy link
Contributor

Which test is flaky?

cagg_insert

Since when has the test been flaky?

February 14, 2023

Link to the failed test run

https://github.com/timescale/timescaledb/actions/runs/4175600633/jobs/7233369995

Log output

diff -u /home/runner/work/timescaledb/timescaledb/tsl/test/isolation/expected/cagg_insert.out /home/runner/work/timescaledb/timescaledb/build/tsl/test/isolation/results/cagg_insert.out
--- /home/runner/work/timescaledb/timescaledb/tsl/test/isolation/expected/cagg_insert.out	2023-02-14 15:55:10.906147850 +0000
+++ /home/runner/work/timescaledb/timescaledb/build/tsl/test/isolation/results/cagg_insert.out	2023-02-14 15:58:22.696072131 +0000
@@ -5,8 +5,8 @@
 step Refresh2: CALL refresh_continuous_aggregate('continuous_view', NULL, 15); <waiting ...>
 step Refresh: CALL refresh_continuous_aggregate('continuous_view', NULL, 15); <waiting ...>
 step UnlockInvalThrEx: ROLLBACK;
+R2: NOTICE:  continuous aggregate "continuous_view" is already up-to-date
 step Refresh2: <... completed>
-R: NOTICE:  continuous aggregate "continuous_view" is already up-to-date
 step Refresh: <... completed>
 
 starting permutation: Ib LockCagg I1 Refresh Ic UnlockCagg

Reason for flakiness

No response
@jchampio jchampio added the flaky-test Issue about a flaky test label Feb 14, 2023
@jnidzwetzki
Copy link
Contributor

Occurred again https://github.com/timescale/timescaledb/actions/runs/5216903228/jobs/9416136011

@mkindahl mkindahl mentioned this issue Sep 28, 2023
Merged
mkindahl added a commit to mkindahl/timescaledb that referenced this issue Sep 28, 2023
@mkindahl
Fix flaky cagg_insert
d78b5e4 
Isolation test `cagg_insert` is flaky because refresh steps can
complete in any order. Adding constraints so that completion is
reported in the same order in all test runs.

Fixes timescale#5331
mkindahl added a commit to mkindahl/timescaledb that referenced this issue Sep 28, 2023
@mkindahl
Fix flaky cagg_insert
b4efe7f 
Isolation test `cagg_insert` is flaky because refresh steps can
complete in any order. Adding constraints so that completion is
reported in the same order in all test runs.

Fixes timescale#5331
mkindahl added a commit that referenced this issue Sep 28, 2023
@mkindahl
Fix flaky cagg_insert
6bef59a 
Isolation test `cagg_insert` is flaky because refresh steps can
complete in any order. Adding constraints so that completion is
reported in the same order in all test runs.

Fixes #5331
konskov pushed a commit to konskov/timescaledb that referenced this issue Oct 6, 2023
@mkindahl @konskov
Fix flaky cagg_insert
b302283 
Isolation test `cagg_insert` is flaky because refresh steps can
complete in any order. Adding constraints so that completion is
reported in the same order in all test runs.

Fixes timescale#5331

(cherry picked from commit 6bef59a)
konskov pushed a commit that referenced this issue Oct 6, 2023
@mkindahl @konskov
Fix flaky cagg_insert
29c0728 
Isolation test `cagg_insert` is flaky because refresh steps can
complete in any order. Adding constraints so that completion is
reported in the same order in all test runs.

Fixes #5331

(cherry picked from commit 6bef59a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
flaky-test Issue about a flaky test
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix flaky cagg_insert mkindahl/timescaledb
2 participants
@jchampio @jnidzwetzki