ksharlandjiev commented on code in PR #56936:
URL: https://github.com/apache/airflow/pull/56936#discussion_r2460920118
##########
providers/amazon/src/airflow/providers/amazon/aws/operators/ssm.py:
##########
@@ -90,7 +90,7 @@ def execute_complete(self, context: Context, event: dict[str,
Any] | None = None
event = validate_execute_complete_event(event)
if event["status"] != "success":
- raise AirflowException(f"Error while running run command: {event}")
+ raise RuntimeError(f"Error while running run command: {event}")
Review Comment:
@o-nikolas I've checked the entire repository. The only place I've found is
the SsmRunCommandTrigger class still raises AirflowException in its run()
method.
I'm happy to revert this and use AirflowException, if this feels safer from
backward compatibility perspective.
--
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]
