o-nikolas commented on code in PR #66833:
URL: https://github.com/apache/airflow/pull/66833#discussion_r3230515668
##########
providers/amazon/tests/system/amazon/aws/example_mwaa_serverless.py:
##########
@@ -117,6 +118,16 @@ def delete_workflow(workflow_arn: str):
)
# [END howto_sensor_mwaa_serverless_workflow_run]
+ # [START howto_operator_mwaa_serverless_update_workflow]
Review Comment:
Please run the system test and verify it is still passing with these
changes. We've had a bit of issues with garbage in and garbage out in our test
harness
##########
providers/amazon/src/airflow/providers/amazon/aws/operators/mwaa_serverless.py:
##########
@@ -159,6 +159,83 @@ def execute(self, context: Context) -> str:
return workflow_arn
+class MwaaServerlessUpdateWorkflowOperator(AwsBaseOperator[AwsBaseHook]):
+ """
+ Update an existing Amazon MWAA Serverless workflow.
+
+ .. seealso::
+ For more information on how to use this operator, take a look at the
guide:
+ :ref:`howto/operator:MwaaServerlessUpdateWorkflowOperator`
+
+ :param workflow_arn: The ARN of the workflow to update. (templated)
+ :param definition_s3_location: Dict with ``Bucket``, ``ObjectKey``, and
optionally
+ ``VersionId`` for the updated YAML definition. (templated)
+ :param role_arn: The execution role ARN. (templated)
+ :param description: Optional updated description. (templated)
+ :param engine_version: Optional engine version number.
+ :param logging_configuration: Optional logging configuration dict with
``LogGroupName``.
+ :param network_configuration: Optional network configuration dict with
+ ``SecurityGroupIds`` and ``SubnetIds``.
Review Comment:
Why not template these?
--
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]
