Skip to content

feat: Add ClickHouse destination connection test #18559

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.

Sign up for GitHub

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

Jump to bottom
Merged
kodiakhq merged 4 commits into from
Jul 15, 2024
Merged

feat: Add ClickHouse destination connection test #18559

kodiakhq merged 4 commits into from
Jul 15, 2024

Conversation

@blesniewski
Copy link
Contributor

@blesniewski blesniewski commented Jul 12, 2024

Summary

Adds connection test to ClickHouse destination

@blesniewski blesniewski requested review from a team and murarustefaan and removed request for a team July 12, 2024 09:50
@blesniewski
Copy link
Contributor Author

blesniewski commented Jul 12, 2024 

❯ cloudquery test-connection pg-clickhouse
Loading spec(s) from pg-clickhouse
Test Connection Results
source       postgresql (cloudquery/postgresql@v6.2.3)  Success
destination  clickhouse (grpc@127.0.0.1:7777)           Success
❯ cloudquery test-connection pg-clickhouse
Loading spec(s) from pg-clickhouse
Test Connection Results
source       postgresql (cloudquery/postgresql@v6.2.3)  Success
destination  clickhouse (grpc@127.0.0.1:7777)           Failure: INVALID_SPEC    invalid spec
json: cannot unmarshal number into Go struct field Spec.ConnectionString of type string
Error: at least one test connection failed
❯ cloudquery test-connection pg-clickhouse
Loading spec(s) from pg-clickhouse
Test Connection Results
source       postgresql (cloudquery/postgresql@v6.2.3)  Success
destination  clickhouse (grpc@127.0.0.1:7777)           Failure: UNAUTHORIZED    failed to verify server version code: 516, message: userna1me: Authentication failed: password is incorrect, or there is no user with such name.
Error: at least one test connection failed
❯ cloudquery test-connection pg-clickhouse
Loading spec(s) from pg-clickhouse
Test Connection Results
source       postgresql (cloudquery/postgresql@v6.2.3)  Success
destination  clickhouse (grpc@127.0.0.1:7777)           Failure: UNREACHABLE     failed to verify server version dial tcp [::1]:19100: connect: connection refused
Error: at least one test connection failed

@blesniewski blesniewski added the automerge Automatically merge once required checks pass label Jul 15, 2024
@kodiakhq kodiakhq bot merged commit d733f1d into main Jul 15, 2024
@kodiakhq kodiakhq bot deleted the feat/clickhouse-destination-connection-test branch July 15, 2024 08:07
@cq-bot cq-bot mentioned this pull request Jul 15, 2024
Merged
kodiakhq bot pushed a commit that referenced this pull request Jul 17, 2024
@cq-bot
chore(main): Release plugins-destination-clickhouse v4.2.0 (#18453)
0af714a 
🤖 I have created a release *beep* *boop*
---


## [4.2.0](plugins-destination-clickhouse-v4.1.4...plugins-destination-clickhouse-v4.2.0) (2024-07-17)


### Features

* Add ClickHouse destination connection test ([#18559](#18559)) ([d733f1d](d733f1d))


### Bug Fixes

* **deps:** Update module github.com/cloudquery/cloudquery-api-go to v1.12.0 ([#18448](#18448)) ([a5850e1](a5850e1))
* **deps:** Update module github.com/cloudquery/codegen to v0.3.17 ([#18491](#18491)) ([b43fd16](b43fd16))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.49.1 ([#18497](#18497)) ([3416eb7](3416eb7))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.49.2 ([#18504](#18504)) ([2d80936](2d80936))
* **deps:** Update module github.com/cloudquery/plugin-sdk/v4 to v4.49.3 ([#18513](#18513)) ([d12da90](d12da90))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@murarustefaan murarustefaan murarustefaan approved these changes

Assignees

No one assigned

Labels

area/plugin/destination/clickhouse automerge Automatically merge once required checks pass

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@blesniewski @murarustefaan