Skip to content

fix(sanity-checks): read dbt_demo DuckDB file with the same image that wrote it - #375

Merged
Pradumnasaraf merged 1 commit into
mainfrom
fix/sanity-check-dbt-demo
Nov 12, 2026
Merged

Pradumnasaraf merged 1 commit into
mainfrom
fix/sanity-check-dbt-demo

Conversation

@Pradumnasaraf

Copy link
Copy Markdown
Member

dbt_demo wrote dbt.duckdb with ghcr.io/kestra-io/dbt-duckdb:latest (DuckDB 2.0) and read it with ghcr.io/kestra-io/duckdb:latest (DuckDB 1.5), which cannot open the newer file format. The python step now uses the same image as the dbt step and exports with DuckDB COPY, since that image has no pandas.

Ran it twice on preview: SUCCESS both times.

@github-project-automation github-project-automation Bot moved this to To review in Pull Requests Nov 12, 2026
@github-actions

github-actions Bot commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor
🔎 OpenGrep ✅ (0 findings)

Found 0 finding(s).

🐷 TruffleHog ✅ (0 secrets)

No secret found.

📦 Artifacts (1 jar)
Name Size Updated Expiration
jar 268.83 KB Nov 12, 26, 8:06:16 AM UTC Nov 19, 26, 8:06:15 AM UTC
🧪 Java Unit Tests ✅ (0 failed, 185 passed, 1 skipped)
TestsPassed ✅Skipped ⚠️FailedTime ⏱
Java Tests Report186 ran185 ✅1 ⚠️0 ❌3m 12s 298ms
📊 Coverage — plugin ✅ (88.34%, no changes)

88.34% line coverage (▬ 0.00pp)
1197/1355 lines covered

Coverage by module

Module Lines Coverage
root 1197/1355 88.34% (▬ 0.00pp)
🔁 Unreleased Commits ✅ (0 commits)

✅ No unreleased commits found.

@github-actions

Copy link
Copy Markdown
Contributor

Tests report quick summary:

success ✅ > tests: 186, success: 185, skipped: 1, failed: 0

unfold for details
Project Status Success Skipped Failed
plugin-dbt success ✅ 185 1 0

@Pradumnasaraf
Pradumnasaraf merged commit c692b7a into main Nov 12, 2026
9 checks passed
@Pradumnasaraf
Pradumnasaraf deleted the fix/sanity-check-dbt-demo branch November 12, 2026 09:17
@github-project-automation github-project-automation Bot moved this from To review to Done in Pull Requests Nov 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants