jroachgolf84 opened a new pull request, #63147:
URL: https://github.com/apache/airflow/pull/63147

   ## Description
   
   Per issue #57891, the following functionality has been requested:
   
   > Support new parameter `sftp_remote_host` in 
[S3ToSFTPOperator](https://github.com/apache/airflow/blob/main/providers/amazon/src/airflow/providers/amazon/aws/transfers/s3_to_sftp.py)
 and 
[SFTPToS3Operator](https://github.com/apache/airflow/blob/main/providers/amazon/src/airflow/providers/amazon/aws/transfers/sftp_to_s3.py).
   > 
   > This parameter, if provided, would override `remote_host` stored in 
`sftp_conn_id`.
   > 
   > This would work the same as the `remote_host` parameter in 
[SFTPOperator](https://github.com/apache/airflow/blob/main/providers/sftp/src/airflow/providers/sftp/operators/sftp.py)
   
   This PR adds the `sftp_remote_host` to both the `SFTPToS3Operator` and 
`S3ToSFTPOperator`. As requested, the implemented changes allow for the host in 
the `sftp_conn_id` to be overridden. This is a very common pattern seen in 
other Operators across Airflow Providers.
   
   closes: #57891 
   
   
   ## Testing
   
   Unit tests were written for these changes for both Operators. These unit 
tests can be executed as follows:
   
   ```bash
   git fetch origin
   git checkout origin/issue-57891
   git pull
   
   breeze testing providers-tests 
providers/amazon/tests/unit/amazon/aws/transfers/test_s3_to_sftp.py 
   
   breeze testing providers-tests 
providers/amazon/tests/unit/amazon/aws/transfers/test_sftp_to_s3.py 
   ```


-- 
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]

Reply via email to