Skip to content

Add non-DAG bundle deploys - #166

Merged
jeremybeard merged 1 commit into
dags-only-fallbacksfrom
non-dag-bundles
Oct 19, 2026
Merged

jeremybeard merged 1 commit into
dags-only-fallbacksfrom
non-dag-bundles

Conversation

@jeremybeard

@jeremybeard jeremybeard commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Adds deploy-type: non-dags (Astro CLI 1.44.0+), which deploys any directory as a non-DAG bundle mounted for DAGs to read. As with dbt, the directory is root-folder. It is required, as is mount-path; non-dags-bundle-type is an optional label.

The bundle deploy step, shared with dbt, also gains:

  • bundle-trigger-paths, the bundle equivalent of image-trigger-paths
  • skip-unchanged support
  • a deploy when the diff base is missing from the checkout
  • always deploying on create-deployment-preview, never on delete-deployment-preview

Validation: deploy-type is now checked, where an unknown value used to deploy nothing. root-folder is required with non-dags, since an empty one would deploy the whole repository. image-name, parse, pytest and build-secrets are rejected with non-dags, as the CLI rejects them alongside --non-dags. A mount-path on other deploy types is a warning.

E2E, on one new bundle deployment: the non-DAG deploy, the skip and fallback paths, trigger-path parsing, a non-DAG deploy to a preview, and the delete guard.

Comment thread action.yaml
fi

for file in $files; do
if [[ $file =~ ^"${{ inputs.root-folder }}".* ]]; then

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

curious q: do we need to normalize the root-folder, I mean stripping any "." or "/" at prefix ("./dbt"), I don't mind to skip it here if we are not doing it in dag/image deploy check?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

It's done this way for all deploy types so will leave it like this.

@jeremybeard
jeremybeard merged commit 2a39f38 into main Oct 19, 2026
20 of 21 checks passed

This branch is waiting to be deployed

1 waiting deployment
e2e-test — 3cd8fe5f Waiting Oct 19, 2026 by jeremybeard via Redact Inputs #370
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.

3 participants