harsh481 commented on PR #55569: URL: https://github.com/apache/airflow/pull/55569#issuecomment-3302098997
> > Hey @o-nikolas / @vincbeck How do I merge these changes into the main branch? Are there any additional steps I need to follow? > > Hey @harsh481 , tests are failing. Can you please take a look and fix them? Examples of failures: > > ``` > ERROR providers/amazon/tests/unit/amazon/aws/transfers/test_s3_to_sftp.py::TestS3ToSFTPOperator::test_s3_to_sftp_operation_confirm_true_default - airflow.exceptions.AirflowException: SSH operator error: exit status = 1 > ERROR providers/amazon/tests/unit/amazon/aws/transfers/test_s3_to_sftp.py::TestS3ToSFTPOperator::test_s3_to_sftp_operation_confirm_false - airflow.exceptions.AirflowException: SSH operator error: exit status = 1 > ``` Hey @vincbeck , Thank you for pointing out the issue in the test. I’ve fixed the SSH connection cleanup and made it consistent with test_s3_to_sftp_operation. Here is the output of tests with latest commit changes: - For `test_s3_to_sftp_operation_confirm_false` : <img width="1050" height="215" alt="Screenshot 2025-09-17 at 2 40 59 PM" src="https://github.com/user-attachments/assets/0efbf866-3a66-4577-afdc-a57db61d0c33"; /> - For `test_s3_to_sftp_operation_confirm_false` : <img width="1050" height="215" alt="Screenshot 2025-09-17 at 2 42 44 PM" src="https://github.com/user-attachments/assets/2f60a965-55a8-4177-8773-96525bf88f41"; /> -- 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]
