search

jaegertracing / jaeger-clickhouse

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

Add integration tests #5

Closed pavolloffay closed 2 years ago

pavolloffay commented 3 years ago

Add integration test that spins up ClickHouse, Jaeger with storage plugin. The test would store and query data.

Jaeger already has storage integration tests - https://github.com/jaegertracing/jaeger/tree/master/plugin/storage/integration. It would be great if they could be reused.

pavolloffay commented 3 years ago

I am working on this one now

EinKrebs commented 3 years ago

@pavolloffay Should we close this issue?

pavolloffay commented 3 years ago

no, we need similar tests that verify query compatibility like in the link above.

It would be also great to create a test for replicated deployment.