-
-
Notifications
You must be signed in to change notification settings - Fork 1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Migration to 2.0.0 fails to connect to Clickhouse when using IPv6 #3173
Comments
👋 @BeryJu Oh, great catch! Thank you! Something like this would probably be a good enough fix: def start(url, max_threads) when is_binary(url) and is_integer(max_threads) do
start_link(
url: url,
queue_target: 500,
queue_interval: 2000,
pool_size: 1,
settings: [
max_insert_threads: max_threads,
send_progress_in_http_headers: 1
],
+ transport_opts: Plausible.ClickHouseRepo.config() |> Keyword.fetch!(:transport_opts)
)
end I'll PR it |
Yeah that's pretty close to my temporary hotfix which was
|
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Past Issues Searched
Issue is a Bug Report
Using official Plausible Cloud hosting or self-hosting?
Self-hosting
Describe the bug
When running
bin/plausible rpc Plausible.DataMigration.NumericIDs.run
when using IPv6 (having setECTO_IPV6
andECTO_CH_IPV6
totrue
), the migration fails:Expected behavior
The migration should work
Having a dig around https://github.com/plausible/analytics/blob/16846b16c8e513f72d7c23459ee879d9249f556d/lib/plausible/data_migration/repo.ex, when adding
on line 20, it does work (requires rebuilding the container)
Screenshots
No response
Environment
The text was updated successfully, but these errors were encountered: