Provide ClickHouse Peers validations as a library by pfcoperez · Pull Request #4288 · PeerDB-io/peerdb

Move `shared#RamdomString` implementation to flow/pkg so it can be used by common code.

pfcoperez

committed

Factor out into flow/pkg ClickHouse service validations so they can be used outside peerdb

pfcoperez

committed

Dependency injection interface for Staging checks in peerdb/pkg

pfcoperez

committed

Factories for Staging checks injected validation. These are used internally at PeerDB and made available for external users of flow/pkg to use

pfcoperez

committed

Move hostname validator to flow/pkg check

pfcoperez

committed

Fix linting issues

pfcoperez

committed

Add `disable_connector_validation` optional field to PeerDB API contract.

pfcoperez

committed

Skip peer validation e2e test

pfcoperez

committed

Remove local debugging session logs

pfcoperez

committed

Since we are using proto3, optional on the boolean flag is not actually required. Absent is not skipping is actually what we want, this commit simplifies the protocol

pfcoperez

committed

Update UI

pfcoperez

committed

Bug: Missing host validation in the checks that are part of `ValidateClickHousePeer`

pfcoperez

committed

Unconditional CH Peer check: Revert "Update UI"

pfcoperez

committed

Unconditional CH Peer check: Revert "Since we are using proto3, optional on the boolean flag is not actually required. Absent is not skipping is actually what we want, this commit simplifies the pr…

pfcoperez

committed

Unconditional CH Peer check: Revert "Skip peer validation e2e test"

pfcoperez

committed

Unconditional CH Peer check: Revert "Add `disable_connector_validation` optional field to PeerDB API contract."

pfcoperez

committed

Let's skip staging validation at ClickHouse connector checks,

pfcoperez

committed

PR Comments: Use pkg/common#RamdonString directly, no need for a proxy to limit the changeset

pfcoperez

committed

PR Comments: Move staging object store definitions to their own package.

pfcoperez

committed

Fix pinned version bump

pfcoperez

committed

Align dependencies with main

pfcoperez

committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Provide ClickHouse Peers validations as a library#4288

Provide ClickHouse Peers validations as a library#4288
pfcoperez wants to merge 21 commits into
mainfrom
DBI-65/shared-ch-validations

Commits on May 12, 2026

Move `shared#RamdomString` implementation to flow/pkg so it can be used by common code.

Factor out into flow/pkg ClickHouse service validations so they can be used outside peerdb

Dependency injection interface for Staging checks in peerdb/pkg

Factories for Staging checks injected validation. These are used internally at PeerDB and made available for external users of flow/pkg to use

Move hostname validator to flow/pkg check

Fix linting issues

Add `disable_connector_validation` optional field to PeerDB API contract.

Skip peer validation e2e test

Remove local debugging session logs

Since we are using proto3, optional on the boolean flag is not actually required. Absent is not skipping is actually what we want, this commit simplifies the protocol

Update UI

Bug: Missing host validation in the checks that are part of `ValidateClickHousePeer`

Unconditional CH Peer check: Revert "Update UI"

Unconditional CH Peer check: Revert "Since we are using proto3, optional on the boolean flag is not actually required. Absent is not skipping is actually what we want, this commit simplifies the pr…

Unconditional CH Peer check: Revert "Skip peer validation e2e test"

Unconditional CH Peer check: Revert "Add `disable_connector_validation` optional field to PeerDB API contract."

Let's skip staging validation at ClickHouse connector checks,

PR Comments: Use pkg/common#RamdonString directly, no need for a proxy to limit the changeset

PR Comments: Move staging object store definitions to their own package.

Fix pinned version bump

Align dependencies with main