Skip to content

ci: disable zero-copy tests for ReplicatedDatabase#83727

Closed
azat wants to merge 2 commits intoClickHouse:masterfrom
azat:ci/fix-zero-copy
Closed

ci: disable zero-copy tests for ReplicatedDatabase#83727
azat wants to merge 2 commits intoClickHouse:masterfrom
azat:ci/fix-zero-copy

Conversation

@azat
Copy link
Copy Markdown
Member

@azat azat commented Jul 14, 2025

The problem is is that Replicated database has multiple shards, each shard will have different path in ZooKeeper, but the same in path for zero-copy locks which will lead to "Lock is lost, node does not exist" errors.

Fixes: #74734

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

@azat azat added the 🍃 green ci 🌿 Fixing flaky tests in CI label Jul 14, 2025
@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Jul 14, 2025

Workflow [PR], commit [99b21c2]

Summary:

job_name test_name status info comment
Stateless tests (amd_msan, 1/4) failure
00804_test_alter_compression_codecs FAIL
Stateless tests (amd_tsan, s3 storage, 2/3) failure
02561_null_as_default_more_formats FAIL
Stress test (amd_ubsan) failure
Killed by signal (in clickhouse-server.log) FAIL
Fatal message in clickhouse-server.log (see fatal_messages.txt) FAIL
Killed by signal (output files) FAIL
Found signal in gdb.log FAIL

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jul 14, 2025
@GrigoryPervakov GrigoryPervakov self-assigned this Jul 14, 2025
@azat
Copy link
Copy Markdown
Member Author

azat commented Jul 14, 2025

The problem is is that Replicated database has multiple shards, each
shard will have different path in ZooKeeper, but the same in path for
zero-copy locks which will lead to "Lock is lost, node does not exist"
errors.

Fixes: ClickHouse#74734
@azat azat force-pushed the ci/fix-zero-copy branch from f3e9749 to d825ac1 Compare July 14, 2025 18:51
@azat
Copy link
Copy Markdown
Member Author

azat commented Jul 15, 2025

Closed in favor #82250

@azat azat closed this Jul 15, 2025
@azat azat deleted the ci/fix-zero-copy branch July 15, 2025 07:47
azat added a commit to azat/ClickHouse that referenced this pull request Aug 21, 2025
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.
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Lock is lost, node does not exist (zero copy)

2 participants