Skip to content

Fix test_keeper_snapshot_on_exit#81922

Merged
antonio2368 merged 1 commit intomasterfrom
fix-test-keeper-snapshot-on-exit
Jun 16, 2025
Merged

Fix test_keeper_snapshot_on_exit#81922
antonio2368 merged 1 commit intomasterfrom
fix-test-keeper-snapshot-on-exit

Conversation

@antonio2368
Copy link
Copy Markdown
Member

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Close #80424

For some reason after restart, node1 raft instance cannot make connection to node2 raft instance when requesting session ID.
I couldn't find a reason why but I removed the place where the issue happens because it's not actually useful. I also increased log level for Raft to make it easier to debug.

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

@clickhouse-gh
Copy link
Copy Markdown
Contributor

clickhouse-gh bot commented Jun 16, 2025

Workflow [PR], commit [cf14f9e]

@clickhouse-gh clickhouse-gh bot added the pr-not-for-changelog This PR should not be mentioned in the changelog label Jun 16, 2025
@hanfei1991 hanfei1991 self-assigned this Jun 16, 2025
@antonio2368 antonio2368 added this pull request to the merge queue Jun 16, 2025
Merged via the queue into master with commit 1c855e9 Jun 16, 2025
121 checks passed
@antonio2368 antonio2368 deleted the fix-test-keeper-snapshot-on-exit branch June 16, 2025 13:53
@robot-clickhouse-ci-2 robot-clickhouse-ci-2 added the pr-synced-to-cloud The PR is synced to the cloud repo label Jun 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

Test keeper_snapshot_on_exit has failed

3 participants