kalluripradeep commented on PR #61338: URL: https://github.com/apache/airflow/pull/61338#issuecomment-3837538188
## Test Fix Applied Fixed the `dag_run_trigger_already_exists` test case by adding the 6th parameter (`run_after=None`) to match the updated `trigger()` method signature. ### Note on Other Test Failures I noticed in previous CI runs there were 2 test failures that appear unrelated to my changes: 1. **Serialization test** (`test_roundtrip_exceptions`) - This test is failing on `TaskDeferred` serialization, which is not related to `DagRunTriggerException` or the `run_after` parameter changes. 2. **SSH Provider test** (`test_execute_defers_to_trigger`) - This test failure is in the SSH provider and unrelated to the `TriggerDagRunOperator` changes. Both failures appear to be pre-existing issues in the codebase. My changes only affect: - `TriggerDagRunOperator` adding `run_after` parameter - `DagRunTriggerException` adding `run_after` parameter - API endpoint `/dag_runs/trigger` adding `run_after` support - Related test updates Please let me know if these failures need to be addressed or if they're known issues. -- 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]
