Skip to content

Conversation

@ephraimbuddy
Copy link
Contributor

In cases where the dag processor component is run in a separate container/pod, dag callback does not work for git dag bundle. This is because the dagbag could not find the exact version to use in running the callback since the versioned repo is not created in the dag processor.

The fix here is to create the versioned repo in the dag processor. In the process, we also need to ensure that the stale bundles are cleaned


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

In cases where the dag processor component is run in a separate container/pod,
dag callback does not work for git dag bundle. This is because the dagbag
could not find the exact version to use in running the callback since the versioned
repo is not created in the dag processor.

The fix here is to create the versioned repo in the dag processor. In the process,
we also need to ensure that the stale bundles are cleaned
@ephraimbuddy ephraimbuddy marked this pull request as ready for review January 28, 2026 20:34
Copy link
Member

Choose a reason for hiding this comment

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

This isn't a GitDagBundle problem. You'd have to do this for every bundle that supports versioning!

Instead, the dag processor should init a bundle at the right version if a callback ends up coming in. Just like workers do when starting a task.

@jedcunningham
Copy link
Member

Closes: #52040

@jedcunningham
Copy link
Member

#60734 is the right fix for this problem.

@ephraimbuddy
Copy link
Contributor Author

Closing in favour of #60734

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.

2 participants