john-jac commented on PR #66046:
URL: https://github.com/apache/airflow/pull/66046#issuecomment-4344957559
Thanks @vincbeck for the review! Addressed both points:
1. **IAM role as external parameter** — now uses
SystemTestContextBuilder().add_variable("ROLE_ARN") instead of creating roles
in the test (following the example_batch.py pattern).
2. **Split into individual tasks** — replaced the monolithic setup/teardown
with separate @task functions: create_bucket, upload_workflow_yaml,
create_workflow, stop_workflow_run, delete_workflow, delete_bucket. Each can be
replaced by an operator later.
--
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]
