Skip to content

Conversation

@jmcarp
Copy link
Contributor

@jmcarp jmcarp commented Oct 23, 2025

ClickHouse doesn't allow us to capture backups unless a backup directory is configured, and we can't configure a backup directory without restarting ClickHouse. To support backup collection for testing purposes, and in case we wind up needing to use backups during production operations, configure a ClickHouse backup directory via clickhouse-admin.

ClickHouse doesn't allow us to capture backups unless a backup directory is
configured, and we can't configure a backup directory without restarting
ClickHouse. To support backup collection for testing purposes, and in case we
wind up needing to use backups during production operations, configure a
ClickHouse backup directory via clickhouse-admin.
@jmcarp jmcarp requested a review from bnaecker October 23, 2025 14:39
Copy link
Collaborator

@bnaecker bnaecker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this looks fine to me. As long as actually creating a backup works, should be good! I think this will close #6612 as well.

@jmcarp
Copy link
Contributor Author

jmcarp commented Oct 23, 2025

Tested locally with analogous config, and I was able to capture a backup.

@jmcarp jmcarp merged commit 7bc5275 into main Oct 23, 2025
16 checks passed
@jmcarp jmcarp deleted the jmcarp/clickhouse-allow-backup branch October 23, 2025 17:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants