Skip to content

Fix test 03237_create_or_replace_view_atomically_with_atomic_engine#80380

Merged
alexey-milovidov merged 1 commit intomasterfrom
fix-test-03237
May 17, 2025
Merged

Fix test 03237_create_or_replace_view_atomically_with_atomic_engine#80380
alexey-milovidov merged 1 commit intomasterfrom
fix-test-03237

Conversation

@alexey-milovidov
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • CI Fix or Improvement (changelog entry is not required)

Closes #70319.

@hanfei1991, see an example, why ddecca2 was not enough.

@alexey-milovidov alexey-milovidov added the 🍃 green ci 🌿 Fixing flaky tests in CI label May 16, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented May 16, 2025

Workflow [PR], commit [9362e3f]

@clickhouse-gh clickhouse-gh bot added the pr-ci label May 16, 2025
@alexey-milovidov alexey-milovidov self-assigned this May 17, 2025
@alexey-milovidov alexey-milovidov added this pull request to the merge queue May 17, 2025
Merged via the queue into master with commit f62a2ac May 17, 2025
120 checks passed
@alexey-milovidov alexey-milovidov deleted the fix-test-03237 branch May 17, 2025 02:13
@robot-ch-test-poll2 robot-ch-test-poll2 added the pr-synced-to-cloud The PR is synced to the cloud repo label May 17, 2025
@alexey-milovidov
Copy link
Copy Markdown
Member Author

@hanfei1991, this was wrong by yet another reason: you forgot to redirect stderr to stdout, and you forgot to filter other lines in the output. I will fix it.

@alexey-milovidov
Copy link
Copy Markdown
Member Author

@hanfei1991, @evillique, see #80412

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

Labels

🍃 green ci 🌿 Fixing flaky tests in CI pr-ci pr-synced-to-cloud The PR is synced to the cloud repo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test 03237_create_or_replace_view_atomically_with_atomic_engine is flaky

2 participants