power-edge commented on PR #69087: URL: https://github.com/apache/airflow/pull/69087#issuecomment-4949819852
Closing this. Since this PR was opened, `main` reworked callback bundle handling (in the "initialize versioned bundles for callbacks" change) and now explicitly treats an unversioned callback request (`bundle_version=None`) as one that should **not** trigger `bundle.initialize()` — codified by the `test_prepare_callback_bundle_skips_initialize_for_unversioned_request` test, which asserts `initialize` is *not* called in exactly the case this PR was initializing it. That is the opposite of the behavior here, so this approach is superseded by the current design. The underlying question of whether skipping initialization for `bundle_version=None` is intended or a bug is still worth confirming; I'll follow up on the linked issue rather than pushing a change that conflicts with the current tests. --- Drafted-by: Claude Code (Opus 4.8) (no human review before posting) -- 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]
