jaegertracing / jaeger-clickhouse

Jaeger ClickHouse storage plugin implementation
Apache License 2.0
249 stars 51 forks source link

[Feature]: Support communication over HTTP to allow connecting to clickhouse proxy #136

Open smunukutla-mycarrier opened 7 months ago

smunukutla-mycarrier commented 7 months ago

Requirement

We require Jaeger-Clickhouse to be able to connect to Clickhouse behind a proxy like Chproxy. To do so, we need the ability to communicate over http. Currently, clickhouse-go/v2 v2.3.0 allows connections over http but there is no support in Jaeger-Clickhouse to create the connection.

Problem

Jaeger-Clickhouse does not provide configuration to allow connection over http to proxied clickhouse instances.

Proposal

Update configuration options to allow Jaeger-Clickhouse plugin to connect over http. Additionally, add options to allow skipping tls verification and enable/disable debug logs. Example config below:

address: chproxy.somewhere.dev
username: someuser
password: **********
database: default
protocol: http
debug: true
insecure_skip_verify: true
replication: false
spans_table:
spans_index_table:
operations_table:

Open questions

No response