Skip to content

Go SDK: pack a bundle that registers no Dags at pack time - #74637

Merged
jason810496 merged 1 commit into
apache:mainfrom
jason810496:feature/go-sdk/pack-bundle-without-dags
Oct 13, 2026
Merged

jason810496 merged 1 commit into
apache:mainfrom
jason810496:feature/go-sdk/pack-bundle-without-dags

Conversation

@jason810496

Copy link
Copy Markdown
Member

Why

A Go Dag definition that generates its Dags at run time, for example from a YAML file in the Dag bundle, registers nothing when airflow-go-pack runs it in a temp build dir, so packing failed with bundle exposes no dags: nothing to pack.

What

  • airflow-go-pack now prints a warning and still writes the bundle when no Dags or task handlers are registered at pack time.
$ go tool airflow-go-pack --output bin/dynamic_dags ./example/dynamic
warning: no Dags or task handlers were registered at pack time; this is expected only if the bundle generates its Dags at run time, and those Dags show the entrypoint source in the Code view
Wrote bundle bin/dynamic_dags (sdk=go/..., task_handler_dags=0, native_dags=0)
  • The bundle metadata schema and spec no longer require dags to be non-empty. It only lists Python Dags with stub TaskHandler, so a bundle of Dag definitions already wrote dags: {}.

Was generative AI tooling used to co-author this PR?
  • Yes, with help of Claude Code (Claude Opus 5.5) following the guidelines

Co-Authored-By: Claude <noreply@anthropic.com>
Comment thread go-sdk/cmd/airflow-go-pack/pack.go Resolved
@guan404ming

Copy link
Copy Markdown
Member

Looks nice!

@jason810496
jason810496 merged commit e043a27 into apache:main Oct 13, 2026
68 of 69 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-4-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
✅ v3-4-test PR Link

vatsrahul1001 pushed a commit that referenced this pull request Oct 17, 2026
…#74637) (#74821)

(cherry picked from commit e043a27)

Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
vatsrahul1001 pushed a commit that referenced this pull request Oct 20, 2026
…#74637) (#74821)

(cherry picked from commit e043a27)

Co-authored-by: Jason(Zhe-You) Liu <68415893+jason810496@users.noreply.github.com>
Co-authored-by: Claude <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants