mitre88 opened a new pull request, #70485: URL: https://github.com/apache/airflow/pull/70485
Two SageMaker operators applied logic to template fields in `__init__`, before Jinja rendering: - `SageMakerProcessingOperator` rejected `output_files_to_xcom` without `wait_for_completion=True` at construction time — a templated `output_files_to_xcom` that renders empty was rejected anyway. The guard now runs at the top of `execute()`. - `SageMakerCreateNotebookOperator` ran `format_tags` over `create_instance_kwargs["tags"]` in `__init__`, so a templated kwargs dict never got its tags formatted (and the operator mutated its own constructor argument). The formatting now happens in `execute()` on a copy, leaving the operator's stored kwargs untouched. Both classes are removed from the exemption list. related: #70296 --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Fable 5) Generated-by: Claude Code (Fable 5) 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]
