shivaam opened a new pull request, #64942:
URL: https://github.com/apache/airflow/pull/64942

   Fix callback queueing skipping bundle init when bundle_version is None
   
   PR #60734 added `bundle.initialize()` in `_add_callback_to_queue` to
   materialize versioned bundles for callbacks. However, the guard
   `bundle.supports_versioning and request.bundle_version` skips
   initialization when `bundle_version` is `None` — which is the
   expected value when `disable_bundle_versioning=True`.
   
   This means DAG and task callbacks on versioned bundle backends
   (e.g. GitDagBundle) with disabled versioning use an uninitialized
   `bundle.path`, causing callback processing to silently fail.
   
   The fix broadens the condition to `bundle.supports_versioning` only.
   This is safe because `initialize()` dispatches internally based on
   `self.version` — when `None`, it refreshes the tracking repo; when a
   commit hash, it materializes the versioned directory. This aligns the
   callback path with `_refresh_dag_bundles`, which initializes
   unconditionally.
   
   closes: #64891
   related: #52040, #60734
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Claude Opus 4.6)
   
   Generated-by: Claude Code (Claude Opus 4.6) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to