Skip to content

ci: test matrix +26.10, -26.7 - #704

Open
polyglotAI-bot wants to merge 1 commit into
mainfrom
polyglot/ci-matrix-26.10
Open

polyglotAI-bot wants to merge 1 commit into
mainfrom
polyglot/ci-matrix-26.10

Conversation

@polyglotAI-bot

@polyglotAI-bot polyglotAI-bot commented Oct 26, 2026 •

Copy link
Copy Markdown
Collaborator

Description

CI/maintenance chore: bring the ClickHouse version coverage of both test suites onto the
support policy — the 3 latest series plus the last 2 LTS series, plus head. No client
behavior changes, so no changelog fragment and no version bump.

Target set derived from ClickHouse utils/list-versions/version_date.tsv:

  • latest (newest 3): 26.10, 26.9, 26.8
  • LTS (newest 2): 26.8, 26.3

Changes

  • .github/workflows/tests.yml (regress-clickhouse, HTTP driver): +26.10, -26.7.
  • .github/workflows/tests-tcp.yml (regress-clickhouse-tcp, TCP driver): +26.10, -26.7.

Both matrices are now [latest, 26.10, 26.9, 26.8, 26.3]. head stays covered by
nightly.yml (HTTP and TCP). The floating latest entry is kept; it satisfies no policy slot.

Notes

  • The support floor stays 26.3, so the policy statement in AGENTS.md needs no edit.
  • "26.10" is quoted, so YAML keeps it a string (unquoted, it parses as the float 26.1).
    Both fixtures parse CLICKHOUSE_VERSION with System.Version, so 26.10 orders after 26.9
    for ClickHouseFeatureMap and TcpServerFeatures.
  • clickhouse/clickhouse-server:latest currently resolves to the same image digest as 26.10.
  • Version coverage only. Single-version pins elsewhere are untouched.

Test

  • Both workflow files parse as YAML; the matrices resolve to [latest, 26.10, 26.9, 26.8, 26.3].
  • docker manifest inspect clickhouse/clickhouse-server:<tag> succeeds for 26.10, 26.9, 26.8,
    26.3, latest and head.
  • No residual references to 26.7 remain in .github/.

🤖 Generated with Claude Code


Note

Low Risk
Workflow-only version matrix tweak with no runtime or client behavior changes.

Overview
Updates the ClickHouse version matrix for HTTP (tests.yml) and TCP (tests-tcp.yml) regression jobs so CI tracks current support policy: three latest series plus two LTS, alongside the floating latest tag.

Both regress-clickhouse matrices change from including 26.7 to 26.10, yielding [latest, 26.10, 26.9, 26.8, 26.3]. No application or driver code changes—only which server image tags run in CI.

Reviewed by Cursor Bugbot for commit ccd3f41. Bugbot is set up for automated code reviews on this repo. Configure here.

Bring the ClickHouse version coverage of the HTTP and TCP suites onto
the support policy (3 latest series + 2 latest LTS series + head):
26.10, 26.9, 26.8, 26.3. head stays covered by nightly.yml, and the
floating latest entry is kept.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 26, 2026 05:36

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

Workflow validation remains outside the completed automated review scope, so manual confirmation is needed before approval.

No open issues 🎉

What changed in this PR

Updates ClickHouse server-version coverage for both HTTP and native TCP regression suites to match the stated support policy. This is a CI-only maintenance change with no client behavior changes.

Changes:

  • Replaces 26.7 with quoted "26.10" in both version matrices.
  • Retains latest, 26.9, 26.8, and the 26.3 support floor.

ClickHouse C# Client Compliance Checklist

Check Status Notes
Protocol compatibility preserved? Yes No protocol changes.
Type system changes tested comprehensively? N/A No type changes.
Async patterns correct? N/A No async changes.
Existing tests untouched? Yes Test implementations unchanged.
Connection string backward compatible? N/A No connection-string changes.
Error messages clear and actionable? N/A No message changes.
Thread safety reviewed? N/A No concurrency changes.
Public docs updated for public-facing changes? N/A No public-facing behavior changes.
Changelog fragment added? N/A CI maintenance only.
Duplicated client logic updated consistently? N/A No client logic changes.
File Description
.github/​workflows/​tests.yml Replaces HTTP regression coverage of 26.7 with 26.10.
.github/​workflows/​tests-tcp.yml Applies the same version replacement to TCP regression coverage.

Review effort: Balanced

@codecov

codecov Bot commented Oct 26, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

This branch has not been deployed

No deployments
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.

2 participants