The GitHub Actions job "Tests (AMD)" on airflow.git/skip-import-when-rebuilding-callback-from-serialized-data has failed. Run started by GitHub user potiuk (triggered by potiuk).
Head commit for run: 373f881a0f91d64a4d6c6d82f7ebb93f7d0ed81b / Jarek Potiuk <[email protected]> Do not import the stored path when rebuilding a Callback from serialized data Callback.get_callback_path imports the module named by a dotted-path string in order to check that it resolves to a callable. That check is best effort: the ImportError is logged and swallowed, and the path is returned either way, because the callable may only exist on the host that will run it. Rebuilding a Callback from its serialized form went through the same path, so deserializing one imported the module named in the stored data. The path was already checked when the Callback was first created, and reconstruction happens in components that never call the callback themselves, so importing there is neither needed nor wanted. Carry a path read back from serialized data in a private str subclass, and return it unchanged instead of resolving it a second time. Construction from a Dag author's callable or dotted path is unaffected, including the dot-path shape check, which still applies to stored paths as well. Generated-by: Claude Opus 5 (1M context) following the guidelines at https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions Report URL: https://github.com/apache/airflow/actions/runs/30491359574 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
