Skip to content

Fix differential CI reference version drift - #14

Merged
ShawnChen-Sirius merged 1 commit into
chdb-io:mainfrom
ShawnChen-Sirius:fix/differential-ci-20260813
Aug 14, 2026
Merged

Fix differential CI reference version drift#14
ShawnChen-Sirius merged 1 commit into
chdb-io:mainfrom
ShawnChen-Sirius:fix/differential-ci-20260813

Conversation

@ShawnChen-Sirius

@ShawnChen-Sirius ShawnChen-Sirius commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

Fixes the scheduled differential CI failure caused by chdb-core 26.7.0 moving chDB's ClickHouse baseline to 26.7.2 while the workflow still extracted a fixed clickhouse-server:26.5.1.882 reference binary.

The differential job now installs the chDB test stack before choosing the reference binary, derives the ClickHouse Docker tag from SELECT version(), and falls back from X.Y.Z to X.Y when the exact patch tag is unavailable. The version-skew guard stays in place, but future chdb-core baseline bumps will pull a matching reference binary instead of failing on a stale hard-coded tag.

Verified with:

  • pytest -q tests/differential
  • ruff check tests/differential
  • git diff --check
  • confirmed clickhouse/clickhouse-server:26.7.2 exists on Docker Hub

Note

Fix differential CI version drift by detecting chDB's ClickHouse version at runtime

  • The differential CI job previously used a hardcoded ClickHouse server tag, which could drift out of sync with the chDB build being tested.
  • A new 'Detect chDB ClickHouse version' step runs SELECT version() via chdb and exports CHDB_CLICKHOUSE_VERSION, CLICKHOUSE_TAG_PATCH, and CLICKHOUSE_TAG_MINOR into the GitHub environment.
  • The 'Extract clickhouse binary from server image' step now iterates over these derived tags instead of a fixed list, and emits a version-specific error if no matching tag is found.
  • Test dependency installation (pip install -e .[dev,test-differential]) is moved earlier in the job, before the version detection step that requires chdb to be importable.

Macroscope summarized 5c4e96f.

@ShawnChen-Sirius
ShawnChen-Sirius merged commit 875b015 into chdb-io:main Aug 14, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant