Skip to content

tests: remove 02432_s3_parallel_parts_cleanup (uses zero-copy)#86006

Merged
azat merged 1 commit intoClickHouse:masterfrom
azat:less-zero-copy-on-ci
Aug 22, 2025
Merged

tests: remove 02432_s3_parallel_parts_cleanup (uses zero-copy)#86006
azat merged 1 commit intoClickHouse:masterfrom
azat:less-zero-copy-on-ci

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Aug 21, 2025

Follow up for #82250, this test fails with DatabaseReplicated, another option is #83727 (includes explanation why it is broken for this case), but we removed all other tests so better to remove.

But I left the integration tests, since likely they will not cause CI problems.

CI: https://s3.amazonaws.com/clickhouse-test-reports/json.html?PR=85984&sha=d53bffdffa261ba32ec27e686a59f01840f96c53&name_0=PR&name_1=Stateless%20tests%20%28amd_binary%2C%20old%20analyzer%2C%20s3%20storage%2C%20DatabaseReplicated%2C%20parallel%29

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Follow up for ClickHouse#82250, this test fails with DatabaseReplicated, another
option is ClickHouse#83727, but we removed all other tests so better to remove.

But I left the integration tests, since likely they will not cause CI
problems.
@azat azat added the 🍃 green ci 🌿 Fixing flaky tests in CI label Aug 21, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Aug 21, 2025

Workflow [PR], commit [bbeb267]

Summary:

job_name test_name status info comment
Stateless tests (amd_binary, old analyzer, s3 storage, DatabaseReplicated, parallel) failure
03595_alter_drop_column_comment_if_exists FAIL
Stateless tests (amd_msan, parallel, 1/2) failure
03364_estimate_compression_ratio FAIL

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Aug 21, 2025
@serxa serxa self-assigned this Aug 22, 2025
@azat azat enabled auto-merge August 22, 2025 06:41
@azat
Copy link
Copy Markdown
Member Author

azat commented Aug 22, 2025

CI issues are known

@azat azat added this pull request to the merge queue Aug 22, 2025
Merged via the queue into ClickHouse:master with commit 8737f2c Aug 22, 2025
121 of 122 checks passed
@azat azat deleted the less-zero-copy-on-ci branch August 22, 2025 06:53
@robot-clickhouse-ci-1 robot-clickhouse-ci-1 added the pr-synced-to-cloud The PR is synced to the cloud repo label Aug 22, 2025
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-not-for-changelog This PR should not be mentioned in the changelog 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.

3 participants