Skip to content

ci: use dbt parse for commit manifest #9629

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
jeff-dude wants to merge 3 commits into
base: main
Choose a base branch
from
Open

ci: use dbt parse for commit manifest #9629

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
92e0fd5
ci: use dbt parse for commit manifest
jeff-dude May 1, 2026
9f729c0
Remove unnecessary cluster check before dbt parse
cursoragent May 1, 2026
f39b8a7
Merge branch 'main' into ci/parse-commit-manifest
jeff-dude May 8, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 2 additions & 5 deletions .github/workflows/commit_manifest.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,11 +46,8 @@ jobs:
working-directory: ${{env.PROJECT_DIR}}
run: "uv run dbt deps"

- name: Run dbt debug on Trino if dunesql engine is used, and retry until cluster is available
run: "uv run bash ./scripts/ensure_cluster.sh"

- name: dbt compile to create prod manifest from main
run: "uv run dbt compile --target-path . $PROFILE --project-dir ${PROJECT_DIR}"
- name: dbt parse to create prod manifest from main
Comment thread
jeff-dude marked this conversation as resolved.
run: "uv run dbt parse --target-path . $PROFILE --project-dir ${PROJECT_DIR}"

- name: copy old manifest locally
run: "aws s3 cp $S3_LOCATION/manifest.json $S3_LOCATION/previous_manifest.json"
Expand Down
Loading