build: make DuckDB 1.5.5 the fleet default - #1029
Merged
Merged
Conversation
Co-authored-by: Shelley <shelley@exe.dev>
Test Impact PlanDeterministic summary of how this PR changes tests, CI runners, and coverage-risk signals. Summary
Signals
Coverage risk: neutral or increased No coverage-reduction warnings detected. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Promotes DuckDB 1.5.5 from canary to the fleet default, following #1028 (which published the 1.5.5 canary images and moved CI/e2e onto 1.5.5).
1.5.5is nowdefault: truein both the build and manifest matrices — it publishes the unsuffixed fleet tags (:latest,:<sha>) and dispatches Charts.1.5.3stays in the matrix as a non-default rollback row, still publishing…-duckdb1.5.3suffixed tags, so rolling a tenant back is a config-storeimagere-point, and a full fleet rollback is a revert of this PR.default:truerow.ducklake_version_matrix.shcomment updated (default + rollback versions).No code changes:
go.mod, the Dockerfiles, the e2e/scenario image pins, and the mw-dev harness assertions all moved to 1.5.5 in #1028.This PR's e2e-mw-dev run is the gate: it builds a 1.5.5 image and exercises the full activation pipeline (control plane → STS → worker pod → DuckDB → DuckLake on real S3/cnpg) on the mw-dev cluster, including the engine-version (
v1.5.5) and fork-extension SHA assertions added in #1028.Known 1.5.5 behavioral note (from #1028 testing): internal catalog schemas (e.g.
system,temp) are now listed byduckdb_schemas()/pg_catalog.pg_namespace. No production impact surfaced in the test suite; worth a look during review/canary.Validation
default: true(1.5.5) in both matrices.tests/integrationsuite and DuckLake concurrency benchmark pass on 1.5.5.